Created attachment 161292 [details]
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: %%PORTDOCS%%%%DOCSDIR%%/faq
Error: Orphaned: %%PORTDOCS%%%%DOCSDIR%%/handbook
===> Checking for items in pkg-plist which are not in STAGEDIR
- Add these missing links to plist
- Add NO_ARCH
Created attachment 161293 [details]
Updated patch: also fixes plist for TXT format
Looking at textproc/docproj ( a BUILD_DEPENDS), we have ONLY_FOR_ARCHS= amd64 i386, so we should prefer to add the same knob instead of No_ARCH. Am I right?
> Looking at textproc/docproj ( a BUILD_DEPENDS), we have ONLY_FOR_ARCHS=
> amd64 i386, so we should prefer to add the same knob instead of No_ARCH. Am
> I right?
No. ONLY_FOR_ARCHS in docproj (which is essentially a metaport) doesn't seem correct at all. I guess it was propagated through fop from java which is ONLY_FOR_ARCHS=i386 amd64, and such propagation is totally wrong.
Regarding freebsd-doc-en, there's just no way for documentation to be limited to specific architectures, so there may never be any ONLY_FOR_ARCHS for this port.
And for the note, NO_ARCH and ONLY_FOR_ARCHS don't conflict and may happily live together.
pkg-plist.txt bit still needs to be committed. Also after the latest update it doesn't build with PS:
(In reply to Dmitry Marakasov from comment #4)
For the pkg-plist, I forgot it. I'll fix it soon.
Regarding the break you experience, I think you hit a xsltproc issue.
Here I can't reproduce it, xsltproc --version gives
"Using libxml 20902, libxslt 10128 and libexslt 817"
Your logs indicate:
"You can adjust xsltMaxDepth (--maxdepth) in order to raise the maximum number of nested template calls and variables/params (currently set to 3000)."
Here, by default --maxdepth is set to 6000.
A commit references this bug:
Date: Sat Oct 10 10:47:14 UTC 2015
New revision: 398987
Remove duplicate plist entries.
Submitted by: Dmitry Marakasov <amdmi3@FreeBSD.org>
Approved by: doceng (implicit)