[openwrt/openwrt] base-files: fix backwards compatiblity of rc.common EXTRA_COMMANDS

LEDE Commits lede-commits at lists.infradead.org
Tue Nov 17 11:59:35 EST 2020


neoraider pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/3f1109bf2af2cf4977689a33ea481afaaac8dc98

commit 3f1109bf2af2cf4977689a33ea481afaaac8dc98
Author: Matthias Schiffer <mschiffer at universe-factory.net>
AuthorDate: Mon Nov 16 17:30:43 2020 +0100

    base-files: fix backwards compatiblity of rc.common EXTRA_COMMANDS
    
    Avoid needlessly breaking old initscripts that set EXTRA_COMMANDS. This
    will aid in debugging (as it simplifies reverting to an older version of
    a package) and unbreaks third-party feeds (and packages that maintain
    their OpenWrt initscripts as part of the software's repo instead of the
    OpenWrt feed like fastd).
    
    Without this, initscripts that set EXTRA_COMMANDS become completely
    unusable, as all default commands like start/stop cease working.
    
    Fixes: 1a69f50dc627 ("base-files: fix rc.common help alignment")
    Signed-off-by: Matthias Schiffer <mschiffer at universe-factory.net>
---
 package/base-files/Makefile            |  2 +-
 package/base-files/files/etc/rc.common | 13 +++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/package/base-files/Makefile b/package/base-files/Makefile
index b546eb7e0b..e13578860f 100644
--- a/package/base-files/Makefile
+++ b/package/base-files/Makefile
@@ -12,7 +12,7 @@ include $(INCLUDE_DIR)/version.mk
 include $(INCLUDE_DIR)/feeds.mk
 
 PKG_NAME:=base-files
-PKG_RELEASE:=236
+PKG_RELEASE:=237
 PKG_FLAGS:=nonshared
 
 PKG_FILE_DEPENDS:=$(PLATFORM_DIR)/ $(GENERIC_PLATFORM_DIR)/base-files/
diff --git a/package/base-files/files/etc/rc.common b/package/base-files/files/etc/rc.common
index d3d91ef250..f39b69464e 100755
--- a/package/base-files/files/etc/rc.common
+++ b/package/base-files/files/etc/rc.common
@@ -62,15 +62,15 @@ depends() {
 	return 0
 }
 
-EXTRA_HELP=""
-EXTRA_COMMANDS="boot shutdown depends"
+ALL_HELP=""
+ALL_COMMANDS="boot shutdown depends"
 extra_command() {
 	local cmd="$1"
 	local help="$2"
 
 	local extra="$(printf "%-16s%s" "${cmd}" "${help}")"
-	EXTRA_HELP="${EXTRA_HELP}\t${extra}\n"
-	EXTRA_COMMANDS="${EXTRA_COMMANDS} ${cmd}"
+	ALL_HELP="${ALL_HELP}\t${extra}\n"
+	ALL_COMMANDS="${ALL_COMMANDS} ${cmd}"
 }
 
 help() {
@@ -79,7 +79,7 @@ Syntax: $initscript [command]
 
 Available commands:
 EOF
-	echo -e "$EXTRA_HELP"
+	echo -e "$ALL_HELP"
 }
 
 # for procd
@@ -175,6 +175,7 @@ extra_command "enabled" "Check if service is started on boot"
 	}
 }
 
-ALL_COMMANDS="${EXTRA_COMMANDS}"
+ALL_COMMANDS="${ALL_COMMANDS} ${EXTRA_COMMANDS}"
+ALL_HELP="${ALL_HELP}${EXTRA_HELP}"
 list_contains ALL_COMMANDS "$action" || action=help
 $action "$@"



More information about the lede-commits mailing list