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

Partially revert "Disable localisation when SOURCE_DATE_EPOCH is set (#10949)" #11343

Merged
merged 5 commits into from
Apr 21, 2023

Conversation

AA-Turner
Copy link
Member

Alternative to @jayaddison's #11306.

The comment at #9778 (comment) posits that the reproducability issue is already fixed, but the objects.inv reproducability test is failing -- @jayaddison any suggestions?

A

@jayaddison
Copy link
Contributor

@AA-Turner I think that I miswrote the test, based on incorrect assumptions.

It says: the EN and ET inventories should be identical in a reproducible build, but what that means is: check that localisation is disabled.

That's not a viable fix, in my opinion. Ideally we want localisation to continue to be supported: we can't rule-out the system build locale as a variance factor in the output until there is a way to configure and build for multiple locales (essentially: ignore the system build locale, and build for all/a configured list of locales, producing identical results each time).

In short: I don't feel comfortable disabling localisation in reproducible-builds builds entirely - it seems like the wrong approach.

@AA-Turner AA-Turner mentioned this pull request Apr 21, 2023
@AA-Turner AA-Turner changed the title Partially revert GH-9778 Partially revert "Disable localisation when SOURCE_DATE_EPOCH is set (#10949)" Apr 21, 2023
@AA-Turner AA-Turner merged commit aee3c0a into sphinx-doc:master Apr 21, 2023
22 checks passed
@jayaddison
Copy link
Contributor

Thanks @AA-Turner!

@AA-Turner AA-Turner added this to the 6.2.0 milestone Apr 29, 2023
@AA-Turner AA-Turner deleted the revert-9778 branch May 12, 2023 21:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants