[Buildroot] [PATCH] wpa-supplicant: add options to enable the new DBus interface

Arnout Vandecappelle arnout at mind.be
Wed Sep 19 20:03:11 UTC 2012


On 09/19/12 11:02, Sven Neumann wrote:
> Hi,
>
> thanks a lot for your review. I'll make the changes you suggested, but I
> am not sure about the following:
>
> On Tue, 2012-09-18 at 08:12 +0200, Arnout Vandecappelle wrote:
>
>>> +config BR2_PACKAGE_WPA_SUPPLICANT_DBUS_OLD
>>> +	bool "Enable support for old DBus control interface"
>>> +        depends on BR2_PACKAGE_DBUS
>>
>>     Use 1 tab instead of 8 spaces (below as well).
>>
>>    Also, I think it's better to make this
>>       select BR2_PACKAGE_DBUS
>>       depends on BR2_TOOLCHAIN_HAS_THREADS
>>
>>    Also, since this only installs a .service file, and it used to be
>> always enabled before, I would make at least one of the two
>> DBus interfaces enabled by default.
>
> If I follow this suggestion and enable one of the DBus interfaces by
> default, then selecting wpa-supplicant would select DBus, which it did
> not do before. Is that really desired?

  Actually, on second thought, it makes more sense to depend on dbus
after all: .service files are installed if dbus is enabled.  That's also
the old behaviour.

  Regards,
  Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F


More information about the buildroot mailing list