Re: cvs2svn.py patch

Arkadiusz Miskiewicz <misiek@pld.ORG.PL> writes:

> It's better than it was before. I've never seen a cvs file with single quotes
> in name but... ;) Anyway every character in file name could be escaped
> just to be sure but this isn't ,,beauty'' way.

No need to escape all characters. If the argument is enclosed in
single quotes, then only single quotes need to be escaped. Just do

      pipe = os.popen('co -q -p%s \'%s\'' %
                          (r, string.replace(f, "'","'\"'\"'")),
                      'r', 102400)

and it should be ok for all characters (although NUL characters could
still be a problem, but they are not valid in filenames anyway).

  // Marcus

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Apr 14 19:14:55 2002