Merge lp://staging/~mvo/software-center/py3-compat-for-license-keyhelper into lp://staging/software-center

Proposed by Michael Vogt
Status: Merged
Merged at revision: 3197
Proposed branch: lp://staging/~mvo/software-center/py3-compat-for-license-keyhelper
Merge into: lp://staging/software-center
Diff against target: 247 lines (+18/-24)
19 files modified
softwarecenter/backend/__init__.py (+0/-6)
softwarecenter/backend/channel_impl/aptchannels.py (+1/-1)
softwarecenter/backend/piston/scaclient.py (+1/-1)
softwarecenter/db/application.py (+1/-1)
softwarecenter/ui/gtk3/app.py (+1/-1)
softwarecenter/ui/gtk3/aptd_gtk3.py (+1/-1)
softwarecenter/ui/gtk3/models/appstore2.py (+1/-1)
softwarecenter/ui/gtk3/models/pendingstore.py (+1/-1)
softwarecenter/ui/gtk3/panes/softwarepane.py (+1/-1)
softwarecenter/ui/gtk3/panes/viewswitcher.py (+1/-1)
softwarecenter/ui/gtk3/views/appdetailsview.py (+1/-1)
softwarecenter/ui/gtk3/views/purchaseview.py (+1/-1)
softwarecenter/ui/gtk3/widgets/apptreeview.py (+1/-1)
softwarecenter/ui/gtk3/widgets/buttons.py (+1/-1)
softwarecenter/ui/gtk3/widgets/recommendations.py (+1/-1)
softwarecenter/ui/qml/pkglist.py (+1/-1)
tests/test_purchase_backend.py (+1/-1)
tests/test_testutils.py (+1/-1)
utils/update-software-center-channels (+1/-1)
To merge this branch: bzr merge lp://staging/~mvo/software-center/py3-compat-for-license-keyhelper
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+125124@code.staging.launchpad.net

Description of the change

This branch move the get_install_backend() import out of softwarecenter/backend/__init__.py and updates the rest of the tree to follow that. The rational is that the software-center-aptdaemon-plugin needs to be compatible with python3 and it imports softwarecenter.backend.piston.scagent. This import will (implicitely) import the get_install_backend code which in turn imports a whole lot more that is not quite python3 ready.

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches