Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not perform any timezone conversions if SOURCE_DATE_EPOCH is set #13203

Merged
merged 3 commits into from
Jan 2, 2025

Conversation

mitya57
Copy link
Contributor

@mitya57 mitya57 commented Jan 2, 2025

Subject: Do not perform any timezone conversions if SOURCE_DATE_EPOCH is set

Feature or Bugfix

  • Bugfix

Detail

Since PR #12907, various packages which use Sphinx stopped passing reproducibility test in Debian. In that test, we try to build the package in various environments (which includes varying the time zone), and make sure that the output is byte-to-byte identical.

However, that PR made html_last_updated_time_zone == "local" by default, which means the generated output depends on the time zone the package is built in. To solve this problem, I propose to not do any timezone conversion when SOURCE_DATE_EPOCH is set. When that variable is set, people most likely want a reproducible result, which should not depend on the time zone.

Relates

@AA-Turner AA-Turner added type:enhancement enhance or introduce a new feature internals:config labels Jan 2, 2025
@AA-Turner AA-Turner added this to the 8.2.0 milestone Jan 2, 2025
@AA-Turner AA-Turner merged commit de4d317 into sphinx-doc:master Jan 2, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internals:config type:enhancement enhance or introduce a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants