Skip to contentSkip to conversation

busybox: add comment that other packages are needed

PATCH New Arnout Vandecappelle (Essensium/Mind) 2 participants 2 messages Jan 17, 2017 – Jan 20, 2017

lore
Share & tools
lore

Arnout Vandecappelle (Essensium/Mind)

author
#0 lore raw

PATCH 1/1 busybox: add comment that other packages are needed

Inspired on the confusion in [1], we want to warn the user that building without busybox may not lead to a working system.

[1] https://bugs.busybox.net/show_bug.cgi?id=9526

Signed-off-by Arnout Vandecappelle (Essensium/Mind) <[email protected]>

---
Better wording is certainly welcome!

---
 package/busybox/Config.in | 3 +++
 1 file changed, 3 insertions(+)
package/busybox/Config.in +3 -0
index 7ef3f0eda8..504cd8a65a 100644
@@ -77,6 +77,9 @@ endif
 
 if !BR2_PACKAGE_BUSYBOX # kconfig doesn't support else
 
+comment "You may need to enable other packages to get a working system"
+comment "You better know what you're doing!"
+
 # This option is not an option of Busybox, it can be selected even if
 # Busybox is not enabled. This dummy option ensures that packages that
 # depend on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS are visible when Busybox
Signature · 1 line
2.11.0
Applied
#1 lore raw

PATCH 1/1 busybox: add comment that other packages are needed

Hello,

Quoted 11 lines · Arnout Vandecappelle (Essensium/Mind)

Inspired on the confusion in [1], we want to warn the user that building without busybox may not lead to a working system.

[1] https://bugs.busybox.net/show_bug.cgi?id=9526

Signed-off-by Arnout Vandecappelle (Essensium/Mind) <[email protected]>

---
Better wording is certainly welcome!

---
 package/busybox/Config.in | 3 +++
 1 file changed, 3 insertions(+)

Applied to master, thanks.

Thomas

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