[Buildroot] [PATCH 2/2] New package: feh
yegorslists at googlemail.com
Wed Oct 12 09:20:34 UTC 2011
On Tue, Oct 11, 2011 at 7:29 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
> On Tuesday 11 October 2011 12:32:31, yegorslists at googlemail.com wrote:
>> +config BR2_PACKAGE_FEH
>> + bool "feh"
>> + depends on BR2_PACKAGE_XLIB_LIBXINERAMA
>> + depends on BR2_PACKAGE_XLIB_LIBXT
> depends on BR2_PACKAGE_XORG7 and select these two?
>> + select BR2_PACKAGE_IMLIB2
> Redundant, giblib already selects imlib2.
>> + select BR2_PACKAGE_IMLIB2_PNG
> Here we have a problem that probably occurs in other places as well, and I
> don't see a good solution.
> The reason to select BR2_PACKAGE_IMLIB2_PNG is probably because ImLib2 is
> completely useless if you don't select any backend. However, the choice for
> png is arbitrary.
> You could put a depends on instead of a select, but then the user has to go
> to two places to get it configured.
> You could repeat the backend options here, but then you get a lot of
> You could select all backends but then ImLib2 is overconfigured.
> You could make png default y for ImLib2, but then the user still has to go
> to two places if he needs gif instead of png.
> At least, the user should be directed to the configuration menu for ImLib2
> so he can select the required backend there.
> One final option is to create a new top-level menu called 'Features' or
> something, which contains items like BR2_FEATURE_PNG, BR2_FEATURE_GIF.
> imlib2/Config.in could then set BR2_PACKAGE_IMLIB2_PNG default y if
> BR2_FEATURE_PNG. But this feels a bit heavy-handed to me.
Yes. Such cases are rather tricky. Let's wait for comments from Peter
More information about the buildroot