Hi everyone!

Following on from [1], this patch prevents the error "a peg revision
is not allowed here" when svn_apply_autoprops.py handles filenames
containing `@`. Stack Overflow [2] gives the impression that these
filenames are rare -- I have them because I have code that works with
npm type packages like [3].

The patch still appears routine to me, but maybe I've put the fix in
the wrong place or something like that.

[1] https://lists.apache.org/thread/k0o0ytr6h74wf92p4xyg8tvq5g7h4tj7
[2] 
https://stackoverflow.com/questions/757435/how-to-escape-characters-in-subversion-managed-file-names
[3] https://www.npmjs.com/package/@types/node

[[[
svn_apply_autoprops.py: Support @-containing filenames.

* contrib/client-side/svn_apply_autoprops.py
  (filter_walk): Append `@` to filenames containing `@`.
]]]

Best regards,
Khairul
Index: contrib/client-side/svn_apply_autoprops.py
===================================================================
--- contrib/client-side/svn_apply_autoprops.py  (revision 1917391)
+++ contrib/client-side/svn_apply_autoprops.py  (working copy)
@@ -147,6 +147,8 @@ def filter_walk(autoprop_lines, dirname, filenames
     for prop in prop_list:
       command = ['svn', 'propset', prop[0], prop[1]]
       for f in matching_filenames:
+        if '@' in f:
+          f += '@'
         command += ["%s/%s" % (dirname, f)]
 
       status = subprocess.call(command)

Reply via email to