[Buildroot] [PATCH 1/1] wpa-supplicant: Add NL80211 support option
thomas.petazzoni at free-electrons.com
Fri Aug 8 08:41:51 UTC 2014
Dear Jean-Baptiste Theou,
On Thu, 7 Aug 2014 13:52:33 -0700, Jean-Baptiste Theou wrote:
> IMHO, wpa_supplicant should provide the support of NL80211 by default
> ('depends on BR2_PACKAGE_LIBNL) since it's the new standard.
> I wasn't expected to have to select LIBNL manually.
There is a balance to find between:
* Adding suboptions to package to enable support for optional features
(what you did).
Advantages : more obvious to the user, you can enable an optional
feature in package A, but not in package B (like you can use OpenSSL
support for your web server, but not necessarily for all other
packages in your system)
Drawbacks : maintenance burden due to more Config.in options all
over the place.
* Make packages automatically enable optional features when the
necessary dependencies are available.
Advantages : less Config.in options to add everywhere.
Drawbacks : less obvious to the user, less flexible (enabling
OpenSSL makes it used in all packages that can optionally use it).
You could propose a patch that extends the Config.in help text of
wpa_supplicant to indicate this optional dependency, though we don't
have a specific policy about this.
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
More information about the buildroot