Merge lp://qastaging/~sergiusens/snapcraft/1502449 into lp://qastaging/~snappy-dev/snapcraft/core
Proposed by
Sergio Schvezov
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Sergio Schvezov | ||||
| Approved revision: | 230 | ||||
| Merged at revision: | 231 | ||||
| Proposed branch: | lp://qastaging/~sergiusens/snapcraft/1502449 | ||||
| Merge into: | lp://qastaging/~snappy-dev/snapcraft/core | ||||
| Diff against target: |
45 lines (+18/-10) 1 file modified
snapcraft/cmds.py (+18/-10) |
||||
| To merge this branch: | bzr merge lp://qastaging/~sergiusens/snapcraft/1502449 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| John Lenton (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Build packages failing to install on the host or not found should be the cause of of a failure
Description of the change
I didn't write a unit test for this as I noticed I was mocking the world.
To post a comment you must log in.
Revision history for this message
| John Lenton (chipaca) | # |
review:
Approve
Revision history for this message
| John Lenton (chipaca) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Ah, oops. inline.