Kayvan> 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. >>
Kayvan> Looks fine. There seems to be a spurious Control-M at the end Kayvan> of one of the lines, but I don't think it matters. ok corrected ;) (it might be confusing on a linux box) Jean Marc this is then the final patch for a problem which looked so simple and still took some time to get resolved ;) Soeren ====================================================================== Politicians are like football coaches. They have to be smart enough to understand the game, and dumb enough to think it's important.
