From 35eaed8d07bdc73a83990bc306fcdb08b2e70eaf Mon Sep 17 00:00:00 2001 From: Thomas De Schampheleire Date: Fri, 20 Dec 2013 22:31:30 +0100 Subject: Config.in files: use if/endif instead of 'depends on' for main symbol 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 Signed-off-by: Thomas Petazzoni --- package/lcdproc/Config.in | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'package/lcdproc') diff --git a/package/lcdproc/Config.in b/package/lcdproc/Config.in index 5635a0209..f5a8501e5 100644 --- a/package/lcdproc/Config.in +++ b/package/lcdproc/Config.in @@ -9,15 +9,17 @@ config BR2_PACKAGE_LCDPROC http://lcdproc.org/ +if BR2_PACKAGE_LCDPROC + config BR2_PACKAGE_LCDPROC_DRIVERS string "List of drivers to build" default "all" - depends on BR2_PACKAGE_LCDPROC help Specify a comma-separated list of lcdproc drivers to be built config BR2_PACKAGE_LCDPROC_MENUS bool "Menu support" - depends on BR2_PACKAGE_LCDPROC help Enable support for lcdproc menus + +endif -- cgit v1.2.3-70-g09d2