Created attachment 144661 [details] Patch to change RUN_DEPENDS to BUILD_DEPENDS Installing textproc/docbook using portmaster fails like this: ===> Fetching all distfiles required by docbook-xml-5.0_2 for building ===> Extracting for docbook-xml-5.0_2 => SHA256 Checksum OK for docbkx412.zip. => SHA256 Checksum OK for docbook-xml-4.2.zip. => SHA256 Checksum OK for docbook-xml-4.3.zip. => SHA256 Checksum OK for docbook-xml-4.4.zip. => SHA256 Checksum OK for docbook-xml-4.5.zip. => SHA256 Checksum OK for docbook-5.0.zip. ===> Patching for docbook-xml-5.0_2 ===> Configuring for docbook-xml-5.0_2 ===> Staging for docbook-xml-5.0_2 echo "CATALOG \"/usr/local/share/xml/xmlcharent/catalog\"" > /usr/ports/textproc/docbook-xml/work/stage/usr/local/share/xml/docbook/catalog /usr/local/bin/xmlcatmgr -c /usr/ports/textproc/docbook-xml/work/stage/usr/local/share/xml/docbook/catalog.xml create make: exec(/usr/local/bin/xmlcatmgr) failed (No such file or directory) *** Error code 1 The problem is that xmlcatmgr is a RUN_DEPENDS but it is needed at installation time. I don't see any runtime dependency, so the attach patch builddep.patch changes it to a BUILD_DEPENDS.
Minor correction: I meant to say "installing textproc/docbook-xml".
*** Bug 191240 has been marked as a duplicate of this bug. ***
Also affected - installing textproc/xmlcatmgr before textproc/docbook-xml fixes install with portmaster.
The problem is still relevant, a patch works fine
+1 on the patch. I hit this build error today while trying to install devel/git on a clean 10.1-RELEASE install.
I'll handle this.
Maintainer timeout
Maintainer timeout, open up for anyone to take
A commit references this bug: Author: robak Date: Tue Mar 3 13:54:44 UTC 2015 New revision: 380367 URL: https://svnweb.freebsd.org/changeset/ports/380367 Log: textproc/docbook-xml: fix build - Change RUN_DEPENDS to BUILD_DEPENDS PR: 191866 Submitted by: Martin Simmons <martin@lispworks.com> MFH: 2015Q1 Changes: head/textproc/docbook-xml/Makefile
Committed, thanks for your work!
As antoine@ pointed out, this in fact wasnt a proper patch, and the commit was reverted back.
With the change reverted I've tested both devel/git and textproc/docbook and I cant reproduce the reported issues, so I am closing this PR as unable to reproduce.
For correctness, the change wasn't just reverted -- the RUN_DEPENDS was added back, but the new BUILD_DEPENDS remains. I'm not sure why a port containing data files depends on anything at run time though...
I can't reproduce the failure when BUILD_DEPENDS are removed. The RUN_DEPENDS are installed before staging occurs: ===> Patching for docbook-xml-5.0_3 ===> Configuring for docbook-xml-5.0_3 ===> Staging for docbook-xml-5.0_3 ===> docbook-xml-5.0_3 depends on file: /usr/local/share/xml/xmlcharent/catalog.xml - not found ===> Verifying install for /usr/local/share/xml/xmlcharent/catalog.xml in /usr/ports/textproc/xmlcharent ===> xmlcharent-0.3_2 depends on file: /usr/local/sbin/pkg - found => xmlcharent-0.3.zip doesn't seem to exist in /portdistfiles/. => Attempting to fetch http://www.oasis-open.org/docbook/xmlcharent/0.3/xmlcharent-0.3.zip ...
Note that I don't use portmaster, I just type "make" in textproc/docbook-xml
Sorry, my mistake -- the RUN_DEPENDS is needed because xmlcatmgr is used in the @exec and @unexec clauses of pkg-plist. The BUILD_DEPENDS is needed for portmaster because that doesn't install the RUN_DEPENDS before staging.