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

[Makefile] Support reproducible builds #127

Merged
merged 1 commit into from
Oct 30, 2019

Conversation

Foxboron
Copy link
Contributor

Embedding build times is generally poor form as it introduces
undeterministic builds. To ensure the binaries are reproducible, we
should be looking for SOURCE_DATE_EPOCH in the build environment as
specified by the reproducible builds project.

https://reproducible-builds.org/docs/source-date-epoch/

Embedding build times is generally poor form as it introduces
undeterministic builds. To ensure the binaries are reproducible, we
should be looking for `SOURCE_DATE_EPOCH` in the build environment as
specified by the reproducible builds project.

https://reproducible-builds.org/docs/source-date-epoch/
@sagikazarmark
Copy link
Owner

Hey @Foxboron!

Thanks a lot, this is great. TBH I care little about the build time, because the commit hash provides authoritative information. (Probably a build number would be better, because that could identify actual builds in the build system)

But with this addition I think we can leave it there.

Thanks again!

@sagikazarmark sagikazarmark merged commit 49debc2 into sagikazarmark:master Oct 30, 2019
@Foxboron
Copy link
Contributor Author

Yes, we completely agree. I however felt that supporting SOURCE_DATE_EPOCH was a less intrusive change compared to removing the build date.

@sagikazarmark
Copy link
Owner

I opened an issue so that I don't forget about it. But eventually I will probably just remove it.

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.

2 participants