[alsa-devel] [PATCH - hwmixvolume 1/9] hwmixvolume: replace PyGTK with gobject-introspection

Takashi Sakamoto o-takashi at sakamocchi.jp
Tue Aug 14 06:24:14 CEST 2018


Hi,

On Aug 10 2018 23:56, Takashi Iwai wrote:
> On Wed, 08 Aug 2018 17:56:29 +0200,
> Emmanuel Gil Peyrot wrote:
>>
>> This doesn’t work yet, we require Gtk 3.0 rather than 2.0 and the API
>> changed quite a lot.
>>
>> Signed-off-by: Emmanuel Gil Peyrot <linkmauve at linkmauve.fr>
> 
> Thanks for the patches.  Most of changes look good, but also most of
> patches have no change log at all.  That's already a bad sign.
> 
> Could you give some details, especially why the change is needed, in
> each patch and resubmit as a patchset?  At best, send with a cover
> letter and a thread as a patch set at the next time (git-send-email
> can manage it well).

In my opinion, the aim of this patchset is porting from Gtk+2.0 to 
Gtk+3.0, according to EOL of python bindings. Original script uses 
PyGtk[1], which is an application of Gtk+2.0, but PyGtk is already at 
the end of life and usage of PyGObject[2] to access to metadata of 
Gtk+3.0 is a fresh way instead. In this point, Emmanuel's proposal is 
preferable in 2018.

Anyway, I encourage Emmanuel to re-post this patchset with enough 
comment according to maintainer's advice. I'll review his next post for 
merging.

[1] https://github.com/GNOME/pygtk
[2] https://gitlab.gnome.org/GNOME/pygobject


Thanks

Takashi Sakamoto


More information about the Alsa-devel mailing list