This is the mail archive of the cygwin@cygwin.com mailing list for the Cygwin project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi, Script attached; comments below. On Wed, 11 Sep 2002, Paul Johnston wrote: > Hi, > Brilliant to see so much progress on this! > Are you sure that "CYGWIN_9*)" correctly catches Windows ME? No, I'm not. I'm incorporating Warren Young's suggestion. Unless someone with ME can confirm that 'uname -s' returns CYGWIN_9*? Nicholas? > I think we're safe with the case issue, given that cygpath -W seems to > return the correct case. Yes, but we are assuming that the file names are all lowercase. > However, I think adding this belt-and-braces check might be a good idea: > > if [ ! -d "$WINETC" ] > then > echo "ERROR: Directory $WINETC not found" >&2 > exit 2 > fi Done, but without the error code. Should we still print a message, though? > Apart from that, seems just about ready to go. Not sure if the change > history is necessary though, at least not until it's first checked-in. > > Paul The change history was for me to keep track of what I've done. It should probably be deleted when this is submitted as an official patch... This is my alternative to setting up a cvs repository for one file (probably should have, though, same effort, and fewer complaints :-) ). On Wed, 11 Sep 2002, Warren Young wrote: > Paul Johnston wrote: > > > > Are you sure that "CYGWIN_9*)" correctly catches Windows ME? > > You might take a look at my alternate patch in the patches list. It > doesn't rely on that sort of thing -- it relies instead on well-known > environment variables: $SYSTEMROOT and $WINDIR. It still has some > OS-specific knowledge in it, in that it knows that only NT-derived OSes > define SYSTEMROOT and uses WINDIR as a fallback for 9x systems. I don't > know which approach is superior. I've incorporated yours for now. If people confirm that `uname -s` returns either CYGWIN_NT* or CYGWIN_9* on all relevant systems, we could revert to that mechanism. > > However, I think adding this belt-and-braces check might be a good idea: > > I'm not a fan of the "if X, scream and die" bit in this original script > or the one you've proposed. Postinstall scripts shouldn't fail. I > think they should give best effort, and if they can't do what you want, > they should simply quietly skip that bit. Besides, this patch may be > put into some other postinstall script, and you don't want to avoid > doing the other tasks in that script just because the /etc/hosts part > can't find the Windows system directory. Well, we should at least refrain from making the symlinks then... On another note, I'm not sure why you were checking for the presence of the windows files. We could still create dangling symlinks if they are missing, which will allow the files to be created by editing/saving the symlink. Igor -- http://cs.nyu.edu/~pechtcha/ |\ _,,,---,,_ pechtcha@cs.nyu.edu ZZZzz /,`.-'`' -. ;-;;,_ igor@watson.ibm.com |,4- ) )-,_. ,\ ( `'-' Igor Pechtchanski '---''(_/--' `-'\_) fL a.k.a JaguaR-R-R-r-r-r-.-.-. Meow! It took the computational power of three Commodore 64s to fly to the moon. It takes a 486 to run Windows 95. Something is wrong here. -- SC sig file
Attachment:
make-etc-links.sh
Description: Text document
Attachment:
make-etc-links.sh-0.4-diff
Description: Text document
-- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Bug reporting: http://cygwin.com/bugs.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |