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

Allow to override build date with SOURCE_DATE_EPOCH #3112

Merged
merged 1 commit into from
Aug 6, 2023

Conversation

bmwiedemann
Copy link
Contributor

Allow to override build date with SOURCE_DATE_EPOCH to make builds reproducible.

See https://reproducible-builds.org/ for why this is good and https://reproducible-builds.org/specs/source-date-epoch/ for the definition of this variable.

Without this patch, I got variations in our openmpi5 package:

+++ new//usr/lib64/mpi/gcc/openmpi5/share/doc/pmix/html/developers/frameworks.html      2023-07-25 00:00:00.000000000 +0000
@@ -109,7 +109,7 @@
 – most services and functionality are implemented through MCA
 components.</p>
 <p>Here is a list of all the component frameworks in PMIx as of
-this writing August-2023:</p>
+this writing September-2039:</p>

This patch was done while working on reproducible builds for openSUSE.

Signed-off-by: Bernhard M. Wiedemann bwiedemann suse de

in order to make builds reproducible.
See https://reproducible-builds.org/ for why this is good
and https://reproducible-builds.org/specs/source-date-epoch/
for the definition of this variable.

This patch was done while working on reproducible builds for openSUSE.

Signed-off-by: Bernhard M. Wiedemann <[email protected]>
@rhc54
Copy link
Contributor

rhc54 commented Aug 3, 2023

We already look for and set SOURCE_DATE_EPOCH during configure. @jsquyres - do we need to simply capture that setting in the docs generator?

@rhc54
Copy link
Contributor

rhc54 commented Aug 4, 2023

bot:ibm:retest

@rhc54
Copy link
Contributor

rhc54 commented Aug 4, 2023

Interestingly enough, this patch now allows someone to build the PMIx docs and claim copyright into the future years by simply setting an envar - see open-mpi/ompi#11847 which raises that issue.

Should there be logic to disallow passing a future date?

@rhc54
Copy link
Contributor

rhc54 commented Aug 6, 2023

Guess we allow this envar to override the build date elsewhere, so might as well be consistent - though I still have no idea how you got the error you report (unless you deliberately set your system clock to some bizarre future time?).

@rhc54 rhc54 merged commit 7271b21 into openpmix:master Aug 6, 2023
@bmwiedemann bmwiedemann deleted the date2 branch August 20, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants