content-hub

Merge lp://qastaging/~mterry/content-hub/ual-parse into lp://qastaging/content-hub

  1. ual-parse
  2. Merge into trunk
Proposed by Michael Terry
Status: Approved
Approved by: Ken VanDine
Approved revision: 323
Proposed branch: lp://qastaging/~mterry/content-hub/ual-parse
Merge into: lp://qastaging/content-hub
Diff against target: 99 lines (+21/-17)
3 files modified
src/com/ubuntu/content/hub.cpp (+8/-2)
src/com/ubuntu/content/service/registry.cpp (+9/-12)
src/com/ubuntu/content/utils.cpp (+4/-3)
To merge this branch: bzr merge lp://qastaging/~mterry/content-hub/ual-parse
Reviewer Review Type Date Requested Status
system-apps-ci-bot continuous-integration Needs Fixing
Ken VanDine Approve
Review via email: mp+318791@code.qastaging.launchpad.net

Commit message

Use ubuntu-app-launch to parse AppIDs, instead of doing it ourselves.

Description of the change

In an effort to allow UAL flexibility to change appid formats in the future, I'm trying to remove instances of appid parsing elsewhere, and filter it all through UAL.

Tests weren't passing for me, but they aren't on trunk either. So I wasn't sure if I broke anything or not.

To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Tests pass for me and look good. Thanks!

review: Approve
Reply
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:323
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/10/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2391/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2391/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/10/rebuild

review: Needs Fixing (continuous-integration)
Reply
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

FAILED: Continuous integration, rev:323
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/11/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/2393/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/2393
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2205/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=zesty/2205/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2205/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/2205
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=zesty/2205/artifact/output/*zip*/output.zip
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/2205/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=zesty/2205/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-content-hub-ci/11/rebuild

review: Needs Fixing (continuous-integration)
Reply

Unmerged revisions

323. By Michael Terry

Use find, not parse, so that legacy apps are still supported

322. By Michael Terry

Use UAL to parse appids rather than embedding format information in content hub

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