rbt patch subcommand to support --print option

Review Request #4525 — Created Sept. 3, 2013 and submitted

Information

RBTools
master

Reviewers

rbt patch subcommand to support --print option. Option to stdout allows more flexibility. i.e. in case the patching fails, or the user wants to inspect without patching. This change adds --print as an option to 'patch' subcommand.

Yes with private installation.

Description From Last Updated

How would you feel about --print instead of --stdout?

daviddavid

Can you change this to be "print patch to stdout instead of applying" ?

daviddavid
reviewbot
  1. This is a review from Review Bot.
    Tool: PEP8 Style Checker
    Processed Files:
    rbtools/commands/patch.py
    Ignored Files:

  2. 
      
reviewbot
  1. This is a review from Review Bot.
    Tool: Pyflakes
    Processed Files:
    rbtools/commands/patch.py
    Ignored Files:

  2. 
      
david
  1. 
      
  2. rbtools/commands/patch.py (Diff revision 1)
     
     
    Show all issues

    How would you feel about --print instead of --stdout?

  3. rbtools/commands/patch.py (Diff revision 1)
     
     
    Show all issues

    Can you change this to be "print patch to stdout instead of applying" ?

  4. 
      
SU
reviewbot
  1. This is a review from Review Bot.
    Tool: PEP8 Style Checker
    Processed Files:
    rbtools/commands/patch.py
    Ignored Files:

  2. 
      
reviewbot
  1. This is a review from Review Bot.
    Tool: Pyflakes
    Processed Files:
    rbtools/commands/patch.py
    Ignored Files:

  2. 
      
SU
SU
Review request changed
Status:
Completed
Change Summary:

Pushed to rbtools master (f5ec3b6). Thanks!

david
  1. Ship It!

  2.