[Buildroot] [PATCH] Config.in files: use if/endif instead of 'depends on' for main symbol
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Wed Dec 25 11:27:11 UTC 2013
Dear Thomas De Schampheleire,
On Fri, 20 Dec 2013 22:31:30 +0100, Thomas De Schampheleire wrote:
> In the Config.in file of package foo, it often happens that there are other
> symbols besides BR2_PACKAGE_FOO. Typically, these symbols only make sense
> when foo itself is enabled. There are two ways to express this: with
> depends on BR2_PACKAGE_FOO
> in each extra symbol, or with
> if BR2_PACKAGE_FOO
> ...
> endif
> around the entire set of extra symbols.
>
> The if/endif approach avoids the repetition of 'depends on' statements on
> multiple symbols, so this is clearly preferred. But even when there is only
> one extra symbol, if/endif is a more logical choice:
> - it is future-proof for when extra symbols are added
> - it allows to have just one strategy instead of two (less confusion)
>
> This patch modifies the Config.in files accordingly.
>
> Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
Applied, thanks.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list