Merge lp://qastaging/~mterry/snapcraft/run-as-list into lp://qastaging/~snappy-dev/snapcraft/core
Proposed by
Michael Terry
Status: | Merged |
---|---|
Approved by: | Michael Terry |
Approved revision: | 78 |
Merged at revision: | 78 |
Proposed branch: | lp://qastaging/~mterry/snapcraft/run-as-list |
Merge into: | lp://qastaging/~snappy-dev/snapcraft/core |
Diff against target: |
168 lines (+26/-37) 7 files modified
snapcraft/__init__.py (+1/-4) snapcraft/cmds.py (+4/-4) snapcraft/common.py (+3/-4) snapcraft/plugins/autotools_project.py (+8/-12) snapcraft/plugins/cmake_project.py (+7/-11) snapcraft/plugins/make_project.py (+2/-1) tests/plainbox/data/local-plugin/parts/plugins/local_plugin.py (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~mterry/snapcraft/run-as-list |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Vogt (community) | Approve | ||
Review via email: mp+264650@code.qastaging.launchpad.net |
Commit message
Make all commands given to our run helper be lists, rather than allowing strings.
Description of the change
Make all commands given to our run helper be lists, rather than allowing strings.
To post a comment you must log in.
Looks great! One small thing I noticed inline but its unrelated to the branch so feel free to ignore.