mediaplayer-app

Merge lp:~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft into lp: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:~artmello/mediaplayer-app/mediaplayer-app-changes_snapcraft
Merge into: lp:mediaplayer-app
Prerequisite: lp:~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:~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.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:~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
=== modified file 'mediaplayer-app.wrapper'
--- mediaplayer-app.wrapper 2016-09-06 19:22:48 +0000
+++ mediaplayer-app.wrapper 2016-09-29 20:22:20 +0000
@@ -8,16 +8,8 @@
8 ARCH="$SNAP_ARCH-linux-gnu"8 ARCH="$SNAP_ARCH-linux-gnu"
9fi9fi
1010
11export LD_LIBRARY_PATH=$SNAP/usr/lib/$ARCH:$LD_LIBRARY_PATH
12
13# Qt Libs
14export LD_LIBRARY_PATH=$SNAP/usr/lib/$ARCH/qt5/libs:$LD_LIBRARY_PATH
15
16# Qt Modules11# Qt Modules
17export QT_PLUGIN_PATH=$SNAP/usr/lib/$ARCH/qt5/plugins
18export QML2IMPORT_PATH=$SNAP/usr/share/mediaplayer-app/qml/12export QML2IMPORT_PATH=$SNAP/usr/share/mediaplayer-app/qml/
19export QML2_IMPORT_PATH=$QML2_IMPORT_PATH:$SNAP/usr/lib/$ARCH/qt5/qml/13export QML2_IMPORT_PATH=$QML2_IMPORT_PATH:$SNAP/usr/lib/$ARCH/qt5/qml/
2014
21echo QT_PLUGIN_PATH=$QT_PLUGIN_PATH15exec "$SNAP/bin/desktop-launch" "mediaplayer-app" "$@"
22
23exec "$SNAP/usr/bin/mediaplayer-app" "$@"
2416
=== modified file 'snapcraft.yaml'
--- snapcraft.yaml 2016-09-08 20:56:15 +0000
+++ snapcraft.yaml 2016-09-29 20:22:20 +0000
@@ -5,11 +5,12 @@
5 A QML media player that can play audio/video sources.5 A QML media player that can play audio/video sources.
6 Please find the source at:6 Please find the source at:
7 https://code.launchpad.net/mediaplayer-app7 https://code.launchpad.net/mediaplayer-app
8grade: stable
8confinement: strict9confinement: strict
910
10apps:11apps:
11 mediaplayer-app:12 mediaplayer-app:
12 command: usr/bin/mediaplayer-app13 command: bin/mediaplayer-app
13 plugs: [mpris]14 plugs: [mpris]
1415
15parts:16parts:
@@ -63,8 +64,11 @@
63 snap:64 snap:
64 - $unwanted65 - $unwanted
6566
67 after: [desktop-qt5]
68
66 environment:69 environment:
67 plugin: copy70 source: .
68 files:71 plugin: dump
72 organize:
69 mediaplayer-app.wrapper: bin/mediaplayer-app73 mediaplayer-app.wrapper: bin/mediaplayer-app
70 debian/copyright: usr/share/doc/mediaplayer-app/copyright74 debian/copyright: usr/share/doc/mediaplayer-app/copyright

Subscribers

People subscribed via source and target branches