[Buildroot] [PATCH v9 03/14] libgtk2: bump to version 2.24.18

Spenser Gilliland spenser at gillilanding.com
Thu May 30 17:06:13 UTC 2013


Thomas,

> Please mention explicitly in the commit log that you are dropping
> DirectFB support for good. Generally speaking, your commit log would
> need to be a little bit more detailed about what you're doing. No need
> to give gazillion of details, but highlighting the important changes is
> good when the bump is not simply a mechanical bump.
>
> Also, I believe more parts of libgtk2.mk can be removed if you remove
> DirectFB support:
>
> ifeq ($(BR2_PACKAGE_DIRECTFB),y)
>         LIBGTK2_CONF_OPT += --with-gdktarget=directfb
>         LIBGTK2_DEPENDENCIES += directfb
> endif
>
> and also all the code that was enclosed in a ifeq
> ($(BR2_PACKAGE_XORG7),y) can now go out of this conditional.

This has been fixed in the newest version.

> Or maybe you want to do two separate patches: one that drops the
> DirectFB support, and then one that bumps to 2.24.18.

I've update the patch title to

libgtk2: bump to version 2.24.18 and drop DirectFB

the commit message states.

 * Drop DirectFB support from libgtk2
 * bump libgtk2 to version 2.24.18

> Thanks!

Thanks,
Spenser


--
Spenser Gilliland
Computer Engineer
Doctoral Candidate



More information about the buildroot mailing list