Merge lp://qastaging/~milo/linaro-image-tools/hwpack-read into lp://qastaging/linaro-image-tools/11.11

Proposed by Milo Casagrande
Status: Merged
Approved by: James Tunnicliffe
Approved revision: 544
Merged at revision: 546
Proposed branch: lp://qastaging/~milo/linaro-image-tools/hwpack-read
Merge into: lp://qastaging/linaro-image-tools/11.11
Diff against target: 459 lines (+360/-9)
6 files modified
linaro-media-create (+30/-7)
linaro_image_tools/hwpack/hwpack_reader.py (+192/-0)
linaro_image_tools/hwpack/tests/__init__.py (+1/-0)
linaro_image_tools/hwpack/tests/test_hwpack_reader.py (+114/-0)
linaro_image_tools/media_create/__init__.py (+6/-2)
linaro_image_tools/utils.py (+17/-0)
To merge this branch: bzr merge lp://qastaging/~milo/linaro-image-tools/hwpack-read
Reviewer Review Type Date Requested Status
James Tunnicliffe (community) Approve
Linaro Infrastructure Pending
Review via email: mp+116606@code.qastaging.launchpad.net

Description of the change

With this merge, there is the initial support for hwpack metadata reading, before creating an image with l-m-c.

What has been added here is:
- A new class for reading hwpack, that makes use of the HwpackHandler to open the tarball files
- New tests for the class
- New command line option --read-hwpack that reads the hwpack metadata and prints out information about the supported boards and bootlaoders

I also fixed a couple of PEP8 issues.
Notes:
- There is one test that will fail, and is related to pyflakes, looks like it is bug 812958 (I used getter and setter)
- Some of the required command line options have been set to False, since if we want to just read the hwpack and see what is supported, we should not require all the information. Since there is still work in progress on this, the bits handling the necessary command line options is still missing.

To post a comment you must log in.
542. By Milo Casagrande

Added option check.

543. By Milo Casagrande

Fixed PEP8 warning.

544. By Milo Casagrande

Fixed tests and imports.

Revision history for this message
James Tunnicliffe (dooferlad) wrote :

Looks fine now. Thanks for the changes.

review: Approve
545. By Milo Casagrande

Fixed annoying pyflakes warnings.

546. By Milo Casagrande

Fixed assertEqual tests.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches