Ubuntu
plainbox-provider-checkbox package

Merge lp://qastaging/~sylvain-pineau/ubuntu/trusty/plainbox-provider-checkbox/namespace_fix into lp://qastaging/ubuntu/trusty/plainbox-provider-checkbox

  1. Trusty (14.04)
  2. namespace_fix
  3. Merge into trusty
Proposed by Sylvain Pineau
Status: Merged
Merge reported by: Sebastien Bacher
Merged at revision: not available
Proposed branch: lp://qastaging/~sylvain-pineau/ubuntu/trusty/plainbox-provider-checkbox/namespace_fix
Merge into: lp://qastaging/ubuntu/trusty/plainbox-provider-checkbox
Diff against target: 31 lines (+11/-1)
3 files modified
checkbox.provider (+1/-1)
debian/patches/provider_namespace.patch (+9/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp://qastaging/~sylvain-pineau/ubuntu/trusty/plainbox-provider-checkbox/namespace_fix
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Ubuntu Development Team Pending
Review via email: mp+212629@code.qastaging.launchpad.net

Description of the change

Fix the namespace of the provider to comply with checkbox-ng 0.2.2-1 settings (once synced from Debian)

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

This looks good and should land. It will also be safe to drop on the next sync from Debian (once we re-release those packages). The problem is caused by hard-coded string in checkbox-ng that looks for '2013.com.canonical.certification' while the older packages still have '2013.com.canonical' (without .certification) and stuff would just crash.

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

FYI: this got merged and uploaded into Ubuntu but this is not reflected here

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