Skip to content

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
@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.

2 participants