/mosquitto

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

Building man pages fails if docbook.xsl is in a different location #1139

Closed
cjmayo opened this Issue Feb 1, 2019 · 1 comment

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  1.5.6
2 participants
@cjmayo @ralight
@cjmayo
Copy link

cjmayo commented Feb 1, 2019

Reported before at #105 and
https://bugs.launchpad.net/mosquitto/+bug/1269967

Can be made universal by using:

man/html.xsl

-	<xsl:import href="/usr/share/xml/docbook/stylesheet/docbook-xsl/html/docbook.xsl"/>
+	<xsl:import href="http://docbook.sourceforge.net/release/xsl/current/html/docbook.xsl"/>

man/manpage.xsl

-	<xsl:import href="/usr/share/xml/docbook/stylesheet/docbook-xsl/manpages/docbook.xsl"/>
+	<xsl:import href="http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"/>

because xsltproc will use the system catalog to find the local path.

Can pass --nonet option to ensure the Internet is not used.

config.mk

-XSLTPROC=xsltproc
+XSLTPROC=xsltproc --nonet

@ralight ralight added this to the 1.5.6 milestone Feb 3, 2019

@ralight

This comment has been minimized.

Sign in to view
Copy link
Contributor

ralight commented Feb 3, 2019

A massive thanks for this, I know it has annoyed people for a long while. I've made the change on my local branch and it will be part of 1.5.6 soon.

@ralight ralight closed this Feb 3, 2019

ralight added a commit that referenced this issue Feb 8, 2019

@ralight
Man page can now be built on any system. 
Closes #1139. Huge thanks to Chris Mayo.
ce1d0ae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment