Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 13:13   ` Peter Korsgaard
  2016-07-04  9:24 ` [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection Yann E. MORIN
                   ` (81 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 boot/at91bootstrap/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/at91bootstrap/Config.in b/boot/at91bootstrap/Config.in
index 57fdeda..d131a6a 100644
--- a/boot/at91bootstrap/Config.in
+++ b/boot/at91bootstrap/Config.in
@@ -26,7 +26,7 @@ config BR2_TARGET_AT91BOOTSTRAP_BOARD
 	  This is used to do a make <board>_config
 
 choice
-	prompt "Boot Memory"
+	bool "Boot Memory"
 	default BR2_TARGET_AT91BOOTSTRAP_DATAFLASH
 	help
 	  Select Chip for which AT91 bootstrap should be built
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 13:19   ` Peter Korsgaard
  2016-07-04 14:07   ` Peter Korsgaard
  2016-07-04  9:24 ` [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
                   ` (80 subsequent siblings)
  82 siblings, 2 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Remove the superfluous config option, and make all modules default to
'y', which provides exactly the same functionality.

simplify (and slightly beautify) the handling of modules in the .mk:
since the modules' symbols are always meaningful now, we can get rid of
the else-clause, and always build the list of modules using the Kconfig
options.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Arnout Vandecappelle <arnout@mind.be>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

---
Changes v1 -> v2:
  - remove spurious comment leftover from RFC  (Thomas)

Changes RFC -> v1:
  - treat the 'angular' module on its own, as was done previosuly
    (Arnout)
  - drop useless comments in Config.in  (Arnout)
  - don't re-order list, it was already alphabetically sorted
---
 package/angularjs/Config.in    | 19 +++++++++----------
 package/angularjs/angularjs.mk | 21 ++++++++++++++-------
 2 files changed, 23 insertions(+), 17 deletions(-)

diff --git a/package/angularjs/Config.in b/package/angularjs/Config.in
index 206b3d8..2a0b48c 100644
--- a/package/angularjs/Config.in
+++ b/package/angularjs/Config.in
@@ -7,16 +7,9 @@ menuconfig BR2_PACKAGE_ANGULARJS
 
 if BR2_PACKAGE_ANGULARJS
 
-config BR2_ANGULARJS_MODULES
-	bool "angularjs modules"
-	help
-	  Select which modules to install. If disabled, all modules
-	  will be installed.
-
-if BR2_ANGULARJS_MODULES
-
 config BR2_ANGULARJS_MODULE_ANIMATE
 	bool "animate"
+	default y
 	help
 	  The ngAnimate module provides support for CSS-based animations
 	  (keyframes and transitions) as well as JavaScript-based animations
@@ -26,6 +19,7 @@ config BR2_ANGULARJS_MODULE_ANIMATE
 
 config BR2_ANGULARJS_MODULE_ARIA
 	bool "aria"
+	default y
 	help
 	  The ngAria module provides support for common ARIA attributes that
 	  convey state or semantic information about the application for users
@@ -33,18 +27,21 @@ config BR2_ANGULARJS_MODULE_ARIA
 
 config BR2_ANGULARJS_MODULE_COOKIES
 	bool "cookies"
+	default y
 	help
 	  The ngCookies module provides a convenient wrapper for reading and
 	  writing browser cookies.
 
 config BR2_ANGULARJS_MODULE_MESSAGE_FORMAT
 	bool "message-format"
+	default y
 	help
 	  The ngMessageFormat module is used recognize MessageFormat extensions
 	  in interpolation expressions.
 
 config BR2_ANGULARJS_MODULE_MESSAGES
 	bool "messages"
+	default y
 	help
 	  The ngMessages module provides enhanced support for displaying
 	  messages within templates (typically within forms or when rendering
@@ -52,28 +49,30 @@ config BR2_ANGULARJS_MODULE_MESSAGES
 
 config BR2_ANGULARJS_MODULE_RESOURCE
 	bool "resource"
+	default y
 	help
 	  The ngResource module provides interaction support with RESTful
 	  services via the $resource service.
 
 config BR2_ANGULARJS_MODULE_ROUTE
 	bool "route"
+	default y
 	help
 	  The ngRoute module provides routing and deeplinking services and
 	  directives for angular apps.
 
 config BR2_ANGULARJS_MODULE_SANITIZE
 	bool "sanitize"
+	default y
 	help
 	  The ngSanitize module provides functionality to sanitize HTML.
 
 config BR2_ANGULARJS_MODULE_TOUCH
 	bool "touch"
+	default y
 	help
 	  The ngTouch module provides touch events and other helpers for
 	  touch-enabled devices. The implementation is based on jQuery Mobile
 	  touch event handling (jquerymobile.com).
 
 endif
-
-endif
diff --git a/package/angularjs/angularjs.mk b/package/angularjs/angularjs.mk
index 60dbdb6..1f9a8fa 100644
--- a/package/angularjs/angularjs.mk
+++ b/package/angularjs/angularjs.mk
@@ -17,18 +17,25 @@ define ANGULARJS_EXTRACT_CMDS
 	rmdir $(@D)/angular-$(ANGULARJS_VERSION)
 endef
 
+# This list mirrors the list of available modules from Config.in.
+# Keep this list in sync and in the same (alphabetical) order as the
+# one in Config.in.
+ANGULARJS_MODULES = \
+	animate \
+	aria \
+	cookies \
+	messages \
+	message-format \
+	resource \
+	route \
+	sanitize \
+	touch
+
 ANGULARJS_FILES = angular
 
-ANGULARJS_MODULES = animate aria cookies message-format messages resource \
-	route sanitize touch
-
-ifeq ($(BR2_ANGULARJS_MODULES),y)
 ANGULARJS_FILES += $(foreach mod,$(ANGULARJS_MODULES),\
 			$(if $(BR2_ANGULARJS_MODULE_$(call UPPERCASE,$(mod))),\
 				angular-$(mod)))
-else
-ANGULARJS_FILES += $(foreach mod,$(ANGULARJS_MODULES),angular-$(mod))
-endif
 
 define ANGULARJS_INSTALL_TARGET_CMDS
 	$(foreach f,$(ANGULARJS_FILES),\
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 15:14   ` Peter Korsgaard
  2016-07-04  9:24 ` [Buildroot] [PATCH 04/83 v2] package/gd: " Yann E. MORIN
                   ` (79 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Keep the existing sub-menus, because there are a lot of entries in those
sub-menus. Using comments to separate them is not a totally satifactory
solution.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/collectd/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/collectd/Config.in b/package/collectd/Config.in
index 71301ec..6405af6 100644
--- a/package/collectd/Config.in
+++ b/package/collectd/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_COLLECTD
+menuconfig BR2_PACKAGE_COLLECTD
 	bool "collectd"
 	# Uses fork()
 	depends on BR2_USE_MMU
@@ -39,6 +39,7 @@ config BR2_PACKAGE_COLLECTD_VALUE
 	bool "value"
 	help
 	  Select values by their data sources' values.
+
 endmenu
 
 menu "misc plugins"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 04/83 v2] package/gd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (2 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 15:10   ` Peter Korsgaard
  2016-07-04  9:24 ` [Buildroot] [PATCH 05/83 v2] package/gpsd: " Yann E. MORIN
                   ` (78 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - typo in title  (Arnout)
---
 package/gd/Config.in | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/package/gd/Config.in b/package/gd/Config.in
index 28b0b0f..092bc06 100644
--- a/package/gd/Config.in
+++ b/package/gd/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_GD
+menuconfig BR2_PACKAGE_GD
 	bool "gd"
 	help
 	  GD is a graphics library. It allows your code to quickly
@@ -12,7 +12,7 @@ config BR2_PACKAGE_GD
 
 if BR2_PACKAGE_GD
 
-menu "gd tools"
+comment "gd tools"
 
 config BR2_PACKAGE_GD_ANNOTATE
 	bool "annotate"
@@ -57,6 +57,4 @@ config BR2_PACKAGE_GD_WEBPNG
 	bool "webpng"
 	select BR2_PACKAGE_LIBPNG
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 05/83 v2] package/gpsd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (3 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 04/83 v2] package/gd: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 15:18   ` Peter Korsgaard
  2016-07-04  9:24 ` [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in Yann E. MORIN
                   ` (77 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Since there were multiple sub-menus, their prompts are turned into
comments.

Some option were missing a prompt, so add one; switch integer options to
use the 'int' type instead of 'string'. Add separating lines around
options.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/gpsd/Config.in | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/package/gpsd/Config.in b/package/gpsd/Config.in
index b4008eb..876ab9b 100644
--- a/package/gpsd/Config.in
+++ b/package/gpsd/Config.in
@@ -2,7 +2,7 @@ comment "gpsd needs a toolchain w/ threads, dynamic library"
 	depends on BR2_USE_MMU
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
-config BR2_PACKAGE_GPSD
+menuconfig BR2_PACKAGE_GPSD
 	bool "gpsd"
 	# Uses fork()
 	depends on BR2_USE_MMU
@@ -25,7 +25,7 @@ config BR2_PACKAGE_GPSD_DEVICES
 	string "Where to look for GPSes"
 	default "/dev/ttyS1"
 
-menu "Features"
+comment "Features"
 
 config BR2_PACKAGE_GPSD_CLIENT_DEBUG
 	bool "client debugging support"
@@ -50,36 +50,41 @@ config BR2_PACKAGE_GPSD_PPS
 
 config BR2_PACKAGE_GPSD_USER
 	bool "GPSD privilege revocation user"
+
 config BR2_PACKAGE_GPSD_USER_VALUE
-	string ""
+	string "user to run as"
 	default "nobody"
 	depends on BR2_PACKAGE_GPSD_USER
 
 config BR2_PACKAGE_GPSD_GROUP
 	bool "GPSD privilege revocation group"
+
 config BR2_PACKAGE_GPSD_GROUP_VALUE
-	string ""
+	string "group to run as"
 	default "nobody"
 	depends on BR2_PACKAGE_GPSD_GROUP
 
 config BR2_PACKAGE_GPSD_FIXED_PORT_SPEED
 	bool "compile with fixed serial port speed"
+
 config BR2_PACKAGE_GPSD_FIXED_PORT_SPEED_VALUE
-	string ""
+	int "serial port speed"
 	default "9600"
 	depends on BR2_PACKAGE_GPSD_FIXED_PORT_SPEED
 
 config BR2_PACKAGE_GPSD_MAX_CLIENT
 	bool "compile with limited maximum clients"
+
 config BR2_PACKAGE_GPSD_MAX_CLIENT_VALUE
-	string "compile with limited maximum clients"
+	int "maximum number of clients"
 	default "10"
 	depends on BR2_PACKAGE_GPSD_MAX_CLIENT
 
 config BR2_PACKAGE_GPSD_MAX_DEV
 	bool "compile with maximum allowed devices"
+
 config BR2_PACKAGE_GPSD_MAX_DEV_VALUE
-	string "compile with maximum allowed devices"
+	int "maximum allowed devices"
 	default "2"
 	depends on BR2_PACKAGE_GPSD_MAX_DEV
 
@@ -94,9 +99,7 @@ config BR2_PACKAGE_GPSD_CONTROLSEND
 config BR2_PACKAGE_GPSD_SQUELCH
 	bool "squelch gpsd_report and gpsd_hexdump to save cpu"
 
-endmenu
-
-menu "Protocols"
+comment "Protocols"
 
 config BR2_PACKAGE_GPSD_AIVDM
 	bool "Aivdm"
@@ -231,6 +234,4 @@ config BR2_PACKAGE_GPSD_UBX
 	help
 	  uBlox UBX binary support
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (4 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 05/83 v2] package/gpsd: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 20:41   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom Yann E. MORIN
                   ` (76 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

... and make it a menuconfig instead of a 'config'+'menu'.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in        | 12 ------------
 package/jquery/Config.in | 17 ++++++++++++++++-
 2 files changed, 16 insertions(+), 13 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 67810b0..d312347 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1055,18 +1055,6 @@ menu "Javascript"
 	source "package/explorercanvas/Config.in"
 	source "package/flot/Config.in"
 	source "package/jquery/Config.in"
-if BR2_PACKAGE_JQUERY
-menu "External jQuery plugins"
-	source "package/jquery-datetimepicker/Config.in"
-	source "package/jquery-keyboard/Config.in"
-	source "package/jquery-mobile/Config.in"
-	source "package/jquery-sidebar/Config.in"
-	source "package/jquery-sparkline/Config.in"
-	source "package/jquery-ui/Config.in"
-	source "package/jquery-ui-themes/Config.in"
-	source "package/jquery-validation/Config.in"
-endmenu
-endif
 	source "package/jsmin/Config.in"
 	source "package/json-javascript/Config.in"
 endmenu
diff --git a/package/jquery/Config.in b/package/jquery/Config.in
index 92f5604..3133702 100644
--- a/package/jquery/Config.in
+++ b/package/jquery/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_JQUERY
+menuconfig BR2_PACKAGE_JQUERY
 	bool "jQuery"
 	help
 	  jQuery is a fast and concise JavaScript Library that
@@ -6,3 +6,18 @@ config BR2_PACKAGE_JQUERY
 	  animating, and Ajax interactions for rapid web development.
 
 	  http://jquery.com
+
+if BR2_PACKAGE_JQUERY
+
+comment "External jQuery plugins"
+
+source "package/jquery-datetimepicker/Config.in"
+source "package/jquery-keyboard/Config.in"
+source "package/jquery-mobile/Config.in"
+source "package/jquery-sidebar/Config.in"
+source "package/jquery-sparkline/Config.in"
+source "package/jquery-ui/Config.in"
+source "package/jquery-ui-themes/Config.in"
+source "package/jquery-validation/Config.in"
+
+endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (5 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 20:42   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
                   ` (75 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Tests programs are not that important; at least, they're probably less
important than the drivers selection.

Move it down to after the drivers selection.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/libdrm/Config.in | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/package/libdrm/Config.in b/package/libdrm/Config.in
index 7c2a828..7bc73fd 100644
--- a/package/libdrm/Config.in
+++ b/package/libdrm/Config.in
@@ -18,11 +18,6 @@ config BR2_PACKAGE_LIBDRM_ENABLE_ATOMIC
 	bool
 	select BR2_PACKAGE_LIBATOMIC_OPS if !BR2_TOOLCHAIN_HAS_SYNC_4
 
-config BR2_PACKAGE_LIBDRM_INSTALL_TESTS
-	bool "Install test programs"
-	help
-	  This option allows to install the libdrm test programs.
-
 menu "DRM Drivers"
 
 config BR2_PACKAGE_LIBDRM_INTEL
@@ -99,6 +94,11 @@ config BR2_PACKAGE_LIBDRM_VC4
 
 endmenu
 
+config BR2_PACKAGE_LIBDRM_INSTALL_TESTS
+	bool "Install test programs"
+	help
+	  This option allows to install the libdrm test programs.
+
 endif
 
 comment "libdrm needs a toolchain w/ threads"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (6 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:35   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 09/83 v2] package/mono: " Yann E. MORIN
                   ` (74 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Transform the menu prompt into a comment.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - drop the comment  (Arnout)
---
 package/libdrm/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/libdrm/Config.in b/package/libdrm/Config.in
index 7bc73fd..3f39e27 100644
--- a/package/libdrm/Config.in
+++ b/package/libdrm/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LIBDRM
+menuconfig BR2_PACKAGE_LIBDRM
 	bool "libdrm"
 	select BR2_PACKAGE_LIBPTHREAD_STUBS
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libpthread-stubs
@@ -18,8 +18,6 @@ config BR2_PACKAGE_LIBDRM_ENABLE_ATOMIC
 	bool
 	select BR2_PACKAGE_LIBATOMIC_OPS if !BR2_TOOLCHAIN_HAS_SYNC_4
 
-menu "DRM Drivers"
-
 config BR2_PACKAGE_LIBDRM_INTEL
 	bool "intel"
 	select BR2_PACKAGE_LIBPCIACCESS
@@ -92,8 +90,6 @@ config BR2_PACKAGE_LIBDRM_VC4
 	help
 	  Install vc4 (Raspberry Pi) driver.
 
-endmenu
-
 config BR2_PACKAGE_LIBDRM_INSTALL_TESTS
 	bool "Install test programs"
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 09/83 v2] package/mono: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (7 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:38   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 10/83 v2] package/ncftp: " Yann E. MORIN
                   ` (73 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

This is a bit overkill, as there is only one sub-item. However, this
commit is not about judging whether it makes sense or not to have a
sub-menu, but to cleanup the 'config'+'menu' situation.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in      | 5 -----
 package/mono/Config.in | 8 +++++++-
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index d312347..819224b 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -569,11 +569,6 @@ endif
 	source "package/micropython-lib/Config.in"
 	source "package/moarvm/Config.in"
 	source "package/mono/Config.in"
-if BR2_PACKAGE_MONO
-menu "Mono libraries/modules"
-	source "package/mono-gtksharp3/Config.in"
-endmenu
-endif
 	source "package/nodejs/Config.in"
 	source "package/perl/Config.in"
 if BR2_PACKAGE_PERL
diff --git a/package/mono/Config.in b/package/mono/Config.in
index d45ad37..4f1f78d 100644
--- a/package/mono/Config.in
+++ b/package/mono/Config.in
@@ -3,7 +3,7 @@ config BR2_PACKAGE_MONO_ARCH_SUPPORTS
 	default y if (BR2_arm || BR2_armeb || BR2_i386 || BR2_mips || \
 		      BR2_mipsel || BR2_powerpc || BR2_x86_64)
 
-config BR2_PACKAGE_MONO
+menuconfig BR2_PACKAGE_MONO
 	bool "mono"
 	depends on BR2_PACKAGE_MONO_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_THREADS
@@ -14,6 +14,12 @@ config BR2_PACKAGE_MONO
 
 	  http://download.mono-project.com/sources/mono/
 
+if BR2_PACKAGE_MONO
+
+source "package/mono-gtksharp3/Config.in"
+
+endif
+
 comment "mono needs a toolchain w/ threads, dynamic library"
 	depends on BR2_PACKAGE_MONO_ARCH_SUPPORTS
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 10/83 v2] package/ncftp: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (8 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 09/83 v2] package/mono: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:39   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 11/83 v2] package/nfs-utils: " Yann E. MORIN
                   ` (72 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - drop the comment  (Arnout)
---
 package/ncftp/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/ncftp/Config.in b/package/ncftp/Config.in
index 57586e7..2569dc5 100644
--- a/package/ncftp/Config.in
+++ b/package/ncftp/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_NCFTP
+menuconfig BR2_PACKAGE_NCFTP
 	bool "ncftp"
 	# fork()
 	depends on BR2_USE_MMU
@@ -10,8 +10,6 @@ config BR2_PACKAGE_NCFTP
 
 if BR2_PACKAGE_NCFTP
 
-menu "ncFTP tools selection"
-
 config BR2_PACKAGE_NCFTP_GET
 	bool "NcFTPGet - command-line utility program"
 	default y
@@ -32,6 +30,4 @@ config BR2_PACKAGE_NCFTP_BOOKMARKS
 	bool "NcFTPBookmarks - ncurses based Bookmark Editor"
 	select BR2_PACKAGE_NCURSES
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 11/83 v2] package/nfs-utils: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (9 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 10/83 v2] package/ncftp: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:40   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 12/83 v2] package/nodejs: " Yann E. MORIN
                   ` (71 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/nfs-utils/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/nfs-utils/Config.in b/package/nfs-utils/Config.in
index 1273092..7476fdb 100644
--- a/package/nfs-utils/Config.in
+++ b/package/nfs-utils/Config.in
@@ -2,7 +2,7 @@ comment "nfs-utils needs a toolchain w/ threads"
 	depends on BR2_USE_MMU
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
 
-config BR2_PACKAGE_NFS_UTILS
+menuconfig BR2_PACKAGE_NFS_UTILS
 	bool "nfs-utils"
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libtirpc, rpcbind
 	select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC
@@ -15,8 +15,6 @@ config BR2_PACKAGE_NFS_UTILS
 
 if BR2_PACKAGE_NFS_UTILS
 
-menu "NFS utilities selection"
-
 config BR2_PACKAGE_NFS_UTILS_RPCDEBUG
 	bool "rpcdebug"
 	help
@@ -32,6 +30,4 @@ config BR2_PACKAGE_NFS_UTILS_RPC_RQUOTAD
 	help
 	  NFS remote quota server
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 12/83 v2] package/nodejs: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (10 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 11/83 v2] package/nfs-utils: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 13/83 v2] package/ola: " Yann E. MORIN
                   ` (70 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1;
  - drop the comment  (Arnout)
---
 package/nodejs/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/nodejs/Config.in b/package/nodejs/Config.in
index 432792f..f0b1f0e 100644
--- a/package/nodejs/Config.in
+++ b/package/nodejs/Config.in
@@ -6,7 +6,7 @@ comment "nodejs needs a toolchain w/ C++, dynamic library, threads, gcc >= 4.8,
 	depends on !BR2_INSTALL_LIBSTDCPP || BR2_STATIC_LIBS || !BR2_TOOLCHAIN_HAS_THREADS || \
 		!BR2_HOST_GCC_AT_LEAST_4_8 || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || !BR2_USE_WCHAR
 
-config BR2_PACKAGE_NODEJS
+menuconfig BR2_PACKAGE_NODEJS
 	bool "nodejs"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_INSTALL_LIBSTDCPP
@@ -46,8 +46,6 @@ config BR2_PACKAGE_NODEJS_VERSION_STRING
 	default "6.2.1"		if BR2_PACKAGE_NODEJS_V8_ARCH_SUPPORTS
 	default "0.10.45"
 
-menu "Module Selection"
-
 config BR2_PACKAGE_NODEJS_NPM
 	bool "NPM for the target"
 	select BR2_PACKAGE_OPENSSL
@@ -102,6 +100,4 @@ config BR2_PACKAGE_NODEJS_MODULES_ADDITIONAL_DEPS
 	  'libcurl' here, to ensure that buildroot builds the libcurl package,
 	  and does so before building your node modules.
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 13/83 v2] package/ola: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (11 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 12/83 v2] package/nodejs: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:38   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 14/83 v2] package/perl: " Yann E. MORIN
                   ` (69 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Since there were multiple sub-menus, their prompts are turned into
comments.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/ola/Config.in | 14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/package/ola/Config.in b/package/ola/Config.in
index 55cea65..edbd612 100644
--- a/package/ola/Config.in
+++ b/package/ola/Config.in
@@ -3,7 +3,7 @@ comment "ola needs a toolchain w/ C++, threads, wchar, dynamic library"
 		|| !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 	depends on BR2_PACKAGE_PROTOBUF_ARCH_SUPPORTS
 
-config BR2_PACKAGE_OLA
+menuconfig BR2_PACKAGE_OLA
 	bool "ola (open lighting architecture)"
 	select BR2_PACKAGE_PROTOBUF
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
@@ -22,7 +22,7 @@ config BR2_PACKAGE_OLA
 
 if BR2_PACKAGE_OLA
 
-menu "bindings and interface"
+comment "bindings and interface"
 
 config BR2_PACKAGE_OLA_WEB
 	bool "http interface"
@@ -42,9 +42,7 @@ config BR2_PACKAGE_OLA_SLP
 	help
 	  Build OLA with slp tools.
 
-endmenu
-
-menu "tests and examples"
+comment "tests and examples"
 
 config BR2_PACKAGE_OLA_EXAMPLES
 	bool "examples"
@@ -58,9 +56,7 @@ config BR2_PACKAGE_OLA_RDM_TESTS
 	help
 	  Build OLA RDM tests.
 
-endmenu
-
-menu "plugin selections"
+comment "plugin selections"
 
 config BR2_PACKAGE_OLA_PLUGIN_E131
 	bool "acn E131"
@@ -130,6 +126,4 @@ config BR2_PACKAGE_OLA_PLUGIN_USBPRO
 	help
 	  Build UsbPro plugin for OLA.
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 14/83 v2] package/perl: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (12 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 13/83 v2] package/ola: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection Yann E. MORIN
                   ` (68 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Move the perl library/modules inclusion to the perl's Config.in.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in      | 47 -----------------------------------------------
 package/perl/Config.in | 48 +++++++++++++++++++++++++++++++++++++++++++++++-
 2 files changed, 47 insertions(+), 48 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 819224b..a72dd91 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -571,53 +571,6 @@ endif
 	source "package/mono/Config.in"
 	source "package/nodejs/Config.in"
 	source "package/perl/Config.in"
-if BR2_PACKAGE_PERL
-menu "Perl libraries/modules"
-	source "package/perl-crypt-openssl-random/Config.in"
-	source "package/perl-crypt-openssl-rsa/Config.in"
-	source "package/perl-datetime-tiny/Config.in"
-	source "package/perl-db-file/Config.in"
-	source "package/perl-digest-hmac/Config.in"
-	source "package/perl-digest-sha1/Config.in"
-	source "package/perl-encode-detect/Config.in"
-	source "package/perl-encode-locale/Config.in"
-	source "package/perl-file-listing/Config.in"
-	source "package/perl-file-util/Config.in"
-	source "package/perl-gd/Config.in"
-	source "package/perl-gdgraph/Config.in"
-	source "package/perl-gdtextutil/Config.in"
-	source "package/perl-html-parser/Config.in"
-	source "package/perl-html-tagset/Config.in"
-	source "package/perl-http-cookies/Config.in"
-	source "package/perl-http-daemon/Config.in"
-	source "package/perl-http-date/Config.in"
-	source "package/perl-http-message/Config.in"
-	source "package/perl-http-negotiate/Config.in"
-	source "package/perl-io-html/Config.in"
-	source "package/perl-io-socket-ssl/Config.in"
-	source "package/perl-json-tiny/Config.in"
-	source "package/perl-libwww-perl/Config.in"
-	source "package/perl-lwp-mediatypes/Config.in"
-	source "package/perl-mail-dkim/Config.in"
-	source "package/perl-mailtools/Config.in"
-	source "package/perl-mime-base64/Config.in"
-	source "package/perl-mojolicious/Config.in"
-	source "package/perl-net-dns/Config.in"
-	source "package/perl-net-http/Config.in"
-	source "package/perl-net-ssleay/Config.in"
-	source "package/perl-netaddr-ip/Config.in"
-	source "package/perl-path-tiny/Config.in"
-	source "package/perl-time-hires/Config.in"
-	source "package/perl-timedate/Config.in"
-	source "package/perl-try-tiny/Config.in"
-	source "package/perl-uri/Config.in"
-	source "package/perl-www-robotrules/Config.in"
-	source "package/perl-xml-libxml/Config.in"
-	source "package/perl-xml-namespacesupport/Config.in"
-	source "package/perl-xml-sax/Config.in"
-	source "package/perl-xml-sax-base/Config.in"
-endmenu
-endif
 	source "package/php/Config.in"
 if BR2_PACKAGE_PHP
 if !BR2_STATIC_LIBS
diff --git a/package/perl/Config.in b/package/perl/Config.in
index cdd5aab..635a6f8 100644
--- a/package/perl/Config.in
+++ b/package/perl/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_PERL
+menuconfig BR2_PACKAGE_PERL
 	bool "perl"
 	depends on BR2_USE_MMU # fork()
 	help
@@ -24,4 +24,50 @@ config BR2_PACKAGE_PERL_MODULES
 	  Leave empty for all modules (as far as the external libraries
 	  are available).
 
+comment "Perl libraries/modules"
+
+source "package/perl-crypt-openssl-random/Config.in"
+source "package/perl-crypt-openssl-rsa/Config.in"
+source "package/perl-datetime-tiny/Config.in"
+source "package/perl-db-file/Config.in"
+source "package/perl-digest-hmac/Config.in"
+source "package/perl-digest-sha1/Config.in"
+source "package/perl-encode-detect/Config.in"
+source "package/perl-encode-locale/Config.in"
+source "package/perl-file-listing/Config.in"
+source "package/perl-file-util/Config.in"
+source "package/perl-gd/Config.in"
+source "package/perl-gdgraph/Config.in"
+source "package/perl-gdtextutil/Config.in"
+source "package/perl-html-parser/Config.in"
+source "package/perl-html-tagset/Config.in"
+source "package/perl-http-cookies/Config.in"
+source "package/perl-http-daemon/Config.in"
+source "package/perl-http-date/Config.in"
+source "package/perl-http-message/Config.in"
+source "package/perl-http-negotiate/Config.in"
+source "package/perl-io-html/Config.in"
+source "package/perl-io-socket-ssl/Config.in"
+source "package/perl-json-tiny/Config.in"
+source "package/perl-libwww-perl/Config.in"
+source "package/perl-lwp-mediatypes/Config.in"
+source "package/perl-mail-dkim/Config.in"
+source "package/perl-mailtools/Config.in"
+source "package/perl-mime-base64/Config.in"
+source "package/perl-mojolicious/Config.in"
+source "package/perl-net-dns/Config.in"
+source "package/perl-net-http/Config.in"
+source "package/perl-net-ssleay/Config.in"
+source "package/perl-netaddr-ip/Config.in"
+source "package/perl-path-tiny/Config.in"
+source "package/perl-time-hires/Config.in"
+source "package/perl-timedate/Config.in"
+source "package/perl-try-tiny/Config.in"
+source "package/perl-uri/Config.in"
+source "package/perl-www-robotrules/Config.in"
+source "package/perl-xml-libxml/Config.in"
+source "package/perl-xml-namespacesupport/Config.in"
+source "package/perl-xml-sax/Config.in"
+source "package/perl-xml-sax-base/Config.in"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (13 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 14/83 v2] package/perl: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:37   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 16/83 v2] package/php: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
                   ` (67 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/php/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/php/Config.in b/package/php/Config.in
index 32faa32..eee8d26 100644
--- a/package/php/Config.in
+++ b/package/php/Config.in
@@ -9,8 +9,6 @@ config BR2_PACKAGE_PHP
 
 if BR2_PACKAGE_PHP
 
-source "package/php/Config.ext"
-
 config BR2_PACKAGE_PHP_CLI
 	bool
 
@@ -66,4 +64,6 @@ config BR2_PACKAGE_PHP_SAPI_CLI_FPM
 
 endchoice
 
+source "package/php/Config.ext"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 16/83 v2] package/php: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (14 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 17/83 v2] package/python: " Yann E. MORIN
                   ` (66 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Move the external extensions selection into the bundled extensions'
Config.in.

Make the menu prompt a comment, reword it so that it is more in line
with the following comments.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in      | 16 ----------------
 package/php/Config.ext | 20 ++++++++++++++++++--
 package/php/Config.in  |  2 +-
 3 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index a72dd91..8f1c870 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -572,22 +572,6 @@ endif
 	source "package/nodejs/Config.in"
 	source "package/perl/Config.in"
 	source "package/php/Config.in"
-if BR2_PACKAGE_PHP
-if !BR2_STATIC_LIBS
-menu "External php extensions"
-	source "package/php-geoip/Config.in"
-	source "package/php-gnupg/Config.in"
-	source "package/php-imagick/Config.in"
-	source "package/php-memcached/Config.in"
-	source "package/php-ssh2/Config.in"
-	source "package/php-yaml/Config.in"
-	source "package/php-zmq/Config.in"
-endmenu
-endif
-if BR2_STATIC_LIBS
-comment "External PHP extensions need a toolchain w/ dynamic library"
-endif
-endif
 	source "package/python/Config.in"
 	source "package/python3/Config.in"
 if BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3
diff --git a/package/php/Config.ext b/package/php/Config.ext
index 82aaf41..0395cb5 100644
--- a/package/php/Config.ext
+++ b/package/php/Config.ext
@@ -1,4 +1,4 @@
-menu "Extensions"
+comment "Generic extensions"
 
 config BR2_PACKAGE_PHP_EXT_CALENDAR
 	bool "Calendar"
@@ -389,4 +389,20 @@ config BR2_PACKAGE_PHP_EXT_XSL
 	help
 	  XSL transformation support
 
-endmenu
+if !BR2_STATIC_LIBS
+
+comment "External extensions"
+
+source "package/php-geoip/Config.in"
+source "package/php-gnupg/Config.in"
+source "package/php-imagick/Config.in"
+source "package/php-memcached/Config.in"
+source "package/php-ssh2/Config.in"
+source "package/php-yaml/Config.in"
+source "package/php-zmq/Config.in"
+
+endif # !STATIC_LIBS
+
+if BR2_STATIC_LIBs
+comment "External PHP extensions need a toolchain w/ dynamic library"
+endif
diff --git a/package/php/Config.in b/package/php/Config.in
index eee8d26..7e79179 100644
--- a/package/php/Config.in
+++ b/package/php/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_PHP
+menuconfig BR2_PACKAGE_PHP
 	bool "php"
 	help
 	  PHP  is a widely-used general-purpose scripting
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 17/83 v2] package/python: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (15 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 16/83 v2] package/php: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 18/83 v2] package/python3: " Yann E. MORIN
                   ` (65 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/python/Config.in | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/package/python/Config.in b/package/python/Config.in
index fa83e77..40a2ef4 100644
--- a/package/python/Config.in
+++ b/package/python/Config.in
@@ -2,7 +2,7 @@ comment "python needs a toolchain w/ wchar, threads, dynamic library"
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
-config BR2_PACKAGE_PYTHON
+menuconfig BR2_PACKAGE_PYTHON
 	bool "python"
 	depends on BR2_USE_WCHAR
 	# uses fork()
@@ -34,7 +34,7 @@ config BR2_PACKAGE_PYTHON_PY_PYC
 
 endchoice
 
-menu "core python modules"
+comment "core python modules"
 
 comment "The following modules are unusual or require extra libraries"
 
@@ -127,6 +127,4 @@ config BR2_PACKAGE_PYTHON_HASHLIB
 	help
 	  hashlib support in Python
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 18/83 v2] package/python3: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (16 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 17/83 v2] package/python: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 19/83 v2] package/python{, 3}: move inclusion of external python modules Yann E. MORIN
                   ` (64 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/python3/Config.in | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/package/python3/Config.in b/package/python3/Config.in
index 0f4fc88..aadc8b6 100644
--- a/package/python3/Config.in
+++ b/package/python3/Config.in
@@ -2,7 +2,7 @@ comment "python3 needs a toolchain w/ wchar, threads, dynamic library"
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
-config BR2_PACKAGE_PYTHON3
+menuconfig BR2_PACKAGE_PYTHON3
 	bool "python3"
 	depends on !BR2_PACKAGE_PYTHON
 	depends on BR2_USE_WCHAR
@@ -35,7 +35,7 @@ config BR2_PACKAGE_PYTHON3_PY_PYC
 
 endchoice
 
-menu "core python3 modules"
+comment "core python3 modules"
 
 comment "The following modules are unusual or require extra libraries"
 
@@ -104,6 +104,4 @@ config BR2_PACKAGE_PYTHON3_ZLIB
 	help
 	  zlib support in Python3
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 19/83 v2] package/python{, 3}: move inclusion of external python modules
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (17 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 18/83 v2] package/python3: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
                   ` (63 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Introduce a Config.ext for external python modules, that both python and
python include from there respective sub-menus.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in          | 155 ---------------------------------------------
 package/python/Config.ext  | 155 +++++++++++++++++++++++++++++++++++++++++++++
 package/python/Config.in   |   2 +
 package/python3/Config.ext |   1 +
 package/python3/Config.in  |   2 +
 5 files changed, 160 insertions(+), 155 deletions(-)
 create mode 100644 package/python/Config.ext
 create mode 120000 package/python3/Config.ext

diff --git a/package/Config.in b/package/Config.in
index 8f1c870..dcf1525 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -574,161 +574,6 @@ endif
 	source "package/php/Config.in"
 	source "package/python/Config.in"
 	source "package/python3/Config.in"
-if BR2_PACKAGE_PYTHON || BR2_PACKAGE_PYTHON3
-menu "External python modules"
-	source "package/python-alsaaudio/Config.in"
-	source "package/python-argh/Config.in"
-	source "package/python-autobahn/Config.in"
-	source "package/python-backports-abc/Config.in"
-	source "package/python-beautifulsoup4/Config.in"
-	source "package/python-bottle/Config.in"
-	source "package/python-can/Config.in"
-	source "package/python-cbor/Config.in"
-	source "package/python-certifi/Config.in"
-	source "package/python-cffi/Config.in"
-	source "package/python-characteristic/Config.in"
-	source "package/python-cheetah/Config.in"
-	source "package/python-cherrypy/Config.in"
-	source "package/python-click/Config.in"
-	source "package/python-coherence/Config.in"
-	source "package/python-configobj/Config.in"
-	source "package/python-configshell-fb/Config.in"
-	source "package/python-crc16/Config.in"
-	source "package/python-crossbar/Config.in"
-	source "package/python-cryptography/Config.in"
-	source "package/python-cssselect/Config.in"
-	source "package/python-daemon/Config.in"
-	source "package/python-dataproperty/Config.in"
-	source "package/python-dateutil/Config.in"
-	source "package/python-dialog/Config.in"
-	source "package/python-django/Config.in"
-	source "package/python-docopt/Config.in"
-	source "package/python-dominate/Config.in"
-	source "package/python-dpkt/Config.in"
-	source "package/python-ecdsa/Config.in"
-	source "package/python-engineio/Config.in"
-	source "package/python-enum/Config.in"
-	source "package/python-enum34/Config.in"
-	source "package/python-flask/Config.in"
-	source "package/python-flask-jsonrpc/Config.in"
-	source "package/python-flask-login/Config.in"
-	source "package/python-flup/Config.in"
-	source "package/python-gobject/Config.in"
-	source "package/python-html5lib/Config.in"
-	source "package/python-httplib2/Config.in"
-	source "package/python-id3/Config.in"
-	source "package/python-idna/Config.in"
-	source "package/python-iniparse/Config.in"
-	source "package/python-iowait/Config.in"
-	source "package/python-ipaddr/Config.in"
-	source "package/python-ipaddress/Config.in"
-	source "package/python-ipy/Config.in"
-	source "package/python-ipython/Config.in"
-	source "package/python-itsdangerous/Config.in"
-	source "package/python-jinja2/Config.in"
-	source "package/python-json-schema-validator/Config.in"
-	source "package/python-keyring/Config.in"
-	source "package/python-libconfig/Config.in"
-	source "package/python-lmdb/Config.in"
-	source "package/python-lxml/Config.in"
-	source "package/python-mad/Config.in"
-	source "package/python-mako/Config.in"
-	source "package/python-markdown/Config.in"
-	source "package/python-markupsafe/Config.in"
-	source "package/python-meld3/Config.in"
-	source "package/python-mistune/Config.in"
-	source "package/python-msgpack/Config.in"
-	source "package/python-netaddr/Config.in"
-	source "package/python-netifaces/Config.in"
-	source "package/python-networkmanager/Config.in"
-	source "package/python-nfc/Config.in"
-	source "package/python-numpy/Config.in"
-	source "package/python-paho-mqtt/Config.in"
-	source "package/python-pam/Config.in"
-	source "package/python-paramiko/Config.in"
-	source "package/python-pathtools/Config.in"
-	source "package/python-pathvalidate/Config.in"
-	source "package/python-pexpect/Config.in"
-	source "package/python-posix-ipc/Config.in"
-	source "package/python-protobuf/Config.in"
-	source "package/python-psutil/Config.in"
-	source "package/python-ptyprocess/Config.in"
-	source "package/python-pyasn/Config.in"
-	source "package/python-pyasn-modules/Config.in"
-	source "package/python-pycli/Config.in"
-	source "package/python-pycparser/Config.in"
-	source "package/python-pycrypto/Config.in"
-	source "package/python-pydal/Config.in"
-	source "package/python-pyftpdlib/Config.in"
-	source "package/python-pygame/Config.in"
-	source "package/python-pygments/Config.in"
-	source "package/python-pyinotify/Config.in"
-	source "package/python-pymysql/Config.in"
-	source "package/python-pynacl/Config.in"
-	source "package/python-pyopenssl/Config.in"
-	source "package/python-pyparsing/Config.in"
-	source "package/python-pyparted/Config.in"
-	source "package/python-pypcap/Config.in"
-	source "package/python-pyqt/Config.in"
-	source "package/python-pyratemp/Config.in"
-	source "package/python-pyro/Config.in"
-	source "package/python-pyroute2/Config.in"
-	source "package/python-pysendfile/Config.in"
-	source "package/python-pysmb/Config.in"
-	source "package/python-pysnmp/Config.in"
-	source "package/python-pysnmp-apps/Config.in"
-	source "package/python-pysnmp-mibs/Config.in"
-	source "package/python-pysocks/Config.in"
-	source "package/python-pytablewriter/Config.in"
-	source "package/python-pytrie/Config.in"
-	source "package/python-pytz/Config.in"
-	source "package/python-pyudev/Config.in"
-	source "package/python-pyusb/Config.in"
-	source "package/python-pyxb/Config.in"
-	source "package/python-pyyaml/Config.in"
-	source "package/python-pyzmq/Config.in"
-	source "package/python-requests/Config.in"
-	source "package/python-rpi-gpio/Config.in"
-	source "package/python-rtslib-fb/Config.in"
-	source "package/python-scapy3k/Config.in"
-	source "package/python-sdnotify/Config.in"
-	source "package/python-serial/Config.in"
-	source "package/python-service-identity/Config.in"
-	source "package/python-setproctitle/Config.in"
-	source "package/python-setuptools/Config.in"
-	source "package/python-shutilwhich/Config.in"
-	source "package/python-simplejson/Config.in"
-	source "package/python-singledispatch/Config.in"
-	source "package/python-sip/Config.in"
-	source "package/python-six/Config.in"
-	source "package/python-smbus-cffi/Config.in"
-	source "package/python-socketio/Config.in"
-	source "package/python-spidev/Config.in"
-	source "package/python-thrift/Config.in"
-	source "package/python-tomako/Config.in"
-	source "package/python-tornado/Config.in"
-	source "package/python-treq/Config.in"
-	source "package/python-twisted/Config.in"
-	source "package/python-txaio/Config.in"
-	source "package/python-u-msgpack/Config.in"
-	source "package/python-ubjson/Config.in"
-	source "package/python-ujson/Config.in"
-	source "package/python-urllib3/Config.in"
-	source "package/python-urwid/Config.in"
-	source "package/python-versiontools/Config.in"
-	source "package/python-watchdog/Config.in"
-	source "package/python-web2py/Config.in"
-	source "package/python-webpy/Config.in"
-	source "package/python-werkzeug/Config.in"
-	source "package/python-ws4py/Config.in"
-	source "package/python-wsaccel/Config.in"
-	source "package/python-xlrd/Config.in"
-	source "package/python-xlsxwriter/Config.in"
-	source "package/python-xlutils/Config.in"
-	source "package/python-xlwt/Config.in"
-	source "package/python-zope-interface/Config.in"
-endmenu
-endif
 	source "package/ruby/Config.in"
 	source "package/tcl/Config.in"
 if BR2_PACKAGE_TCL
diff --git a/package/python/Config.ext b/package/python/Config.ext
new file mode 100644
index 0000000..6dfcc1e
--- /dev/null
+++ b/package/python/Config.ext
@@ -0,0 +1,155 @@
+# This file is included both from python and python3
+
+comment "External python modules"
+
+source "package/python-alsaaudio/Config.in"
+source "package/python-argh/Config.in"
+source "package/python-autobahn/Config.in"
+source "package/python-backports-abc/Config.in"
+source "package/python-beautifulsoup4/Config.in"
+source "package/python-bottle/Config.in"
+source "package/python-can/Config.in"
+source "package/python-cbor/Config.in"
+source "package/python-certifi/Config.in"
+source "package/python-cffi/Config.in"
+source "package/python-characteristic/Config.in"
+source "package/python-cheetah/Config.in"
+source "package/python-cherrypy/Config.in"
+source "package/python-click/Config.in"
+source "package/python-coherence/Config.in"
+source "package/python-configobj/Config.in"
+source "package/python-configshell-fb/Config.in"
+source "package/python-crc16/Config.in"
+source "package/python-crossbar/Config.in"
+source "package/python-cryptography/Config.in"
+source "package/python-cssselect/Config.in"
+source "package/python-daemon/Config.in"
+source "package/python-dataproperty/Config.in"
+source "package/python-dateutil/Config.in"
+source "package/python-dialog/Config.in"
+source "package/python-django/Config.in"
+source "package/python-docopt/Config.in"
+source "package/python-dominate/Config.in"
+source "package/python-dpkt/Config.in"
+source "package/python-ecdsa/Config.in"
+source "package/python-engineio/Config.in"
+source "package/python-enum/Config.in"
+source "package/python-enum34/Config.in"
+source "package/python-flask/Config.in"
+source "package/python-flask-jsonrpc/Config.in"
+source "package/python-flask-login/Config.in"
+source "package/python-flup/Config.in"
+source "package/python-gobject/Config.in"
+source "package/python-html5lib/Config.in"
+source "package/python-httplib2/Config.in"
+source "package/python-id3/Config.in"
+source "package/python-idna/Config.in"
+source "package/python-iniparse/Config.in"
+source "package/python-iowait/Config.in"
+source "package/python-ipaddr/Config.in"
+source "package/python-ipaddress/Config.in"
+source "package/python-ipy/Config.in"
+source "package/python-ipython/Config.in"
+source "package/python-itsdangerous/Config.in"
+source "package/python-jinja2/Config.in"
+source "package/python-json-schema-validator/Config.in"
+source "package/python-keyring/Config.in"
+source "package/python-libconfig/Config.in"
+source "package/python-lmdb/Config.in"
+source "package/python-lxml/Config.in"
+source "package/python-mad/Config.in"
+source "package/python-mako/Config.in"
+source "package/python-markdown/Config.in"
+source "package/python-markupsafe/Config.in"
+source "package/python-meld3/Config.in"
+source "package/python-mistune/Config.in"
+source "package/python-msgpack/Config.in"
+source "package/python-netaddr/Config.in"
+source "package/python-netifaces/Config.in"
+source "package/python-networkmanager/Config.in"
+source "package/python-nfc/Config.in"
+source "package/python-numpy/Config.in"
+source "package/python-paho-mqtt/Config.in"
+source "package/python-pam/Config.in"
+source "package/python-paramiko/Config.in"
+source "package/python-pathtools/Config.in"
+source "package/python-pathvalidate/Config.in"
+source "package/python-pexpect/Config.in"
+source "package/python-posix-ipc/Config.in"
+source "package/python-protobuf/Config.in"
+source "package/python-psutil/Config.in"
+source "package/python-ptyprocess/Config.in"
+source "package/python-pyasn/Config.in"
+source "package/python-pyasn-modules/Config.in"
+source "package/python-pycli/Config.in"
+source "package/python-pycparser/Config.in"
+source "package/python-pycrypto/Config.in"
+source "package/python-pydal/Config.in"
+source "package/python-pyftpdlib/Config.in"
+source "package/python-pygame/Config.in"
+source "package/python-pygments/Config.in"
+source "package/python-pyinotify/Config.in"
+source "package/python-pymysql/Config.in"
+source "package/python-pynacl/Config.in"
+source "package/python-pyopenssl/Config.in"
+source "package/python-pyparsing/Config.in"
+source "package/python-pyparted/Config.in"
+source "package/python-pypcap/Config.in"
+source "package/python-pyqt/Config.in"
+source "package/python-pyratemp/Config.in"
+source "package/python-pyro/Config.in"
+source "package/python-pyroute2/Config.in"
+source "package/python-pysendfile/Config.in"
+source "package/python-pysmb/Config.in"
+source "package/python-pysnmp/Config.in"
+source "package/python-pysnmp-apps/Config.in"
+source "package/python-pysnmp-mibs/Config.in"
+source "package/python-pysocks/Config.in"
+source "package/python-pytablewriter/Config.in"
+source "package/python-pytrie/Config.in"
+source "package/python-pytz/Config.in"
+source "package/python-pyudev/Config.in"
+source "package/python-pyusb/Config.in"
+source "package/python-pyxb/Config.in"
+source "package/python-pyyaml/Config.in"
+source "package/python-pyzmq/Config.in"
+source "package/python-requests/Config.in"
+source "package/python-rpi-gpio/Config.in"
+source "package/python-rtslib-fb/Config.in"
+source "package/python-scapy3k/Config.in"
+source "package/python-sdnotify/Config.in"
+source "package/python-serial/Config.in"
+source "package/python-service-identity/Config.in"
+source "package/python-setproctitle/Config.in"
+source "package/python-setuptools/Config.in"
+source "package/python-shutilwhich/Config.in"
+source "package/python-simplejson/Config.in"
+source "package/python-singledispatch/Config.in"
+source "package/python-sip/Config.in"
+source "package/python-six/Config.in"
+source "package/python-smbus-cffi/Config.in"
+source "package/python-socketio/Config.in"
+source "package/python-spidev/Config.in"
+source "package/python-thrift/Config.in"
+source "package/python-tomako/Config.in"
+source "package/python-tornado/Config.in"
+source "package/python-treq/Config.in"
+source "package/python-twisted/Config.in"
+source "package/python-txaio/Config.in"
+source "package/python-u-msgpack/Config.in"
+source "package/python-ubjson/Config.in"
+source "package/python-ujson/Config.in"
+source "package/python-urllib3/Config.in"
+source "package/python-urwid/Config.in"
+source "package/python-versiontools/Config.in"
+source "package/python-watchdog/Config.in"
+source "package/python-web2py/Config.in"
+source "package/python-webpy/Config.in"
+source "package/python-werkzeug/Config.in"
+source "package/python-ws4py/Config.in"
+source "package/python-wsaccel/Config.in"
+source "package/python-xlrd/Config.in"
+source "package/python-xlsxwriter/Config.in"
+source "package/python-xlutils/Config.in"
+source "package/python-xlwt/Config.in"
+source "package/python-zope-interface/Config.in"
diff --git a/package/python/Config.in b/package/python/Config.in
index 40a2ef4..6563cd6 100644
--- a/package/python/Config.in
+++ b/package/python/Config.in
@@ -127,4 +127,6 @@ config BR2_PACKAGE_PYTHON_HASHLIB
 	help
 	  hashlib support in Python
 
+source "package/python/Config.ext"
+
 endif
diff --git a/package/python3/Config.ext b/package/python3/Config.ext
new file mode 120000
index 0000000..1b3df59
--- /dev/null
+++ b/package/python3/Config.ext
@@ -0,0 +1 @@
+../python/Config.ext
\ No newline at end of file
diff --git a/package/python3/Config.in b/package/python3/Config.in
index aadc8b6..f221a54 100644
--- a/package/python3/Config.in
+++ b/package/python3/Config.in
@@ -104,4 +104,6 @@ config BR2_PACKAGE_PYTHON3_ZLIB
 	help
 	  zlib support in Python3
 
+source "package/python3/Config.ext"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (18 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 19/83 v2] package/python{, 3}: move inclusion of external python modules Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:37   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 21/83 v2] package/tcl: " Yann E. MORIN
                   ` (62 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/sdl_image/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/sdl_image/Config.in b/package/sdl_image/Config.in
index 1b98066..0c4a018 100644
--- a/package/sdl_image/Config.in
+++ b/package/sdl_image/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_SDL_IMAGE
+menuconfig BR2_PACKAGE_SDL_IMAGE
 	bool "SDL_image"
 	depends on BR2_PACKAGE_SDL
 	help
@@ -10,8 +10,6 @@ config BR2_PACKAGE_SDL_IMAGE
 
 if BR2_PACKAGE_SDL_IMAGE
 
-menu "SDL_image file format support"
-
 config BR2_PACKAGE_SDL_IMAGE_BMP
 	bool "enable BMP file format support"
 	default y
@@ -56,6 +54,4 @@ config BR2_PACKAGE_SDL_IMAGE_XPM
 config BR2_PACKAGE_SDL_IMAGE_XV
 	bool "enable XV file format support"
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 21/83 v2] package/tcl: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (19 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 22/83 v2] package/tiff: " Yann E. MORIN
                   ` (61 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Move inclusion of Tcl libraries/modules to Tcl's Config.in so that they
appear in Tcl's sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in     | 6 ------
 package/tcl/Config.in | 7 ++++++-
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index dcf1525..ebc8f6f 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -576,12 +576,6 @@ endif
 	source "package/python3/Config.in"
 	source "package/ruby/Config.in"
 	source "package/tcl/Config.in"
-if BR2_PACKAGE_TCL
-menu "Tcl libraries/modules"
-	source "package/expect/Config.in"
-	source "package/tcllib/Config.in"
-endmenu
-endif
 endmenu
 
 menu "Libraries"
diff --git a/package/tcl/Config.in b/package/tcl/Config.in
index f1fa054..dacd1a8 100644
--- a/package/tcl/Config.in
+++ b/package/tcl/Config.in
@@ -2,7 +2,7 @@ comment "tcl needs a toolchain w/ threads, dynamic library"
 	depends on BR2_USE_MMU
 	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
-config BR2_PACKAGE_TCL
+menuconfig BR2_PACKAGE_TCL
 	bool "tcl"
 	# fork()
 	depends on BR2_USE_MMU
@@ -36,4 +36,9 @@ config BR2_PACKAGE_TCL_SHLIB_ONLY
 
 	  Saves ~14kb.
 
+comment "Tcl libraries/modules"
+
+source "package/expect/Config.in"
+source "package/tcllib/Config.in"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 22/83 v2] package/tiff: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (20 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 21/83 v2] package/tcl: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:38   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 23/83 v2] package/udpcast: " Yann E. MORIN
                   ` (60 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/tiff/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/tiff/Config.in b/package/tiff/Config.in
index 7d66dd2..6e5c176 100644
--- a/package/tiff/Config.in
+++ b/package/tiff/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_TIFF
+menuconfig BR2_PACKAGE_TIFF
 	bool "tiff"
 	help
 	  Library for handling TIFF (Tag Image File Format) images.
@@ -6,7 +6,6 @@ config BR2_PACKAGE_TIFF
 	  http://www.remotesensing.org/libtiff/
 
 if BR2_PACKAGE_TIFF
-menu "tiff Options"
 
 config BR2_PACKAGE_TIFF_CCITT
 	bool "CCITT Group 3 & 4 support"
@@ -62,7 +61,6 @@ config BR2_PACKAGE_TIFF_OLD_JPEG
 config BR2_PACKAGE_TIFF_JBIG
 	bool "JBIG compression"
 	default y
-endmenu
 
 config BR2_PACKAGE_TIFF_UTILITIES
 	bool "tiff utilities"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 23/83 v2] package/udpcast: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (21 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 22/83 v2] package/tiff: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04 21:38   ` Thomas Petazzoni
  2016-07-04  9:24 ` [Buildroot] [PATCH 24/83 v2] package/gstreamer: include plugins from gstreamer's real Config.in Yann E. MORIN
                   ` (59 subsequent siblings)
  82 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

This is a bit overkill, as there is only two sub-items. However, this
commit is not about judging whether it makes sense or not to have a
sub-menu, but to cleanup the 'config'+'menu' situation.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/udpcast/Config.in | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/package/udpcast/Config.in b/package/udpcast/Config.in
index 0424db7..a61829a 100644
--- a/package/udpcast/Config.in
+++ b/package/udpcast/Config.in
@@ -2,7 +2,7 @@ comment "udpcast needs a toolchain w/ threads"
 	depends on BR2_USE_MMU
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
 
-config BR2_PACKAGE_UDPCAST
+menuconfig BR2_PACKAGE_UDPCAST
 	bool "udpcast"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
@@ -14,8 +14,6 @@ config BR2_PACKAGE_UDPCAST
 
 if BR2_PACKAGE_UDPCAST
 
-menu "udpcast tools selection"
-
 config BR2_PACKAGE_UDPCAST_SENDER
 	bool "sender"
 	help
@@ -26,6 +24,4 @@ config BR2_PACKAGE_UDPCAST_RECEIVER
 	help
 	  The udpcast receiver.
 
-endmenu
-
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 24/83 v2] package/gstreamer: include plugins from gstreamer's real Config.in
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (22 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 23/83 v2] package/udpcast: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 25/83 v2] package/gstreamer1: include plugins from gstreamer1's " Yann E. MORIN
                   ` (58 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

It is more logical to have a single if-clause, rather than repeat it in
another file, possiblty brealking indentation like it previously was.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/gstreamer/Config.in           | 12 ------------
 package/gstreamer/gstreamer/Config.in | 10 ++++++++++
 2 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/package/gstreamer/Config.in b/package/gstreamer/Config.in
index c6292f4..476f838 100644
--- a/package/gstreamer/Config.in
+++ b/package/gstreamer/Config.in
@@ -1,14 +1,2 @@
 # Gstreamer 0.10.x & Plugins
 source "package/gstreamer/gstreamer/Config.in"
-
-if BR2_PACKAGE_GSTREAMER
-source "package/gstreamer/gst-plugins-base/Config.in"
-source "package/gstreamer/gst-plugins-good/Config.in"
-source "package/gstreamer/gst-plugins-bad/Config.in"
-source "package/gstreamer/gst-plugins-ugly/Config.in"
-source "package/gstreamer/gst-ffmpeg/Config.in"
-source "package/gstreamer/gst-dsp/Config.in"
-source "package/gstreamer/gst-fsl-plugins/Config.in"
-source "package/gstreamer/gst-omapfb/Config.in"
-source "package/gstreamer/gst-plugin-x170/Config.in"
-endif
diff --git a/package/gstreamer/gstreamer/Config.in b/package/gstreamer/gstreamer/Config.in
index eb1249a..aee48b1 100644
--- a/package/gstreamer/gstreamer/Config.in
+++ b/package/gstreamer/gstreamer/Config.in
@@ -33,4 +33,14 @@ config BR2_PACKAGE_GSTREAMER_PLUGIN_REGISTRY
 	  Enable support for the GStreamer plugin registry. This may increase
 	  the launch-time for a GStreamer application.
 
+source "package/gstreamer/gst-plugins-base/Config.in"
+source "package/gstreamer/gst-plugins-good/Config.in"
+source "package/gstreamer/gst-plugins-bad/Config.in"
+source "package/gstreamer/gst-plugins-ugly/Config.in"
+source "package/gstreamer/gst-ffmpeg/Config.in"
+source "package/gstreamer/gst-dsp/Config.in"
+source "package/gstreamer/gst-fsl-plugins/Config.in"
+source "package/gstreamer/gst-omapfb/Config.in"
+source "package/gstreamer/gst-plugin-x170/Config.in"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 25/83 v2] package/gstreamer1: include plugins from gstreamer1's real Config.in
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (23 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 24/83 v2] package/gstreamer: include plugins from gstreamer's real Config.in Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 26/83 v2] docs/manual: add example about sub-options Yann E. MORIN
                   ` (57 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

It is more logical to have a single if-clause, rather than repeat it in
another file, possiblty brealking indentation like it previously was.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/gstreamer1/Config.in            | 11 -----------
 package/gstreamer1/gstreamer1/Config.in |  9 +++++++++
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/package/gstreamer1/Config.in b/package/gstreamer1/Config.in
index ea35ecc..ca3240f 100644
--- a/package/gstreamer1/Config.in
+++ b/package/gstreamer1/Config.in
@@ -1,13 +1,2 @@
 # Gstreamer 1.x & Plugins
 source "package/gstreamer1/gstreamer1/Config.in"
-
-if BR2_PACKAGE_GSTREAMER1
-source "package/gstreamer1/gst1-plugins-base/Config.in"
-source "package/gstreamer1/gst1-plugins-good/Config.in"
-source "package/gstreamer1/gst1-plugins-bad/Config.in"
-source "package/gstreamer1/gst1-plugins-ugly/Config.in"
-source "package/gstreamer1/gst1-imx/Config.in"
-source "package/gstreamer1/gst1-libav/Config.in"
-source "package/gstreamer1/gst1-validate/Config.in"
-source "package/gstreamer1/gst-omx/Config.in"
-endif
diff --git a/package/gstreamer1/gstreamer1/Config.in b/package/gstreamer1/gstreamer1/Config.in
index 274171b..99aadb4 100644
--- a/package/gstreamer1/gstreamer1/Config.in
+++ b/package/gstreamer1/gstreamer1/Config.in
@@ -52,4 +52,13 @@ config BR2_PACKAGE_GSTREAMER1_INSTALL_TOOLS
 	  Install the gst-launch and gst-inspect tools. This will take up
 	  additional space on the target.
 
+source "package/gstreamer1/gst1-plugins-base/Config.in"
+source "package/gstreamer1/gst1-plugins-good/Config.in"
+source "package/gstreamer1/gst1-plugins-bad/Config.in"
+source "package/gstreamer1/gst1-plugins-ugly/Config.in"
+source "package/gstreamer1/gst1-imx/Config.in"
+source "package/gstreamer1/gst1-libav/Config.in"
+source "package/gstreamer1/gst1-validate/Config.in"
+source "package/gstreamer1/gst-omx/Config.in"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 26/83 v2] docs/manual: add example about sub-options
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (24 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 25/83 v2] package/gstreamer1: include plugins from gstreamer1's " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 27/83 v2] docs/manual: use 'menuconfig' when there are more than 5 sub-options Yann E. MORIN
                   ` (56 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - wrap example at 72-chars  (Arnout)
---
 docs/manual/adding-packages-directory.txt | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/docs/manual/adding-packages-directory.txt b/docs/manual/adding-packages-directory.txt
index dc66f00..0ff9eda 100644
--- a/docs/manual/adding-packages-directory.txt
+++ b/docs/manual/adding-packages-directory.txt
@@ -54,6 +54,31 @@ examples in other packages. The syntax of the +Config.in+ file is the same
 as the one for the kernel Kconfig file. The documentation for this syntax is
 available at http://kernel.org/doc/Documentation/kbuild/kconfig-language.txt[]
 
+For example:
+
+---------------------------
+config BR2_PACKAGE_LIBFOO
+	bool "libfoo"
+	help
+	  This is a comment that explains what libfoo is.
+
+	  http://foosoftware.org/libfoo/
+
+if BR2_PACKAGE_LIBFOO
+
+config BR2_PACKAGE_LIBFOO_BAR
+	bool "bar support"
+	help
+	  This is a comment that briefly explains what bar support is,
+	  if it's not obvious.
+
+endif # BR2_PACKAGE_LIBFOO
+---------------------------
+
+The sub-options will be cleanly indented below the main symbol when it
+is enabled, and will be properly hidden when the symbol is disabled.
+
+
 Finally you have to add your new +libfoo/Config.in+ to
 +package/Config.in+ (or in a category subdirectory if you decided to
 put your package in one of the existing categories). The files
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 27/83 v2] docs/manual: use 'menuconfig' when there are more than 5 sub-options
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (25 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 26/83 v2] docs/manual: add example about sub-options Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 28/83 v2] package/boost: use 'menuconfig' Yann E. MORIN
                   ` (55 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

When a package has a lot of sub-options, the layout in menuconfig (the
mconf UI) is quickly getting messy.

Using 'menuconfig' (the kconfig keyword) will automatically create a
sub-menu, which provides a cleaner layout.

Document that, with 5 or more options, a 'menuconfig' should be used
instead of a simple 'config'.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Note: 5 is arbitrary. We could tweak it to a bit more, but on small
terminals (the smallest supported is 80x24), there are only 10 lines
for the menu entries, so 5 looks like a good compromise. YMMV.

---
Changes RFC -> v1:
  - wrap example at 72-chars  (Arnout)
---
 docs/manual/adding-packages-directory.txt | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/docs/manual/adding-packages-directory.txt b/docs/manual/adding-packages-directory.txt
index 0ff9eda..25f9cfc 100644
--- a/docs/manual/adding-packages-directory.txt
+++ b/docs/manual/adding-packages-directory.txt
@@ -79,6 +79,34 @@ The sub-options will be cleanly indented below the main symbol when it
 is enabled, and will be properly hidden when the symbol is disabled.
 
 
+If your package has 5 or more options, then you should use +menuconfig+
+to define the main symbol; this will ensure that all the package's
+sub-options are automaticaly classified in a sub-menu:
+
+---------------------------
+menuconfig BR2_PACKAGE_LIBFOO
+	bool "libfoo"
+	help
+	  This is a comment that explains what libfoo is.
+
+	  http://foosoftware.org/libfoo/
+
+if BR2_PACKAGE_LIBFOO
+
+config BR2_PACKAGE_LIBFOO_BAR
+	bool "bar support"
+	help
+	  This is a comment that briefly explains what bar support is,
+	  if it's not obvious.
+
+endif # BR2_PACKAGE_LIBFOO
+---------------------------
+
+In any case, ensure that there is no +comment+ or other symbol in-between
+the main symbol (be it +config+ or +menuconfig+) and the +if+ clause,
+otherwise 'Kconfig' will not properly classify the sub-options.
+
+
 Finally you have to add your new +libfoo/Config.in+ to
 +package/Config.in+ (or in a category subdirectory if you decided to
 put your package in one of the existing categories). The files
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 28/83 v2] package/boost: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (26 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 27/83 v2] docs/manual: use 'menuconfig' when there are more than 5 sub-options Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 29/83 v2] package/cairo: " Yann E. MORIN
                   ` (54 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

boost has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/boost/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/boost/Config.in b/package/boost/Config.in
index 51d4ed7..9501373 100644
--- a/package/boost/Config.in
+++ b/package/boost/Config.in
@@ -1,7 +1,7 @@
 comment "boost needs a toolchain w/ C++, threads, wchar"
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS || !BR2_USE_WCHAR
 
-config BR2_PACKAGE_BOOST
+menuconfig BR2_PACKAGE_BOOST
 	bool "boost"
 	depends on BR2_INSTALL_LIBSTDCPP
 	# Boost could theorically be built with threading=single, but
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 29/83 v2] package/cairo: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (27 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 28/83 v2] package/boost: use 'menuconfig' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 30/83 v2] package/connman: " Yann E. MORIN
                   ` (53 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

cairo has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/cairo/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/cairo/Config.in b/package/cairo/Config.in
index 0c87a29..abedee2 100644
--- a/package/cairo/Config.in
+++ b/package/cairo/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_CAIRO
+menuconfig BR2_PACKAGE_CAIRO
 	bool "cairo"
 	select BR2_PACKAGE_PIXMAN
 	select BR2_PACKAGE_FONTCONFIG
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 30/83 v2] package/connman: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (28 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 29/83 v2] package/cairo: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 31/83 v2] package/dejavu: " Yann E. MORIN
                   ` (52 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

connman has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/connman/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/connman/Config.in b/package/connman/Config.in
index f723a31..23adf85 100644
--- a/package/connman/Config.in
+++ b/package/connman/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_CONNMAN
+menuconfig BR2_PACKAGE_CONNMAN
 	bool "connman"
 	select BR2_PACKAGE_DBUS
 	select BR2_PACKAGE_LIBGLIB2
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 31/83 v2] package/dejavu: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (29 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 30/83 v2] package/connman: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 32/83 v2] package/directfb: " Yann E. MORIN
                   ` (51 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

dejavu has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/dejavu/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/dejavu/Config.in b/package/dejavu/Config.in
index 5d21e56..8e114f0 100644
--- a/package/dejavu/Config.in
+++ b/package/dejavu/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_DEJAVU
+menuconfig BR2_PACKAGE_DEJAVU
 	bool "DejaVu fonts"
 	help
 	  The DejaVu fonts are a font family based on the Vera Fonts.
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 32/83 v2] package/directfb: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (30 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 31/83 v2] package/dejavu: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 33/83 v2] package/dnsmasq: " Yann E. MORIN
                   ` (50 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

directfb has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/directfb/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/directfb/Config.in b/package/directfb/Config.in
index f0bcaaa..e842594 100644
--- a/package/directfb/Config.in
+++ b/package/directfb/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_DIRECTFB
+menuconfig BR2_PACKAGE_DIRECTFB
 	bool "directfb"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_INSTALL_LIBSTDCPP
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 33/83 v2] package/dnsmasq: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (31 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 32/83 v2] package/directfb: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 34/83 v2] package/dovecot: " Yann E. MORIN
                   ` (49 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

dnsmasq has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/dnsmasq/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/dnsmasq/Config.in b/package/dnsmasq/Config.in
index b4f34b8..d78c0be 100644
--- a/package/dnsmasq/Config.in
+++ b/package/dnsmasq/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_DNSMASQ
+menuconfig BR2_PACKAGE_DNSMASQ
 	bool "dnsmasq"
 	help
 	  A lightweight DNS and DHCP server. It is intended to provide
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 34/83 v2] package/dovecot: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (32 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 33/83 v2] package/dnsmasq: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 35/83 v2] package/dropbear: " Yann E. MORIN
                   ` (48 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

dovecot has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/dovecot/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/dovecot/Config.in b/package/dovecot/Config.in
index 194efb6..f27f5f2 100644
--- a/package/dovecot/Config.in
+++ b/package/dovecot/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_DOVECOT
+menuconfig BR2_PACKAGE_DOVECOT
 	bool "dovecot"
 	depends on !BR2_STATIC_LIBS
 	depends on BR2_USE_MMU # fork()
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 35/83 v2] package/dropbear: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (33 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 34/83 v2] package/dovecot: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 36/83 v2] package/e2fsprogs: " Yann E. MORIN
                   ` (47 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

dropbear has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/dropbear/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/dropbear/Config.in b/package/dropbear/Config.in
index d92361f..ce1a73b 100644
--- a/package/dropbear/Config.in
+++ b/package/dropbear/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_DROPBEAR
+menuconfig BR2_PACKAGE_DROPBEAR
 	bool "dropbear"
 	select BR2_PACKAGE_ZLIB if !BR2_PACKAGE_DROPBEAR_SMALL
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 36/83 v2] package/e2fsprogs: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (34 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 35/83 v2] package/dropbear: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 37/83 v2] package/efl: " Yann E. MORIN
                   ` (46 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

e2fsprogs has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/e2fsprogs/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/e2fsprogs/Config.in b/package/e2fsprogs/Config.in
index 4db11e1..ddbc90a 100644
--- a/package/e2fsprogs/Config.in
+++ b/package/e2fsprogs/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_E2FSPROGS
+menuconfig BR2_PACKAGE_E2FSPROGS
 	bool "e2fsprogs"
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_USE_MMU # util-linux/libblkid
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 37/83 v2] package/efl: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (35 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 36/83 v2] package/e2fsprogs: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 38/83 v2] package/erlang: include Erlang libraries from erlang's Config.in Yann E. MORIN
                   ` (45 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

EFL has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/efl/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/efl/Config.in b/package/efl/Config.in
index 4ab1ec4..f6b7fb9 100644
--- a/package/efl/Config.in
+++ b/package/efl/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_EFL
+menuconfig BR2_PACKAGE_EFL
 	bool "efl"
 	 # g++ issue with 4.4.5, tested with g++ 4.7.2
 	depends on BR2_HOST_GCC_AT_LEAST_4_7
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 38/83 v2] package/erlang: include Erlang libraries from erlang's Config.in
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (36 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 37/83 v2] package/efl: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 39/83 v2] package/flot: use 'menuconfig' Yann E. MORIN
                   ` (44 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

erlang now has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in        | 16 ----------------
 package/erlang/Config.in | 17 ++++++++++++++++-
 2 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index ebc8f6f..0e52f54 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -491,22 +491,6 @@ endmenu
 menu "Interpreter languages and scripting"
 	source "package/enscript/Config.in"
 	source "package/erlang/Config.in"
-if BR2_PACKAGE_ERLANG
-menu "Erlang libraries/modules"
-	source "package/erlang-goldrush/Config.in"
-	source "package/erlang-lager/Config.in"
-	source "package/erlang-p1-cache-tab/Config.in"
-	source "package/erlang-p1-iconv/Config.in"
-	source "package/erlang-p1-sip/Config.in"
-	source "package/erlang-p1-stringprep/Config.in"
-	source "package/erlang-p1-stun/Config.in"
-	source "package/erlang-p1-tls/Config.in"
-	source "package/erlang-p1-utils/Config.in"
-	source "package/erlang-p1-xml/Config.in"
-	source "package/erlang-p1-yaml/Config.in"
-	source "package/erlang-p1-zlib/Config.in"
-endmenu
-endif
 	source "package/ficl/Config.in"
 	source "package/gauche/Config.in"
 	source "package/guile/Config.in"
diff --git a/package/erlang/Config.in b/package/erlang/Config.in
index 11407db..b4bb7e0 100644
--- a/package/erlang/Config.in
+++ b/package/erlang/Config.in
@@ -10,7 +10,7 @@ comment "erlang needs a toolchain w/ dynamic library"
 	depends on BR2_PACKAGE_ERLANG_ARCH_SUPPORTS
 	depends on BR2_STATIC_LIBS
 
-config BR2_PACKAGE_ERLANG
+menuconfig BR2_PACKAGE_ERLANG
 	bool "erlang"
 	depends on BR2_USE_MMU # fork()
 	depends on !BR2_STATIC_LIBS
@@ -43,4 +43,19 @@ config BR2_PACKAGE_ERLANG_MEGACO
 	  size so if you do not need it then it is recommended not to
 	  enable it.
 
+comment "Erlang libraries/modules"
+
+source "package/erlang-goldrush/Config.in"
+source "package/erlang-lager/Config.in"
+source "package/erlang-p1-cache-tab/Config.in"
+source "package/erlang-p1-iconv/Config.in"
+source "package/erlang-p1-sip/Config.in"
+source "package/erlang-p1-stringprep/Config.in"
+source "package/erlang-p1-stun/Config.in"
+source "package/erlang-p1-tls/Config.in"
+source "package/erlang-p1-utils/Config.in"
+source "package/erlang-p1-xml/Config.in"
+source "package/erlang-p1-yaml/Config.in"
+source "package/erlang-p1-zlib/Config.in"
+
 endif # BR2_PACKAGE_ERLANG
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 39/83 v2] package/flot: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (37 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 38/83 v2] package/erlang: include Erlang libraries from erlang's Config.in Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 40/83 v2] package/gnuradio: " Yann E. MORIN
                   ` (43 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

flot has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/flot/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/flot/Config.in b/package/flot/Config.in
index 4c0d3ec..0037ff2 100644
--- a/package/flot/Config.in
+++ b/package/flot/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_FLOT
+menuconfig BR2_PACKAGE_FLOT
 	bool "flot"
 	select BR2_PACKAGE_JQUERY
 	help
@@ -18,8 +18,6 @@ config BR2_PACKAGE_FLOT
 
 if BR2_PACKAGE_FLOT
 
-comment "flot plugins"
-
 config BR2_PACKAGE_FLOT_CANVAS
 	bool "canvas"
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 40/83 v2] package/gnuradio: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (38 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 39/83 v2] package/flot: use 'menuconfig' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 41/83 v2] package/gnuradio: split long omment Yann E. MORIN
                   ` (42 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

gnuradio has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/gnuradio/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in
index 50f5e7f..9efe9d3 100644
--- a/package/gnuradio/Config.in
+++ b/package/gnuradio/Config.in
@@ -3,7 +3,7 @@ comment "gnuradio needs a toolchain w/ C++, NPTL, wchar, dynamic library"
 	depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
 		!BR2_TOOLCHAIN_HAS_THREADS_NPTL || BR2_STATIC_LIBS
 
-config BR2_PACKAGE_GNURADIO
+menuconfig BR2_PACKAGE_GNURADIO
 	bool "gnuradio"
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on !BR2_STATIC_LIBS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 41/83 v2] package/gnuradio: split long omment
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (39 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 40/83 v2] package/gnuradio: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 42/83 v2] package/google-material-design-icons: use 'menuconfig' Yann E. MORIN
                   ` (41 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

It does not fit even is a wider-than-normal terminal.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1;
  - tabulate comment  (Arnout)
  - slightly tweak sentence  (Arnout)
  - follow alphabetical ordering

---
Note to Arnout: I did not exactly use your proposal to fix the sentence,
because it was too wide or would have required yet another line.
---
 package/gnuradio/Config.in | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/package/gnuradio/Config.in b/package/gnuradio/Config.in
index 9efe9d3..4bc71b9 100644
--- a/package/gnuradio/Config.in
+++ b/package/gnuradio/Config.in
@@ -67,8 +67,11 @@ config BR2_PACKAGE_GNURADIO_UTILS
 	help
 	  Misc python utilities
 
-comment "gr-fft, -filter, -analog, -channels, -digital, -trellis, -pager, -qtgui depends fftw's single precision"
-	depends on !BR2_PACKAGE_FFTW_PRECISION_SINGLE
+if !BR2_PACKAGE_FFTW_PRECISION_SINGLE
+comment "gr-analog, gr-channels, gr-digital, gr-fft,i"
+comment "gr-filter, gr-pager, gr-trellis and gr-qtgui"
+comment "all depend on fftw with single precision.   "
+endif
 
 if BR2_PACKAGE_FFTW_PRECISION_SINGLE
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 42/83 v2] package/google-material-design-icons: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (40 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 41/83 v2] package/gnuradio: split long omment Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 43/83 v2] package/gr-osmosdr: " Yann E. MORIN
                   ` (40 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

google-material-design-icons has more than 5 options, so use a
'menuconfig' to group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/google-material-design-icons/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/google-material-design-icons/Config.in b/package/google-material-design-icons/Config.in
index a200429..d7be363 100644
--- a/package/google-material-design-icons/Config.in
+++ b/package/google-material-design-icons/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_GOOGLE_MATERIAL_DESIGN_ICONS
+menuconfig BR2_PACKAGE_GOOGLE_MATERIAL_DESIGN_ICONS
 	bool "google-material-design-icons"
 	help
 	  Material design icons are the official icon set from Google that
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 43/83 v2] package/gr-osmosdr: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (41 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 42/83 v2] package/google-material-design-icons: use 'menuconfig' Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 44/83 v2] package/gstreamer: " Yann E. MORIN
                   ` (39 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

gr-osmosdr has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/gr-osmosdr/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/gr-osmosdr/Config.in b/package/gr-osmosdr/Config.in
index aa6ef0d..56d7fe7 100644
--- a/package/gr-osmosdr/Config.in
+++ b/package/gr-osmosdr/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_GR_OSMOSDR
+menuconfig BR2_PACKAGE_GR_OSMOSDR
 	bool "gr-osmosdr"
 	depends on BR2_PACKAGE_GNURADIO
 	select BR2_PACKAGE_GNURADIO_BLOCKS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 44/83 v2] package/gstreamer: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (42 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 43/83 v2] package/gr-osmosdr: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 45/83 v2] package/gstreamer1: " Yann E. MORIN
                   ` (38 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

gstreamer has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - fix commit log+title  (Arnout)
  - move later in the series, after we document that 5-or-more
    sub-options require a menuconfig
---
 package/gstreamer/gstreamer/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/gstreamer/gstreamer/Config.in b/package/gstreamer/gstreamer/Config.in
index aee48b1..cba8774 100644
--- a/package/gstreamer/gstreamer/Config.in
+++ b/package/gstreamer/gstreamer/Config.in
@@ -2,7 +2,7 @@ comment "gstreamer 0.10 needs a toolchain w/ wchar, threads"
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
 
-config BR2_PACKAGE_GSTREAMER
+menuconfig BR2_PACKAGE_GSTREAMER
 	bool "gstreamer 0.10"
 	depends on BR2_USE_WCHAR # glib2
 	depends on BR2_TOOLCHAIN_HAS_THREADS # glib2
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 45/83 v2] package/gstreamer1: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (43 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 44/83 v2] package/gstreamer: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 46/83 v2] package/igh-ethercat: " Yann E. MORIN
                   ` (37 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

gstreamer1 has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

---
Changes RFC -> v1:
  - fix commit log+title  (Arnout)
  - move later in the series, after we document that 5-or-more
    sub-options require a menuconfig
---
 package/gstreamer1/gstreamer1/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/gstreamer1/gstreamer1/Config.in b/package/gstreamer1/gstreamer1/Config.in
index 99aadb4..6475287 100644
--- a/package/gstreamer1/gstreamer1/Config.in
+++ b/package/gstreamer1/gstreamer1/Config.in
@@ -2,7 +2,7 @@ comment "gstreamer 1.x needs a toolchain w/ wchar, threads"
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS
 
-config BR2_PACKAGE_GSTREAMER1
+menuconfig BR2_PACKAGE_GSTREAMER1
 	bool "gstreamer 1.x"
 	depends on BR2_USE_WCHAR # glib2
 	depends on BR2_TOOLCHAIN_HAS_THREADS # glib2
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 46/83 v2] package/igh-ethercat: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (44 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 45/83 v2] package/gstreamer1: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 47/83 v2] package/imlib2: " Yann E. MORIN
                   ` (36 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

igh-ethercat has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/igh-ethercat/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/igh-ethercat/Config.in b/package/igh-ethercat/Config.in
index e600ef5..696e550 100644
--- a/package/igh-ethercat/Config.in
+++ b/package/igh-ethercat/Config.in
@@ -1,7 +1,7 @@
 comment "igh-ethercat needs a Linux kernel to be built"
 	depends on !BR2_LINUX_KERNEL
 
-config BR2_PACKAGE_IGH_ETHERCAT
+menuconfig BR2_PACKAGE_IGH_ETHERCAT
 	bool "igh-ethercat"
 	depends on BR2_LINUX_KERNEL
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 47/83 v2] package/imlib2: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (45 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 46/83 v2] package/igh-ethercat: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 48/83 v2] package/ipsec-tools: " Yann E. MORIN
                   ` (35 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

imlib2 has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/imlib2/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/imlib2/Config.in b/package/imlib2/Config.in
index 7aac074..0b10623 100644
--- a/package/imlib2/Config.in
+++ b/package/imlib2/Config.in
@@ -1,7 +1,7 @@
 comment "imlib2 needs a toolchain w/ dynamic library"
 	depends on BR2_STATIC_LIBS
 
-config BR2_PACKAGE_IMLIB2
+menuconfig BR2_PACKAGE_IMLIB2
 	bool "imlib2"
 	select BR2_PACKAGE_FREETYPE
 	depends on !BR2_STATIC_LIBS # dlopen()
@@ -39,4 +39,5 @@ config BR2_PACKAGE_IMLIB2_X
 	depends on BR2_PACKAGE_XORG7
 	select BR2_PACKAGE_XLIB_LIBXEXT
 	select BR2_PACKAGE_XLIB_LIBX11
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 48/83 v2] package/ipsec-tools: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (46 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 47/83 v2] package/imlib2: " Yann E. MORIN
@ 2016-07-04  9:24 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 49/83 v2] package/lftp: " Yann E. MORIN
                   ` (34 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:24 UTC (permalink / raw)
  To: buildroot

ipsec-tools has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/ipsec-tools/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/ipsec-tools/Config.in b/package/ipsec-tools/Config.in
index 36c5a65..b0d0765 100644
--- a/package/ipsec-tools/Config.in
+++ b/package/ipsec-tools/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_IPSEC_TOOLS
+menuconfig BR2_PACKAGE_IPSEC_TOOLS
 	bool "ipsec-tools"
 	depends on BR2_USE_MMU # fork()
 	depends on !BR2_TOOLCHAIN_USES_MUSL # Use __P() macro all over the tree
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 49/83 v2] package/lftp: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (47 preceding siblings ...)
  2016-07-04  9:24 ` [Buildroot] [PATCH 48/83 v2] package/ipsec-tools: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 50/83 v2] package/libgtk3: " Yann E. MORIN
                   ` (33 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

lftp has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/lftp/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/lftp/Config.in b/package/lftp/Config.in
index fb4ac13..b83a256 100644
--- a/package/lftp/Config.in
+++ b/package/lftp/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LFTP
+menuconfig BR2_PACKAGE_LFTP
 	bool "lftp"
 	depends on BR2_USE_WCHAR
 	depends on BR2_INSTALL_LIBSTDCPP
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 50/83 v2] package/libgtk3: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (48 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 49/83 v2] package/lftp: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 51/83 v2] package/libiio: " Yann E. MORIN
                   ` (32 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

libgtk3 has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/libgtk3/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/libgtk3/Config.in b/package/libgtk3/Config.in
index 39105ac..a9fac0c 100644
--- a/package/libgtk3/Config.in
+++ b/package/libgtk3/Config.in
@@ -10,7 +10,7 @@ comment "libgtk3 needs an OpenGL or OpenGL EGL backend provided by mesa3d"
 	depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL && \
 		!BR2_PACKAGE_HAS_LIBGL
 
-config BR2_PACKAGE_LIBGTK3
+menuconfig BR2_PACKAGE_LIBGTK3
 	bool "libgtk3"
 	select BR2_PACKAGE_ATK
 	select BR2_PACKAGE_CAIRO
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 51/83 v2] package/libiio: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (49 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 50/83 v2] package/libgtk3: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 52/83 v2] package/lighttpd: " Yann E. MORIN
                   ` (31 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

libiio has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/libiio/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/libiio/Config.in b/package/libiio/Config.in
index 28a43f9..d556293 100644
--- a/package/libiio/Config.in
+++ b/package/libiio/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LIBIIO
+menuconfig BR2_PACKAGE_LIBIIO
 	bool "libiio"
 	select BR2_PACKAGE_LIBIIO_LOCAL_BACKEND if !BR2_PACKAGE_LIBIIO_NETWORK_BACKEND
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 52/83 v2] package/lighttpd: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (50 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 51/83 v2] package/libiio: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 53/83 v2] package/linux-firmware: " Yann E. MORIN
                   ` (30 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

lighttpd has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/lighttpd/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/lighttpd/Config.in b/package/lighttpd/Config.in
index f62ccb9..6672ec1 100644
--- a/package/lighttpd/Config.in
+++ b/package/lighttpd/Config.in
@@ -2,7 +2,7 @@ comment "lighttpd needs a toolchain w/ dynamic library"
 	depends on BR2_USE_MMU
 	depends on BR2_STATIC_LIBS
 
-config BR2_PACKAGE_LIGHTTPD
+menuconfig BR2_PACKAGE_LIGHTTPD
 	bool "lighttpd"
 	depends on BR2_USE_MMU # fork()
 	depends on !BR2_STATIC_LIBS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 53/83 v2] package/linux-firmware: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (51 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 52/83 v2] package/lighttpd: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 54/83 v2] package/lldpd: " Yann E. MORIN
                   ` (29 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

linux-firmware has more than 5 options, so use a 'menuconfig' to group
the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/linux-firmware/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/linux-firmware/Config.in b/package/linux-firmware/Config.in
index f56f31d..537273f 100644
--- a/package/linux-firmware/Config.in
+++ b/package/linux-firmware/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LINUX_FIRMWARE
+menuconfig BR2_PACKAGE_LINUX_FIRMWARE
 	bool "linux-firmware"
 	help
 	  This package provides various binary firmware files (closed
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 54/83 v2] package/lldpd: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (52 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 53/83 v2] package/linux-firmware: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 55/83 v2] package/lm-sensors: " Yann E. MORIN
                   ` (28 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

lldpd has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/lldpd/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/lldpd/Config.in b/package/lldpd/Config.in
index 9a9517b..a1ffc54 100644
--- a/package/lldpd/Config.in
+++ b/package/lldpd/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LLDPD
+menuconfig BR2_PACKAGE_LLDPD
 	bool "lldpd"
 	depends on BR2_USE_MMU # fork()
 	select BR2_PACKAGE_LIBEVENT
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 55/83 v2] package/lm-sensors: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (53 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 54/83 v2] package/lldpd: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 56/83 v2] package/lua{, jit}: move inclusion of external libraries/modules Yann E. MORIN
                   ` (27 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

lm-sensors has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/lm-sensors/Config.in | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/package/lm-sensors/Config.in b/package/lm-sensors/Config.in
index 7065c4c..e6dd085 100644
--- a/package/lm-sensors/Config.in
+++ b/package/lm-sensors/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LM_SENSORS
+menuconfig BR2_PACKAGE_LM_SENSORS
 	bool "lm-sensors"
 	help
 	  Lm-sensors is a hardware health monitoring package for
@@ -9,7 +9,6 @@ config BR2_PACKAGE_LM_SENSORS
 	  http://www.lm-sensors.org/
 
 if BR2_PACKAGE_LM_SENSORS
-comment "lm-sensors tools"
 
 config BR2_PACKAGE_LM_SENSORS_SENSORS
 	bool "sensors"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 56/83 v2] package/lua{, jit}: move inclusion of external libraries/modules
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (54 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 55/83 v2] package/lm-sensors: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 57/83 v2] package/mtd: use 'menuconfig' Yann E. MORIN
                   ` (26 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Introduce a Config.ext for external Lua external libraries/modules, that
both lua and luajit include from their respective sub-menus.

lua and luajit now have more than 5 options, so use a 'menuconfig' to
group the options in sub-menus.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in         | 48 --------------------------------------------
 package/lua/Config.ext    | 51 +++++++++++++++++++++++++++++++++++++++++++++++
 package/lua/Config.in     |  4 +++-
 package/luajit/Config.ext |  1 +
 package/luajit/Config.in  |  4 +++-
 5 files changed, 58 insertions(+), 50 deletions(-)
 create mode 100644 package/lua/Config.ext
 create mode 120000 package/luajit/Config.ext

diff --git a/package/Config.in b/package/Config.in
index 0e52f54..6fcd6e6 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -500,54 +500,6 @@ menu "Interpreter languages and scripting"
 	source "package/lua/Config.in"
 	source "package/luainterpreter/Config.in"
 	source "package/luajit/Config.in"
-if BR2_PACKAGE_HAS_LUAINTERPRETER && !BR2_STATIC_LIBS
-# lua modules are dynamically loaded, so not available on static builds
-menu "Lua libraries/modules"
-	source "package/argparse/Config.in"
-	source "package/cgilua/Config.in"
-	source "package/copas/Config.in"
-	source "package/cosmo/Config.in"
-	source "package/coxpcall/Config.in"
-	source "package/dado/Config.in"
-	source "package/lbase64/Config.in"
-	source "package/ljlinenoise/Config.in"
-	source "package/ljsyscall/Config.in"
-	source "package/lpeg/Config.in"
-	source "package/lpty/Config.in"
-	source "package/lrandom/Config.in"
-	source "package/lsqlite3/Config.in"
-	source "package/lua-cjson/Config.in"
-	source "package/lua-coat/Config.in"
-	source "package/lua-coatpersistent/Config.in"
-	source "package/lua-csnappy/Config.in"
-	source "package/lua-ev/Config.in"
-	source "package/lua-iconv/Config.in"
-	source "package/lua-messagepack/Config.in"
-	source "package/lua-msgpack-native/Config.in"
-	source "package/lua-periphery/Config.in"
-	source "package/lua-testmore/Config.in"
-	source "package/luabitop/Config.in"
-	source "package/luacrypto/Config.in"
-	source "package/luaexpat/Config.in"
-	source "package/luaexpatutils/Config.in"
-	source "package/luafilesystem/Config.in"
-	source "package/luajson/Config.in"
-	source "package/lualogging/Config.in"
-	source "package/luaposix/Config.in"
-	source "package/luasec/Config.in"
-	source "package/luasocket/Config.in"
-	source "package/luasql-sqlite3/Config.in"
-	source "package/lunit/Config.in"
-	source "package/luv/Config.in"
-	source "package/luvi/Config.in"
-	source "package/lzlib/Config.in"
-	source "package/orbit/Config.in"
-	source "package/rings/Config.in"
-	source "package/turbolua/Config.in"
-	source "package/wsapi/Config.in"
-	source "package/xavante/Config.in"
-endmenu
-endif
 	source "package/lutok/Config.in"
 	source "package/micropython/Config.in"
 	source "package/micropython-lib/Config.in"
diff --git a/package/lua/Config.ext b/package/lua/Config.ext
new file mode 100644
index 0000000..2141d1a
--- /dev/null
+++ b/package/lua/Config.ext
@@ -0,0 +1,51 @@
+# This file is included both from lua and luajit
+
+if !BR2_STATIC_LIBS
+
+comment "Lua libraries/modules"
+
+source "package/argparse/Config.in"
+source "package/cgilua/Config.in"
+source "package/copas/Config.in"
+source "package/cosmo/Config.in"
+source "package/coxpcall/Config.in"
+source "package/dado/Config.in"
+source "package/lbase64/Config.in"
+source "package/ljlinenoise/Config.in"
+source "package/ljsyscall/Config.in"
+source "package/lpeg/Config.in"
+source "package/lpty/Config.in"
+source "package/lrandom/Config.in"
+source "package/lsqlite3/Config.in"
+source "package/lua-cjson/Config.in"
+source "package/lua-coat/Config.in"
+source "package/lua-coatpersistent/Config.in"
+source "package/lua-csnappy/Config.in"
+source "package/lua-ev/Config.in"
+source "package/lua-iconv/Config.in"
+source "package/lua-messagepack/Config.in"
+source "package/lua-msgpack-native/Config.in"
+source "package/lua-periphery/Config.in"
+source "package/lua-testmore/Config.in"
+source "package/luabitop/Config.in"
+source "package/luacrypto/Config.in"
+source "package/luaexpat/Config.in"
+source "package/luaexpatutils/Config.in"
+source "package/luafilesystem/Config.in"
+source "package/luajson/Config.in"
+source "package/lualogging/Config.in"
+source "package/luaposix/Config.in"
+source "package/luasec/Config.in"
+source "package/luasocket/Config.in"
+source "package/luasql-sqlite3/Config.in"
+source "package/lunit/Config.in"
+source "package/luv/Config.in"
+source "package/luvi/Config.in"
+source "package/lzlib/Config.in"
+source "package/orbit/Config.in"
+source "package/rings/Config.in"
+source "package/turbolua/Config.in"
+source "package/wsapi/Config.in"
+source "package/xavante/Config.in"
+
+endif # !static
diff --git a/package/lua/Config.in b/package/lua/Config.in
index 711fa89..99637dc 100644
--- a/package/lua/Config.in
+++ b/package/lua/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_LUA
+menuconfig BR2_PACKAGE_LUA
 	bool "lua"
 	select BR2_PACKAGE_HAS_LUAINTERPRETER
 	help
@@ -68,4 +68,6 @@ config BR2_PACKAGE_LUA_LINENOISE
 
 endchoice
 
+source "package/lua/Config.ext"
+
 endif
diff --git a/package/luajit/Config.ext b/package/luajit/Config.ext
new file mode 120000
index 0000000..62d6109
--- /dev/null
+++ b/package/luajit/Config.ext
@@ -0,0 +1 @@
+../lua/Config.ext
\ No newline at end of file
diff --git a/package/luajit/Config.in b/package/luajit/Config.in
index 8ce528b..4916b21 100644
--- a/package/luajit/Config.in
+++ b/package/luajit/Config.in
@@ -5,7 +5,7 @@ config BR2_PACKAGE_LUAJIT_ARCH_SUPPORTS
 		BR2_powerpc || BR2_arm || BR2_armeb || \
 		((BR2_mips || BR2_mipsel) && !BR2_MIPS_SOFT_FLOAT)
 
-config BR2_PACKAGE_LUAJIT
+menuconfig BR2_PACKAGE_LUAJIT
 	bool "luajit"
 	select BR2_PACKAGE_HAS_LUAINTERPRETER
 	depends on !BR2_STATIC_LIBS # dlopen
@@ -32,6 +32,8 @@ config BR2_PACKAGE_PROVIDES_LUAINTERPRETER
 config BR2_PACKAGE_LUAINTERPRETER_ABI_VERSION
 	default "5.1"
 
+source "package/luajit/Config.ext"
+
 endif
 
 comment "luajit needs a toolchain w/ dynamic library"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 57/83 v2] package/mtd: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (55 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 56/83 v2] package/lua{, jit}: move inclusion of external libraries/modules Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 58/83 v2] package/ncurses: " Yann E. MORIN
                   ` (25 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

mtd has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/mtd/Config.in | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/package/mtd/Config.in b/package/mtd/Config.in
index 6e4346f..5add9c1 100644
--- a/package/mtd/Config.in
+++ b/package/mtd/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_MTD
+menuconfig BR2_PACKAGE_MTD
 	bool "mtd, jffs2 and ubi/ubifs tools"
 	help
 	  Build mtd, jffs2 and ubi/ubifs tools
@@ -6,7 +6,6 @@ config BR2_PACKAGE_MTD
 	  http://www.linux-mtd.infradead.org/
 
 if BR2_PACKAGE_MTD
-comment "MTD tools selection"
 
 config BR2_PACKAGE_MTD_DOCFDISK
 	bool "docfdisk"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 58/83 v2] package/ncurses: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (56 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 57/83 v2] package/mtd: use 'menuconfig' Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 59/83 v2] package/ndisc6: " Yann E. MORIN
                   ` (24 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

ncurses has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/ncurses/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/ncurses/Config.in b/package/ncurses/Config.in
index 44713f9..0887b33 100644
--- a/package/ncurses/Config.in
+++ b/package/ncurses/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_NCURSES
+menuconfig BR2_PACKAGE_NCURSES
 	bool "ncurses"
 	help
 	  The Ncurses (new curses) library is a free software emulation of
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 59/83 v2] package/ndisc6: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (57 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 58/83 v2] package/ncurses: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 60/83 v2] package/netsnmp: " Yann E. MORIN
                   ` (23 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

ndisc6 has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/ndisc6/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/ndisc6/Config.in b/package/ndisc6/Config.in
index 321bfbc..966ec91 100644
--- a/package/ndisc6/Config.in
+++ b/package/ndisc6/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_NDISC6
+menuconfig BR2_PACKAGE_NDISC6
 	bool "ndisc6 tools"
 	depends on BR2_USE_MMU # fork()
 	select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT_IF_LOCALE
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 60/83 v2] package/netsnmp: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (58 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 59/83 v2] package/ndisc6: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 61/83 v2] package/ntp: " Yann E. MORIN
                   ` (22 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

netsnmp has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/netsnmp/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/netsnmp/Config.in b/package/netsnmp/Config.in
index 57cc6b9..3cef6f3 100644
--- a/package/netsnmp/Config.in
+++ b/package/netsnmp/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_NETSNMP
+menuconfig BR2_PACKAGE_NETSNMP
 	bool "netsnmp"
 	depends on BR2_USE_MMU # fork()
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 61/83 v2] package/ntp: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (59 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 60/83 v2] package/netsnmp: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 62/83 v2] package/nvidia-driver: " Yann E. MORIN
                   ` (21 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

ntp has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/ntp/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/ntp/Config.in b/package/ntp/Config.in
index 1763225..2b8ecc9 100644
--- a/package/ntp/Config.in
+++ b/package/ntp/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_NTP
+menuconfig BR2_PACKAGE_NTP
 	bool "ntp"
 	select BR2_PACKAGE_LIBEVENT
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 62/83 v2] package/nvidia-driver: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (60 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 61/83 v2] package/ntp: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 63/83 v2] package/openobex: " Yann E. MORIN
                   ` (20 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

nvidia-driver has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/nvidia-driver/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/nvidia-driver/Config.in b/package/nvidia-driver/Config.in
index 04499f6..8632835 100644
--- a/package/nvidia-driver/Config.in
+++ b/package/nvidia-driver/Config.in
@@ -2,7 +2,7 @@ comment "nvidia-driver needs a glibc toolchain"
 	depends on BR2_i386 || BR2_x86_64
 	depends on !BR2_TOOLCHAIN_USES_GLIBC
 
-config BR2_PACKAGE_NVIDIA_DRIVER
+menuconfig BR2_PACKAGE_NVIDIA_DRIVER
 	bool "nvidia-driver"
 	depends on BR2_i386 || BR2_x86_64
 	depends on BR2_TOOLCHAIN_USES_GLIBC
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 63/83 v2] package/openobex: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (61 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 62/83 v2] package/nvidia-driver: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 64/83 v2] package/openocd: " Yann E. MORIN
                   ` (19 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

openobex has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/openobex/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/openobex/Config.in b/package/openobex/Config.in
index ed1b81f..bd965d4 100644
--- a/package/openobex/Config.in
+++ b/package/openobex/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_OPENOBEX
+menuconfig BR2_PACKAGE_OPENOBEX
 	bool "openobex"
 	help
 	  Free open source implementation of the Object Exchange (OBEX)
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 64/83 v2] package/openocd: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (62 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 63/83 v2] package/openobex: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 65/83 v2] package/poco: " Yann E. MORIN
                   ` (18 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

openocd has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/openocd/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/openocd/Config.in b/package/openocd/Config.in
index 2313671..66d3826 100644
--- a/package/openocd/Config.in
+++ b/package/openocd/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_OPENOCD
+menuconfig BR2_PACKAGE_OPENOCD
 	bool "openocd"
 	help
 	  OpenOCD - Open On-Chip Debugger
@@ -7,8 +7,6 @@ config BR2_PACKAGE_OPENOCD
 
 if BR2_PACKAGE_OPENOCD
 
-comment "Adapters"
-
 config BR2_PACKAGE_OPENOCD_FTDI
 	bool "MPSSE mode of FTDI based devices"
 	select BR2_PACKAGE_LIBUSB
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 65/83 v2] package/poco: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (63 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 64/83 v2] package/openocd: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 66/83 v2] package/qemu: " Yann E. MORIN
                   ` (17 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

poco has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/poco/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/poco/Config.in b/package/poco/Config.in
index dfdc8d1..1e7ca21 100644
--- a/package/poco/Config.in
+++ b/package/poco/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_POCO
+menuconfig BR2_PACKAGE_POCO
 	bool "poco"
 	depends on BR2_INSTALL_LIBSTDCPP
 	depends on BR2_USE_WCHAR
@@ -15,8 +15,6 @@ config BR2_PACKAGE_POCO
 
 if BR2_PACKAGE_POCO
 
-comment "poco components"
-
 config BR2_PACKAGE_POCO_XML
 	bool "xml"
 	select BR2_PACKAGE_EXPAT
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 66/83 v2] package/qemu: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (64 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 65/83 v2] package/poco: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 67/83 v2] package/quagga: " Yann E. MORIN
                   ` (16 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Qemu has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/qemu/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/qemu/Config.in b/package/qemu/Config.in
index ea6b946..9c3dcfe 100644
--- a/package/qemu/Config.in
+++ b/package/qemu/Config.in
@@ -9,7 +9,7 @@ comment "QEMU requires a toolchain with wchar, threads"
 	depends on BR2_USE_MMU
 	depends on !(BR2_TOOLCHAIN_HAS_THREADS && BR2_USE_WCHAR)
 
-config BR2_PACKAGE_QEMU
+menuconfig BR2_PACKAGE_QEMU
 	bool "QEMU"
 	depends on BR2_PACKAGE_QEMU_ARCH_SUPPORTS_TARGET
 	depends on BR2_TOOLCHAIN_HAS_THREADS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 67/83 v2] package/quagga: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (65 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 66/83 v2] package/qemu: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 68/83 v2] package/sdl: " Yann E. MORIN
                   ` (15 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

quagga has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/quagga/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/quagga/Config.in b/package/quagga/Config.in
index 56b8cad..dd575f8 100644
--- a/package/quagga/Config.in
+++ b/package/quagga/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_QUAGGA
+menuconfig BR2_PACKAGE_QUAGGA
 	bool "quagga"
 	depends on BR2_USE_MMU # fork()
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 68/83 v2] package/sdl: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (66 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 67/83 v2] package/quagga: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 69/83 v2] package/sdl: include SDL external libraries from sdl's Config.in Yann E. MORIN
                   ` (14 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

SDL has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/sdl/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/sdl/Config.in b/package/sdl/Config.in
index 65ac8a2..e0a4437 100644
--- a/package/sdl/Config.in
+++ b/package/sdl/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_SDL
+menuconfig BR2_PACKAGE_SDL
 	bool "SDL"
 	help
 	  Simple DirectMedia Layer - SDL is a library that allows
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 69/83 v2] package/sdl: include SDL external libraries from sdl's Config.in
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (67 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 68/83 v2] package/sdl: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 70/83 v2] package/sdl_*: remove redundant dependency Yann E. MORIN
                   ` (13 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/Config.in     | 6 ------
 package/sdl/Config.in | 9 +++++++++
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 6fcd6e6..b329fc9 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -274,12 +274,6 @@ endif
 	source "package/ocrad/Config.in"
 	source "package/psplash/Config.in"
 	source "package/sdl/Config.in"
-	source "package/sdl_gfx/Config.in"
-	source "package/sdl_image/Config.in"
-	source "package/sdl_mixer/Config.in"
-	source "package/sdl_net/Config.in"
-	source "package/sdl_sound/Config.in"
-	source "package/sdl_ttf/Config.in"
 	source "package/sdl2/Config.in"
 
 comment "Other GUIs"
diff --git a/package/sdl/Config.in b/package/sdl/Config.in
index e0a4437..bba089c 100644
--- a/package/sdl/Config.in
+++ b/package/sdl/Config.in
@@ -27,4 +27,13 @@ config BR2_PACKAGE_SDL_X11
 	select BR2_PACKAGE_XLIB_LIBX11
 	select BR2_PACKAGE_XLIB_LIBXEXT
 
+comment "External libraries"
+
+source "package/sdl_gfx/Config.in"
+source "package/sdl_image/Config.in"
+source "package/sdl_mixer/Config.in"
+source "package/sdl_net/Config.in"
+source "package/sdl_sound/Config.in"
+source "package/sdl_ttf/Config.in"
+
 endif
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 70/83 v2] package/sdl_*: remove redundant dependency
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (68 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 69/83 v2] package/sdl: include SDL external libraries from sdl's Config.in Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 71/83 v2] package/squashfs: use 'menuconfig' Yann E. MORIN
                   ` (12 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Now that those packages are directly included in an if-SDL clause, there
is no need to expressly depend on SDL.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/sdl_gfx/Config.in   | 1 -
 package/sdl_image/Config.in | 1 -
 package/sdl_mixer/Config.in | 1 -
 package/sdl_net/Config.in   | 1 -
 package/sdl_sound/Config.in | 1 -
 package/sdl_ttf/Config.in   | 1 -
 6 files changed, 6 deletions(-)

diff --git a/package/sdl_gfx/Config.in b/package/sdl_gfx/Config.in
index b5d5cac..0f4518b 100644
--- a/package/sdl_gfx/Config.in
+++ b/package/sdl_gfx/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_SDL_GFX
 	bool "SDL_gfx"
-	depends on BR2_PACKAGE_SDL
 	help
 	  The SDL_gfx library is an extension to the SDL library which
 	  provides basic antialiased drawing routines such as lines,
diff --git a/package/sdl_image/Config.in b/package/sdl_image/Config.in
index 0c4a018..d9278d7 100644
--- a/package/sdl_image/Config.in
+++ b/package/sdl_image/Config.in
@@ -1,6 +1,5 @@
 menuconfig BR2_PACKAGE_SDL_IMAGE
 	bool "SDL_image"
-	depends on BR2_PACKAGE_SDL
 	help
 	  SDL_image is an image file loading library. It loads images
 	  as SDL surfaces, and supports the following formats:
diff --git a/package/sdl_mixer/Config.in b/package/sdl_mixer/Config.in
index 42dfe63..8d558ed 100644
--- a/package/sdl_mixer/Config.in
+++ b/package/sdl_mixer/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_SDL_MIXER
 	bool "SDL_mixer"
-	depends on BR2_PACKAGE_SDL
 	help
 	  SDL_mixer is a sample multi-channel audio mixer library.
 	  It supports any number of simultaneously playing channels of
diff --git a/package/sdl_net/Config.in b/package/sdl_net/Config.in
index 2001d23..6fddba5 100644
--- a/package/sdl_net/Config.in
+++ b/package/sdl_net/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_SDL_NET
 	bool "SDL_net"
-	depends on BR2_PACKAGE_SDL
 	help
 	  SDL_net is a small, low-level, cross-platform network library, that
 	  can be used with the Simple DirectMedia Layer library (SDL).
diff --git a/package/sdl_sound/Config.in b/package/sdl_sound/Config.in
index 6da1985..723f9ac 100644
--- a/package/sdl_sound/Config.in
+++ b/package/sdl_sound/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_SDL_SOUND
 	bool "SDL_sound"
-	depends on BR2_PACKAGE_SDL
 	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	help
 	  SDL_sound is a library that handles the decoding of several
diff --git a/package/sdl_ttf/Config.in b/package/sdl_ttf/Config.in
index efc92e5..be56139 100644
--- a/package/sdl_ttf/Config.in
+++ b/package/sdl_ttf/Config.in
@@ -1,6 +1,5 @@
 config BR2_PACKAGE_SDL_TTF
 	bool "SDL_TTF"
-	depends on BR2_PACKAGE_SDL
 	select BR2_PACKAGE_FREETYPE
 	help
 	  SDL_ttf is a sample TrueType font library. It allows you to
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 71/83 v2] package/squashfs: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (69 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 70/83 v2] package/sdl_*: remove redundant dependency Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 72/83 v2] package/sqlite: " Yann E. MORIN
                   ` (11 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

squashfs has 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/squashfs/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/squashfs/Config.in b/package/squashfs/Config.in
index 70c0fc9..cec57f9 100644
--- a/package/squashfs/Config.in
+++ b/package/squashfs/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_SQUASHFS
+menuconfig BR2_PACKAGE_SQUASHFS
 	bool "squashfs"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 72/83 v2] package/sqlite: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (70 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 71/83 v2] package/squashfs: use 'menuconfig' Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 73/83 v2] package/transmission: " Yann E. MORIN
                   ` (10 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

sqlite has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/sqlite/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/sqlite/Config.in b/package/sqlite/Config.in
index c9fde53..c7d6c70 100644
--- a/package/sqlite/Config.in
+++ b/package/sqlite/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_SQLITE
+menuconfig BR2_PACKAGE_SQLITE
 	bool "sqlite"
 	help
 	  SQLite is a small C library that implements a self-contained,
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 73/83 v2] package/transmission: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (71 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 72/83 v2] package/sqlite: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 74/83 v2] package/valgrind: " Yann E. MORIN
                   ` (9 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

transmission has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/transmission/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/transmission/Config.in b/package/transmission/Config.in
index b47c36f..92928c7 100644
--- a/package/transmission/Config.in
+++ b/package/transmission/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_TRANSMISSION
+menuconfig BR2_PACKAGE_TRANSMISSION
 	bool "transmission"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
@@ -45,6 +45,7 @@ config BR2_PACKAGE_TRANSMISSION_GTK
 
 comment "transmission-gtk needs a toolchain w/ locale"
 	depends on BR2_PACKAGE_LIBGTK2 && !BR2_ENABLE_LOCALE
+
 endif
 
 comment "transmission needs a toolchain w/ threads"
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 74/83 v2] package/valgrind: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (72 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 73/83 v2] package/transmission: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 75/83 v2] package/weston: " Yann E. MORIN
                   ` (8 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Valgrind has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/valgrind/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/valgrind/Config.in b/package/valgrind/Config.in
index 40e219d..d38f3fa 100644
--- a/package/valgrind/Config.in
+++ b/package/valgrind/Config.in
@@ -1,7 +1,7 @@
 comment "valgrind needs a toolchain w/ dynamic library"
 	depends on BR2_STATIC_LIBS
 
-config BR2_PACKAGE_VALGRIND
+menuconfig BR2_PACKAGE_VALGRIND
 	bool "valgrind"
 	depends on BR2_ARM_CPU_ARMV7A || BR2_i386 || \
 		   BR2_mips || BR2_mipsel || BR2_mips64 || BR2_mips64el || \
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 75/83 v2] package/weston: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (73 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 74/83 v2] package/valgrind: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 76/83 v2] package/wpa-supplicant: " Yann E. MORIN
                   ` (7 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

weston has 5 options, so use a 'menuconfig' to group the options in a
sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/weston/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/weston/Config.in b/package/weston/Config.in
index 0812279..810a9a0 100644
--- a/package/weston/Config.in
+++ b/package/weston/Config.in
@@ -3,7 +3,7 @@ comment "weston needs udev and a toolchain w/ locale, threads, dynamic library,
 		!BR2_TOOLCHAIN_HEADERS_AT_LEAST_3_0 || BR2_STATIC_LIBS || \
 		!BR2_ENABLE_LOCALE
 
-config BR2_PACKAGE_WESTON
+menuconfig BR2_PACKAGE_WESTON
 	bool "weston"
 	select BR2_PACKAGE_WAYLAND
 	select BR2_PACKAGE_WAYLAND_PROTOCOLS
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 76/83 v2] package/wpa-supplicant: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (74 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 75/83 v2] package/weston: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 77/83 v2] package/xenomai: " Yann E. MORIN
                   ` (6 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

wpa-supplicant has more than 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 package/wpa_supplicant/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/wpa_supplicant/Config.in b/package/wpa_supplicant/Config.in
index 85520b6..c4e312a 100644
--- a/package/wpa_supplicant/Config.in
+++ b/package/wpa_supplicant/Config.in
@@ -1,4 +1,4 @@
-config BR2_PACKAGE_WPA_SUPPLICANT
+menuconfig BR2_PACKAGE_WPA_SUPPLICANT
 	bool "wpa_supplicant"
 	depends on BR2_USE_MMU # fork()
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 77/83 v2] package/xenomai: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (75 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 76/83 v2] package/wpa-supplicant: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 78/83 v2] boot/at91bootstrap3: " Yann E. MORIN
                   ` (5 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Xenomai has 5-or-more options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/xenomai/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/xenomai/Config.in b/package/xenomai/Config.in
index d4c8a89..3736303 100644
--- a/package/xenomai/Config.in
+++ b/package/xenomai/Config.in
@@ -3,7 +3,7 @@ comment "xenomai needs a toolchain w/ threads"
 	depends on BR2_i386 || BR2_x86_64 || BR2_arm || \
 		   BR2_bfin || BR2_powerpc || BR2_sh4
 
-config BR2_PACKAGE_XENOMAI
+menuconfig BR2_PACKAGE_XENOMAI
 	bool "Xenomai Userspace"
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_i386 || BR2_x86_64 || BR2_arm || \
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 78/83 v2] boot/at91bootstrap3: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (76 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 77/83 v2] package/xenomai: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 79/83 v2] boot/barebox: " Yann E. MORIN
                   ` (4 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

at91bootstrap3 has more than 5 options, so use a 'menuconfig' to
group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 boot/at91bootstrap3/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/at91bootstrap3/Config.in b/boot/at91bootstrap3/Config.in
index b8ebe80..6f4050c 100644
--- a/boot/at91bootstrap3/Config.in
+++ b/boot/at91bootstrap3/Config.in
@@ -1,4 +1,4 @@
-config BR2_TARGET_AT91BOOTSTRAP3
+menuconfig BR2_TARGET_AT91BOOTSTRAP3
 	depends on BR2_arm926t || BR2_cortex_a5
 	bool "AT91 Bootstrap 3"
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig)
@ 2016-07-04  9:25 Yann E. MORIN
  2016-07-04  9:24 ` [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type Yann E. MORIN
                   ` (82 more replies)
  0 siblings, 83 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

Hello All!

This series cleans up the layout of the menuconfig UI. Although it looks
to be pretty long (83 patches), it really only boils down to mainly
three different changes :

  - patches 1-25: changes use of 'config'+'menu' into a 'menuconfig'
    (the Kconfig keywords) to automatically arrange the sub-options into
    sub-menus;

  - patches 26-27: document that packages with 5 or more options should
    use sub-menus (with 'menuconfig') to store their sub-options;

  - patches 28-83: change a lot of packages according to the new rule.

A few interspersed, minor changes also provide further cleanups.


Using 'menuconfig' (like bar, below) instead of 'config'+'menu' (like
foo, below) provides a nicer layout (but that's just my taste!):

    [*] foo
        foo sub-menu --->
    [*] bar --->


Finally, moving 5-or-more sub-options to sub-menus also provides for a
nicer layout. Of course, I choose 5 arbitrarily; we can refine this
value, but 5 looked obvious to me.

This series however does not undo the 'menuconfig' of packages with less
than 5 sub-options, when such 'menuconfig' (or 'config'+'menu') were
already present before the series. This is left for another series later
on (this one is already big enough).


Last note: I did try to hunt for all packages, but it is not possible
to either:
  - review all Config.in, or
  - see all packages in the menuconfig UI (because of arch dependencies
    for example)

So, I may have missed a few... These can be fixed later.


Many thanks to Arnout for reviewing the series twice! :-)


Changes v1 -> v2:
  - slight cleanups  (Thomas)

Changes RFC -> v1:
  - don't turn some 'menu' into 'comment' when useless  (Arnout)
  - treat the 'angular' module separately  (Arnout)
  - follow our wrapping rules in the manual  (Arnout)
  - re-format the multi-line gnuradio comment  (Arnout)
  - tyopes  (Arnout)
  - re-order the gstreamer{,1} use of menuconfig after it is documented
  - use menuconfig for linux-firmware as well
  - re-order the post-manual patches alphabetically
  - the Kconfig syntax fixup in at91bootstrap


Regards,
Yann E. MORIN.


The following changes since commit d45b3064f11b532895c04a15f4c283eceea41220

  package/wireless_tools: Pretty fixes (2016-07-04 11:13:05 +0200)


are available in the git repository at:

  git://git.buildroot.org/~ymorin/git/buildroot.git

for you to fetch changes up to 779b15791a114a8e6ee44e85122e1dd4baaf3196

  boot/uboot: use 'menuconfig' (2016-07-04 11:19:13 +0200)


----------------------------------------------------------------
Yann E. MORIN (83):
      boot/at91bootstrap: choices must have a type
      package/angularjs: simplify modules selection
      package/collectd: use 'menuconfig' instead of 'config'+'menu'
      package/gd: use 'menuconfig' instead of 'config'+'menu'
      package/gpsd: use 'menuconfig' instead of 'config'+'menu'
      package/jquery: include external plugins from jquery's Config.in
      package/libdrm: move tests programs to the bottom
      package/libdrm: use 'menuconfig' instead of 'config'+'menu'
      package/mono: use 'menuconfig' instead of 'config'+'menu'
      package/ncftp: use 'menuconfig' instead of 'config'+'menu'
      package/nfs-utils: use 'menuconfig' instead of 'config'+'menu'
      package/nodejs: use 'menuconfig' instead of 'config'+'menu'
      package/ola: use 'menuconfig' instead of 'config'+'menu'
      package/perl: use 'menuconfig' instead of 'config'+'menu'
      package/php: move list of extensions after interface selection
      package/php: use 'menuconfig' instead of 'config'+'menu'
      package/python: use 'menuconfig' instead of 'config'+'menu'
      package/python3: use 'menuconfig' instead of 'config'+'menu'
      package/python{,3}: move inclusion of external python modules
      package/sdl_image: use 'menuconfig' instead of 'config'+'menu'
      package/tcl: use 'menuconfig' instead of 'config'+'menu'
      package/tiff: use 'menuconfig' instead of 'config'+'menu'
      package/udpcast: use 'menuconfig' instead of 'config'+'menu'
      package/gstreamer: include plugins from gstreamer's real Config.in
      package/gstreamer1: include plugins from gstreamer1's real Config.in
      docs/manual: add example about sub-options
      docs/manual: use 'menuconfig' when there are more than 5 sub-options
      package/boost: use 'menuconfig'
      package/cairo: use 'menuconfig'
      package/connman: use 'menuconfig'
      package/dejavu: use 'menuconfig'
      package/directfb: use 'menuconfig'
      package/dnsmasq: use 'menuconfig'
      package/dovecot: use 'menuconfig'
      package/dropbear: use 'menuconfig'
      package/e2fsprogs: use 'menuconfig'
      package/efl: use 'menuconfig'
      package/erlang: include Erlang libraries from erlang's Config.in
      package/flot: use 'menuconfig'
      package/gnuradio: use 'menuconfig'
      package/gnuradio: split long omment
      package/google-material-design-icons: use 'menuconfig'
      package/gr-osmosdr: use 'menuconfig'
      package/gstreamer: use 'menuconfig'
      package/gstreamer1: use 'menuconfig'
      package/igh-ethercat: use 'menuconfig'
      package/imlib2: use 'menuconfig'
      package/ipsec-tools: use 'menuconfig'
      package/lftp: use 'menuconfig'
      package/libgtk3: use 'menuconfig'
      package/libiio: use 'menuconfig'
      package/lighttpd: use 'menuconfig'
      package/linux-firmware: use 'menuconfig'
      package/lldpd: use 'menuconfig'
      package/lm-sensors: use 'menuconfig'
      package/lua{,jit}: move inclusion of external libraries/modules
      package/mtd: use 'menuconfig'
      package/ncurses: use 'menuconfig'
      package/ndisc6: use 'menuconfig'
      package/netsnmp: use 'menuconfig'
      package/ntp: use 'menuconfig'
      package/nvidia-driver: use 'menuconfig'
      package/openobex: use 'menuconfig'
      package/openocd: use 'menuconfig'
      package/poco: use 'menuconfig'
      package/qemu: use 'menuconfig'
      package/quagga: use 'menuconfig'
      package/sdl: use 'menuconfig'
      package/sdl: include SDL external libraries from sdl's Config.in
      package/sdl_*: remove redundant dependency
      package/squashfs: use 'menuconfig'
      package/sqlite: use 'menuconfig'
      package/transmission: use 'menuconfig'
      package/valgrind: use 'menuconfig'
      package/weston: use 'menuconfig'
      package/wpa-supplicant: use 'menuconfig'
      package/xenomai: use 'menuconfig'
      boot/at91bootstrap3: use 'menuconfig'
      boot/barebox: use 'menuconfig'
      boot/grub: use 'menuconfig'
      boot/mxs-bootlets: use 'menuconfig'
      boot/syslinux: use 'menuconfig'
      boot/uboot: use 'menuconfig'

 boot/at91bootstrap/Config.in                   |   2 +-
 boot/at91bootstrap3/Config.in                  |   2 +-
 boot/barebox/Config.in                         |   3 +-
 boot/grub/Config.in                            |  10 +-
 boot/mxs-bootlets/Config.in                    |   2 +-
 boot/syslinux/Config.in                        |   2 +-
 boot/uboot/Config.in                           |   3 +-
 docs/manual/adding-packages-directory.txt      |  53 +++++
 package/Config.in                              | 311 -------------------------
 package/angularjs/Config.in                    |  19 +-
 package/angularjs/angularjs.mk                 |  21 +-
 package/boost/Config.in                        |   2 +-
 package/cairo/Config.in                        |   2 +-
 package/collectd/Config.in                     |   3 +-
 package/connman/Config.in                      |   2 +-
 package/dejavu/Config.in                       |   2 +-
 package/directfb/Config.in                     |   2 +-
 package/dnsmasq/Config.in                      |   2 +-
 package/dovecot/Config.in                      |   2 +-
 package/dropbear/Config.in                     |   2 +-
 package/e2fsprogs/Config.in                    |   2 +-
 package/efl/Config.in                          |   2 +-
 package/erlang/Config.in                       |  17 +-
 package/flot/Config.in                         |   4 +-
 package/gd/Config.in                           |   6 +-
 package/gnuradio/Config.in                     |   9 +-
 package/google-material-design-icons/Config.in |   2 +-
 package/gpsd/Config.in                         |  25 +-
 package/gr-osmosdr/Config.in                   |   2 +-
 package/gstreamer/Config.in                    |  12 -
 package/gstreamer/gstreamer/Config.in          |  12 +-
 package/gstreamer1/Config.in                   |  11 -
 package/gstreamer1/gstreamer1/Config.in        |  11 +-
 package/igh-ethercat/Config.in                 |   2 +-
 package/imlib2/Config.in                       |   3 +-
 package/ipsec-tools/Config.in                  |   2 +-
 package/jquery/Config.in                       |  17 +-
 package/lftp/Config.in                         |   2 +-
 package/libdrm/Config.in                       |  14 +-
 package/libgtk3/Config.in                      |   2 +-
 package/libiio/Config.in                       |   2 +-
 package/lighttpd/Config.in                     |   2 +-
 package/linux-firmware/Config.in               |   2 +-
 package/lldpd/Config.in                        |   2 +-
 package/lm-sensors/Config.in                   |   3 +-
 package/lua/Config.ext                         |  51 ++++
 package/lua/Config.in                          |   4 +-
 package/luajit/Config.ext                      |   1 +
 package/luajit/Config.in                       |   4 +-
 package/mono/Config.in                         |   8 +-
 package/mtd/Config.in                          |   3 +-
 package/ncftp/Config.in                        |   6 +-
 package/ncurses/Config.in                      |   2 +-
 package/ndisc6/Config.in                       |   2 +-
 package/netsnmp/Config.in                      |   2 +-
 package/nfs-utils/Config.in                    |   6 +-
 package/nodejs/Config.in                       |   6 +-
 package/ntp/Config.in                          |   2 +-
 package/nvidia-driver/Config.in                |   2 +-
 package/ola/Config.in                          |  14 +-
 package/openobex/Config.in                     |   2 +-
 package/openocd/Config.in                      |   4 +-
 package/perl/Config.in                         |  48 +++-
 package/php/Config.ext                         |  20 +-
 package/php/Config.in                          |   6 +-
 package/poco/Config.in                         |   4 +-
 package/python/Config.ext                      | 155 ++++++++++++
 package/python/Config.in                       |   6 +-
 package/python3/Config.ext                     |   1 +
 package/python3/Config.in                      |   6 +-
 package/qemu/Config.in                         |   2 +-
 package/quagga/Config.in                       |   2 +-
 package/sdl/Config.in                          |  11 +-
 package/sdl_gfx/Config.in                      |   1 -
 package/sdl_image/Config.in                    |   7 +-
 package/sdl_mixer/Config.in                    |   1 -
 package/sdl_net/Config.in                      |   1 -
 package/sdl_sound/Config.in                    |   1 -
 package/sdl_ttf/Config.in                      |   1 -
 package/sqlite/Config.in                       |   2 +-
 package/squashfs/Config.in                     |   2 +-
 package/tcl/Config.in                          |   7 +-
 package/tiff/Config.in                         |   4 +-
 package/transmission/Config.in                 |   3 +-
 package/udpcast/Config.in                      |   6 +-
 package/valgrind/Config.in                     |   2 +-
 package/weston/Config.in                       |   2 +-
 package/wpa_supplicant/Config.in               |   2 +-
 package/xenomai/Config.in                      |   2 +-
 89 files changed, 532 insertions(+), 507 deletions(-)
 create mode 100644 package/lua/Config.ext
 create mode 120000 package/luajit/Config.ext
 create mode 100644 package/python/Config.ext
 create mode 120000 package/python3/Config.ext

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 79/83 v2] boot/barebox: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (77 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 78/83 v2] boot/at91bootstrap3: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 80/83 v2] boot/grub: " Yann E. MORIN
                   ` (3 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

barebox has more than 5 options, so use a 'menuconfig' to
group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
---
 boot/barebox/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/boot/barebox/Config.in b/boot/barebox/Config.in
index fd5cb8c..90aaede 100644
--- a/boot/barebox/Config.in
+++ b/boot/barebox/Config.in
@@ -1,4 +1,4 @@
-config BR2_TARGET_BAREBOX
+menuconfig BR2_TARGET_BAREBOX
 	bool "Barebox"
 	help
 	  The Barebox bootloader, formerly known as U-Boot v2.
@@ -6,6 +6,7 @@ config BR2_TARGET_BAREBOX
 	  http://www.barebox.org
 
 if BR2_TARGET_BAREBOX
+
 choice
 	prompt "version"
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 80/83 v2] boot/grub: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (78 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 79/83 v2] boot/barebox: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 81/83 v2] boot/mxs-bootlets: " Yann E. MORIN
                   ` (2 subsequent siblings)
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

grub has more than 5 options, so use a 'menuconfig' to
group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 boot/grub/Config.in | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/boot/grub/Config.in b/boot/grub/Config.in
index 4db8642..3ae688f 100644
--- a/boot/grub/Config.in
+++ b/boot/grub/Config.in
@@ -1,4 +1,4 @@
-config BR2_TARGET_GRUB
+menuconfig BR2_TARGET_GRUB
 	bool "grub"
 	depends on BR2_i386 || BR2_x86_64
 	select BR2_HOSTARCH_NEEDS_IA32_COMPILER
@@ -52,7 +52,7 @@ config BR2_TARGET_GRUB_DISKLESS
 	help
 	  enable diskless support
 
-menu "filesystem drivers"
+comment "Filesystem drivers"
 
 config BR2_TARGET_GRUB_FS_EXT2
 	bool "ext2"
@@ -86,9 +86,7 @@ config BR2_TARGET_GRUB_FS_XFS
 	help
 	  Enable support for the XFS filesystem in Grub.
 
-endmenu
-
-menu "network drivers"
+comment "Network drivers"
 
 config BR2_TARGET_GRUB_3c595
 	bool "3Com595 driver"
@@ -195,6 +193,4 @@ config BR2_TARGET_GRUB_w89c840
 	help
 	  enable Winbond W89c840 driver
 
-endmenu
-
 endif # BR2_TARGET_GRUB
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 81/83 v2] boot/mxs-bootlets: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (79 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 80/83 v2] boot/grub: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 82/83 v2] boot/syslinux: " Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 83/83 v2] boot/uboot: " Yann E. MORIN
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

mxs-bootlets has 5 options, so use a 'menuconfig' to
group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 boot/mxs-bootlets/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/mxs-bootlets/Config.in b/boot/mxs-bootlets/Config.in
index 4bb07a0..2712d8d 100644
--- a/boot/mxs-bootlets/Config.in
+++ b/boot/mxs-bootlets/Config.in
@@ -1,4 +1,4 @@
-config BR2_TARGET_MXS_BOOTLETS
+menuconfig BR2_TARGET_MXS_BOOTLETS
 	bool "mxs-bootlets"
 	depends on BR2_arm
 	help
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 82/83 v2] boot/syslinux: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (80 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 81/83 v2] boot/mxs-bootlets: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  2016-07-04  9:25 ` [Buildroot] [PATCH 83/83 v2] boot/uboot: " Yann E. MORIN
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

syslinux can show up to 5 options, so use a 'menuconfig' to group the
options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 boot/syslinux/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/boot/syslinux/Config.in b/boot/syslinux/Config.in
index 6067e9c..ad0704a 100644
--- a/boot/syslinux/Config.in
+++ b/boot/syslinux/Config.in
@@ -1,4 +1,4 @@
-config BR2_TARGET_SYSLINUX
+menuconfig BR2_TARGET_SYSLINUX
 	bool "syslinux"
 	depends on BR2_i386 || BR2_x86_64
 	select BR2_HOSTARCH_NEEDS_IA32_COMPILER
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 83/83 v2] boot/uboot: use 'menuconfig'
  2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
                   ` (81 preceding siblings ...)
  2016-07-04  9:25 ` [Buildroot] [PATCH 82/83 v2] boot/syslinux: " Yann E. MORIN
@ 2016-07-04  9:25 ` Yann E. MORIN
  82 siblings, 0 replies; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04  9:25 UTC (permalink / raw)
  To: buildroot

uboot has more than 5 options, so use a 'menuconfig' to
group the options in a sub-menu.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
 boot/uboot/Config.in | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
index 9ffbb51..8e5ecda 100644
--- a/boot/uboot/Config.in
+++ b/boot/uboot/Config.in
@@ -1,9 +1,10 @@
-config BR2_TARGET_UBOOT
+menuconfig BR2_TARGET_UBOOT
 	bool "U-Boot"
 	help
 	  Build "Das U-Boot" Boot Monitor
 
 if BR2_TARGET_UBOOT
+
 choice
 	prompt "Build system"
 	default BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type
  2016-07-04  9:24 ` [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type Yann E. MORIN
@ 2016-07-04 13:13   ` Peter Korsgaard
  0 siblings, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 13:13 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > ---
 >  boot/at91bootstrap/Config.in | 2 +-
 >  1 file changed, 1 insertion(+), 1 deletion(-)

 > diff --git a/boot/at91bootstrap/Config.in b/boot/at91bootstrap/Config.in
 > index 57fdeda..d131a6a 100644
 > --- a/boot/at91bootstrap/Config.in
 > +++ b/boot/at91bootstrap/Config.in
 > @@ -26,7 +26,7 @@ config BR2_TARGET_AT91BOOTSTRAP_BOARD
 >  	  This is used to do a make <board>_config
 
 >  choice
 > -	prompt "Boot Memory"
 > +	bool "Boot Memory"

We use prompts in several other places in Buildroot:

git grep 'prompt "' **/Config.in | wc -l
89

And we have "always" done so without issues, so I don't think we need to
change this now.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection
  2016-07-04  9:24 ` [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection Yann E. MORIN
@ 2016-07-04 13:19   ` Peter Korsgaard
  2016-07-04 14:07   ` Peter Korsgaard
  1 sibling, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 13:19 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Remove the superfluous config option, and make all modules default to
 > 'y', which provides exactly the same functionality.

 > simplify (and slightly beautify) the handling of modules in the .mk:
 > since the modules' symbols are always meaningful now, we can get rid of
 > the else-clause, and always build the list of modules using the Kconfig
 > options.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Cc: Arnout Vandecappelle <arnout@mind.be>
 > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

 > ---
 > Changes v1 -> v2:
 >   - remove spurious comment leftover from RFC  (Thomas)

 > Changes RFC -> v1:
 >   - treat the 'angular' module on its own, as was done previosuly
 >     (Arnout)
 >   - drop useless comments in Config.in  (Arnout)
 >   - don't re-order list, it was already alphabetically sorted
 > ---
 >  package/angularjs/Config.in    | 19 +++++++++----------
 >  package/angularjs/angularjs.mk | 21 ++++++++++++++-------
 >  2 files changed, 23 insertions(+), 17 deletions(-)

 > diff --git a/package/angularjs/Config.in b/package/angularjs/Config.in
 > index 206b3d8..2a0b48c 100644
 > --- a/package/angularjs/Config.in
 > +++ b/package/angularjs/Config.in
 > @@ -7,16 +7,9 @@ menuconfig BR2_PACKAGE_ANGULARJS
 
 >  if BR2_PACKAGE_ANGULARJS
 
 > -config BR2_ANGULARJS_MODULES
 > -	bool "angularjs modules"
 > -	help
 > -	  Select which modules to install. If disabled, all modules
 > -	  will be installed.
 > -
 > -if BR2_ANGULARJS_MODULES
 > -
 >  config BR2_ANGULARJS_MODULE_ANIMATE
 >  	bool "animate"
 > +	default y
 >  	help
 >  	  The ngAnimate module provides support for CSS-based animations
 >  	  (keyframes and transitions) as well as JavaScript-based animations
 > @@ -26,6 +19,7 @@ config BR2_ANGULARJS_MODULE_ANIMATE
 
 >  config BR2_ANGULARJS_MODULE_ARIA
 >  	bool "aria"
 > +	default y
 >  	help
 >  	  The ngAria module provides support for common ARIA attributes that
 >  	  convey state or semantic information about the application for users
 > @@ -33,18 +27,21 @@ config BR2_ANGULARJS_MODULE_ARIA
 
 >  config BR2_ANGULARJS_MODULE_COOKIES
 >  	bool "cookies"
 > +	default y
 >  	help
 >  	  The ngCookies module provides a convenient wrapper for reading and
 >  	  writing browser cookies.
 
 >  config BR2_ANGULARJS_MODULE_MESSAGE_FORMAT
 >  	bool "message-format"
 > +	default y
 >  	help
 >  	  The ngMessageFormat module is used recognize MessageFormat extensions
 >  	  in interpolation expressions.
 
 >  config BR2_ANGULARJS_MODULE_MESSAGES
 >  	bool "messages"
 > +	default y
 >  	help
 >  	  The ngMessages module provides enhanced support for displaying
 >  	  messages within templates (typically within forms or when rendering
 > @@ -52,28 +49,30 @@ config BR2_ANGULARJS_MODULE_MESSAGES
 
 >  config BR2_ANGULARJS_MODULE_RESOURCE
 >  	bool "resource"
 > +	default y
 >  	help
 >  	  The ngResource module provides interaction support with RESTful
 >  	  services via the $resource service.
 
 >  config BR2_ANGULARJS_MODULE_ROUTE
 >  	bool "route"
 > +	default y
 >  	help
 >  	  The ngRoute module provides routing and deeplinking services and
 >  	  directives for angular apps.
 
 >  config BR2_ANGULARJS_MODULE_SANITIZE
 >  	bool "sanitize"
 > +	default y
 >  	help
 >  	  The ngSanitize module provides functionality to sanitize HTML.
 
 >  config BR2_ANGULARJS_MODULE_TOUCH
 >  	bool "touch"
 > +	default y
 >  	help
 >  	  The ngTouch module provides touch events and other helpers for
 >  	  touch-enabled devices. The implementation is based on jQuery Mobile
 >  	  touch event handling (jquerymobile.com).
 
 >  endif
 > -
 > -endif
 > diff --git a/package/angularjs/angularjs.mk b/package/angularjs/angularjs.mk
 > index 60dbdb6..1f9a8fa 100644
 > --- a/package/angularjs/angularjs.mk
 > +++ b/package/angularjs/angularjs.mk
 > @@ -17,18 +17,25 @@ define ANGULARJS_EXTRACT_CMDS
 >  	rmdir $(@D)/angular-$(ANGULARJS_VERSION)
 >  endef
 
 > +# This list mirrors the list of available modules from Config.in.
 > +# Keep this list in sync and in the same (alphabetical) order as the
 > +# one in Config.in.
 > +ANGULARJS_MODULES = \
 > +	animate \
 > +	aria \
 > +	cookies \
 > +	messages \
 > +	message-format \
 > +	resource \
 > +	route \
 > +	sanitize \
 > +	touch

I like the Config.in change, but how about using something like we do in
E.G. mtd-utils for the options - E.G.:

ANGULAR_MODULES_y = angular
ANGULAR_MODULES_$(BR2_ANGULARJS_MODULE_ANIMATE) += angular-animate
ANGULAR_MODULES_$(BR2_ANGULARJS_MODULE_ARIA) += angular-aria
...

define ANGULARJS_INSTALL_TARGET_CMDS
	$(foreach f,$(ANGULARJS_MODULES_y),\
		$(INSTALL) -m 0644 -D $(@D)/$(f).min.js \
			$(TARGET_DIR)/var/www/$(f).js$(sep))
endef

That seems more robust than this list.

While we're at it, it is quite unfortunate that these options aren't
prefixed by BR2_PACKAGE_ :/

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection
  2016-07-04  9:24 ` [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection Yann E. MORIN
  2016-07-04 13:19   ` Peter Korsgaard
@ 2016-07-04 14:07   ` Peter Korsgaard
  1 sibling, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 14:07 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Remove the superfluous config option, and make all modules default to
 > 'y', which provides exactly the same functionality.

 > simplify (and slightly beautify) the handling of modules in the .mk:
 > since the modules' symbols are always meaningful now, we can get rid of
 > the else-clause, and always build the list of modules using the Kconfig
 > options.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Cc: Arnout Vandecappelle <arnout@mind.be>
 > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

As discussed, the angular sub modules are really small compared to
angular:

ls -lahS target/var/www
total 228K
-rw-r--r-- 1 peko peko 142K Jul  4 16:06 angular.js
-rw-r--r-- 1 peko peko  23K Jul  4 16:06 angular-animate.js
-rw-r--r-- 1 peko peko  11K Jul  4 16:06 angular-message-format.js
-rw-r--r-- 1 peko peko 5.9K Jul  4 16:06 angular-sanitize.js
-rw-r--r-- 1 peko peko 4.3K Jul  4 16:06 angular-route.js
drwxr-xr-x 2 peko peko 4.0K Jul  4 16:06 .
drwxr-xr-x 5 peko peko 4.0K Jul  4 15:58 ..
-rw-r--r-- 1 peko peko 3.7K Jul  4 16:06 angular-aria.js
-rw-r--r-- 1 peko peko 3.6K Jul  4 16:06 angular-resource.js
-rw-r--r-- 1 peko peko 3.6K Jul  4 16:06 angular-touch.js
-rw-r--r-- 1 peko peko 2.6K Jul  4 16:06 angular-messages.js
-rw-r--r-- 1 peko peko 1.7K Jul  4 16:06 angular-loader.js
-rw-r--r-- 1 peko peko 1.4K Jul  4 16:06 angular-cookies.js

So committed after removing the suboptions, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 04/83 v2] package/gd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 04/83 v2] package/gd: " Yann E. MORIN
@ 2016-07-04 15:10   ` Peter Korsgaard
  0 siblings, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 15:10 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > ---
 > Changes RFC -> v1:
 >   - typo in title  (Arnout)
 > ---
 >  package/gd/Config.in | 6 ++----
 >  1 file changed, 2 insertions(+), 4 deletions(-)

 > diff --git a/package/gd/Config.in b/package/gd/Config.in
 > index 28b0b0f..092bc06 100644
 > --- a/package/gd/Config.in
 > +++ b/package/gd/Config.in
 > @@ -1,4 +1,4 @@
 > -config BR2_PACKAGE_GD
 > +menuconfig BR2_PACKAGE_GD
 >  	bool "gd"
 >  	help
 >  	  GD is a graphics library. It allows your code to quickly
 > @@ -12,7 +12,7 @@ config BR2_PACKAGE_GD
 
 >  if BR2_PACKAGE_GD
 
 > -menu "gd tools"
 > +comment "gd tools"

I don't think this comment really adds any value, so I've dropped this
and committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04 15:14   ` Peter Korsgaard
  2016-07-04 15:18     ` Yann E. MORIN
  0 siblings, 1 reply; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 15:14 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Keep the existing sub-menus, because there are a lot of entries in those
 > sub-menus. Using comments to separate them is not a totally satifactory
 > solution.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

I'm not quite sure I think this is an improvement. We used to have:

  [ ] collectd

Which when selected becomes:

  [*] collectd
       match plugins  --->
       misc plugins  --->
       read plugins  --->
       target plugins  --->
       write plugins  --->

Which looks sensible to me.

With the change we instead have:

[ ] collectd  ----

(and if you press enter you get to an empty menu). When enabled you
again have to press enter to get to the submenus.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04 15:14   ` Peter Korsgaard
@ 2016-07-04 15:18     ` Yann E. MORIN
  2016-07-04 15:28       ` Peter Korsgaard
  0 siblings, 1 reply; 103+ messages in thread
From: Yann E. MORIN @ 2016-07-04 15:18 UTC (permalink / raw)
  To: buildroot

Peter, All,

On 2016-07-04 17:14 +0200, Peter Korsgaard spake thusly:
> >>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:
> 
>  > Keep the existing sub-menus, because there are a lot of entries in those
>  > sub-menus. Using comments to separate them is not a totally satifactory
>  > solution.
> 
>  > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> I'm not quite sure I think this is an improvement. We used to have:
> 
>   [ ] collectd
> 
> Which when selected becomes:
> 
>   [*] collectd
>        match plugins  --->
>        misc plugins  --->
>        read plugins  --->
>        target plugins  --->
>        write plugins  --->
> 
> Which looks sensible to me.
> 
> With the change we instead have:
> 
> [ ] collectd  ----

You did notice the trailing '---' which means that the menu is empty, I
guess? ;-)

But as we discussed live about this series: meh... ;-)

Regards,
Yann E. MORIN.

> (and if you press enter you get to an empty menu). When enabled you
> again have to press enter to get to the submenus.
> 
> -- 
> Bye, Peter Korsgaard

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 05/83 v2] package/gpsd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 05/83 v2] package/gpsd: " Yann E. MORIN
@ 2016-07-04 15:18   ` Peter Korsgaard
  0 siblings, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 15:18 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Since there were multiple sub-menus, their prompts are turned into
 > comments.

 > Some option were missing a prompt, so add one; switch integer options to
 > use the 'int' type instead of 'string'. Add separating lines around
 > options.

 > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

Committed, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04 15:18     ` Yann E. MORIN
@ 2016-07-04 15:28       ` Peter Korsgaard
  0 siblings, 0 replies; 103+ messages in thread
From: Peter Korsgaard @ 2016-07-04 15:28 UTC (permalink / raw)
  To: buildroot

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

Hi,

 >> Which looks sensible to me.
 >> 
 >> With the change we instead have:
 >> 
 >> [ ] collectd  ----

 > You did notice the trailing '---' which means that the menu is empty, I
 > guess? ;-)

Yes. I do find it quite stupid that enter isn't just a no-op when the
option isn't enabled, but I guess that's just kconfig :/

-- 
Venlig hilsen,
Peter Korsgaard 

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in
  2016-07-04  9:24 ` [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in Yann E. MORIN
@ 2016-07-04 20:41   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 20:41 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:17 +0200, Yann E. MORIN wrote:
> ... and make it a menuconfig instead of a 'config'+'menu'.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

I very much prefer to have all packages included from the top-level
package/Config.in, as much as possible. So I don't really like this
change and prefer to keep things as they are today.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom
  2016-07-04  9:24 ` [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom Yann E. MORIN
@ 2016-07-04 20:42   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 20:42 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:18 +0200, Yann E. MORIN wrote:
> Tests programs are not that important; at least, they're probably less
> important than the drivers selection.
> 
> Move it down to after the drivers selection.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/libdrm/Config.in | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04 21:35   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:35 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:19 +0200, Yann E. MORIN wrote:
> Transform the menu prompt into a comment.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> ---
> Changes RFC -> v1:
>   - drop the comment  (Arnout)
> ---
>  package/libdrm/Config.in | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

I applied, but I discovered afterwards that the result now looks worse
than it was: there is now no indication that the
intel/nouveau/amdgpu/... options are DRM drivers.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection
  2016-07-04  9:24 ` [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection Yann E. MORIN
@ 2016-07-04 21:37   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:37 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:26 +0200, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/php/Config.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
@ 2016-07-04 21:37   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:37 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:31 +0200, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/sdl_image/Config.in | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

Applied to master, thanks. In addition, I have changed the prompts of
the sub-options to be less verbose.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 22/83 v2] package/tiff: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 22/83 v2] package/tiff: " Yann E. MORIN
@ 2016-07-04 21:38   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:38 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:33 +0200, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/tiff/Config.in | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 23/83 v2] package/udpcast: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 23/83 v2] package/udpcast: " Yann E. MORIN
@ 2016-07-04 21:38   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:38 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:34 +0200, Yann E. MORIN wrote:
> This is a bit overkill, as there is only two sub-items. However, this
> commit is not about judging whether it makes sense or not to have a
> sub-menu, but to cleanup the 'config'+'menu' situation.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Instead, I've replaced this by a commit that simply removes the
menu ... endmenu.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 09/83 v2] package/mono: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 09/83 v2] package/mono: " Yann E. MORIN
@ 2016-07-04 21:38   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:38 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:20 +0200, Yann E. MORIN wrote:

> -config BR2_PACKAGE_MONO
> +menuconfig BR2_PACKAGE_MONO
>  	bool "mono"
>  	depends on BR2_PACKAGE_MONO_ARCH_SUPPORTS
>  	depends on BR2_TOOLCHAIN_HAS_THREADS
> @@ -14,6 +14,12 @@ config BR2_PACKAGE_MONO
>  
>  	  http://download.mono-project.com/sources/mono/
>  
> +if BR2_PACKAGE_MONO
> +
> +source "package/mono-gtksharp3/Config.in"
> +
> +endif

I want to keep the package inclusions as much as possible in
package/Config.in, so I've not applied this patch, and for the same
reason, I have not applied:

[PATCH 14/83 v2] package/perl: use 'menuconfig' instead of 'config'+'menu'
[PATCH 16/83 v2] package/php: use 'menuconfig' instead of 'config'+'menu'
[PATCH 19/83 v2] package/python{, 3}: move inclusion of external python modules
[PATCH 21/83 v2] package/tcl: use 'menuconfig' instead of 'config'+'menu'

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 13/83 v2] package/ola: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 13/83 v2] package/ola: " Yann E. MORIN
@ 2016-07-04 21:38   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:38 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:24 +0200, Yann E. MORIN wrote:
> Since there were multiple sub-menus, their prompts are turned into
> comments.
> 
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/ola/Config.in | 14 ++++----------
>  1 file changed, 4 insertions(+), 10 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 10/83 v2] package/ncftp: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 10/83 v2] package/ncftp: " Yann E. MORIN
@ 2016-07-04 21:39   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:39 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:21 +0200, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> 
> ---
> Changes RFC -> v1:
>   - drop the comment  (Arnout)
> ---
>  package/ncftp/Config.in | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

I've replaced this by a commit that removes the menu ... endmenu and
simplifies the prompt of the sub-options.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

* [Buildroot] [PATCH 11/83 v2] package/nfs-utils: use 'menuconfig' instead of 'config'+'menu'
  2016-07-04  9:24 ` [Buildroot] [PATCH 11/83 v2] package/nfs-utils: " Yann E. MORIN
@ 2016-07-04 21:40   ` Thomas Petazzoni
  0 siblings, 0 replies; 103+ messages in thread
From: Thomas Petazzoni @ 2016-07-04 21:40 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  4 Jul 2016 11:24:22 +0200, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
>  package/nfs-utils/Config.in | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)

I've instead applied a patch that removes the menu...endmenu, because
it was a bit silly to have a sub-menu for just 3 options.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 103+ messages in thread

end of thread, other threads:[~2016-07-04 21:40 UTC | newest]

Thread overview: 103+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-04  9:25 [Buildroot] [PATCH 00/83 v2] packages: use menuconfig to group sub-options (branch yem/menuconfig) Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 01/83 v2] boot/at91bootstrap: choices must have a type Yann E. MORIN
2016-07-04 13:13   ` Peter Korsgaard
2016-07-04  9:24 ` [Buildroot] [PATCH 02/83 v2] package/angularjs: simplify modules selection Yann E. MORIN
2016-07-04 13:19   ` Peter Korsgaard
2016-07-04 14:07   ` Peter Korsgaard
2016-07-04  9:24 ` [Buildroot] [PATCH 03/83 v2] package/collectd: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
2016-07-04 15:14   ` Peter Korsgaard
2016-07-04 15:18     ` Yann E. MORIN
2016-07-04 15:28       ` Peter Korsgaard
2016-07-04  9:24 ` [Buildroot] [PATCH 04/83 v2] package/gd: " Yann E. MORIN
2016-07-04 15:10   ` Peter Korsgaard
2016-07-04  9:24 ` [Buildroot] [PATCH 05/83 v2] package/gpsd: " Yann E. MORIN
2016-07-04 15:18   ` Peter Korsgaard
2016-07-04  9:24 ` [Buildroot] [PATCH 06/83 v2] package/jquery: include external plugins from jquery's Config.in Yann E. MORIN
2016-07-04 20:41   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 07/83 v2] package/libdrm: move tests programs to the bottom Yann E. MORIN
2016-07-04 20:42   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 08/83 v2] package/libdrm: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
2016-07-04 21:35   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 09/83 v2] package/mono: " Yann E. MORIN
2016-07-04 21:38   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 10/83 v2] package/ncftp: " Yann E. MORIN
2016-07-04 21:39   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 11/83 v2] package/nfs-utils: " Yann E. MORIN
2016-07-04 21:40   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 12/83 v2] package/nodejs: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 13/83 v2] package/ola: " Yann E. MORIN
2016-07-04 21:38   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 14/83 v2] package/perl: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 15/83 v2] package/php: move list of extensions after interface selection Yann E. MORIN
2016-07-04 21:37   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 16/83 v2] package/php: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 17/83 v2] package/python: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 18/83 v2] package/python3: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 19/83 v2] package/python{, 3}: move inclusion of external python modules Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 20/83 v2] package/sdl_image: use 'menuconfig' instead of 'config'+'menu' Yann E. MORIN
2016-07-04 21:37   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 21/83 v2] package/tcl: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 22/83 v2] package/tiff: " Yann E. MORIN
2016-07-04 21:38   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 23/83 v2] package/udpcast: " Yann E. MORIN
2016-07-04 21:38   ` Thomas Petazzoni
2016-07-04  9:24 ` [Buildroot] [PATCH 24/83 v2] package/gstreamer: include plugins from gstreamer's real Config.in Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 25/83 v2] package/gstreamer1: include plugins from gstreamer1's " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 26/83 v2] docs/manual: add example about sub-options Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 27/83 v2] docs/manual: use 'menuconfig' when there are more than 5 sub-options Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 28/83 v2] package/boost: use 'menuconfig' Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 29/83 v2] package/cairo: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 30/83 v2] package/connman: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 31/83 v2] package/dejavu: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 32/83 v2] package/directfb: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 33/83 v2] package/dnsmasq: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 34/83 v2] package/dovecot: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 35/83 v2] package/dropbear: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 36/83 v2] package/e2fsprogs: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 37/83 v2] package/efl: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 38/83 v2] package/erlang: include Erlang libraries from erlang's Config.in Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 39/83 v2] package/flot: use 'menuconfig' Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 40/83 v2] package/gnuradio: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 41/83 v2] package/gnuradio: split long omment Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 42/83 v2] package/google-material-design-icons: use 'menuconfig' Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 43/83 v2] package/gr-osmosdr: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 44/83 v2] package/gstreamer: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 45/83 v2] package/gstreamer1: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 46/83 v2] package/igh-ethercat: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 47/83 v2] package/imlib2: " Yann E. MORIN
2016-07-04  9:24 ` [Buildroot] [PATCH 48/83 v2] package/ipsec-tools: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 49/83 v2] package/lftp: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 50/83 v2] package/libgtk3: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 51/83 v2] package/libiio: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 52/83 v2] package/lighttpd: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 53/83 v2] package/linux-firmware: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 54/83 v2] package/lldpd: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 55/83 v2] package/lm-sensors: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 56/83 v2] package/lua{, jit}: move inclusion of external libraries/modules Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 57/83 v2] package/mtd: use 'menuconfig' Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 58/83 v2] package/ncurses: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 59/83 v2] package/ndisc6: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 60/83 v2] package/netsnmp: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 61/83 v2] package/ntp: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 62/83 v2] package/nvidia-driver: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 63/83 v2] package/openobex: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 64/83 v2] package/openocd: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 65/83 v2] package/poco: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 66/83 v2] package/qemu: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 67/83 v2] package/quagga: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 68/83 v2] package/sdl: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 69/83 v2] package/sdl: include SDL external libraries from sdl's Config.in Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 70/83 v2] package/sdl_*: remove redundant dependency Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 71/83 v2] package/squashfs: use 'menuconfig' Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 72/83 v2] package/sqlite: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 73/83 v2] package/transmission: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 74/83 v2] package/valgrind: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 75/83 v2] package/weston: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 76/83 v2] package/wpa-supplicant: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 77/83 v2] package/xenomai: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 78/83 v2] boot/at91bootstrap3: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 79/83 v2] boot/barebox: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 80/83 v2] boot/grub: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 81/83 v2] boot/mxs-bootlets: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 82/83 v2] boot/syslinux: " Yann E. MORIN
2016-07-04  9:25 ` [Buildroot] [PATCH 83/83 v2] boot/uboot: " Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox