It seems fine, except that I am concerned about checking for the exact
string CYGWIN_NT-5.0

What happens with the ``uname -s'' string changes to CYGWIN_NT-5.1?

I think it might be better to have a case statement there:

sysname=`uname -s`
case "$sysname" in
  CYGWIN*)
    [... code ...] ;;
  *)
    [... default case here ...];;
esac

-- 
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)

Attachment: cygwin_detection_patch.
Description: Binary data

Attachment: msg40627/pgp00000.pgp
Description: PGP signature

Reply via email to