Merge lp://qastaging/~james-w/linaro-image-tools/package-fetcher-config-manager into lp://qastaging/linaro-image-tools/11.11
Proposed by
James Westby
| Status: | Merged |
|---|---|
| Approved by: | Michael Hudson-Doyle |
| Approved revision: | 113 |
| Merged at revision: | 69 |
| Proposed branch: | lp://qastaging/~james-w/linaro-image-tools/package-fetcher-config-manager |
| Merge into: | lp://qastaging/linaro-image-tools/11.11 |
| Prerequisite: | lp://qastaging/~james-w/linaro-image-tools/architecture-in-filename |
| Diff against target: |
43 lines (+15/-0) 2 files modified
hwpack/packages.py (+7/-0) hwpack/tests/test_packages.py (+8/-0) |
| To merge this branch: | bzr merge lp://qastaging/~james-w/linaro-image-tools/package-fetcher-config-manager |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michael Hudson-Doyle (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Hi,
Another small branch. PackageFetcher has a lifecycle, so making it suport
the context manager protocol makes sense. This branch does that and just adds
a smoke test that it can be used like that, and relies on the exiting tests
for the prepare and cleanup methods to test that they do the correct thing.
Thanks,
James
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Also fine.
merge approved
Cheers,
mwh