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

Make the build reproducible #74

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

Conversation

lamby
Copy link

@lamby lamby commented Jul 24, 2023

Whilst working on the Reproducible Builds effort I noticed that babosa could not be built reproducibly.

This is because the __dir__ gets expanded when building/rendering the .gemspec file which is then shipped in the binary package. As this path is dependent on the path in which you build babosa, this makes the build unreproducible.

I think that cert_chain is only needed if signing_key is needed, so moving it under the same conditional seems to be safe. The goal, however, is to prevent cert_chain from being included in regular, "normal", builds of the package.

I originally filed this in Debian as bug #1041840.

Whilst working on the Reproducible Builds effort [0], I noticed that
babosa could not be built reproducibly.

This is because the __dir__ gets expanded when building/rendering the .gemspec
file which is then shipped in the binary package. As this path is dependent on
the path in which you build babosa, this makes the build unreproducible.

I think that cert_chain is only needed if signing_key is needed, so moving it
under the same conditional seems to be safe. The goal, however, is to prevent
cert_chain from being included in regular, "normal", builds of the package.

I originally filed this in Debian as bug #1041840 [1].

 [0] https://reproducible-builds.org/
 [1] https://bugs.debian.org/1041840
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.

1 participant