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

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Jul 24, 2023

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

This is due to the testsuite not completely cleaning up after itself, so these files get installed into the guidata Debian package. However, as they create nondeterministic contents this makes the package unreproducible.

I originally filed this in Debian as bug #1041842. The patch is slightly different there as I didn't notice the conditional block until now; this patch should be preferred instead.

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

This is due to the testsuite not completely cleaning up after itself, so these
files get installed into the guidata Debian package. However, as they create
nondetermistic contents this makes the package unreproducible.

I originally filed this in Debian as bug #1041842 [1]. The patch is slightly
different there as I didn't notice the conditional block until now; this patch
should be prefered instead.

 [0] https://reproducible-builds.org/
 [1] https://bugs.debian.org/1041842
Copy link
Collaborator

@PierreRaybaut PierreRaybaut left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

@PierreRaybaut PierreRaybaut merged commit 1b235b1 into PlotPyStack:master Jul 26, 2023
3 checks passed
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