Re: [tz] [tz-announce] 2021b release of tz code and data available
On 9/26/21 3:38 AM, Michael H Deckers wrote:
Link America/Port_of_Spain America/Virgin in line 755 of backzone should be converted to Link America/St_Thomas America/Virgin
That would create a link-to-a-link (America/Virgin → America/St_Thomas → America/Puerto_Rico), which would cause "make check_links" to fail. This maintainer check was put in several years ago, when some downstream software misbehaved with a link-to-a-link in one of our releases. And I suspect that even zic is not robust in the presence of arbitrary links-to-links, though I haven't tested this. I do see the semantic issue behind the change you're proposing, as America/Virgin is an old alias for America/St_Thomas. If we wanted to add support for links-to-links, I suspect someone would have to code up and test changes to zic.c, and we would definitely need to give downstream users considerable time to adjust to the change. I'm not sure it's worth it merely to support this very minor issue.
On 2021-09-26 15:37, Paul Eggert wrote:
That would create a link-to-a-link (America/Virgin → America/St_Thomas → America/Puerto_Rico),
No, it would not. I mean the link in the file "backzone", line 755, and America/St_Thomas is not a link in backzone. Michael Deckers.
On 9/26/21 1:42 PM, Michael H Deckers wrote:
No, it would not. I mean the link in the file "backzone",
Oh, sorry, I was so focused on 2021a-to-2021b changes that I assumed your report was about 'backward'. You're right, 'backzone' is incorrect here. Proposed patch attached, and installed in the development repository. Thanks for following up.
participants (2)
-
Michael H Deckers -
Paul Eggert