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: Allow to override build date+host+user #1560

Merged
merged 1 commit into from Dec 6, 2019

Conversation

bmwiedemann
Copy link
Contributor

@bmwiedemann bmwiedemann commented Dec 6, 2019

Allow to override build date+host+user 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.

Also switch to UTC to be independent of timezone.
Also use ISO 8601 date format to be easier to parse.

Note: This date call is designed to work with different flavors
of date (GNU, BSD and others).
If only GNU (Linux) support is needed, the patch can be simplified.

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

For reproducible builds, it would also be a good solution to just drop one or both of these date+user+host to avoid the added complexity.

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.

Also switch to UTC to be independent of timezone.
Also use ISO 8601 date format to be easier to parse.

Note: This date call is designed to work with different flavors
of date (GNU, BSD and others).
If only GNU (Linux) support is needed, the patch can be simplified.

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

Signed-off-by: Bernhard M. Wiedemann <bwiedemann@suse.de>
@rhc54
Copy link
Contributor

rhc54 commented Dec 6, 2019

Thanks!

@rhc54 rhc54 merged commit 3999206 into openpmix:master Dec 6, 2019
@jjhursey
Copy link
Member

jjhursey commented Dec 6, 2019

This should be PR'ed to the v3.1 branch. We can get it in the pipeline for v3.1.5

@jjhursey
Copy link
Member

jjhursey commented Dec 6, 2019

Ah Ralph beat me to it :) PR #1561

@bmwiedemann bmwiedemann deleted the date branch December 8, 2019 19:16
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

3 participants