Issue 423: allow file names with $ in them

Review Request #305 — Created March 25, 2008 and submitted

Information

Review Board SVN (deprecated)

Reviewers

*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 -- 
chipx86
  1. Looks good. Committed in r1236. Thanks!
  2. 
      
Loading...