On Thu, Jul 11, 2002 at 08:01:25PM +0200, Soeren Pietsch wrote: > Kayvan>> It seems fine, except that I am concerned about checking for the exact > Kayvan>> string CYGWIN_NT-5.0 > > Kayvan>> What happens with the ``uname -s'' string changes to CYGWIN_NT-5.1? > > Kayvan>> I think it might be better to have a case statement there: > > Kayvan>> sysname=`uname -s` > Kayvan>> case "$sysname" in > Kayvan>> CYGWIN*) > Kayvan>> [... code ...] ;; > Kayvan>> *) > Kayvan>> [... default case here ...];; > Kayvan>> esac > > I neglected my work today a bit and came up with the following patch. > What does it look like to you Kayvan? > >From my point of view it should work. >
Looks fine. There seems to be a spurious Control-M at the end of one of the lines, but I don't think it matters. -- Kayvan A. Sylvan | Proud husband of | Father to my kids: Sylvan Associates, Inc. | Laura Isabella Sylvan | Katherine Yelena (8/8/89) http://sylvan.com/~kayvan | "crown of her husband" | Robin Gregory (2/28/92)
msg40800/pgp00000.pgp
Description: PGP signature
