Ok - good point on the stat change Alan. I think this is what you're after :
http://cr.openjdk.java.net/~coffeys/webrev.7149608.jdk8.2/
regards,
Sean.
On 12/03/12 11:04, Alan Bateman wrote:
On 09/03/2012 16:00, Seán Coffey wrote:
Issue seen on somewhat irregular linux system configuration where
/etc/localtime is a symbolic link to a directory outside of
/usr/share/zoneinfo.
In past, when a symbolic link was seen, the end target file was
assumed to be under /usr/share/zoneinfo and a string comparison match
was attempted. We need to factor in the case where symbolic link
points to any other location and match the end target file with a TZ
file from
/usr/share/zoneinfo via findZoneinfoFile function.
webrev : http://cr.openjdk.java.net/~coffeys/webrev.7149608.jdk8/
bug report : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7149608
regards,
Sean.
cc'ing i18n-dev as that is where the TZ code is maintained.
The changes look okay but I think it would be more efficient to fstat
the zone info file after opening it.
-Alan