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.Kudrna@mff.cuni.cz>
User-Agent: Thunderbird 1.5 (Windows/20051201)
MIME-Version: 1.0
To: cygwin@cygwin.com
Subject: Re: bug in cygwin_conv_to_posix_path() caused by period in win32  path
References: <468A1024.8000007@mff.cuni.cz> <468A441A.2000009@cygwin.com> <468B657B.3050306@mff.cuni.cz> <20070704213927.GC16838@trixie.casa.cgf.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@cygwin.com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.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/

