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

Use RELEASE_NOTES date instead of build date #1289

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bmwiedemann
Copy link

@bmwiedemann bmwiedemann commented Jul 8, 2024

Use RELEASE_NOTES date instead of build date
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 and BSD date.

Alternatively, https://reproducible-builds.org/docs/source-date-epoch/#makefile could be implemented.

This patch was done while working on reproducible builds for openSUSE, sponsored by the NLnet NGI0 fund.

@DevinCarr
Copy link
Contributor

DevinCarr commented Aug 6, 2024

I think for this repo, we update the RELEASE_NOTES file with every release, so if you are looking for a stable timestamp to reference for the build date, it would be better to use that file. How does that sound?
Otherwise, this change seems fine and I'll be happy to help merge this.

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 and BSD date.

This patch was done while working on reproducible builds for openSUSE, sponsored by the NLnet NGI0 fund.
@bmwiedemann
Copy link
Author

updated the commit

@DevinCarr DevinCarr self-requested a review August 22, 2024 16:31
@DevinCarr DevinCarr changed the title Use ChangeLog date instead of build date Use RELEASENOTES date instead of build date Aug 22, 2024
@DevinCarr DevinCarr changed the title Use RELEASENOTES date instead of build date Use RELEASE_NOTES date instead of build date Aug 22, 2024
@bmwiedemann
Copy link
Author

So is something missing or is it good to merge?

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.

3 participants