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

- Raw text -