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

configury: make build reproducible #8136

Merged
merged 2 commits into from Oct 28, 2020
Merged

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Oct 26, 2020

If defined, use SOURCE_DATE_EPOCH environment variable make the build
Reproducible by forcing timestamps. See
https://reproducible-builds.org/docs/source-date-epoch/ for more
information.

Thanks Bernhard M. Wiedemann for bringing this to our attention.

Fixes #3759

This is mostly a rebase of #3779 by @ggouaillardet with some feedback from @jsquyres applied.

@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@awlauria
Copy link
Contributor

ok to test

config/opal_functions.m4 Outdated Show resolved Hide resolved
This code was invoked twice.  Leave it solely in OPAL_CONFIGURE_SETUP,
which is invoked before OPAL_BASIC_SETUP.

Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
@jsquyres
Copy link
Member

jsquyres commented Oct 26, 2020

@bmwiedemann I took the liberty of pushing to this branch:

  1. I removed some redundant code from opal_functions.m4 (which affected this PR)
  2. I changed to $srcdir/getdate.sh in a few places
  3. I fixed the perms on getdate.sh

It seems to work. Can you confirm?

README Outdated Show resolved Hide resolved
If defined, use SOURCE_DATE_EPOCH environment variable; make the build
Reproducible by forcing timestamps.  See
https://reproducible-builds.org/docs/source-date-epoch/ for more
information.

Thanks Bernhard M. Wiedemann for bringing this to our attention.

Fixes open-mpi#3759

Signed-off-by: Gilles Gouaillardet <gilles@rist.or.jp>
Signed-off-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
@jsquyres jsquyres merged commit 00ccc9d into open-mpi:master Oct 28, 2020
6 checks passed
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.

builds of openmpi are not yet reproducible
5 participants