Merge lp://qastaging/~milo/linaro-image-tools/boards-support into lp://qastaging/linaro-image-tools/11.11
Proposed by
Milo Casagrande
| Status: | Merged |
|---|---|
| Approved by: | James Tunnicliffe |
| Approved revision: | 551 |
| Merged at revision: | 538 |
| Proposed branch: | lp://qastaging/~milo/linaro-image-tools/boards-support |
| Merge into: | lp://qastaging/linaro-image-tools/11.11 |
| Diff against target: |
3436 lines (+1871/-432) 15 files modified
linaro-hwpack-install (+20/-5) linaro_image_tools/hwpack/builder.py (+145/-48) linaro_image_tools/hwpack/config.py (+495/-205) linaro_image_tools/hwpack/hardwarepack.py (+170/-25) linaro_image_tools/hwpack/hardwarepack_format.py (+9/-0) linaro_image_tools/hwpack/hwpack_convert.py (+15/-3) linaro_image_tools/hwpack/hwpack_fields.py (+3/-0) linaro_image_tools/hwpack/tests/__init__.py (+1/-0) linaro_image_tools/hwpack/tests/test_config.py (+10/-8) linaro_image_tools/hwpack/tests/test_config_v3.py (+738/-0) linaro_image_tools/hwpack/tests/test_hardwarepack.py (+133/-86) linaro_image_tools/hwpack/tests/test_hwpack_converter.py (+24/-0) linaro_image_tools/media_create/boards.py (+27/-27) linaro_image_tools/media_create/chroot_utils.py (+12/-2) linaro_image_tools/media_create/tests/test_media_create.py (+69/-23) |
| To merge this branch: | bzr merge lp://qastaging/~milo/linaro-image-tools/boards-support |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| James Tunnicliffe (community) | Approve | ||
| Stevan Radaković | Pending | ||
| Linaro Infrastructure | Pending | ||
|
Review via email:
|
|||
Description of the change
With this branch I added support for the boards section in the Config class and metadata file.
I reworked a couple of sections in the metadata file, where we were modifying fields value. Now that happens in the builder.py file, reworking the necessary fields there, and in metadata we just reset the whole attribute that holds the modified values.
I reworked a little bit the builder class in order to clean up also the methods call.
To post a comment you must log in.
Revision history for this message
| Milo Casagrande (milo) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
This branch applies to the one that is already in review. Once that gets in, this one would be easier to read.