From: Greg Louis Date: Tue, 13 Sep 2005 11:07:20 GMT Subject: Re: [PATCH] Redirect cd output to /dev/null, was: git-clone seems dead Message-ID: <20050913110720.GA4826@athame.dynamicro.on.ca> In-Reply-To: <7v3boaawra.fsf@assigned-by-dhcp.cox.net> On 20050912 (Mon) at 1253:13 -0700, Junio C Hamano wrote: > Greg Louis writes: > > > Or (2+1==3) detect CDPATH in the same places as (1), complain with a > > brief explanation ("don't export CDPATH; it's really only useful in > > interactive shells") and unset it. > > I like your wording above, so probably that is what is going to > happen. Thanks for the suggestion. You're very welcome. > CDPATH has two problems. 1. It takes scripts to unexpected > places (somebody had CDPATH=..:../..:$HOME and the "cd" in > git-clone.sh:get_repo_base took him to $HOME/.git when he said > "clone foo bar" to clone a repository in "foo" which had > "foo/.git"). CDPATH mechanism does not implicitly give "." at > the beginning of CDPATH, which is the most irritating part. > 2. the extra echo when it does its thing. > > I was under the impression that the patch fixed 2. without doing > anything about 1., but I may be mistaken. Oh, I agree entirely that unsetting CDPATH altogether is cleaner and more complete (and much less ugly) than putting &>/dev/null all over the place. I should have said something like, "My patch is _philosophically_ about equivalent to (1)" to make it clear I was referring to the context, ie the alternative approaches you had proposed. -- | G r e g L o u i s | gpg public key: 0x400B1AA86D9E3E64 | | http://www.bgl.nu/~glouis | (on my website or any keyserver) | | http://wecanstopspam.org in signatures helps fight junk email. |