Merge lp:~james-w/tarmac/exit-status into lp:tarmac
Status: | Rejected |
---|---|
Rejected by: | dobey |
Proposed branch: | lp:~james-w/tarmac/exit-status |
Merge into: | lp:tarmac |
Prerequisite: | lp:~james-w/tarmac/break-up-do-merges |
Diff against target: |
54 lines (+9/-4) 4 files modified
bin/tarmac (+3/-1) tarmac/bin/__init__.py (+1/-1) tarmac/bin/commands.py (+3/-0) tarmac/bin/registry.py (+2/-2) |
To merge this branch: | bzr merge lp:~james-w/tarmac/exit-status |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
dobey | Disapprove | ||
Paul Hummer | Approve | ||
Review via email: mp+144810@code.launchpad.net |
Commit message
Return exit status of the tarmac script.
Description of the change
This allows the commands to set an exit status for the whole
tarmac script. We need it so that we can communicate some status
information to jenkins.
(from the u1 fork)
Unmerged revisions
- 424. By James Westby
-
Merged break-up-do-merges into exit-status.
- 423. By James Westby
-
Merged break-up-do-merges into exit-status.
- 422. By James Westby
-
Merged break-up-do-merges into exit-status.
- 421. By James Westby
-
Merged break-up-do-merges into exit-status.
- 420. By James Westby
-
Merged break-up-do-merges into exit-status.
- 419. By James Westby
-
Allow the command to set the exit status.
- 418. By James Westby
-
Remove the option that isn't defined yet.
- 417. By James Westby
-
Break up the very large do_merges method somewhat.
- 416. By James Westby
-
Add the commit message plugin that sets [r=<reviewer>] and bug info.
- 415. By James Westby
-
Decode stderr/stdout from the command, ignoring errors.
We were seeing UnicodeDecodeErrors in Jenkins on occaision, so
we changed the code to be more robust, and to discard anything
that couldn't be decoded from utf-8.
The prerequisite lp:~james-w/tarmac/break-up-do-merges has not yet been merged into lp:tarmac.