• 
      

    post-review: support for plain mercurial repos

    Review Request #869 — Created May 18, 2009 and discarded

    Information

    cr
    Review Board SVN (deprecated)

    Reviewers

    Adds a client for plain mercurial repos (not hgsubversion).
    - renamed the hgsubversion class to MercurialSVNClient
    - added a copy named MercurialClient using plain hg command lines
    
    
    tested with:
    $ hg --version
    Mercurial Distributed SCM (version 1.2.1)
    
    
    chipx86
    1. 
        
    2. /trunk/rbtools/scripts/post-review (Diff revision 1)
       
       
      End with a period.
    3. /trunk/rbtools/scripts/post-review (Diff revision 1)
       
       
      Space after the comma.
      
      file_name should be filename.
    4. /trunk/rbtools/scripts/post-review (Diff revision 1)
       
       
      Should specify "r" as the second parameter.
    5. /trunk/rbtools/scripts/post-review (Diff revision 1)
       
       
       
      We end up creating this thing twice in here, with much of the same data. Can you refactor the code before this such that we eventually fall out into this one statement, building using the path variables calculated above?
      
      For example, instead of doing:
      
          if not os.path.exists(...):
              return RepositoryInfo(...)
      
          # Other code here
      
      
      Do:
      
          if os.path.exists(...):
              # Other code here
      
          return RepositoryInfo(...)
    6. /trunk/rbtools/scripts/post-review (Diff revision 1)
       
       
      No blank line here.
    7. 
        
    CR
    1. As far as I can tell the current svn version (-r 2045) already supports hg and hg svn repos.
      
    2.