Created attachment 144285 [details] poudriere build log for graphics/dia graphics/dia fails to build in poudriere (current test, arch powerpc.powerpc64), with a series of errors in translations and docs. Log is attached.
over to maintainers
Created attachment 146450 [details] Exclude doc dir from build when DOCS option is off
Comment on attachment 146450 [details] Exclude doc dir from build when DOCS option is off >Index: graphics/dia/Makefile >=================================================================== >--- graphics/dia/Makefile (revision 366396) >+++ graphics/dia/Makefile (working copy) >@@ -32,6 +32,10 @@ > DOCS_BUILD_DEPENDS= ${LOCALBASE}/share/xsl/docbook/manpages/docbook.xsl:${PORTSDIR}/textproc/docbook-xsl > .include <bsd.port.options.mk> > >+.if !${PORT_OPTIONS:MDOCS} >+EXTRA_PATCHES= ${FILESDIR}/extra-doc >+.endif >+ > .if ${ARCH} == "i386" > LIB_DEPENDS+= libEMF.so:${PORTSDIR}/graphics/libemf > CONFIGURE_ARGS+= --enable-libemf >Index: graphics/dia/files/extra-doc >=================================================================== >--- graphics/dia/files/extra-doc (revision 0) >+++ graphics/dia/files/extra-doc (working copy) >@@ -0,0 +1,11 @@ >+--- Makefile.in 2014-08-28 14:59:33.851569012 +0300 >++++ Makefile.in 2014-08-28 14:54:49.747589247 +0300 >+@@ -276,7 +276,7 @@ >+ top_builddir = @top_builddir@ >+ top_srcdir = @top_srcdir@ >+ uidatadir = @uidatadir@ >+-SUBDIRS = lib objects plug-ins shapes app bindings samples po sheets data doc tests installer >++SUBDIRS = lib objects plug-ins shapes app bindings samples po sheets data tests installer >+ gnomedatadir = $(datadir) >+ >+ #desktop_in_in_files = dia.desktop.in.in > >Property changes on: graphics/dia/files/extra-doc >___________________________________________________________________ >Added: svn:mime-type >## -0,0 +1 ## >+text/plain >\ No newline at end of property >Added: svn:eol-style >## -0,0 +1 ## >+native >\ No newline at end of property >Index: graphics/dia/pkg-plist >=================================================================== >--- graphics/dia/pkg-plist (revision 366396) >+++ graphics/dia/pkg-plist (working copy) >@@ -1812,6 +1812,6 @@ > @dirrm share/dia/shapes/Assorted > @dirrm share/dia/shapes > @dirrm share/dia >-@dirrmtry man/fr/man1 >-@dirrmtry man/fr >+%%PORTDOCS%%@dirrmtry man/fr/man1 >+%%PORTDOCS%%@dirrmtry man/fr > @dirrm lib/dia
Created attachment 148629 [details] patch-docs.diff update patch
Created attachment 148630 [details] patch-docs.diff use DOCS_EXTRA_PATCHES_OFF
Created attachment 150721 [details] Remove docs from SUBDIRS when DOCS option is off Rebase patch.
Why the option is named DOCS and not MANPAGES ? Nothing is installed under ${DOCSDIR}. Otherwise, restoring REINPLACE_CMD from before ports r350986 maybe better. I cannot add |regression| keyword for some reason.
(In reply to Jan Beich from comment #7) > Why the option is named DOCS and not MANPAGES ? Nothing is installed under > ${DOCSDIR}. Nevermind, I forgot share/doc/dia would be %%DOCSDIR%% in pkg-plist.
Created attachment 150788 [details] Restore workaround for DOCS option (which was removed in r350986) (In reply to Jan Beich from comment #7) > Otherwise, restoring REINPLACE_CMD from before ports r350986 maybe better. I think so, too (it was overlooked by me).
Created attachment 172096 [details] patch-sheets_checktrans.py I rewrote checktrans.py to support new version of xml.sax. Just put this patch to "files/" folder.
Created attachment 172097 [details] Makefile.patch Also, this port has a missing dependencies. This patch fixes it.
Is this still relevant?
I stopped building ports with non-default options (where able) long ago, so this is no longer relevant to me, but the bug likely still exists. I wouldn't oppose closing it, though.