Merge lp:~bjornt/landscape-client/apt-facade-changer-optional into lp:~landscape/landscape-client/trunk
Proposed by
Björn Tillenius
Status: | Merged |
---|---|
Approved by: | Free Ekanayaka |
Approved revision: | 407 |
Merged at revision: | 404 |
Proposed branch: | lp:~bjornt/landscape-client/apt-facade-changer-optional |
Merge into: | lp:~landscape/landscape-client/trunk |
Diff against target: |
143 lines (+50/-13) 4 files modified
landscape/package/reporter.py (+1/-3) landscape/package/taskhandler.py (+2/-2) landscape/package/tests/test_reporter.py (+1/-1) landscape/package/tests/test_taskhandler.py (+46/-7) |
To merge this branch: | bzr merge lp:~bjornt/landscape-client/apt-facade-changer-optional |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Free Ekanayaka (community) | Approve | ||
Thomas Herve (community) | Approve | ||
Review via email: mp+82262@code.launchpad.net |
Description of the change
Use AptFacade in PackageChanger, if USE_APT_FACADE=1 is set.
I've removed the use_apt_facade parameter to run_task_handler() and
moved the check to inside that method. Everything that uses
run_task_handler() should be compatible with AptFacade.
To post a comment you must log in.
[1] You may want to use EnvironSaverHelper to make your os.environ changes safer.
+1!