Skip to content

Adjust whitespace for better symmetry between Dockerfiles and cut image size in half by keeping fetch and compile in the same layer so the Python source goes away properly #1

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
Dec 31, 2015

Conversation

tianon
Copy link

@tianon tianon commented Dec 30, 2015

No description provided.

@tianon
Copy link
Author

tianon commented Dec 30, 2015

Before: ~163.2 MB
After: ~89.05 MB

| while read dep; do \
apk info --installed -q "$dep" && echo "$dep"; \
done \
| sort -u \
Copy link
Author

Choose a reason for hiding this comment

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

Is this second sort -u actually necessary? Our while loop has just filtered the output of the first sort, so this one should be a no-op.

Copy link
Author

Choose a reason for hiding this comment

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

Nevermind, figured out something simpler that skips the loop. 😄

@tianon tianon force-pushed the alpine-tweaks branch 2 times, most recently from b7ef92d to f4cf226 Compare December 30, 2015 23:36
…ge size in half by keeping fetch and compile in the same layer so the Python source goes away properly

Before: ~163.2 MB
After: ~89.05 MB
ncopa added a commit that referenced this pull request Dec 31, 2015
Adjust whitespace for better symmetry between Dockerfiles and cut image size in half by keeping fetch and compile in the same layer so the Python source goes away properly
@ncopa ncopa merged commit a418b4e into ncopa:alpine Dec 31, 2015
@ncopa
Copy link
Owner

ncopa commented Dec 31, 2015

thanks! 👍

@tianon tianon deleted the alpine-tweaks branch December 31, 2015 18:04
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