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

Closed
wants to merge 1 commit into from

Conversation

bmwiedemann
Copy link

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 only works with GNU date.

Also use UTC to be independent of timezone.

Locales still can cause %B to vary. ISO 8601 date format would help there.

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 only works with GNU date.

Also use UTC to be independent of timezone.

Locales still can cause %B to vary. ISO 8601 date format would help there.
@okirch
Copy link
Owner

okirch commented Jul 3, 2023

Thank you for the patch, Bernd! I've applied a slightly different change. I fixed it in https://github.com/okirch/microconf (and then rebuilt the microconf subdir in pcr-oracle) - making SOURCE_DATE_EPOCH do what you want it to do, but leaving the original code unchanged for how. I did a quick smoke test and it appears to work correctly. I'd appreciate a quick nod from your end if you agree :-)

@okirch
Copy link
Owner

okirch commented Jul 17, 2023

@bmwiedemann did you have an opportunity to check if my alternative fix addresses the issue correctly?

@bmwiedemann
Copy link
Author

I can confirm that the new pcr-oracle-0.4.6 package builds reproducibly now.

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

2 participants