Message-ID: <4B27DE97.8000101@x-ray.at>
Date: Tue, 15 Dec 2009 20:08:07 +0100
From: Reini Urban <rurban@x-ray.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@cygwin.com
Subject: Re: Suggested patch for cygwin perl 5.10
References: <4B27B922.6070107@cs.umass.edu>
In-Reply-To: <4B27B922.6070107@cs.umass.edu>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.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