[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Windows patches for the next release

From: Christopher Faylor
Subject: Re: Windows patches for the next release
Date: Fri, 4 Oct 2013 01:06:58 -0400
User-agent: Mutt/1.5.20 (2009-06-14)

On Thu, Oct 03, 2013 at 07:05:42PM +0300, Eli Zaretskii wrote:
>> Date: Thu, 3 Oct 2013 01:23:15 -0400
>> From: Christopher Faylor <address@hidden>
>> >@@ -2001,13 +2005,17 @@ abspath (const char *name, char *apath)
>> >     }
>> >   else
>> >     {
>> >+#ifdef __CYGWIN__
>> >+      if (STOP_SET (name[0], MAP_PATHSEP))
>> >+   root_len = 1;
>> >+#endif
>> I've eyeballed the patch even though I won't be releasing a version of
>> make which has this turned on.
>> I think the above unnecessary if HAVE_DOS_PATHS is true and should be
>> #if defined(__CYGWIN__) && defined(HAVE_DOS_PATHS)
>You are right, I pushed a fix.

You got it even with my garbled sentence.  Thanks.

>> I'm not sure it's correct to consider \a to be an absolute path in the
>> same way that /a is since they mean different things to Cygwin but I
>> guess that's not something that I have to worry about.
>I thought about that as well.  I actually think that Cygwin should not
>consider a backslash as a directory separator, even if HAVE_DOS_PATHS
>is defined.  What do you think?

Hmm.  I'm not sure.  It wouldn't bother me but there may be people who
want to be able to use backslashes.

>Anyway, such a change modifies behavior (under HAVE_DOS_PATHS), and
>its prerequisite is to use STOP_SET everywhere in Make.  So I think I
>will make that change after the release (unless you, or someone else

Sounds good.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]