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

Merged
merged 1 commit into from
Jul 7, 2020
Merged

Make the build reproducible #14

merged 1 commit into from
Jul 7, 2020

Conversation

mr-c
Copy link

@mr-c mr-c commented May 6, 2020

Hi,

Whilst working on the Reproducible Builds effort [0] we noticed that
ataqv could not be built reproducibly.

This is because it embedded the absolute build path when generating
a manual page:

│ │ │ │ │ \fB-t\fR TEMPLATE_DIRECTORY, \fB--template-directory\fR TEMPLATE_DIRECTORY
│ │ │ │ │ The location of the web app directory template.
│ │ │ │ │ (default:
│ │ │ │ │ -\fI,/build/1st/ataqv-1.1/\fP.1+ds/src/scripts/src/web)
│ │ │ │ │ +\fI,/build/2/ataqv-1.1/\fP.1+ds/2nd/src/scripts/src/web)

[0] https://reproducible-builds.org/

(Original report from @lamby: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=959714 )

@lamby
Copy link
Contributor

lamby commented May 6, 2020

👍

@@ -570,6 +570,9 @@ def write_metrics(indent, metrics):
if __name__ == '__main__':
args = parse_arguments()

if args.template_directory == 'auto':
Copy link
Contributor

Choose a reason for hiding this comment

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

Looking at this now, it maybe nicer to use None, but that would look confusing in the --help output.

@umijh
Copy link
Contributor

umijh commented Jun 14, 2020

👍 I applied this change (after syncing up the Debian package version) and the mkarv output is fine. And I learned something about reproducible Python. Thanks @lamby and @mr-c!

@porchard
Copy link
Contributor

porchard commented Jul 7, 2020

Thanks @lamby and @mr-c for the change and thanks @umijh for testing!

@porchard porchard merged commit 8fbd929 into ParkerLab:master Jul 7, 2020
@mr-c mr-c deleted the reproducible_build branch July 8, 2020 08:38
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.

4 participants