Looks good. Committed in r1236. Thanks!
Issue 423: allow file names with $ in them
Review Request #305 — Created March 25, 2008 and submitted
|Review Board SVN (deprecated)|
*This fix should probably be ported to the other SCM tools [I don't have a way to test]* The problem is that execute() winds up performing shell expansion of $NAME, and the quotes around the execute command line weren't correct to prevent shell expansion.
This change has been tested on Perforce on files with $ in them --