[36/75] cd /wrkdirs/usr/ports/audio/vmpk/work/.build/man && /usr/local/bin/xsltproc --nonet --xinclude --xincludestyle --output /wrkdirs/usr/ports/audio/vmpk/work/.build/man/vmpk.1 http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl /wrkdirs/usr/ports/audio/vmpk/work/vmpk-0.6.2a/man/vmpk.xml
FAILED: man/vmpk.1 
cd /wrkdirs/usr/ports/audio/vmpk/work/.build/man && /usr/local/bin/xsltproc --nonet --xinclude --xincludestyle --output /wrkdirs/usr/ports/audio/vmpk/work/.build/man/vmpk.1 http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl /wrkdirs/usr/ports/audio/vmpk/work/vmpk-0.6.2a/man/vmpk.xml
I/O error : Attempt to load network entity http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
warning: failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
cannot parse http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl

Fixed http download problem.
Added MANPAGES option.

Harbormaster completed remote builds in B12666: Diff 35117.Nov 11 2017, 7:02 PM

In D13045#271154, @tcberner wrote:

Thanks for catching this! I fixed it.

In D13045#271203, @yuri wrote:

In D13045#271154, @tcberner wrote:

Thanks for catching this! I fixed it.

No worries. Don't you need to add some magic to that option?

In D13045#271209, @tcberner wrote:

No worries. Don't you need to add some magic to that option?

They already have vmpk.1 file, but were still rebuilding it the hard way by default for some reason.

Downloading files like that, especially from http:// URLs, is a security problem. It is a MITM attack vulnerability.
A lot of projects have such problems, and linux packaging frameworks never catch such problems.
cmake itself has this problem, built into their specs. They allow ad-hoc fetch commands without the required fingerprint parameter.

I verified, it builds correctly now with MANPAGES on and off.

Maybe add MANPAGES to options default sensible defaults and all :D -- I leave that up to you.

This revision is now accepted and ready to land.Nov 11 2017, 7:41 PM

Closed by commit rP453996: New port: audio/vmpk: Virtual MIDI Piano Keyboard (authored by yuri). · Explain WhyNov 11 2017, 8:01 PM

This revision was automatically updated to reflect the committed changes.

mat added inline comments.Nov 24 2017, 4:51 PM