Merge lp:~bjornt/landscape-client/update-internal-repos-only into lp:~landscape/landscape-client/trunk
Status: | Merged |
---|---|
Approved by: | Geoff Teale |
Approved revision: | 533 |
Merged at revision: | 534 |
Proposed branch: | lp:~bjornt/landscape-client/update-internal-repos-only |
Merge into: | lp:~landscape/landscape-client/trunk |
Diff against target: | 0 lines |
To merge this branch: | bzr merge lp:~bjornt/landscape-client/update-internal-repos-only |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Geoff Teale (community) | Approve | ||
Thomas Herve (community) | Approve | ||
Review via email: mp+98414@code.launchpad.net |
Description of the change
Update only internal facade repositories in AptFacade.
if force_reload_
in the facade-specific .list file will be updated, other repositories in
sources.list and sources.list.d won't.
This will only work if python-apt is new enough. For older version
(e.g. on Lucid) everything will still be updated. There doesn't seem to
be a better way of determining whether python-apt is new enough apart
from trying to call the update() method.
I do think the parameter name force_reload_
filed bug 960217 about moving the refetch_
parameter to update() and I will rethink the names then.
[1] pyflakes:
landscape/ package/ facade. py:5: 'time' imported but unused
Looks great, +1!