[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