qml plugin: make it combinable with apps which use other build systems to install their files

Bug #1510008 reported by Daniel Holbach on 2015-10-26
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Snapcraft
Undecided
Unassigned

Bug Description

Alan put together a branch which builds lp:music-app using the cmake plugin (https://bazaar.launchpad.net/~popey/+junk/music-app-snap/view/head:/snapcraft.yaml). This works just fine, but a lot of dependencies are not bundled, so I tried to use the qml plugin. This works fine, but the music-app expects some files in the right place (like translations), for which it uses cmake.

Trying to use both (snapcraft.yaml snippet below), you run into the problem that both try to install the same files (the qml plugin installs * - https://bazaar.launchpad.net/~snappy-dev/snapcraft/core/view/head:/snapcraft/plugins/qml.py#L67), which breaks the build. Maybe the qml plugin could be more selective in what exactly it tries to bundle?

parts:
  music-build:
    plugin: cmake
    source: https://launchpad.net/music-app
    source-type: bzr
    snap:
      - icon.png
  music-app:
    plugin: music-qml
    after: music-build

Ted Gould (ted) wrote :

I'm a bit confused. If snapcraft sees that two parts have the same file in the same location it won't have an issue with that. So it wouldn't matter whether all the packages were installed or not.

Also, the snapcraft.yaml snippet here includes a custom plugin, which you should attach, but also shouldn't be needed. The version in the repo doesn't have the custom plugin.

Last comment, this should really be in the repo for the music app. There's no reason to make another repository just for the snapcraft file.

Daniel Holbach (dholbach) wrote :

Ok... should something like this work lp:~dholbach/music-app/initial-snapcraft-work?

Leo Arias (elopio) wrote :
Changed in snapcraft:
status: New → Invalid
Changed in snapcraft:
status: Invalid → Triaged
Sergio Schvezov (sergiusens) wrote :

We are going to remove the qml plugin

Changed in snapcraft:
status: Triaged → Invalid
status: Invalid → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers