Add snapcraft.yaml, dependencies for building juju as a snap #5956

Merged
merged 14 commits into from Aug 11, 2016

Conversation

Projects
None yet
3 participants
Owner

nskaggs commented Aug 9, 2016

Upstreaming snapcraft.yaml file. The snapcraft.yaml file I'm checking in here re-pulls the source, as the plugin doesn't seem to yet support using the local source. The readme also could be improved once the plugin is improved in order to mention how to make a developer build and share it via the snap store.

Owner

nskaggs commented Aug 9, 2016

In order to try this, you need the godeps plugin. Since there still hasn't been a new release of snapcraft, grab it from master.

sudo curl -o /usr/lib/python3/dist-packages/snapcraft/plugins/godeps.py https://raw.githubusercontent.com/snapcore/snapcraft/master/snapcraft/plugins/godeps.py

@nskaggs nskaggs changed the title from Add snapcraft.yaml for building juju as a snap to Add snapcraft.yaml, dependencies for building juju as a snap Aug 9, 2016

Owner

nskaggs commented Aug 9, 2016

Added juju/cmd#40, juju/romulus#59, juju/utils#230 PR's in support

jujubot added a commit to juju/cmd that referenced this pull request Aug 9, 2016

Merge pull request #40 from nskaggs/master
Update gnuflag dependency

In support of juju/juju#5956
Contributor

natefinch commented Aug 9, 2016

LGTM after adding in the updates to juju/cmd and juju/utils, once they get the same update to remove launchpad deps.

jujubot added a commit to juju/utils that referenced this pull request Aug 9, 2016

Merge pull request #230 from nskaggs/master
update tomb dependency

In support of juju/juju#5956
Owner

nskaggs commented Aug 9, 2016

$$merge$$

Contributor

jujubot commented Aug 9, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 9, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8687

Owner

nskaggs commented Aug 10, 2016

$$merge$$

Contributor

jujubot commented Aug 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 10, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8696

Owner

nskaggs commented Aug 10, 2016

$$merge$$

Contributor

jujubot commented Aug 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 10, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8697

Owner

nskaggs commented Aug 10, 2016

$$merge$$

Contributor

jujubot commented Aug 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8699

Owner

nskaggs commented Aug 10, 2016

$$merge$$

Contributor

jujubot commented Aug 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8700

Owner

nskaggs commented Aug 10, 2016

$$merge$$

Contributor

jujubot commented Aug 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8702

Contributor

jujubot commented Aug 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8706

jujubot added a commit that referenced this pull request Aug 11, 2016

Merge pull request #5974 from nskaggs/snapcraft-only
Add snapcraft.yaml, information on building juju as snap

See #5956
Owner

nskaggs commented Aug 11, 2016

Splitting up this PR in an attempt to isolate what is causing the unit test failure.

Owner

nskaggs commented Aug 11, 2016

$$merge$$

Contributor

jujubot commented Aug 11, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 11, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8733

@jujubot jujubot merged commit 911de19 into juju:master Aug 11, 2016

@nskaggs nskaggs referenced this pull request Aug 11, 2016

Merged

update romulus depends #5982

jujubot added a commit that referenced this pull request Aug 11, 2016

Merge pull request #5982 from nskaggs/update-romulus-depends
update romulus depends

This is in support or #5956
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment