Alternative path to tzdb.dat

[Andrew Hughes]

On 30 September 2017 at 21:06, Fridrich Strba <fridrich.strba at suse.com> wrote:
> Hello, Andrew,
>
> On 30/09/17 20:41, Andrew Hughes wrote:
>> We did try to get the original patch upstream some time ago, but they
>> were reluctant to accept such a change if a symlink would do. We could
>> try again though.
>
> Yeah, the symlink is less then optimal solution. You have to symlink in
> post-install phase, and then you have to have the system-wide tzdb.dat
> present. That one needs a tzdb tool that needs Java. So the chicken-egg
> problem comes.
>
> With this patch, one can filter out the timezone-java package in build
> environment when building the tzdb tool and still the user has the
> advantage of fresh timezone information as the system-wide one is
> updated on every release of the data...
>
>>
>> The only thing I note about the patch is that you retrieve homeDir via
>> a doPrivileged call, when you're already inside a doPrivileged block.
>> Was this intentional?
>
> Nah, just an oversight. Should be reading more of the context. Attached
> is the new, simplified patch.
>
> Cheers
>
> F.


Thanks. I've filed a bug and will try and include it in the upcoming
release, 3.6.0.

https://icedtea.classpath.org/bugzilla/show_bug.cgi?id=3469
-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Web Site: http://fuseyism.com
Twitter: https://twitter.com/gnu_andrew_java
PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222