Skip to content

Delete Python test files like in Python image #23

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

Merged
merged 1 commit into from
Apr 30, 2018

Conversation

JayH5
Copy link
Contributor

@JayH5 JayH5 commented Apr 28, 2018

There are other files that could be cleaned up here too (e.g. there's at least one bundled pip in lib-python), but this is a start. This doesn't delete .pyc/.pyo files like in the Python image since PyPy doesn't produce those.

This saves ~15MB of uncompressed image size for PyPy 2 for me.

@tianon
Copy link
Member

tianon commented Apr 30, 2018

Nice! Any reason you've chosen /usr/local/lib-python explicitly instead of just doing all of /usr/local like the Python images do?

@JayH5
Copy link
Contributor Author

JayH5 commented Apr 30, 2018

Just that I don’t know that much about the PyPy source but lib-python seems to be the vendored Python stdlib which we do clean in the Python image.

@tianon
Copy link
Member

tianon commented Apr 30, 2018

Good enough for me, thanks! 👍

@tianon tianon merged commit 802584a into docker-library:master Apr 30, 2018
tianon added a commit to infosiftr/stackbrew that referenced this pull request May 3, 2018
- `buildpack-deps`: add `netbase`
- `gcc`: 8.1.0
- `ghost`: 1.22.5, 0.11.13
- `haproxy`: 1.7.11
- `mariadb`: 10.0.35
- `mongo`: 3.7.9
- `mysql`: docker-library/mysql#402
- `openjdk`: 11-ea-11, 8u171
- `php`: docker-library/php#627, docker-library/php#632
- `pypy`: docker-library/pypy#23
- `python`: add `netbase`, 3.7.0b4
- `rabbitmq`: 3.7.5-rc.1
- `rocket.chat`: 0.64.1
@JayH5 JayH5 deleted the rm-python-tests branch May 16, 2018 21:05
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