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

Fix relative target of manpage links. #13

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Feb 15, 2021

Whilst working on the Reproducible Builds effort we noticed that mocassin could not be built reproducibly.

This led me to discover that the mocassinOutput(1), mocassinPlot(1) and mocassinWarm(1) manpages were linking to the absolute location for the mocassin(1) manpage, so they would not work when distributed:

For example mocassinOutput.1.gz linked to, on my machine:

/home/lamby/temp/cdt.20210215121553.ZpEolGOptg.repro.mocassin/build-a/mocassin-2.02.73.1/debian/tmp/usr/share/man/man1/mocassin.1.gz

Patch attached that uses relative URLs instead. This change would also make the build reproducible. I originally filed this in Debian as #982851.

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

This led me to discover that the mocassinOutput(1), mocassinPlot(1) and
mocassinWarm(1) manpages were linking to the absolute location for the
mocassin(1) manpage, so they would not work when distributed:

For example mocassinOutput.1.gz linked to, on my machine:

  /home/lamby/temp/cdt.20210215121553.ZpEolGOptg.repro.mocassin/build-a/mocassin-2.02.73.1/debian/tmp/usr/share/man/man1/mocassin.1.gz

Patch attached that uses relative URLs instead. This change would also make the
build reproducible. I originally filed this in Debian as #982851 [1].

 [0] https://reproducible-builds.org/
 [1] https://bugs.debian.org/982851
@rwesson rwesson merged commit b532cdb into rwesson:master Feb 15, 2021
@rwesson
Copy link
Owner

rwesson commented Feb 16, 2021

Thanks for providing this patch! I have updated the debian package with the new version to close that bug also.

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.

None yet

2 participants