snapcraft.yaml: add desktop-platform-gnome part #46

Merged
merged 3 commits into from Feb 14, 2017

Conversation

Projects
None yet
2 participants
Contributor

3v1n0 commented Feb 9, 2017

It was just called gnome before

As we discussed on IRC:

  • please update the help text on top
  • change the help text from runtime -> platform (same with content and such)
  • please update the name to desktop-gnome-platform
snapcraft.yaml
@@ -41,7 +41,7 @@ description: |
target: ubuntu-app-platform
and then make your apps use it:
plugs: [platform]
- - similarly for the gnome runtime
+ - similarly for the gnome platform based apps
plugs:
gnome-runtime:
@didrocks

didrocks Feb 9, 2017

Owner

use platform here (and below) as we are going to change that to the platform snap
Please change it in the helper as well with backward compat (test the dir existance)

@3v1n0 3v1n0 changed the title from Add desktop-gnome part to snapcraft.yaml: add desktop-platform-gnome part Feb 9, 2017

Here are the changes we discussed about.

echo ""
echo "You can do this with those commands:"
echo "snap install --edge gnome318-udt"
- echo "snap connect $SNAP_NAME:gnome318-runtime gnome318-udt:gnome318-runtime"
+ echo "snap connect $SNAP_NAME:gnome318-platform gnome318-udt:gnome318-platform"
@didrocks

didrocks Feb 10, 2017

Owner

s/gnome318-platform/platform/g (both)

- gnome-runtime:
- content: gnome318-runtime
- default-provider: gnome318-udt:gnome318-runtime
+ gnome-platform:
@didrocks

didrocks Feb 10, 2017

Owner

s/gnome-platform/platform/

- default-provider: gnome318-udt:gnome318-runtime
+ gnome-platform:
+ content: gnome318-platform
+ default-provider: gnome318-udt:gnome318-platform
@didrocks

didrocks Feb 10, 2017

Owner

s/gnome318-platform/platform/

and then make your apps use it:
- plugs: [gnome-runtime]
+ plugs: [gnome-platform]
@didrocks

didrocks Feb 10, 2017

Owner

s/gnome-platform/platform/

@didrocks didrocks merged commit 2289e89 into Ubuntu:master Feb 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment