[Buildroot] [PATCH 3/9] wireshark: enable GUI options

[Gustavo Zacarias <gustavo@zacarias.com.ar>]

On 02/12/15 18:55, Yann E. MORIN wrote:


> And add this variable, too:
>
>      WIRESHARK_HAS_GUI = NO
...
>> +# Qt4 needs accessibility, we don't support it
>> +ifeq ($(BR2_PACKAGE_QT5BASE_WIDGETS),y)
>> +WIRESHARK_CONF_OPTS += --with-qt=5
>> +WIRESHARK_DEPENDENCIES += qt5base
>> +# Seems it expects wrappers and passes a -qt=X parameter for version
>> +WIRESHARK_MAKE_OPTS += \
>> +	MOC="$(HOST_DIR)/usr/bin/moc" \
>> +	RCC="$(HOST_DIR)/usr/bin/rcc" \
>> +	UIC="$(HOST_DIR)/usr/bin/uic"
>> +else
>> +WIRESHARK_CONF_OPTS += --with-qt=no
>
> Ditto.
>
>> +endif
>> +
>> +# No GUI at all
>> +ifeq ($(BR2_PACKAGE_LIBGTK2)$(BR2_PACKAGE_LIBGTK3)$(BR2_PACKAGE_QT5BASE_WIDGETS),)
>
> And then change that condition to:
>
>      ifeq ($(WIRESHARK_HAS_GUI),NO)
>
> which is simpler to manage should there be other toolkit we can use in
> the future.

Hi.
Well, only one gtk and/or qt variant can be built at the same time, and 
i don't expect any other major toolkit to show up in the near future.
I think it can be adjusted in the future if that's the case, but right 
now that's only cosmetic.
If someone fails to specify --disable-wireshark the wireshark 
buildsystem will balk anyway since it's default on and won't detect any 
usable toolkit.
Regards.