On Tue, 25 Feb 2025 17:11:18 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> Hello Naoto, this looks reasonable to me. Do you think we should add a 
>> release note for this change?
>
>> Hello Naoto, this looks reasonable to me. Do you think we should add a 
>> release note for this change?
> 
> Good point. I will draft a release note for this change.

> @naotoj Do you know what was the last Debian release where this file was 
> used? I'm looking at JDK-6456628 to try to understand if any of the distros 
> from that needed this code have any supported releases that might need it.

Here is a quote from this Debian bug report: 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=822733


This now leaves /etc/timezone completely redundant, as you should get
exactly the same answer by readlink /etc/localtime -- and if you do
not get the same answer, you have an inconsistency. /etc/localtime
obviously "wins" for the actual clock (as that's what programs are
reading), but you presumably get the /etc/timezone value in some
"system configuration tool" packages which read /etc/timezone first.


This seems to imply that as of 2016, /etc/timezone is a redundant file. Our 
supported Debian distros (as of JDK23) are Ubuntu 24.04LTS and 24.10, so I 
think we are OK just remove the support for /etc/timezone

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23755#issuecomment-2682711288

Reply via email to