Merge lp:~salgado/linaro-image-tools/check-hwpack-format into lp:linaro-image-tools/11.11
Proposed by
Guilherme Salgado
Status: | Merged |
---|---|
Merged at revision: | 166 |
Proposed branch: | lp:~salgado/linaro-image-tools/check-hwpack-format |
Merge into: | lp:linaro-image-tools/11.11 |
Diff against target: |
88 lines (+26/-2) 3 files modified
hwpack/hardwarepack.py (+7/-1) hwpack/tests/test_hardwarepack.py (+5/-1) linaro-hwpack-install (+14/-0) |
To merge this branch: | bzr merge lp:~salgado/linaro-image-tools/check-hwpack-format |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Westby (community) | Needs Fixing | ||
Review via email: mp+37749@code.launchpad.net |
Description of the change
Check that the hwpack's format is supported before trying to do anything with
it.
Fixes bug 638322
To post a comment you must log in.
Hi,
Looks good to me. I just wonder what
for format in $SUPPORTED_FORMATS; do
requires SUPPORTED_FORMATS to be set to when there are multiple. Does
space-separated work, or does it need to be newline-separated?
Maybe want to put a hint about trying a newer linaro- hwpack- install
script in the error message?
Thanks,
James