[PATCH] compress output of `cd' when installing templates

[PATCH] compress output of `cd' when installing templates

[Gang Chen]

When CDPATH is set in Bash, `cd' writes new working directory to
stdout, which corrupts output of `tar cf - .'.

Signed-off-by: Gang Chen <goncha.ml@gmail.com>

---

 Makefile |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/templates/Makefile b/templates/Makefile
index 8f7f4fe..cb67489 100644
--- a/templates/Makefile
+++ b/templates/Makefile
@@ -44,5 +44,5 @@ clean:
 
 install: all
 	$(INSTALL) -d -m755 $(call shellquote,$(DESTDIR)$(template_dir))
-	(cd blt && $(TAR) cf - .) | \
+	(cd blt > /dev/null && $(TAR) cf - .) | \
 	(cd $(call shellquote,$(DESTDIR)$(template_dir)) && $(TAR) xf -)

-- 
If it makes you happy, it can't be that bad.

Re: [PATCH] compress output of `cd' when installing templates

[Timo Hirvonen]

Gang Chen <goncha.ml@gmail.com> wrote:

> When CDPATH is set in Bash, `cd' writes new working directory to
> stdout, which corrupts output of `tar cf - .'.

Fix bash instead, or just don't export CDPATH.  It is used by bash only
so exporting it is unnecessary.

-- 
http://onion.dynserv.net/~timo/

Re: [PATCH] compress output of `cd' when installing templates

[Junio C Hamano]

Timo Hirvonen <tihirvon@gmail.com> writes:

> Gang Chen <goncha.ml@gmail.com> wrote:
>
>> When CDPATH is set in Bash, `cd' writes new working directory to
>> stdout, which corrupts output of `tar cf - .'.
>
> Fix bash instead, or just don't export CDPATH.  It is used by bash only
> so exporting it is unnecessary.

Thanks.

It is not just unnecessary, but "export CDPATH" anywhere in your
rc scripts is actively *WRONG*.