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

Shorter manifest #74

Closed
wants to merge 4 commits into from
Closed

Conversation

ted-gould
Copy link
Contributor

Shortening the manifest so that snap packages have more of their own stuff. Basically making less assumptions on the platform. I've also made a small shell script to build all the examples and attached the output. I don't see anything odd, but reviewers should check my work ;-)

$ find . -name "*snap" | xargs -n 1 du -h
868K ./godd/godd_1.0_amd64.snap
240K ./downloader-with-wiki-parts/downloader_1.0_amd64.snap
4.0M ./gopaste/gopaste_1.0_amd64.snap
55M ./py2-project/spongeshaker_0_amd64.snap
90M ./webcam-webui/webcam-webui_1_amd64.snap
185M ./tomcat-maven-webapp/tomcat-webapp-demo_1.0_amd64.snap
139M ./qmldemo/qmldemo_1_amd64.snap
86M ./py3-project/spongeshaker_0_amd64.snap
189M ./ros/ros-example_1.0_amd64.snap
176M ./java-hello-world/java-hello-world_0_amd64.snap
72K ./libpipeline/pipelinetest_1.0_amd64.snap
run.txt

debianutils
diffutils
dmsetup
dpkg
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need this included in snaps?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On Tue, 2015-11-03 at 16:52 -0800, sergiusens wrote:

In snapcraft/manifest.txt:

dash
-debconf
-debconf-i18n
-debianutils
-diffutils
-dmsetup
-dpkg
do we need this included in snaps?
I was thinking for people who may use things like dpkg-archiecture to
setup an environment variable. I'd like to find a way to not include it
unless they're using it explicitly, but I don't see a way to do that.
Perhaps we could filter it by default?

@ted-gould ted-gould closed this Nov 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants