From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann E. MORIN Date: Thu, 10 Mar 2016 21:29:13 +0100 Subject: [Buildroot] [PATCH 5/8] linux: add the generic help rules In-Reply-To: <56E0B5F5.9040109@mind.be> References: <56E0B5F5.9040109@mind.be> Message-ID: <20160310202913.GE3424@free.fr> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On 2016-03-10 00:47 +0100, Arnout Vandecappelle spake thusly: > On 03/09/16 23:22, Yann E. MORIN wrote: > >Signed-off-by: "Yann E. MORIN" > >Cc: J?r?me Pouiller > >Cc: Arnout Vandecappelle > >Cc: Thomas Petazzoni > >--- > > Makefile | 6 ------ > > linux/linux.mk | 6 ++++++ > > 2 files changed, 6 insertions(+), 6 deletions(-) > > > >diff --git a/Makefile b/Makefile > >index 8134a14..cdf2dac 100644 > >--- a/Makefile > >+++ b/Makefile > >@@ -933,12 +933,6 @@ help: > > @echo ' -reconfigure - Restart the build from the configure step' > > @echo ' -rebuild - Restart the build from the build step' > > @$(call HELP_RULE,$(PACKAGE_HELP)) > >-ifeq ($(BR2_LINUX_KERNEL),y) > >- @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' > >-endif > > ifeq ($(BR2_TOOLCHAIN_BUILDROOT),y) > > @echo ' uclibc-menuconfig - Run uClibc menuconfig' > > endif > >diff --git a/linux/linux.mk b/linux/linux.mk > >index 7e20255..d530d00 100644 > >--- a/linux/linux.mk > >+++ b/linux/linux.mk > >@@ -8,6 +8,12 @@ LINUX_VERSION = $(call qstrip,$(BR2_LINUX_KERNEL_VERSION)) > > LINUX_LICENSE = GPLv2 > > LINUX_LICENSE_FILES = COPYING > > > >+LINUX_HELP = \ > >+ "linux-menuconfig : Run Linux kernel menuconfig" \ > >+ "linux-savedefconfig : Run Linux kernel savedefconfig" \ > >+ "linux-update-defconfig : Save the Linux configuration to the path specified \ > >+ by BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE" > > So now this will be printed on a single line, instead of being split like > it was before. Well, J?r?me's patch already did that as well I guess. > > Fixing that properly would make the HELP_RULE a bit too complicated, but > starting the continuation line with " : by..." doesn't look too shabby. > > I was also thinking that a define would have been better, but that doesn't > work unfortunately. I have fixed it here. It works for any arbitrary number of needed extra lines. /me expects some flaming for a trivial sed expression... ;-) Regards, Yann E. MORIN. > Regards, > Arnout > > >+ > > # 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)) > > > > > -- > Arnout Vandecappelle arnout at mind be > Senior Embedded Software Architect +32-16-286500 > Essensium/Mind http://www.mind.be > G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven > LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle > GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF -- .-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'