Fix plumbing of change number when using --diff-filename.

Review Request #7501 — Created July 7, 2015 and submitted

Information

RBTools
release-0.6.x

Reviewers

A previous change of mine fixed an exception when trying to use
--diff-filename, but didn't fix an issue where we had regressed the use of
this command-line argument with systems that use change numbers (such as
perforce). This change fixes the plumbing so that we set the change number
correctly even if the user (for some reason) is choosing the create the diff
themselves.

Ran unit tests.

Description From Last Updated

Can you add Args and Returns sections to the docstring? Same with the one in Perforce.

chipx86chipx86

"when"

chipx86chipx86

This bit about what we'd ideally do is probably best suited for a comment and not any public-facing docs.

chipx86chipx86

Do we want to use six for this?

chipx86chipx86

Use self.REVISION_DEFAULT_CLN rather than literal 'default'.

gmyersgmyers
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/commands/post.py
        rbtools/clients/__init__.py
        rbtools/clients/perforce.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/commands/post.py
        rbtools/clients/__init__.py
        rbtools/clients/perforce.py
    
    
  2. 
      
chipx86
  1. 
      
  2. rbtools/clients/__init__.py (Diff revision 1)
     
     
     
    Show all issues

    Can you add Args and Returns sections to the docstring? Same with the one in Perforce.

  3. rbtools/clients/perforce.py (Diff revision 1)
     
     
    Show all issues

    "when"

  4. rbtools/clients/perforce.py (Diff revision 1)
     
     
     
     
     
    Show all issues

    This bit about what we'd ideally do is probably best suited for a comment and not any public-facing docs.

  5. rbtools/clients/perforce.py (Diff revision 1)
     
     
    Show all issues

    Do we want to use six for this?

  6. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        rbtools/commands/post.py
        rbtools/clients/__init__.py
        rbtools/clients/perforce.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        rbtools/commands/post.py
        rbtools/clients/__init__.py
        rbtools/clients/perforce.py
    
    
  2. 
      
gmyers
  1. 
      
  2. rbtools/clients/perforce.py (Diff revision 2)
     
     
    Show all issues

    Use self.REVISION_DEFAULT_CLN rather than literal 'default'.

  3. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-0.6.x (2cf7aaf)