diff mbox series

[05/10] package/petitboot: fix ordering of comments and sub-options

Message ID eae889ce783ef15ffa62e2af3f2d2ef0e8e079dd.1706448055.git.yann.morin.1998@free.fr
State Accepted
Headers show
Series package: fix unmet dependencies (branch yem/misc-fixes) | expand

Commit Message

Yann E. MORIN Jan. 28, 2024, 1:20 p.m. UTC
Currently, the comment about availability is between the package main
symbol and its suboptions, breaking the threading in menuconfig.

Invert the order, and move the comment last. Also enclose the suboptions
between an if-block, as it is more customary.

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
---
 package/petitboot/Config.in | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

Comments

Thomas Petazzoni Feb. 7, 2024, 11:51 a.m. UTC | #1
On Sun, 28 Jan 2024 14:20:54 +0100
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

> Currently, the comment about availability is between the package main
> symbol and its suboptions, breaking the threading in menuconfig.
> 
> Invert the order, and move the comment last. Also enclose the suboptions
> between an if-block, as it is more customary.
> 
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> ---
>  package/petitboot/Config.in | 19 +++++++++++--------
>  1 file changed, 11 insertions(+), 8 deletions(-)

Applied to master, thanks.

Thomas
diff mbox series

Patch

diff --git a/package/petitboot/Config.in b/package/petitboot/Config.in
index 93ed3642c3..f931ab91cd 100644
--- a/package/petitboot/Config.in
+++ b/package/petitboot/Config.in
@@ -19,17 +19,20 @@  config BR2_PACKAGE_PETITBOOT
 
 	  http://www.kernel.org/pub/linux/kernel/people/geoff/petitboot/petitboot.html
 
+if BR2_PACKAGE_PETITBOOT
+
+config BR2_PACKAGE_PETITBOOT_GETTY_PORT
+	string "TTY port(s)"
+	default "console"
+	help
+	  Specify a space-separated list of ports to run the petitboot
+	  UI on. Wildcards are allowed. Example: "hvc* ttys0 ttyS*"
+
+endif # BR2_PACKAGE_PETITBOOT
+
 comment "petitboot needs a toolchain w/ wchar, dynamic library, threads, udev /dev management"
 	depends on BR2_PACKAGE_KEXEC_ARCH_SUPPORTS
 	depends on BR2_USE_MMU
 	depends on !BR2_USE_WCHAR || BR2_STATIC_LIBS || \
 		!BR2_TOOLCHAIN_HAS_THREADS || \
 		!BR2_PACKAGE_HAS_UDEV
-
-config BR2_PACKAGE_PETITBOOT_GETTY_PORT
-	string "TTY port(s)"
-	default "console"
-	depends on BR2_PACKAGE_PETITBOOT
-	help
-	  Specify a space-separated list of ports to run the petitboot
-	  UI on. Wildcards are allowed. Example: "hvc* ttys0 ttyS*"