[PATCH 0/3] infra: make it possible to install without clobbering existing utilities

2017-12-28 Thread Yann E. MORIN
Hello All! Even though there is a hint of code preventing installation of applets over existing utilities, this is flawed in two ways: - first, the non-clobbering conditional code does not account for installation as script wrappers, - second, only one option can be passed to the install

[PATCH 1/3] applets/install: respect noclobber for script wrappers too

2017-12-28 Thread Yann E. MORIN
Simplify the handling of --noclobber so that it applies to all types of installation types, even to script wrappers. Signed-off-by: "Yann E. MORIN" --- applets/install.sh | 20 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/applets/install.sh b/applets/instal

[PATCH 3/3] infra: add rule to install without cloberring existing utilities

2017-12-28 Thread Yann E. MORIN
Signed-off-by: "Yann E. MORIN" --- Makefile.custom | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Makefile.custom b/Makefile.custom index 891c9ced7..28d0ef7bc 100644 --- a/Makefile.custom +++ b/Makefile.custom @@ -46,6 +46,9 @@ ifeq ($(strip $(CONFIG_FEATURE_SUID)),y) @echo endif

[PATCH 2/3] applets/install: accept more than one install option

2017-12-28 Thread Yann E. MORIN
Currently, it is impossible to pass more than one option to the isntall script, so it totally prevents using --noclobber. Signed-off-by: "Yann E. MORIN" --- applets/install.sh | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/applets/install.sh b/

Re: [PATCH] ash: fix a crash on arm when building with clang.

2017-12-28 Thread Yunlian Jiang
Sorry about the confusion. My build system changed the compiler back to gcc when I try to apply the patch and the patch does not help. However, the patch below helps @@ -173,7 +173,11 @@ struct lineedit_statics { }; /* See lineedit_ptr_hack.c */ -extern struct lineedit_statics *const lineedit_