X-Spam-Check-By: sourceware.org
Message-ID: <468D70E6.4070403@mff.cuni.cz>
Date: Fri, 06 Jul 2007 00:29:58 +0200
From: Pavel Kudrna <Pavel DOT Kudrna AT mff DOT cuni DOT cz>
User-Agent: Thunderbird 1.5 (Windows/20051201)
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: bug in cygwin_conv_to_posix_path() caused by period in win32  path
References: <468A1024 DOT 8000007 AT mff DOT cuni DOT cz> <468A441A DOT 2000009 AT cygwin DOT com> <468B657B DOT 3050306 AT mff DOT cuni DOT cz> <20070704213927 DOT GC16838 AT trixie DOT casa DOT cgf DOT cx>
In-Reply-To: <20070704213927.GC16838@trixie.casa.cgf.cx>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

Christopher Faylor wrote:
> I suppose it is possible that we'd consider a nicely written, minimally
> intrusive patch to use these but I'm not 100% certain that it is even
> possible to do this cleanly given the way the environment variable/path
> conversion handling works.
>   
Lets look back to original subject. Function cygwin_conv_to_posix_path()
should not leave passed win32 path almost unconverted:

c:.\  ->  c:./
c:.   ->  c:.

That is clearly incorrect.

I found the present meaning of "C:" defined as "C:\" at the beginning
of path.cc file. So both paths should be converted to /cygdrive/c.

But I see it much better to convert it according to the contents "!C:"
environment variable. Even in the present time when these variables are
without changing inherited between cygwin programs.
Pavel Kudrna

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/