* [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH
@ 2018-12-19 21:04 Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH Markus Mayer
` (3 more replies)
0 siblings, 4 replies; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 21:04 UTC (permalink / raw)
To: buildroot
This series introduces a common way of configuring the system's default
PATH.
We introduce a Kconfig variable called BR2_SYSTEM_DEFAULT_PATH, which
is being used in /etc/profile of the default skeleton and for dropbear
and openssh sessions.
Changes since v1:
- only use a single default (independent of BR2_ROOTFS_MERGED_USR)
- add a guard to system/system.mk checking the default path isn't empty
- replace PATH in etc/profile with place-holder @PATH@ and update regex
- use BR2_SYSTEM_DEFAULT_PATH unconditionally for dropbear
- update commit messages that BR2_SYSTEM_DEFAULT_PATH is a Kconfig variable
and already quoted (which is what we need)
Markus Mayer (4):
system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH
skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH
openssh: use BR2_SYSTEM_DEFAULT_PATH as default PATH
dropbear: use BR2_SYSTEM_DEFAULT_PATH as default PATH
package/dropbear/dropbear.mk | 6 ++++++
package/openssh/openssh.mk | 1 +
package/skeleton-init-common/skeleton-init-common.mk | 1 +
system/Config.in | 9 +++++++++
system/skeleton/etc/profile | 2 +-
system/system.mk | 4 ++++
6 files changed, 22 insertions(+), 1 deletion(-)
--
2.17.1
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH
2018-12-19 21:04 [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH Markus Mayer
@ 2018-12-19 21:04 ` Markus Mayer
2018-12-19 21:32 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH Markus Mayer
` (2 subsequent siblings)
3 siblings, 1 reply; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 21:04 UTC (permalink / raw)
To: buildroot
The configuration option BR2_SYSTEM_DEFAULT_PATH allows the user to
override the default path, which can be used by /etc/profile and some
system daemons.
It defaults to the value previously hard-coded in /etc/profile. This
default should be suitable for most users.
Signed-off-by: Markus Mayer <mmayer@broadcom.com>
---
system/Config.in | 9 +++++++++
system/system.mk | 4 ++++
2 files changed, 13 insertions(+)
diff --git a/system/Config.in b/system/Config.in
index 0f77b9b6721a..afae487a4bd5 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -386,6 +386,15 @@ comment "automatic network configuration via DHCP needs ifupdown or busybox or n
endif # BR2_ROOTFS_SKELETON_DEFAULT
+config BR2_SYSTEM_DEFAULT_PATH
+ string "Set the system's default PATH"
+ default "/bin:/sbin:/usr/bin:/usr/sbin"
+ help
+ Sets the system's default PATH. It is being used in /etc/profile
+ in the skeleton-init-common package and by some daemons.
+
+ The default should work in most cases.
+
config BR2_ENABLE_LOCALE_PURGE
bool "Purge unwanted locales"
default y
diff --git a/system/system.mk b/system/system.mk
index ca6bf1388f39..8db87cb12930 100644
--- a/system/system.mk
+++ b/system/system.mk
@@ -86,3 +86,7 @@ define SYSTEM_REMOUNT_ROOT_INITTAB
$(SED) '/^[^#].*-o remount,rw \/$$/s~^~#~' $(TARGET_DIR)/etc/inittab
endef
endif
+
+ifeq ($(BR_BUILDING)$(BR2_SYSTEM_DEFAULT_PATH),y"")
+$(error BR2_SYSTEM_DEFAULT_PATH can't be empty)
+endif
--
2.17.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH Markus Mayer
@ 2018-12-19 21:04 ` Markus Mayer
2018-12-19 21:34 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 3/4] openssh: " Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 4/4] dropbear: " Markus Mayer
3 siblings, 1 reply; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 21:04 UTC (permalink / raw)
To: buildroot
We substitute the path specified in system/skeleton/etc/profile with
the path specified in the configuration variable
$(BR2_SYSTEM_DEFAULT_PATH).
$(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
quoted, which is exactly what we want.
Signed-off-by: Markus Mayer <mmayer@broadcom.com>
---
package/skeleton-init-common/skeleton-init-common.mk | 1 +
system/skeleton/etc/profile | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/package/skeleton-init-common/skeleton-init-common.mk b/package/skeleton-init-common/skeleton-init-common.mk
index e8a052205268..866273bbd7ee 100644
--- a/package/skeleton-init-common/skeleton-init-common.mk
+++ b/package/skeleton-init-common/skeleton-init-common.mk
@@ -22,6 +22,7 @@ define SKELETON_INIT_COMMON_INSTALL_TARGET_CMDS
$(call SYSTEM_LIB_SYMLINK,$(TARGET_DIR))
$(INSTALL) -m 0644 support/misc/target-dir-warning.txt \
$(TARGET_DIR_WARNING_FILE)
+ $(SED) 's, at PATH@,$(BR2_SYSTEM_DEFAULT_PATH),' $(TARGET_DIR)/etc/profile
endef
# We don't care much about what goes in staging, as long as it is
diff --git a/system/skeleton/etc/profile b/system/skeleton/etc/profile
index 1255d23ff40d..db29e44920e8 100644
--- a/system/skeleton/etc/profile
+++ b/system/skeleton/etc/profile
@@ -1,4 +1,4 @@
-export PATH=/bin:/sbin:/usr/bin:/usr/sbin
+export PATH=@PATH@
if [ "$PS1" ]; then
if [ "`id -u`" -eq 0 ]; then
--
2.17.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 3/4] openssh: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH Markus Mayer
@ 2018-12-19 21:04 ` Markus Mayer
2018-12-19 21:34 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 4/4] dropbear: " Markus Mayer
3 siblings, 1 reply; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 21:04 UTC (permalink / raw)
To: buildroot
We use the configuration option $(BR2_SYSTEM_DEFAULT_PATH) to set the
default PATH in OpenSSH sessions.
$(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
quoted, which is exactly what we want.
Signed-off-by: Markus Mayer <mmayer@broadcom.com>
---
package/openssh/openssh.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/openssh/openssh.mk b/package/openssh/openssh.mk
index 07f3e0d663f6..4ee7727fb9e8 100644
--- a/package/openssh/openssh.mk
+++ b/package/openssh/openssh.mk
@@ -11,6 +11,7 @@ OPENSSH_LICENSE_FILES = LICENCE
OPENSSH_CONF_ENV = LD="$(TARGET_CC)" LDFLAGS="$(TARGET_CFLAGS)"
OPENSSH_CONF_OPTS = \
--sysconfdir=/etc/ssh \
+ --with-default-path=$(BR2_SYSTEM_DEFAULT_PATH) \
--disable-lastlog \
--disable-utmp \
--disable-utmpx \
--
2.17.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 4/4] dropbear: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH Markus Mayer
` (2 preceding siblings ...)
2018-12-19 21:04 ` [Buildroot] [PATCH v2 3/4] openssh: " Markus Mayer
@ 2018-12-19 21:04 ` Markus Mayer
2018-12-19 21:38 ` Yann E. MORIN
3 siblings, 1 reply; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 21:04 UTC (permalink / raw)
To: buildroot
We use the configuration option $(BR2_SYSTEM_DEFAULT_PATH) to set the
default PATH in dropbear sessions.
$(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
quoted, which is exactly what we want.
Signed-off-by: Markus Mayer <mmayer@broadcom.com>
---
package/dropbear/dropbear.mk | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
index 7b1468cfb100..a5a8243bd431 100644
--- a/package/dropbear/dropbear.mk
+++ b/package/dropbear/dropbear.mk
@@ -81,6 +81,12 @@ define DROPBEAR_DISABLE_STANDALONE
echo '#define NON_INETD_MODE 0' >> $(@D)/localoptions.h
endef
+define DROPBEAR_CUSTOM_PATH
+ echo '#define DEFAULT_PATH $(BR2_SYSTEM_DEFAULT_PATH)' >>$(@D)/localoptions.h
+endef
+
+DROPBEAR_POST_EXTRACT_HOOKS += DROPBEAR_CUSTOM_PATH
+
define DROPBEAR_INSTALL_INIT_SYSTEMD
$(INSTALL) -D -m 644 package/dropbear/dropbear.service \
$(TARGET_DIR)/usr/lib/systemd/system/dropbear.service
--
2.17.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH
2018-12-19 21:04 ` [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH Markus Mayer
@ 2018-12-19 21:32 ` Yann E. MORIN
0 siblings, 0 replies; 11+ messages in thread
From: Yann E. MORIN @ 2018-12-19 21:32 UTC (permalink / raw)
To: buildroot
Markus, All,
On 2018-12-19 13:04 -0800, Markus Mayer spake thusly:
> The configuration option BR2_SYSTEM_DEFAULT_PATH allows the user to
> override the default path, which can be used by /etc/profile and some
> system daemons.
>
> It defaults to the value previously hard-coded in /etc/profile. This
> default should be suitable for most users.
>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
> ---
> system/Config.in | 9 +++++++++
> system/system.mk | 4 ++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/system/Config.in b/system/Config.in
> index 0f77b9b6721a..afae487a4bd5 100644
> --- a/system/Config.in
> +++ b/system/Config.in
> @@ -386,6 +386,15 @@ comment "automatic network configuration via DHCP needs ifupdown or busybox or n
>
> endif # BR2_ROOTFS_SKELETON_DEFAULT
>
> +config BR2_SYSTEM_DEFAULT_PATH
> + string "Set the system's default PATH"
> + default "/bin:/sbin:/usr/bin:/usr/sbin"
> + help
> + Sets the system's default PATH. It is being used in /etc/profile
Slight formatting issue here, check-package conplains:
system/Config.in:393: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in)
Otherwise (no need to respin, that can be fixed when applying I hope):
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> + in the skeleton-init-common package and by some daemons.
> +
> + The default should work in most cases.
> +
> config BR2_ENABLE_LOCALE_PURGE
> bool "Purge unwanted locales"
> default y
> diff --git a/system/system.mk b/system/system.mk
> index ca6bf1388f39..8db87cb12930 100644
> --- a/system/system.mk
> +++ b/system/system.mk
> @@ -86,3 +86,7 @@ define SYSTEM_REMOUNT_ROOT_INITTAB
> $(SED) '/^[^#].*-o remount,rw \/$$/s~^~#~' $(TARGET_DIR)/etc/inittab
> endef
> endif
> +
> +ifeq ($(BR_BUILDING)$(BR2_SYSTEM_DEFAULT_PATH),y"")
> +$(error BR2_SYSTEM_DEFAULT_PATH can't be empty)
> +endif
> --
> 2.17.1
>
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 ` [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH Markus Mayer
@ 2018-12-19 21:34 ` Yann E. MORIN
2018-12-19 23:24 ` Markus Mayer
0 siblings, 1 reply; 11+ messages in thread
From: Yann E. MORIN @ 2018-12-19 21:34 UTC (permalink / raw)
To: buildroot
Markus, All,
On 2018-12-19 13:04 -0800, Markus Mayer spake thusly:
> We substitute the path specified in system/skeleton/etc/profile with
> the path specified in the configuration variable
> $(BR2_SYSTEM_DEFAULT_PATH).
>
> $(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
> quoted, which is exactly what we want.
>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
> ---
> package/skeleton-init-common/skeleton-init-common.mk | 1 +
> system/skeleton/etc/profile | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/package/skeleton-init-common/skeleton-init-common.mk b/package/skeleton-init-common/skeleton-init-common.mk
> index e8a052205268..866273bbd7ee 100644
> --- a/package/skeleton-init-common/skeleton-init-common.mk
> +++ b/package/skeleton-init-common/skeleton-init-common.mk
> @@ -22,6 +22,7 @@ define SKELETON_INIT_COMMON_INSTALL_TARGET_CMDS
> $(call SYSTEM_LIB_SYMLINK,$(TARGET_DIR))
> $(INSTALL) -m 0644 support/misc/target-dir-warning.txt \
> $(TARGET_DIR_WARNING_FILE)
> + $(SED) 's, at PATH@,$(BR2_SYSTEM_DEFAULT_PATH),' $(TARGET_DIR)/etc/profile
I think we should keep the warning file the last command executed.
Otherwise:
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> endef
>
> # We don't care much about what goes in staging, as long as it is
> diff --git a/system/skeleton/etc/profile b/system/skeleton/etc/profile
> index 1255d23ff40d..db29e44920e8 100644
> --- a/system/skeleton/etc/profile
> +++ b/system/skeleton/etc/profile
> @@ -1,4 +1,4 @@
> -export PATH=/bin:/sbin:/usr/bin:/usr/sbin
> +export PATH=@PATH@
>
> if [ "$PS1" ]; then
> if [ "`id -u`" -eq 0 ]; then
> --
> 2.17.1
>
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 3/4] openssh: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 ` [Buildroot] [PATCH v2 3/4] openssh: " Markus Mayer
@ 2018-12-19 21:34 ` Yann E. MORIN
0 siblings, 0 replies; 11+ messages in thread
From: Yann E. MORIN @ 2018-12-19 21:34 UTC (permalink / raw)
To: buildroot
Markus, All,
On 2018-12-19 13:04 -0800, Markus Mayer spake thusly:
> We use the configuration option $(BR2_SYSTEM_DEFAULT_PATH) to set the
> default PATH in OpenSSH sessions.
>
> $(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
> quoted, which is exactly what we want.
>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> package/openssh/openssh.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/openssh/openssh.mk b/package/openssh/openssh.mk
> index 07f3e0d663f6..4ee7727fb9e8 100644
> --- a/package/openssh/openssh.mk
> +++ b/package/openssh/openssh.mk
> @@ -11,6 +11,7 @@ OPENSSH_LICENSE_FILES = LICENCE
> OPENSSH_CONF_ENV = LD="$(TARGET_CC)" LDFLAGS="$(TARGET_CFLAGS)"
> OPENSSH_CONF_OPTS = \
> --sysconfdir=/etc/ssh \
> + --with-default-path=$(BR2_SYSTEM_DEFAULT_PATH) \
> --disable-lastlog \
> --disable-utmp \
> --disable-utmpx \
> --
> 2.17.1
>
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 4/4] dropbear: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:04 ` [Buildroot] [PATCH v2 4/4] dropbear: " Markus Mayer
@ 2018-12-19 21:38 ` Yann E. MORIN
0 siblings, 0 replies; 11+ messages in thread
From: Yann E. MORIN @ 2018-12-19 21:38 UTC (permalink / raw)
To: buildroot
Markus, All,
On 2018-12-19 13:04 -0800, Markus Mayer spake thusly:
> We use the configuration option $(BR2_SYSTEM_DEFAULT_PATH) to set the
> default PATH in dropbear sessions.
>
> $(BR2_SYSTEM_DEFAULT_PATH) is a Kconfig string. So it is already
> quoted, which is exactly what we want.
>
> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> package/dropbear/dropbear.mk | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
> index 7b1468cfb100..a5a8243bd431 100644
> --- a/package/dropbear/dropbear.mk
> +++ b/package/dropbear/dropbear.mk
> @@ -81,6 +81,12 @@ define DROPBEAR_DISABLE_STANDALONE
> echo '#define NON_INETD_MODE 0' >> $(@D)/localoptions.h
> endef
>
> +define DROPBEAR_CUSTOM_PATH
> + echo '#define DEFAULT_PATH $(BR2_SYSTEM_DEFAULT_PATH)' >>$(@D)/localoptions.h
> +endef
> +
> +DROPBEAR_POST_EXTRACT_HOOKS += DROPBEAR_CUSTOM_PATH
> +
> define DROPBEAR_INSTALL_INIT_SYSTEMD
> $(INSTALL) -D -m 644 package/dropbear/dropbear.service \
> $(TARGET_DIR)/usr/lib/systemd/system/dropbear.service
> --
> 2.17.1
>
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 21:34 ` Yann E. MORIN
@ 2018-12-19 23:24 ` Markus Mayer
2018-12-20 16:04 ` Yann E. MORIN
0 siblings, 1 reply; 11+ messages in thread
From: Markus Mayer @ 2018-12-19 23:24 UTC (permalink / raw)
To: buildroot
Hi Yann,
On Wed, 19 Dec 2018 at 13:34, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
>
> Markus, All,
>
> > diff --git a/package/skeleton-init-common/skeleton-init-common.mk b/package/skeleton-init-common/skeleton-init-common.mk
> > index e8a052205268..866273bbd7ee 100644
> > --- a/package/skeleton-init-common/skeleton-init-common.mk
> > +++ b/package/skeleton-init-common/skeleton-init-common.mk
> > @@ -22,6 +22,7 @@ define SKELETON_INIT_COMMON_INSTALL_TARGET_CMDS
> > $(call SYSTEM_LIB_SYMLINK,$(TARGET_DIR))
> > $(INSTALL) -m 0644 support/misc/target-dir-warning.txt \
> > $(TARGET_DIR_WARNING_FILE)
> > + $(SED) 's, at PATH@,$(BR2_SYSTEM_DEFAULT_PATH),' $(TARGET_DIR)/etc/profile
>
> I think we should keep the warning file the last command executed.
I am not sure I am following. I am not doing anything to target-dir-warning.txt.
> Otherwise:
>
> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> Regards,
> Yann E. MORIN.
Thanks,
-Markus
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH
2018-12-19 23:24 ` Markus Mayer
@ 2018-12-20 16:04 ` Yann E. MORIN
0 siblings, 0 replies; 11+ messages in thread
From: Yann E. MORIN @ 2018-12-20 16:04 UTC (permalink / raw)
To: buildroot
Markus, All,
On 2018-12-19 15:24 -0800, Markus Mayer spake thusly:
> On Wed, 19 Dec 2018 at 13:34, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
> > > diff --git a/package/skeleton-init-common/skeleton-init-common.mk b/package/skeleton-init-common/skeleton-init-common.mk
> > > index e8a052205268..866273bbd7ee 100644
> > > --- a/package/skeleton-init-common/skeleton-init-common.mk
> > > +++ b/package/skeleton-init-common/skeleton-init-common.mk
> > > @@ -22,6 +22,7 @@ define SKELETON_INIT_COMMON_INSTALL_TARGET_CMDS
> > > $(call SYSTEM_LIB_SYMLINK,$(TARGET_DIR))
> > > $(INSTALL) -m 0644 support/misc/target-dir-warning.txt \
> > > $(TARGET_DIR_WARNING_FILE)
> > > + $(SED) 's, at PATH@,$(BR2_SYSTEM_DEFAULT_PATH),' $(TARGET_DIR)/etc/profile
> > I think we should keep the warning file the last command executed.
> I am not sure I am following. I am not doing anything to target-dir-warning.txt.
Just put your sed command before the warning file:
$(SED) 's, at PATH@,$(BR2_SYSTEM_DEFAULT_PATH),' $(TARGET_DIR)/etc/profile
$(INSTALL) -m 0644 support/misc/target-dir-warning.txt \
$(TARGET_DIR_WARNING_FILE)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2018-12-20 16:04 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-19 21:04 [Buildroot] [PATCH v2 0/4] Allow customization of system default PATH Markus Mayer
2018-12-19 21:04 ` [Buildroot] [PATCH v2 1/4] system cfg: introduce option BR2_SYSTEM_DEFAULT_PATH Markus Mayer
2018-12-19 21:32 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 2/4] skeleton: use BR2_SYSTEM_DEFAULT_PATH as default PATH Markus Mayer
2018-12-19 21:34 ` Yann E. MORIN
2018-12-19 23:24 ` Markus Mayer
2018-12-20 16:04 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 3/4] openssh: " Markus Mayer
2018-12-19 21:34 ` Yann E. MORIN
2018-12-19 21:04 ` [Buildroot] [PATCH v2 4/4] dropbear: " Markus Mayer
2018-12-19 21:38 ` Yann E. MORIN
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.