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 existing build information for version if not embedded with build script #701

Closed
wants to merge 1 commit into from

Conversation

dtscd
Copy link

@dtscd dtscd commented Dec 28, 2022

Go1.12 introduced BuildInfo which embeds build information. It does not embed build date to facilitate reproducable builds by default. If build information is embedded from build script, use the information provided by the Go build system.

Closes #687

… script

Go1.12 introduced BuildInfo which embeds build information. It does
not embed build date to facilitate reproducable builds by default.
If build information is embedded from build script, use the information
provided by the Go build system.
@rfjakob
Copy link
Owner

rfjakob commented Dec 29, 2022

Merged, thanks. I refactored it afterwards, and I had to cut some functionality because https://github.com/rfjakob/gocryptfs/actions/runs/3795050466/jobs/6464850039 .

@rfjakob rfjakob closed this Dec 29, 2022
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.

Consider using runtime/debug.BuildInfo if build info not build with make.sh
2 participants