Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

interface 'platform' doesn't exist #1123

Closed
Quacky2200 opened this issue Jan 16, 2017 · 11 comments · May be fixed by qcif/data-curator#563
Closed

interface 'platform' doesn't exist #1123

Quacky2200 opened this issue Jan 16, 2017 · 11 comments · May be fixed by qcif/data-curator#563

Comments

@Quacky2200
Copy link

electron-builder version 11.2.6

64 bit and 32 bit Linux targets

I am building my snap with this configuration

...
    "linux": {
      "synopsis": "shortened...",
      "category": "Audio",
      "packageCategory": "GNOME;GTK;AudioVideo;Audio;Player",
      "depends": [
        "libappindicator1",
        "libindicator7",
        "libnotify4",
        "notify-osd",
        "wget",
        "unzip",
        "tar"
      ],
      "target": [
        "AppImage",
        "deb",
        "zip",
        "snap"
      ]
    }
  },
  "snap": {
      "confinement": "devmode",
      "grade": "stable"
  }, ...

I had my package reviewed and it has an interface named 'platform' - where did this come from and how do I remove it because it's automatically generated with electron-builder. Do I need to configure my snap differently or is this a bug?

Evan Dandrea evan.dandrea at canonical.com 
Mon Jan 16 09:12:30 UTC 2017

Hi Matthew,

You've set your snap to use the 'platform' interface, but no such interface
exists. If you remove that line and 'unity8', then re-upload, it should
pass review.

Do you recall what you read that referenced a 'platform' interface? If
there's some outdated documentation out there, I'd like to get it fixed.

Let us know if you need any more help, and thanks for snapping Spotify!

@develar
Copy link
Member

develar commented Jan 16, 2017

  1. https://github.com/ubuntu/snapcraft-desktop-helpers/blob/master/snapcraft.yaml#L35

  2. https://insights.ubuntu.com/2016/12/08/using-the-ubuntu-app-platform-content-interface-in-app-snaps/

"Using the ‘platform’ plug (content interface) and specifying its default provider (‘ubuntu-app-platform’)"

Bug: this plug should be not added if ubuntuAppPlatformContent option is not set.

@develar develar added the bug label Jan 16, 2017
@develar
Copy link
Member

develar commented Jan 16, 2017

Workaround until fixed version is not published: #509 (comment)

  1. build snap using electron-builder
  2. edit generated snapcraft.yaml in the dist/linux-unpacked-snap
  3. run snapcraft snap (current working directory must be dist/linux-unpacked-snap).

linux-unpacked-snap is not deleted after build for this purpose.

@Quacky2200
Copy link
Author

Yeah I see that I can edit this, will I need to remove unity7 as well since I'm also removing unity 8?

@develar
Copy link
Member

develar commented Jan 16, 2017

Why unity8 is not suitable? unity7 should be here in any case, I think. Please ask why unity8 is bad.

@Quacky2200
Copy link
Author

It just doesn't pass the reviewing stage on my snap. I will email them, it may just be that unity8 is not ready for my application?

@develar
Copy link
Member

develar commented Jan 16, 2017

unity8 is removed, platform is added only if ubuntuAppPlatformContent is specified. Eager to know about is it valid or not.

@develar
Copy link
Member

develar commented Jan 16, 2017

Fixed in 11.4.3

@develar develar added the snap label Jan 16, 2017
@Quacky2200
Copy link
Author

What is ubuntuAppPlatformContent supposed to refer to? Scopes?

@Quacky2200
Copy link
Author

Thanks.

It looks like they just use it to save space for Qt apps due to Qt being supported mainstream by Ubuntu. Interesting... 👍

@develar
Copy link
Member

develar commented Jan 16, 2017

@Quacky2200 Is is not only Qt, but gconf/gtk/other weird stuff that is required to run electron apps (UI apps). Currently, usability is a nightmare.

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

Successfully merging a pull request may close this issue.

2 participants