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)