X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 	tests=AWL,BAYES_00
X-Spam-Check-By: sourceware.org
Message-ID: <4B27DE97.8000101@x-ray.at>
Date: Tue, 15 Dec 2009 20:08:07 +0100
From: Reini Urban <rurban AT x-ray DOT at>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.1.4) Gecko/20091017 SeaMonkey/2.0
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: Suggested patch for cygwin perl 5.10
References: <4B27B922 DOT 6070107 AT cs DOT umass DOT edu>
In-Reply-To: <4B27B922.6070107@cs.umass.edu>
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

Eliot Moss schrieb:
> Dear cygwin / perl maintainers:
> When running svk, a package written in perl that runs
> "on top of" svn, I have found the attached patch to be
> necessary. Otherwise svk does not handle filenames
> properly. The first change is the essential one. The
> second one just eliminates an annoying and possibly
> confusing/alarming (but I think truly harmless)
> warning about setting something a second time. (The
> affected lines were only recently introduced, I
> believe.) This has been going on for some time, and
> I finally got organized to send this in, rather than
> continuing to edit the file myself after each update.
>
> The file in question is:
>
> /usr/lib/perl5/5.10/i686-cygwin/Cwd.pm

Thanks!

For the future, please create patches -u and not reverse.
We prefer old to new.
   diff -bu old new > new.patch

I'll discuss it upstream with the PathTools maintainers,
as it is quite a dignificant change. Admitted, Cwd is a minefield.
http://search.cpan.org/dist/PathTools/
-- 
Reini Urban
http://phpwiki.org/  http://murbreak.at/

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