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.

Sign up for GitHub

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

Jump to bottom

#24 ADded tzdata package #25

Closed
wants to merge 3 commits into from
Closed

#24 ADded tzdata package #25

wants to merge 3 commits into from

Conversation

krufab
Copy link
Contributor

@krufab krufab commented Mar 27, 2020

Added tzdata package so that the TZ env variable is handled correctly.

docker run --rm -it -e "TZ=Europe/Luxembourg" 2317ceea7eb3 date
Fri Mar 27 22:58:50 CET 2020
docker run --rm -it -e "TZ=Europe/London" 2317ceea7eb3 date
Fri Mar 27 21:58:56 GMT 2020

Fixes #24

Fabio Kruger added 3 commits March 27, 2020 22:57
postfixadmin#24 ADded tzdata package
b024827 
Signed-off-by: Fabio Kruger <krufab@gmail.com>
postfixadmin#24 Added package tzdata also to Dockerfile-alpine.template
4f35868 
Signed-off-by: Fabio Kruger <krufab@gmail.com>
Resolved conflicts
2032396
@krufab
Copy link
Contributor Author

krufab commented Apr 18, 2020

Resolved conflicts.

@DavidGoodwin
Copy link
Member

Looks fine to me.... @J0WI ?

@J0WI
Copy link
Collaborator

J0WI commented Apr 9, 2021

IMHO this should be fixed in docker-library/php#949

@DavidGoodwin
Copy link
Member

ok - then we might as well close this .... with a 'fixed upstream' type comment :) thanks.

@krufab
Copy link
Contributor Author

krufab commented Apr 10, 2021 •
edited
Loading

Pity, I've this mail server set up with nginx, roundcube, postfixadmin and phpmyadmin, and the only image i've to build is this one to add tzdata.
But I rest my case...
https://github.com/krufab/mailpine/blob/main/apps/web/docker-compose.yml#L44
https://github.com/krufab/mailpine/blob/main/apps/web/pa/Dockerfile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Alpine misses the tzdata package and does not handle timezone correctly
3 participants
@krufab @DavidGoodwin @J0WI