[Buildroot] [PATCH 5/8] linux: add the generic help rules
Yann E. MORIN
yann.morin.1998 at free.fr
Thu Mar 10 20:29:13 UTC 2016
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" <yann.morin.1998 at free.fr>
> >Cc: Jérôme Pouiller <jezz at sysmic.org>
> >Cc: Arnout Vandecappelle <arnout at mind.be>
> >Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> >---
> > 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 ' <pkg>-reconfigure - Restart the build from the configure step'
> > @echo ' <pkg>-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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list