All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit branch/2020.02.x] package/busybox: enable flags for use as systemd pager
@ 2020-08-28 14:47 Peter Korsgaard
  2020-08-30  6:22 ` François Perrad
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Korsgaard @ 2020-08-28 14:47 UTC (permalink / raw)
  To: buildroot

commit: https://git.buildroot.net/buildroot/commit/?id=9faba29108e74eb4acab21f5919dfab0288b23ac
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/2020.02.x

If the less package is not enable and systemd is enabled,
then configure the less applet to fully work with systemd.

systemd sets the flags for less in an environment variable
and requires a few options for correct display.

Signed-off-by: Norbert Lange <nolange79@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
(cherry picked from commit c2caf816e9296c4812f83f0a5d16e0e33a264b72)
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 package/busybox/busybox.mk | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/package/busybox/busybox.mk b/package/busybox/busybox.mk
index b9d82ed71e..55aa9b0bb1 100644
--- a/package/busybox/busybox.mk
+++ b/package/busybox/busybox.mk
@@ -237,6 +237,18 @@ define BUSYBOX_SET_SELINUX
 endef
 endif
 
+# enable relevant options to allow the Busybox less applet to be used
+# as a systemd pager
+ifeq ($(BR2_PACKAGE_SYSTEMD):$(BR2_PACKAGE_LESS),y:)
+define BUSYBOX_SET_LESS_FLAGS
+	$(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_DASHCMD)
+	$(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_RAW)
+	$(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_TRUNCATE)
+	$(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_FLAGS)
+	$(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_ENV)
+endef
+endif
+
 ifeq ($(BR2_PACKAGE_BUSYBOX_INDIVIDUAL_BINARIES),y)
 define BUSYBOX_SET_INDIVIDUAL_BINARIES
 	$(call KCONFIG_ENABLE_OPT,CONFIG_BUILD_LIBBUSYBOX,$(BUSYBOX_BUILD_CONFIG))
@@ -338,6 +350,7 @@ define BUSYBOX_KCONFIG_FIXUP_CMDS
 	$(BUSYBOX_SET_INIT)
 	$(BUSYBOX_SET_WATCHDOG)
 	$(BUSYBOX_SET_SELINUX)
+	$(BUSYBOX_SET_LESS_FLAGS)
 	$(BUSYBOX_SET_INDIVIDUAL_BINARIES)
 endef
 

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [git commit branch/2020.02.x] package/busybox: enable flags for use as systemd pager
  2020-08-28 14:47 [Buildroot] [git commit branch/2020.02.x] package/busybox: enable flags for use as systemd pager Peter Korsgaard
@ 2020-08-30  6:22 ` François Perrad
  2020-08-30  6:52   ` Peter Korsgaard
  0 siblings, 1 reply; 3+ messages in thread
From: François Perrad @ 2020-08-30  6:22 UTC (permalink / raw)
  To: buildroot

Le ven. 28 ao?t 2020 ? 18:00, Peter Korsgaard <peter@korsgaard.com> a
?crit :

> commit:
> https://git.buildroot.net/buildroot/commit/?id=9faba29108e74eb4acab21f5919dfab0288b23ac
> branch:
> https://git.buildroot.net/buildroot/commit/?id=refs/heads/2020.02.x
>
> If the less package is not enable and systemd is enabled,
> then configure the less applet to fully work with systemd.
>
> systemd sets the flags for less in an environment variable
> and requires a few options for correct display.
>
> Signed-off-by: Norbert Lange <nolange79@gmail.com>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> (cherry picked from commit c2caf816e9296c4812f83f0a5d16e0e33a264b72)
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  package/busybox/busybox.mk | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> diff --git a/package/busybox/busybox.mk b/package/busybox/busybox.mk
> index b9d82ed71e..55aa9b0bb1 100644
> --- a/package/busybox/busybox.mk
> +++ b/package/busybox/busybox.mk
> @@ -237,6 +237,18 @@ define BUSYBOX_SET_SELINUX
>  endef
>  endif
>
> +# enable relevant options to allow the Busybox less applet to be used
> +# as a systemd pager
> +ifeq ($(BR2_PACKAGE_SYSTEMD):$(BR2_PACKAGE_LESS),y:)
> +define BUSYBOX_SET_LESS_FLAGS
> +       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_DASHCMD)
>

the backport of this patch on the branch 2020.02.x breaks the build of
busybox.
on 2020.02.x, an additional parameter is required with the value :
$(BUSYBOX_BUILD_CONFIG)

note: the backport on 2020.05.x is OK

Fran?ois

+       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_RAW)
> +       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_TRUNCATE)
> +       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_FLAGS)
> +       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_ENV)
> +endef
> +endif
> +
>  ifeq ($(BR2_PACKAGE_BUSYBOX_INDIVIDUAL_BINARIES),y)
>  define BUSYBOX_SET_INDIVIDUAL_BINARIES
>         $(call
> KCONFIG_ENABLE_OPT,CONFIG_BUILD_LIBBUSYBOX,$(BUSYBOX_BUILD_CONFIG))
> @@ -338,6 +350,7 @@ define BUSYBOX_KCONFIG_FIXUP_CMDS
>         $(BUSYBOX_SET_INIT)
>         $(BUSYBOX_SET_WATCHDOG)
>         $(BUSYBOX_SET_SELINUX)
> +       $(BUSYBOX_SET_LESS_FLAGS)
>         $(BUSYBOX_SET_INDIVIDUAL_BINARIES)
>  endef
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20200830/63925ecf/attachment.html>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [git commit branch/2020.02.x] package/busybox: enable flags for use as systemd pager
  2020-08-30  6:22 ` François Perrad
@ 2020-08-30  6:52   ` Peter Korsgaard
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2020-08-30  6:52 UTC (permalink / raw)
  To: buildroot

>>>>> "Fran?ois" == Fran?ois Perrad <francois.perrad@gadz.org> writes:

 > Le ven. 28 ao?t 2020 ? 18:00, Peter Korsgaard <peter@korsgaard.com> a
 > ?crit :

 >> commit:
 >> https://git.buildroot.net/buildroot/commit/?id=9faba29108e74eb4acab21f5919dfab0288b23ac
 >> branch:
 >> https://git.buildroot.net/buildroot/commit/?id=refs/heads/2020.02.x
 >> 
 >> If the less package is not enable and systemd is enabled,
 >> then configure the less applet to fully work with systemd.
 >> 
 >> systemd sets the flags for less in an environment variable
 >> and requires a few options for correct display.
 >> 
 >> Signed-off-by: Norbert Lange <nolange79@gmail.com>
 >> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
 >> (cherry picked from commit c2caf816e9296c4812f83f0a5d16e0e33a264b72)
 >> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
 >> ---
 >> package/busybox/busybox.mk | 13 +++++++++++++
 >> 1 file changed, 13 insertions(+)
 >> 
 >> diff --git a/package/busybox/busybox.mk b/package/busybox/busybox.mk
 >> index b9d82ed71e..55aa9b0bb1 100644
 >> --- a/package/busybox/busybox.mk
 >> +++ b/package/busybox/busybox.mk
 >> @@ -237,6 +237,18 @@ define BUSYBOX_SET_SELINUX
 >> endef
 >> endif
 >> 
 >> +# enable relevant options to allow the Busybox less applet to be used
 >> +# as a systemd pager
 >> +ifeq ($(BR2_PACKAGE_SYSTEMD):$(BR2_PACKAGE_LESS),y:)
 >> +define BUSYBOX_SET_LESS_FLAGS
 >> +       $(call KCONFIG_ENABLE_OPT,CONFIG_FEATURE_LESS_DASHCMD)
 >> 

 > the backport of this patch on the branch 2020.02.x breaks the build of
 > busybox.
 > on 2020.02.x, an additional parameter is required with the value :
 > $(BUSYBOX_BUILD_CONFIG)

Crap, and I just released 2020.02.5 :/ Strange that we didn't have any
autobuilder issues so far.

I'll fix it, thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-08-30  6:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-28 14:47 [Buildroot] [git commit branch/2020.02.x] package/busybox: enable flags for use as systemd pager Peter Korsgaard
2020-08-30  6:22 ` François Perrad
2020-08-30  6:52   ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.