-
Notifications
You must be signed in to change notification settings - Fork 83
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
Has anyone gotten this working on alpine linux? #94
Comments
@bdentino I'm having the same issue. Did you get any further with this issue? |
Running |
We tried to use use here on Alpine and find out that tzset has different results between musl glibc (alpine) and ubuntu glibc.
This simple snippet will print:
When the time zone has Daylight Saving Time, it will return the name for the time zone when Daylight Saving Time is in use for both. Based on this link, I believe that musl glibc is correct and you should not use tzname[1] to validate the time zone, but I am not sure about this. |
@TooTallNate I see that there has been a commit (2739eac) regarding alpine. |
That commit doesn't seem to fix the issue. @TooTallNate here's the debug ouput I get on alpine
|
This is not related to this library, node or other languages, for some reason someone thumbed down @Yumonra07 answer, on install tzdata, and that solves the issue, on other languages as well such as python for instance. |
Hey guys, This issue is still open it seems? My best attempt at the previous two solutions didn't seem to work for me, still getting the 'unknown timezone' error for UTC. |
Having trouble running a node app that uses this library in an alpine-based docker container, as I'm getting the 'unknown timezone' error. Debug output shows it's trying to find a proper zoneinfo directory and failing to do so. Most likely because of this: https://wiki.alpinelinux.org/wiki/Alpine_Linux:FAQ#How_do_I_set_the_local_timezone.3F
Is there any chance of this being supportable on alpine? I'd submit a PR if someone could point me in the right direction, but not sure if it's even feasible without significant rewrites.
The text was updated successfully, but these errors were encountered: