[M3commit] CVS Update: cm3
Coleburn, Randy
rcolebur at SCIRES.COM
Sun Nov 21 02:54:09 CET 2010
Jay:
My recollection about the reason this was coded as a "pushd foo" followed by a "cd .." was to get around a problem in one of the older cmd shell variants.
I agree that "pushd foo\bar" is better than "pushd foo" "cd bar", but note that relative paths are being used in some of my code that you changed, so your example is not exactly the same.
My experience with an older shell (I can't remember which) is that it ran into problems with
Pushd ..\..\..
Popd
when the target of the original pushd did not exist.
ALSO WORD TO THE WISE: IF IT ISN'T BROKE, DON'T FIX IT. This code was working fine. Your "optimization" is trivial and may wind up breaking something (esp. if my recollection is correct).
Regards,
Randy
-----Original Message-----
From: Jay Krell [mailto:jkrell at elego.de]
Sent: Wednesday, November 10, 2010 7:29 AM
To: m3commit at elegosoft.com
Subject: [M3commit] CVS Update: cm3
CVSROOT: /usr/cvs
Changes by: jkrell at birch. 10/11/10 12:28:36
Modified files:
cm3/scripts/dev/windows/: do-cm3.cmd
Log message:
pushd foo
cd bar
is terser and no less clear as
pushd foo\bar
More information about the M3commit
mailing list