Re: [PATCH] Support command line options in svn_apply_autoprops.py
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 : #3723 Gavin "Beau" Baumanis On 04/09/2010, at 11:43 PM, Wei-Yin Chen wrote: > Sorry, don't know why the attachment was removed again. Does this > mailing list have a strict "Content-Type" filter? I'm trying > text/x-patch this time from my linux box. Type > application/octet-stream doesn't seem to pass through. > > From web interface of the mailing 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 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_FILENAME. >> The rest of the command line arguments are treated as paths to process. >> >> [[[ >> Make svn_apply_autoprops.py take command line options. >> Option -c for svn configuration filename, and the rest for paths to process. >> ]]] >> >> Regards, >> Wei-Yin >> >
Re: [PATCH] Support command line options in svn_apply_autoprops.py
Ping. This patch has received no comments. Gavin "Beau" Baumanis On 04/09/2010, at 11:43 PM, Wei-Yin Chen wrote: > Sorry, don't know why the attachment was removed again. Does this > mailing list have a strict "Content-Type" filter? I'm trying > text/x-patch this time from my linux box. Type > application/octet-stream doesn't seem to pass through. > > From web interface of the mailing 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 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_FILENAME. >> The rest of the command line arguments are treated as paths to process. >> >> [[[ >> Make svn_apply_autoprops.py take command line options. >> Option -c for svn configuration filename, and the rest for paths to process. >> ]]] >> >> Regards, >> Wei-Yin >> >
Re: [PATCH] Support command line options in svn_apply_autoprops.py
Try renaming the file as a .txt ? -Hyrum On Sat, Sep 4, 2010 at 8:43 AM, Wei-Yin Chen wrote: > Sorry, don't know why the attachment was removed again. Does this > mailing list have a strict "Content-Type" filter? I'm trying > text/x-patch this time from my linux box. Type > application/octet-stream doesn't seem to pass through. > > From web interface of the mailing 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 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_FILENAME. >> The rest of the command line arguments are treated as paths to process. >> >> [[[ >> Make svn_apply_autoprops.py take command line options. >> Option -c for svn configuration filename, and the rest for paths to process. >> ]]] >> >> Regards, >> Wei-Yin >> >
Re: [PATCH] Support command line options in svn_apply_autoprops.py
Sorry, don't know why the attachment was removed again. Does this
mailing list have a strict "Content-Type" filter? I'm trying
text/x-patch this time from my linux box. Type
application/octet-stream doesn't seem to pass through.
>From web interface of the mailing 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 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_FILENAME.
> The rest of the command line arguments are treated as paths to process.
>
> [[[
> Make svn_apply_autoprops.py take command line options.
> Option -c for svn configuration filename, and the rest for paths to process.
> ]]]
>
> Regards,
> Wei-Yin
>
--- svn_apply_autoprops.py.old 2010-09-04 20:16:28.0 +0800
+++ svn_apply_autoprops.py.option 2010-09-04 20:53:47.0 +0800
@@ -8,12 +8,10 @@
# auto-property to a single pathname, this script will apply all
# matching lines to a single pathname.
#
# To do:
# 1) Switch to using the Subversion Python bindings.
-# 2) Allow a command line option to specify the configuration file to
-#load the auto-properties from.
#
# $HeadURL$
# $LastChangedRevision$
# $LastChangedDate$
# $LastChangedBy$
@@ -137,22 +135,39 @@
print 'Command "%s" failed with exit status %s' \
% (command, status)
sys.exit(1)
def main():
+ from optparse import OptionParser
+ op=OptionParser(usage="%prog [-c config_file] [paths to process...]")
+ op.add_option("-c",
+dest="config_filename",action="store",help="SVN config filename.",metavar="")
+ options, args = op.parse_args()
+ for p in args:
+if not os.path.exists(p):
+ print "Path '%s' doesn't exist." % p
+ sys.exit(1)
+ paths = [p for p in args if os.path.exists(p)]
+ if not paths:
+paths = '.'
+
config_filename = os.path.expandvars(SVN_CONFIG_FILENAME)
+ if options.config_filename:
+config_filename = options.config_filename
try:
fd = file(config_filename)
except IOError:
print "Cannot open svn configuration file '%s' for reading: %s" \
% (config_filename, sys.exc_value.strerror)
+sys.exit(1)
autoprop_lines = get_autoprop_lines(fd)
fd.close()
autoprop_lines = process_autoprop_lines(autoprop_lines)
- os.path.walk('.', filter_walk, autoprop_lines)
+ for p in paths:
+os.path.walk(p, filter_walk, autoprop_lines)
if __name__ == '__main__':
sys.exit(main())
[PATCH] Support command line options in svn_apply_autoprops.py
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_FILENAME.
The rest of the command line arguments are treated as paths to process.
[[[
Make svn_apply_autoprops.py take command line options.
Option -c for svn configuration filename, and the rest for paths to process.
]]]
Regards,
Wei-Yin
--- svn_apply_autoprops.py.old 2010-09-04 20:16:28.0 +0800
+++ svn_apply_autoprops.py.option 2010-09-04 20:53:47.0 +0800
@@ -8,12 +8,10 @@
# auto-property to a single pathname, this script will apply all
# matching lines to a single pathname.
#
# To do:
# 1) Switch to using the Subversion Python bindings.
-# 2) Allow a command line option to specify the configuration file to
-#load the auto-properties from.
#
# $HeadURL$
# $LastChangedRevision$
# $LastChangedDate$
# $LastChangedBy$
@@ -137,22 +135,39 @@
print 'Command "%s" failed with exit status %s' \
% (command, status)
sys.exit(1)
def main():
+ from optparse import OptionParser
+ op=OptionParser(usage="%prog [-c config_file] [paths to process...]")
+ op.add_option("-c",
+dest="config_filename",action="store",help="SVN
config filename.",metavar="")
+ options, args = op.parse_args()
+ for p in args:
+if not os.path.exists(p):
+ print "Path '%s' doesn't exist." % p
+ sys.exit(1)
+ paths = [p for p in args if os.path.exists(p)]
+ if not paths:
+paths = '.'
+
config_filename = os.path.expandvars(SVN_CONFIG_FILENAME)
+ if options.config_filename:
+config_filename = options.config_filename
try:
fd = file(config_filename)
except IOError:
print "Cannot open svn configuration file '%s' for reading: %s" \
% (config_filename, sys.exc_value.strerror)
+sys.exit(1)
autoprop_lines = get_autoprop_lines(fd)
fd.close()
autoprop_lines = process_autoprop_lines(autoprop_lines)
- os.path.walk('.', filter_walk, autoprop_lines)
+ for p in paths:
+os.path.walk(p, filter_walk, autoprop_lines)
if __name__ == '__main__':
sys.exit(main())
