mediaplayer-app

Merge lp://staging/~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft into lp://staging/mediaplayer-app

  1. mediaplayer-app-changes_snapcraft
  2. Merge into trunk
Proposed by Arthur Mello
Status: Approved
Approved by: Jim Hodapp
Approved revision: 423
Proposed branch: lp://staging/~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft
Merge into: lp://staging/mediaplayer-app
Prerequisite: lp://staging/~phablet-team/mediaplayer-app/snap-it-up
Diff against target: 53 lines (+8/-12)
2 files modified
mediaplayer-app.wrapper (+1/-9)
snapcraft.yaml (+7/-3)
To merge this branch: bzr merge lp://staging/~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
system-apps-ci-bot continuous-integration Approve
Review via email: mp+306942@code.staging.launchpad.net

Commit message

Instead of using a wrapper use desktop-launch on snap pkg
Rename copy plugin to dump on snapcraft.yaml

Description of the change

Instead of using a wrapper use desktop-launch on snap pkg
Rename copy plugin to dump on snapcraft.yaml

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:422
https://jenkins.canonical.com/system-apps/job/lp-mediaplayer-app-ci/18/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/1646
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/392
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1646
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1492/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1492
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1492/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-mediaplayer-app-ci/18/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Jim Hodapp (jhodapp) :
review: Needs Information
Reply
lp://staging/~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft updated
423. By Arthur Mello

Add again the mediaplayer-app.wrapper to handle a missing var
Remove unnecessary info from snapcraft.yaml

Revision history for this message
Arthur Mello (artmello) wrote :

I verified the env variables that are set by the desktop-launch script. It was almost the same as the one that we were setting on the wrapper. Only one that is missing is QML2IMPORT_PATH, so I keep the wrapper to set that.

It seems during my tests that setting the paths on organize with the dump plugin is enough to correct place them on the final snap. So I remove the stage/snap sets

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

PASSED: Continuous integration, rev:423
https://jenkins.canonical.com/system-apps/job/lp-mediaplayer-app-ci/19/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/1684
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/400
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1684
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1530/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1530
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1530/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-mediaplayer-app-ci/19/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Jim Hodapp (jhodapp) wrote :

LGTM

review: Approve (code)
Reply
Revision history for this message
Jim Hodapp (jhodapp) wrote :

Any plans to land this MR still Arthur?

Reply

Unmerged revisions

423. By Arthur Mello

Add again the mediaplayer-app.wrapper to handle a missing var
Remove unnecessary info from snapcraft.yaml

422. By Arthur Mello

Instead of using a wrapper use desktop-launch on snap pkg
Rename copy plugin to dump on snapcraft.yaml

421. By Arthur Mello

Merge with lp:~phablet-team/mediaplayer-app/snap-it-up

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