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 #7814

Closed
wants to merge 1 commit into from

Conversation

bmwiedemann
Copy link
Contributor

Allow to override build date with SOURCE_DATE_EPOCH
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 date call is designed to work with all variants of date.

Also use UTC to be independent of timezone.

As an alternative, the date could be dropped to reduce complexity.

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

@trilinos/

Motivation

Stakeholder Feedback

Testing

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 date call is designed to work with all variants of date.

Also use UTC to be independent of timezone.

This PR was done while working on reproducible builds for openSUSE.
@trilinos-autotester
Copy link
Contributor

Status Flag 'Pre-Test Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging
WARNING: NO REVIEWERS HAVE BEEN REQUESTED FOR THIS PULL REQUEST!

@bmwiedemann
Copy link
Contributor Author

@crtrott can you please review this? It seems, you added the date in commit 4dcdead

@jhux2
Copy link
Member

jhux2 commented Aug 29, 2020

@bmwiedemann This should probably be opened in the Kokkos project, https://github.com/kokkos/kokkos, since Kokkos is snapshotted into Trilinos.

@bmwiedemann
Copy link
Contributor Author

Forwarded it to kokkos/kokkos#3327

@william76
Copy link
Contributor

@bmwiedemann @jhux2
Can this PR be closed if it's been passed over to Kokkos?

@jhux2
Copy link
Member

jhux2 commented Sep 15, 2020

Closing.

@jhux2 jhux2 closed this Sep 15, 2020
@crtrott
Copy link
Member

crtrott commented Sep 18, 2020

We are removing the date. Btw. this wasn't affecting Trilinos anymore since January, since Makefile.kokkos was not used after the 3.0 release.

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.

None yet

5 participants