[Buildroot] [PATCH v3 4/9] package/screen: add /usr/bin/screen to /etc/shells
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Thu May 3 21:27:48 UTC 2018
Hello,
On Sat, 3 Feb 2018 15:58:48 +0100, Yann E. MORIN wrote:
> > diff --git a/package/screen/screen.mk b/package/screen/screen.mk
> > index 8d67c04..37d4336 100644
> > --- a/package/screen/screen.mk
> > +++ b/package/screen/screen.mk
> > @@ -17,7 +17,14 @@ SCREEN_INSTALL_TARGET_OPTS = DESTDIR=$(TARGET_DIR) SCREEN=screen install_bin
> > define SCREEN_INSTALL_SCREENRC
> > $(INSTALL) -m 0755 -D $(@D)/etc/screenrc $(TARGET_DIR)/etc/screenrc
> > endef
> > -
>
> Spurious empty line removal.
Indeed, but I believe it's fairly OK, as it allows the hook addition
below to follow the coding style. I've updated the commit log to
include the following paragraph:
While at it, drop an empty line between an existing hook definition
and its registration, to be consistent with the coding style used in
the rest of Buildroot.
Thanks for the review!
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list