Merge lp:~dholbach/snapcraft/1498347 into lp:~snappy-dev/snapcraft/core

Proposed by Daniel Holbach
Status: Merged
Approved by: Sergio Schvezov
Approved revision: 193
Merged at revision: 195
Proposed branch: lp:~dholbach/snapcraft/1498347
Merge into: lp:~snappy-dev/snapcraft/core
Diff against target: 24 lines (+7/-3)
1 file modified
examples/godd/snapcraft.yaml (+7/-3)
To merge this branch: bzr merge lp:~dholbach/snapcraft/1498347
Reviewer Review Type Date Requested Status
Sergio Schvezov Approve
Review via email: mp+271921@code.launchpad.net

Commit message

stage package libgudev-1.0-0, which makes the godd snap actually work

To post a comment you must log in.
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

Thanks for this, but I don't understand :-)

This is strange, it would either be needed as a stage-package or as a build-package; if introducing it as a stage-package (to get a shared lib in the end) you should introduce fileset usage to not have the snap full of development headers and remove the build-package.

review: Needs Information
Revision history for this message
Daniel Holbach (dholbach) wrote :

Not sure I understand. AFAICS you need libgudev-1.0-dev installed as part of build-packages (for the build to pass locally) AND to bundle libudev-1.0-0 for the snap to work on a snappy system.

lp:~dholbach/snapcraft/1498347 updated
193. By Daniel Holbach

merged lp:~mvo/snapcraft/1498347

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

and here's the real real one ;-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'examples/godd/snapcraft.yaml'
2--- examples/godd/snapcraft.yaml 2015-09-21 05:04:31 +0000
3+++ examples/godd/snapcraft.yaml 2015-09-22 13:50:22 +0000
4@@ -5,12 +5,16 @@
5 godd:
6 exec: ./bin/godd
7 summary: Simple dd like tool
8-description: written in go so it is self contained and really portable, you would need to use hw-assign to access devices.
9+description: Written in go with support for device auto-detection via libgudev, you would need to use hw-assign to access devices.
10 icon: icon.png
11
12 parts:
13 godd:
14 type: go-project
15 source: git://github.com/mvo5/godd
16-build-packages:
17- - libgudev-1.0-dev
18+ stage-packages: [libgudev-1.0-dev]
19+ snap:
20+ - usr/lib/x86_64-linux-gnu/libgudev-1.0.so*
21+ - usr/lib/x86_64-linux-gnu/libobject-2.0.so*
22+ - usr/lib/x86_64-linux-gnu/libglib-2.0.so*
23+ - bin/godd*
24\ No newline at end of file

Subscribers

People subscribed via source and target branches