Add -C option to commit patches immediately
Review Request #6122 — Created July 21, 2014 and submitted
| Information | |
|---|---|
| sgallagh | |
| RBTools | |
| master | |
| 2a12a09... | |
| Reviewers | |
| rbtools | |
It is very useful to be able to script a series of patch
applications all at once (such as when reviewing a patch series).
In these cases, it is useful to be able to accept the default
commit message for each patch.
Patched my local RBTools installation with these changes and pulled a patch into my working tree using the -C option. It applied with the default commit as expected.
| Description | From | Last Updated |
|---|---|---|
|
Col: 80 E501 line too long (83 > 79 characters) |
 reviewbot
|
|
|
Col: 80 E501 line too long (83 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (83 > 79 characters) |
reviewbot
|
|
|
I think I'd prefer to have this be called run_editor and have it set to True when -c is passed. |
 david
|
|
|
The long option should probably be --commit-no-edit |
david
|
Change Summary:
Fixed long lines detected by Review Bot.
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+24 -7) |
-
Tool: Pyflakes Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py Tool: PEP8 Style Checker Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py
-
-
rbtools/clients/__init__.py (Diff revision 2) I think I'd prefer to have this be called
run_editorand have it set to True when -c is passed. -
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 3 (+25 -7) |
Tool: Pyflakes Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py Tool: PEP8 Style Checker Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py