From e6b3913cfc4ec7a9f737674a8dcb07ea4155038a Mon Sep 17 00:00:00 2001 From: "Yann E. MORIN" Date: Sat, 24 Jul 2021 23:19:22 +0200 Subject: [PATCH] package: use the generic _HELP_CMDS for kconfig-based packages As Thomas put it: The _HELP_CMDS variable allows packages using the kconfig-package infrastructure to display their specific targets related to the handling of their configuration. However, it was not consistently used and handled by the different packages. So, this commit switches all the kconfig-based package to use the generic help helper. As a consequence: - all kconfig packages now advetise their kconfig-related actions, where some were previously missing: at91bootstrap3, linux-backports, swupdate, xvisor; - busybox advertises it does not support defconfig files; - the 'foo-savedfconfig' action is no longer advertised: it is to be considered an internal implementation detail. Signed-off-by: Yann E. MORIN Cc: Thomas Petazzoni Signed-off-by: Arnout Vandecappelle (Essensium/Mind) --- boot/barebox/barebox/barebox.mk | 5 ----- boot/uboot/uboot.mk | 6 ------ linux/linux.mk | 7 ------- package/busybox/busybox.mk | 5 +---- package/uclibc/uclibc.mk | 4 ---- 5 files changed, 1 insertion(+), 26 deletions(-) diff --git a/boot/barebox/barebox/barebox.mk b/boot/barebox/barebox/barebox.mk index 39afb0fccb..6a5a80de34 100644 --- a/boot/barebox/barebox/barebox.mk +++ b/boot/barebox/barebox/barebox.mk @@ -4,10 +4,5 @@ # ################################################################################ -define BAREBOX_HELP_CMDS - @echo ' barebox-menuconfig - Run barebox menuconfig' - @echo ' barebox-savedefconfig - Run barebox savedefconfig' -endef - # Instantiate the barebox package $(eval $(barebox-package)) diff --git a/boot/uboot/uboot.mk b/boot/uboot/uboot.mk index 9c2261963d..4f3c9b7c3a 100644 --- a/boot/uboot/uboot.mk +++ b/boot/uboot/uboot.mk @@ -296,12 +296,6 @@ UBOOT_KCONFIG_EDITORS = menuconfig xconfig gconfig nconfig # override again. In addition, host-ccache is not ready at kconfig # time, so use HOSTCC_NOCCACHE. UBOOT_KCONFIG_OPTS = $(UBOOT_MAKE_OPTS) HOSTCC="$(HOSTCC_NOCCACHE)" HOSTLDFLAGS="" -define UBOOT_HELP_CMDS - @echo ' uboot-menuconfig - Run U-Boot menuconfig' - @echo ' uboot-savedefconfig - Run U-Boot savedefconfig' - @echo ' uboot-update-defconfig - Save the U-Boot configuration to the path specified' - @echo ' by BR2_TARGET_UBOOT_CUSTOM_CONFIG_FILE' -endef endif # BR2_TARGET_UBOOT_BUILD_SYSTEM_LEGACY UBOOT_CUSTOM_DTS_PATH = $(call qstrip,$(BR2_TARGET_UBOOT_CUSTOM_DTS_PATH)) diff --git a/linux/linux.mk b/linux/linux.mk index 1457228eb9..61fdc0c76c 100644 --- a/linux/linux.mk +++ b/linux/linux.mk @@ -16,13 +16,6 @@ LINUX_CPE_ID_VENDOR = linux LINUX_CPE_ID_PRODUCT = linux_kernel LINUX_CPE_ID_PREFIX = cpe:2.3:o -define LINUX_HELP_CMDS - @echo ' linux-menuconfig - Run Linux kernel menuconfig' - @echo ' linux-savedefconfig - Run Linux kernel savedefconfig' - @echo ' linux-update-defconfig - Save the Linux configuration to the path specified' - @echo ' by BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE' -endef - # Compute LINUX_SOURCE and LINUX_SITE from the configuration ifeq ($(BR2_LINUX_KERNEL_CUSTOM_TARBALL),y) LINUX_TARBALL = $(call qstrip,$(BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION)) diff --git a/package/busybox/busybox.mk b/package/busybox/busybox.mk index 413939e28d..68e0b8a72c 100644 --- a/package/busybox/busybox.mk +++ b/package/busybox/busybox.mk @@ -11,10 +11,6 @@ BUSYBOX_LICENSE = GPL-2.0, bzip2-1.0.4 BUSYBOX_LICENSE_FILES = LICENSE archival/libarchive/bz/LICENSE BUSYBOX_CPE_ID_VENDOR = busybox -define BUSYBOX_HELP_CMDS - @echo ' busybox-menuconfig - Run BusyBox menuconfig' -endef - BUSYBOX_CFLAGS = \ $(TARGET_CFLAGS) @@ -107,6 +103,7 @@ ifndef BUSYBOX_CONFIG_FILE BUSYBOX_CONFIG_FILE = $(call qstrip,$(BR2_PACKAGE_BUSYBOX_CONFIG)) endif +BUSYBOX_KCONFIG_HAS_DEFCONFIG = NO BUSYBOX_KCONFIG_FILE = $(BUSYBOX_CONFIG_FILE) BUSYBOX_KCONFIG_FRAGMENT_FILES = $(call qstrip,$(BR2_PACKAGE_BUSYBOX_CONFIG_FRAGMENT_FILES)) BUSYBOX_KCONFIG_EDITORS = menuconfig xconfig gconfig diff --git a/package/uclibc/uclibc.mk b/package/uclibc/uclibc.mk index a7c96684d0..5c17eea8dd 100644 --- a/package/uclibc/uclibc.mk +++ b/package/uclibc/uclibc.mk @@ -13,10 +13,6 @@ UCLIBC_INSTALL_STAGING = YES UCLIBC_CPE_ID_VENDOR = uclibc-ng_project UCLIBC_CPE_ID_PRODUCT = uclibc-ng -define UCLIBC_HELP_CMDS - @echo ' uclibc-menuconfig - Run uClibc menuconfig' -endef - # uclibc is part of the toolchain so disable the toolchain dependency UCLIBC_ADD_TOOLCHAIN_DEPENDENCY = NO -- 2.30.2