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

Dockerfiles and build utils for Alpine containers #220

Closed
wants to merge 4 commits into from
Closed

Dockerfiles and build utils for Alpine containers #220

wants to merge 4 commits into from

Conversation

Simone-Zabberoni
Copy link
Contributor

Hi, I've added the dockerfiles for Alpine linux, a build script and a small python example to check them.

The 2 small images are ok.
Please check the "dev" one: I've added every package I found to match the ubuntu version but it's still a different platform

Current image size:
csparpa/pyowm.alpine-py2small latest 75809058cd77 11 minutes ago 81.88 MB
csparpa/pyowm.alpine-py3small latest 095eabc2ff69 17 minutes ago 127.8 MB
csparpa/pyowm.alpine-dev latest e10e00787e77 17 minutes ago 328.6 MB

S.

@csparpa
Copy link
Owner

csparpa commented Dec 12, 2017

Thanks @Simone-Zabberoni !

This actually aims to fix #204

Unfortunately, I've noticed that the Travis CI integration of your Pull Request broke. Can you please fix it?

@csparpa
Copy link
Owner

csparpa commented Dec 12, 2017

Also @Simone-Zabberoni I would ask you if you could open the same PR on the develop branch instead of the master branch

The latter is intended for production use only

Thank you very much!

@Simone-Zabberoni
Copy link
Contributor Author

Ok, I'll update the PR on delevop.

The CI integration failed on all pyenv but 3.5 with the same error:

pyenv: python3.5: command not found

The `python3.5' command exists in these Python versions:
  3.5
  3.5.3

even if I did not touch the code, seems to be an issue between Tox and Travis.

I found something here, the suggestion is to switch to tox-travis.
Unfortunately I'm not too skilled on travis yet, but I'll give it a try!

@csparpa
Copy link
Owner

csparpa commented Dec 13, 2017

Grazie!! Good job!

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.

2 participants