[Buildroot] [PATCH 2/2] New package: feh

Yegor Yefremov 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
> duplication.
>
> 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
and Thomas.

Yegor


More information about the buildroot mailing list