Merge lp:~mhall119/click-reviewers-tools/snap-package into lp:click-reviewers-tools

[Michael Hall]

Adds snapcraft.yaml which will produce a classic-confinement snap for these review tools

[Sergio Schvezov (sergiusens)]

If you ever want this to work on trusty or future versions of Ubuntu you will need to build your own python for which snapcraft will setup the correct program loaders. Look at the snapcraft.yaml for snapcraft or conjure-up for more information.

review: Needs Fixing

[Michael Hall (mhall119)]

Why won't this use the python3 bundled in the snap on those systems?

[Jamie Strandboge (jdstrand)]

Thanks for this! Note that there is already a 'review-tools' snap in the store and it is being worked on under a separate project name:

https://launchpad.net/review-tools

It is under 'git' and aims to work with strict confinement since running something on arbitrary input like a squashfs could really benefit from confinement (aka, we want the store to use this snap and benefit from confinement).

That snap (sudo snap install --edge) is non-functional though due to recent changes to the preload part. Help is appreciated if you are interested:

https://code.launchpad.net/~myapps-reviewers/review-tools/+git/review-tools

Disapproving not because snapping the review tools isn't desired, but because this is the wrong project/branch.

review: Disapprove

[Sergio Schvezov (sergiusens)]

To answer Michael for completeness, it will use the python bundled in the snap, but using a python from a stage-packages entry (in the plugin) will use the wrong program loader and be compiled with the wrong library flags making it dlopen things from the host system essentially and potentially causing crashes on systems with different glibc versions.

Unmerged revisions

870. By Michael Hall on 2017-04-11

Add snapcraft.yaml to build snap-reviewers-tools with snap-review command aliased

=== added directory 'snap'
=== added file 'snap/snapcraft.yaml'
--- snap/snapcraft.yaml	1970-01-01 00:00:00 +0000
+++ snap/snapcraft.yaml	2017-04-11 19:24:20 +0000
@@ -0,0 +1,28 @@
+name: snap-reviewers-tools
+version: '0.47'
+summary: Tools to review Snap packages
+description: |
+  These scripts can be used to review snap packages both manually and in a
+  programmatic fashion.
+
+grade: stable
+confinement: classic
+
+apps:
+  snap-review:
+    command: bin/click-review
+    plugs: []
+    aliases: [snap-review]
+    
+parts:
+  click-reviewers-tools:
+    plugin: python
+    python-version: python3
+    source: .
+    stage-packages:
+        - python3-lxml
+        - python3-magic
+        - python3-simplejson
+        - python3-xdg
+        - python3-yaml
+        - squashfs-tools