• 
      

    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)