Mailing-List: contact cygwin-help AT sourceware DOT cygnus DOT com; run by ezmlm List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner AT sources DOT redhat DOT com Delivered-To: mailing list cygwin AT sources DOT redhat DOT com Date: Thu, 22 Feb 2001 12:53:40 -0500 From: Christopher Faylor To: Corinna Vinschen Subject: Re: [ANNOUNCEMENT]: Important change to symbolic link functionality Message-ID: <20010222125340.K11175@redhat.com> Reply-To: cygwin AT cygwin DOT com Mail-Followup-To: Corinna Vinschen References: <20010221232921 DOT X908 AT cygbert DOT vinschen DOT de> <21157045389 DOT 20010222102353 AT logos-m DOT ru> <20010222115142 DOT H908 AT cygbert DOT vinschen DOT de> <20010222120909 DOT I908 AT cygbert DOT vinschen DOT de> <180175177361 DOT 20010222152605 AT logos-m DOT ru> <20010222172713 DOT L908 AT cygbert DOT vinschen DOT de> <20010222113656 DOT I11175 AT redhat DOT com> <20010222183723 DOT R908 AT cygbert DOT vinschen DOT de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.3.11i In-Reply-To: <20010222183723.R908@cygbert.vinschen.de>; from cygwin@cygwin.com on Thu, Feb 22, 2001 at 06:37:23PM +0100 On Thu, Feb 22, 2001 at 06:37:23PM +0100, Corinna Vinschen wrote: >On Thu, Feb 22, 2001 at 11:36:56AM -0500, Christopher Faylor wrote: >> On Thu, Feb 22, 2001 at 05:27:13PM +0100, Corinna Vinschen wrote: >> >The problem is that a shortcut has to contain a DOS path, obviously. >> >The new strategy is as follows: >> >> If this is the case, then we can't use .lnk files as the default symlinks. >> DOS paths are not UNIX paths. The mount table will be bypassed in the symlinks. > >Chris, > >I'm a bit annoyed about your excoriating here. You're changing >the code I have created and you're turning down the patch in >public without even having asked me about the behaviour or having >a deeper look into it. What code am I changing? I sent you a patch and I explained why I didn't feel comfortable with your code. I'm still not comfortable with the goto. I have pointed out other little glitches in your code before and you have done the same for me. Sometimes people miss things. I'm not going to assume that you infallible. I hope you will assume that I am not infallible, too. Right now, I'm trying to get ready to go out of time for a while so I didn't feel like I had the time to do an in-depth review of your changes. But I wanted to make some observations in case we were going down a road that was unsupportable. >If I only had a DOS path in a shortcut I wouldn't have introduced the >patch. I'm not silly. I'm trying to generate useful code. If my >style isn't exactly yours, sorry. But your current criticising sounds >to me exactly the style of criticism we both don't like from the >mailing list. Changes like this should be discussed in public. Witness the gdb-patches and gdb mailing list. You're free to defend yourself and tell me that it doesn't work that way. If you didn't want it publicly discussed then you shouldn't have announced it in cygwin AT cygwin DOT com. I was a little surprised to see that it was announced there before I'd even had a chance to evaluate it or play with it and while I was still asking questions about the code. I still have reservations about storing the dos path in the .lnk file. Even if the cygwin path is stored there, there will exist a potential disconnect between what the windows link is pointing to versus what the cygwin portion is linking to. That could be surprising behavior. I'm sorry if my mail sounded harsh, though. I guess I could have worded it differently. I assumed that someone would notice the same thing that I did with regard to dos paths so I wanted to make the observation ASAP. cgf -- Want to unsubscribe from this list? Check out: http://cygwin.com/ml/#unsubscribe-simple