-
-
Notifications
You must be signed in to change notification settings - Fork 925
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
EEST timezone problem? #3978
Comments
To be clear: my problem here is not "it doesn't works with this concrete string", but rather "it doesn't work with a string it returned itself". |
I vaguely recall a peculiarity in MRI that the 9-argument form of FWIW, it doesn't work in CDT either:
|
Hmm...we do appear to be looking at the timezone, but it doesn't appear to get set correctly. |
Ok, looks like this is in the group of timezones that Joda Time does not support (because they are considered nonstandard):
I believe there's a way to get a time zone from Joda using these legacy abbreviations. If not, we may want to do a fallback to JDK's timezone logic, which I believe supports them. |
FWIW a quick workaround would probably be for you to use the full canonical name of an appropriate EEST city as your TZ environment. Joda Time prefers that daylight savings is calculated rather than set or forced, which is why it doesn't have support for most of the DST-related timezone abbreviations. I'm testing a fallback to JDK timezone support. |
Ok, the JDK classes don't appear to support these older abbreviations either, and it's likely because they were never standardized and have a number of duplicates: https://en.wikipedia.org/wiki/List_of_time_zone_abbreviations At the moment it seems your best option is using a specific, canonical time zone "country/city" identifier as your timezone rather than EEST or EEDT. Every resources I can find basically says these older abbreviations should not be used anymore. |
Hey, thanks for investigation. Currently, I use fallback to UTC offsets to handle this problem JRuby, but it will not work correctly on DST boundaries. |
Yeah I think our only option here is to find a translation table from "established" short abbreviations to their long form equivalents. |
OK, I was wrong. After further investigation, this |
Thanks for following up on this. I wish there were a better option for those old-style time zones. |
Working on it! (That's how I've found I was wrong previously) :) |
:( MRI works as expected (creates Time in "EEST" timezone).
Other timezones (like, for ex, "EET") works on JRuby as expected.
As a gem author, I'm interested in fixing (or creating fallback for) this problem in generic way rather than "install X on your machine" way.
The text was updated successfully, but these errors were encountered: