• 
      

    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.