Merge lp:~bjornt/landscape-client/apt-facade-output into lp:~landscape/landscape-client/trunk
Proposed by
Björn Tillenius
Status: | Merged |
---|---|
Approved by: | Thomas Herve |
Approved revision: | 419 |
Merged at revision: | 409 |
Proposed branch: | lp:~bjornt/landscape-client/apt-facade-output |
Merge into: | lp:~landscape/landscape-client/trunk |
Diff against target: |
302 lines (+205/-12) 2 files modified
landscape/package/facade.py (+26/-3) landscape/package/tests/test_facade.py (+179/-9) |
To merge this branch: | bzr merge lp:~bjornt/landscape-client/apt-facade-output |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Thomas Herve (community) | Approve | ||
Alberto Donato (community) | Approve | ||
Review via email: mp+83138@code.launchpad.net |
Description of the change
Capture the fetch and dpkg output and send it back to the server, both
for succesful and failed operations.
The redirection of stdout/stderr isn't directly pretty, but I asked mvo,
and this is they way it's generally done. It's a bit complicated, since
dpkg runs in a subprocess.
To post a comment you must log in.
Looks good, +1!