Page MenuHomeFreeBSD

tzcode: garbage collect references to tzsetwall
AbandonedPublic

Authored by val_packett.cool on Feb 6 2023, 8:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 23, 10:24 AM
Unknown Object (File)
May 13 2024, 5:26 PM
Unknown Object (File)
May 9 2024, 3:53 AM
Unknown Object (File)
Apr 25 2024, 9:34 PM
Unknown Object (File)
Apr 18 2024, 9:24 PM
Unknown Object (File)
Mar 29 2024, 6:13 AM
Unknown Object (File)
Jan 2 2024, 8:22 PM
Unknown Object (File)
Dec 23 2023, 2:59 AM
Subscribers

Details

Reviewers
des
emaste
Group Reviewers
Contributor Reviews (src)
Summary

It was removed with the recent update to upstream tzcode, but references in the header, symbol map and man pages remained.

Fixes: bc42155199 ("Bring our tzcode up to date.")
Reported by: ld.lld 16 being --no-undefined-version by default
Sponsored by: https://www.patreon.com/valpackett

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 49575
Build 46465: arc lint + arc unit

Event Timeline

I honestly hadn't noticed that the tzcode update removed tzsetwall(3).

On the one hand, it's useless and nobody should be calling it.

On the other hand, we've had it for 30+ years and it's arguably part of our ABI.

On the gripping hand, this was merged to stable/13 eight days ago and nobody (including the package builders) noticed.

What do you think @emaste @imp? Should we reintroduce it as an alias for tzset(3), or implement something ourselves? I think the closest thing to the original spirit is “like tzset(3) but ignores the TZ environment variable, taking only /etc/localtime into account.”

@val_packett.cool can you file a PR instead and abandon this review?

Discussed briefly on IRC - suggestion was to make tzsetwall an alias for tzset, with linker magic to emit a deprecation warning