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

Merged
merged 1 commit into from Nov 2, 2022
Merged

Allow to override build date with SOURCE_DATE_EPOCH #486

merged 1 commit into from Nov 2, 2022

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 works with different variants of date.
Also use UTC to be independent of timezone.

@yxxhero
Copy link
Member

yxxhero commented Oct 31, 2022

@bmwiedemann please fix DCO issue.

@bmwiedemann
Copy link
Contributor Author

added Signed-off-by

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 works with different variants of date.
Also use UTC to be independent of timezone.

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

yxxhero commented Nov 2, 2022

@bmwiedemann do you test it locally?

@bmwiedemann
Copy link
Contributor Author

Yes, I tested it.
It is also based on the Makefile template in https://reproducible-builds.org/docs/source-date-epoch/#makefile

@yxxhero
Copy link
Member

yxxhero commented Nov 2, 2022

@bmwiedemann Thanks for your feedback. I have a question. Can SOURCE_DATE_EPOCH work with goreleaser?

Copy link
Member

@yxxhero yxxhero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yxxhero yxxhero merged commit 008e92d into helmfile:main Nov 2, 2022
12 checks passed
@bmwiedemann bmwiedemann deleted the date branch November 2, 2022 14:11
@bmwiedemann
Copy link
Contributor Author

Can SOURCE_DATE_EPOCH work with goreleaser?

It can be set by any caller of make. And if it is unset, everything will work as before this patch.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants