Re: [PATCH] don't do autoprops on symbolic links

2010-09-29 Thread Stefan Sperling
On Wed, Sep 29, 2010 at 10:56:17AM +1000, Gavin Beau Baumanis wrote: > As a result of no comments for this patch , I have logged it into the issue > tracker so that it doesn't get lost. > > http://subversion.tigris.org/issue-tracker.html > Issue Number : #3722 I've committed the patch. Thanks ve

Re: [PATCH] don't do autoprops on symbolic links

2010-09-28 Thread Gavin Beau Baumanis
As a result of no comments for this patch , I have logged it into the issue tracker so that it doesn't get lost. http://subversion.tigris.org/issue-tracker.html Issue Number : #3722 Gavin "Beau" Baumanis On 04/09/2010, at 10:25 PM, Wei-Yin Chen wrote: > Dear Gavin, > > Thanks. The attachme

Re: [PATCH] don't do autoprops on symbolic links

2010-09-09 Thread Gavin Beau Baumanis
Hi Everyone. This renewed patch has received no comments. Gavin "Beau" Baumanis On 04/09/2010, at 10:25 PM, Wei-Yin Chen wrote: > Dear Gavin, > > Thanks. The attachment was in my sent box, but it's absent in the mailing > archive. Don't know why. > > Per Branko's suggestion, I'm using joi

Re: [PATCH] don't do autoprops on symbolic links

2010-09-04 Thread Wei-Yin Chen
Dear Gavin, Thanks. The attachment was in my sent box, but it's absent in the mailing archive. Don't know why. Per Branko's suggestion, I'm using join this time. In case the attachment gets missing again, it is also embedded in the mail body. Regards, Wei-Yin --- svn_apply_autoprops.py.old 201

Re: [PATCH] don't do autoprops on symbolic links

2010-09-04 Thread Gavin Beau Baumanis
Hi Wei-Yin, Just thought I would mention that you did not attach your updated patch. Gavin "Beau" Baumanis E: ga...@thespidernet.com On 04/09/2010, at 2:19 PM, Wei-Yin Chen wrote: > Hi Stefan, > > Thanks for your comment. I've updated the patch accordingly. > > Regards, > Wei-Yin > > On Fr

Re: [PATCH] don't do autoprops on symbolic links

2010-09-04 Thread Branko Čibej
On 03.09.2010 15:08, Stefan Sperling wrote: > On Fri, Sep 03, 2010 at 08:58:09PM +0800, Wei-Yin Chen wrote: >> Sorry, that line should have been >> matching_filenames = [f for f in matching_filenames if not >> os.path.islink(dirname+'/'+f)] > Hi, > > thanks for your patch! > > I think we should us

Re: [PATCH] don't do autoprops on symbolic links

2010-09-03 Thread Wei-Yin Chen
Hi Stefan, Thanks for your comment. I've updated the patch accordingly. Regards, Wei-Yin On Fri, Sep 3, 2010 at 9:08 PM, Stefan Sperling wrote: > > Hi, > > thanks for your patch! > > I think we should use os.sep instead of '/', because os.sep is more > portable. > > Also, please put spaces aro

Re: [PATCH] don't do autoprops on symbolic links

2010-09-03 Thread Stefan Sperling
On Fri, Sep 03, 2010 at 08:58:09PM +0800, Wei-Yin Chen wrote: > Sorry, that line should have been > matching_filenames = [f for f in matching_filenames if not > os.path.islink(dirname+'/'+f)] Hi, thanks for your patch! I think we should use os.sep instead of '/', because os.sep is more portable.

Re: [PATCH] don't do autoprops on symbolic links

2010-09-03 Thread Wei-Yin Chen
Sorry, that line should have been matching_filenames = [f for f in matching_filenames if not os.path.islink(dirname+'/'+f)] On Fri, Sep 3, 2010 at 8:15 PM, Wei-Yin Chen wrote: > This patch is for the following file. > > https://svn.apache.org/repos/asf/subversion/trunk/contrib/client-side/svn_app

[PATCH] don't do autoprops on symbolic links

2010-09-03 Thread Wei-Yin Chen
This patch is for the following file. https://svn.apache.org/repos/asf/subversion/trunk/contrib/client-side/svn_apply_autoprops.py Log message: Do not apply autoprops on symbolic links in svn_apply_autoprops.py. Index: svn_apply_autoprops.py