Merge lp:~renatofilho/ubuntu-weather-app/add-missing-mir into lp:ubuntu-weather-app

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Andrew Hayzen
Approved revision: 267
Merged at revision: 264
Proposed branch: lp:~renatofilho/ubuntu-weather-app/add-missing-mir
Merge into: lp:ubuntu-weather-app
Diff against target: 29 lines (+3/-11)
1 file modified
snapcraft.yaml (+3/-11)
To merge this branch: bzr merge lp:~renatofilho/ubuntu-weather-app/add-missing-mir
Reviewer Review Type Date Requested Status
Andrew Hayzen Approve
Jenkins Bot continuous-integration Approve
Review via email: mp+321696@code.launchpad.net

Commit message

[snap] propagate args to qmlscene and add missing "mir" into the plug list.

To post a comment you must log in.
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Andrew Hayzen (ahayzen) wrote :

LGTM, thanks :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'snapcraft.yaml'
2--- snapcraft.yaml 2017-03-28 19:28:59 +0000
3+++ snapcraft.yaml 2017-04-03 15:17:37 +0000
4@@ -3,22 +3,14 @@
5 summary: ubuntu weather app.
6 description: |
7 A weather forecast application for Ubuntu with support for multiple online weather data sources.
8+confinement: strict
9 grade: devel
10-confinement: strict
11 architectures: [all]
12
13 apps:
14 ubuntu-weather-app:
15- aliases: [ubuntu-weather-app]
16- command: desktop-launch qmlscene $SNAP/usr/share/ubuntu-weather-app/app/ubuntu-weather-app.qml
17- plugs:
18- - network
19- - location-observe
20- - opengl
21- - platform
22- - unity7
23- - unity8
24- - mir-libs
25+ command: desktop-launch qmlscene "$@" $SNAP/usr/share/ubuntu-weather-app/app/ubuntu-weather-app.qml
26+ plugs: [network, location-observe, opengl, platform, unity7, unity8, mir, mir-libs]
27
28 plugs:
29 platform:

Subscribers

People subscribed via source and target branches