Handle a missing "cvs" binary better.

Review Request #886 — Created June 7, 2009 and submitted

Information

Review Board SVN (deprecated)
trunk

Reviewers

Handle a missing "cvs" binary better.

In our dependency checks, SCMTool and unit tests, we didn't handle a missing
"cvs" binary in the same way we do for many of our other dependencies. I noticed
this because Ubuntu doesn't install CVS by default anymore (hooray!). This
change fixes up all the places so that we don't throw errors.

 
chipx86
  1. Making the following changes and shipping it for RC3, as requested.
  2. /trunk/reviewboard/manage.py (Diff revision 1)
     
     
     
    Blank line here.
  3. /trunk/reviewboard/manage.py (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     
    Needs to check for binaryname and binaryname.exe, since it won't work on Windows otherwise.
    
    Should also space these out more.
  4. /trunk/reviewboard/manage.py (Diff revision 1)
     
     
     
     
     
     
     
    Blank line between each of these.
  5. /trunk/reviewboard/scmtools/cvs.py (Diff revision 1)
     
     
     
    Blank line here.
  6. /trunk/reviewboard/scmtools/cvs.py (Diff revision 1)
     
     
    Should check for cvs and cvs.exe.
  7. /trunk/reviewboard/scmtools/cvs.py (Diff revision 1)
     
     
     
    Blank line here.
  8. 
      
Loading...