Merge lp://qastaging/~roadmr/cdts/1313845-restore-icons into lp://qastaging/cdts

Proposed by Daniel Manrique
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 2374
Merged at revision: 2373
Proposed branch: lp://qastaging/~roadmr/cdts/1313845-restore-icons
Merge into: lp://qastaging/cdts
Diff against target: 133 lines (+92/-0)
2 files modified
plainbox-provider-canonical-driver-test-suite/launcher/icons/scalable/apps/canonical-driver-test-suite.svg (+84/-0)
plainbox-provider-canonical-driver-test-suite/manage.py (+8/-0)
To merge this branch: bzr merge lp://qastaging/~roadmr/cdts/1313845-restore-icons
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Daniel Manrique (community) Approve
Review via email: mp+217967@code.qastaging.launchpad.net

Commit message

Restore the icons that were removed during providerization, and update manage.py to install them in the target system.

Description of the change

Restore the icons that were removed during providerization, and update manage.py to install them in the target system.

An additional packaging branch change is needed to include them in the package.

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

I tested this build with dpkg-buildpackage and sbuild, and installed on a VM where everything worked fine. Since it's rather urgent and pretty simple code-wise, I'll self-approve. Feel free to comment or file bugs in cdts if stuff is borked and needs fixing.

review: Approve
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1. Thanks for finishing this

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