-
Notifications
You must be signed in to change notification settings - Fork 7
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
Dev #3
Dev #3
Conversation
This doesn't solve the main problem commented at #1 (comment). It just removes the mail support (which I was planning anyway) and the apt-get clean, but at the same time, you didn't remove /tmp and other temporary files which should not be necessary. I already worked on a fix that does the same things, I'm still testing and I'll publish this week. |
I made a pull request by mistake my friend, no intended to solve any issue yet. But now that you're on the thread, It will be good to have a glimpse on that fix so I can test and assist myself |
ca-certificates \ | ||
openssl \ | ||
sendmail --no-install-recommends \ | ||
&& rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removal of unnecessary files
Basically you need to upgrade to Docker 1.11 to have this issue, you can read about the problem at: docker-library/php#74 and the solution is to use |
The removal of unnecessary files is in line 40 |
Revert sendmail, remove volume, and refactor dockerfile to reduce size
No description provided.