Re: [PATCH] Support command line options in svn_apply_autoprops.py

2010-09-04 Thread Wei-Yin Chen
g archive, the format of the inlined version looks strange, and the indentation seems wrong, so here's another way to get it: http://pastebin.com/YWhpnZvF On Sat, Sep 4, 2010 at 9:05 PM, Wei-Yin Chen wrote: > > Hi, > > This patch makes trunk/contrib/client-side/svn_apply_autoprops.py t

[PATCH] Support command line options in svn_apply_autoprops.py

2010-09-04 Thread Wei-Yin Chen
Hi, This patch makes trunk/contrib/client-side/svn_apply_autoprops.py take command line options. The usage would be: svn_apply_autoprops.py [-c config_file] [paths to process...] The -c option specifies the configuration file name, and it overrides the setting in environment variable SVN_CONFIG_F

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-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 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/

[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