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

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 renames BUILD_INFO to the standard name
so that distributions do not have to discover the BUILD_INFO variable,
similar to cri-o/cri-o#2643

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

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 renames BUILD_INFO to the standard name
so that distributions do not have to discover the BUILD_INFO variable,
similar to cri-o/cri-o#2643

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

@bmwiedemann Thanks for the PR! Our CI requires commits to be signed. If you did not do '-s' with your git commit, could you please do git commit --amend -s and the repush?

@rh-atomic-bot
Copy link
Collaborator

Can one of the admins verify this patch?
I understand the following commands:

  • bot, add author to whitelist
  • bot, test pull request
  • bot, test pull request once

@bmwiedemann
Copy link
Contributor Author

@TomSweeneyRedHat the commit has a Signed-off-by line (and a gpg sig). I just omitted in the PR message for readability.

@rhatdan
Copy link
Member

rhatdan commented Aug 20, 2019

bot, add author to whitelist

@rhatdan
Copy link
Member

rhatdan commented Aug 20, 2019

LGTM
@rh-atomic-bot r+
Thanks @bmwiedemann

@rh-atomic-bot
Copy link
Collaborator

📌 Commit f8ec5cd has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

Testing commit f8ec5cd with merge fa68ed6...

@TomSweeneyRedHat
Copy link
Member

LGTM assuming happy tests, tyvm @bmwiedemann !

@rh-atomic-bot
Copy link
Collaborator

☀️ Test successful - status-papr, status-travis
Approved by: rhatdan
Pushing fa68ed6 to master...

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

4 participants