Bug 101237 - multimedia/dirac: fix pkg-plist
Summary: multimedia/dirac: fix pkg-plist
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Bruce M Simpson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-08-02 03:10 UTC by Jacula Modyun
Modified: 2006-08-02 13:20 UTC (History)
0 users

See Also:


Attachments
file.diff (64.10 KB, patch)
2006-08-02 03:10 UTC, Jacula Modyun
no flags Details | Diff
file.diff (2.10 KB, patch)
2006-08-02 03:10 UTC, Jacula Modyun
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jacula Modyun 2006-08-02 03:10:14 UTC
If you have doxygen (and eventually graphviz) the pkg-plist
is incorrect.
If you want install the documentation, you have to install
doxygen.
If you want generate inheritance diagrams to the documentation
to have to install graphviz.

Fix: I use this patch:

For multimedia/dirac/Makefile:

For multimedia/dirac/pkg-plist:

And a new multimedia/dirac/files/patch-configure:



pre-install:
	${MKDIR} ${DOCSDIR}/programmers

because the installation process break when the istall.sh create
the programmers directory. This problem is documented in the
wrk/dirac-0.5.0/install.sh--yf8FZLFBRDtu1AC8fjWKryoAT1f7QTvS9JrGNPgCzNpjCq0z
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- Makefile.orig	Fri Jan 14 13:46:47 2005
+++ Makefile	Wed Aug  2 01:36:04 2006
@@ -18,14 +18,44 @@
 USE_GMAKE=	yes
 USE_GNOME=	pkgconfig gnomehack gnometarget
 USE_GCC=	3.4
-PORTDOCS=	README
 
 CONFIGURE_ENV+=	CPPFLAGS=-I${LOCALBASE}/include
 
-post-install:
-.if !defined(NOPORTDOCS)
-	@${MKDIR} ${DOCSDIR}
-	${INSTALL_DATA} ${WRKSRC}/README ${DOCSDIR}
+OPTIONS=	DOXYGEN "Define DOXYGEN to build documentation" off \
+		DOT "Define DOT to gerate inheritance diagrams" off
+
+.include <bsd.port.pre.mk>
+
+.if (defined(WITH_DOT) && defined(WITHOUT_DOXYGEN))
+IGNORE=If you set WITH_DOT=yes, you have to set WITH_DOXYGEN=yes.
+.endif
+
+.ifdef(WITH_DOXYGEN)
+
+PLIST_SUB+=	WITH_DOXYGEN=""
+BUILD_DEPENDS+=	doxygen:${PORTSDIR}/devel/doxygen
+
+pre-install:
+	${MKDIR} ${DOCSDIR}/programmers
+
+.ifdef(WITH_DOT)
+PLIST_SUB+=	WITH_DOT=""
+PLIST_SUB+=	WITHOUT_DOT="@comment "
+BUILD_DEPENDS+=	dot:${PORTSDIR}/graphics/graphviz
+.else
+CONFIGURE_ENV+=	HAVE_DOT=false
+PLIST_SUB+=	WITH_DOT="@comment "
+PLIST_SUB+=	WITHOUT_DOT=""
+.endif
+
+.else
+
+CONFIGURE_ENV+=	HAVE_DOXYGEN=false HAVE_LATEX=false HAVE_DVIPDFM=false HAVE_DOT=false
+PLIST_SUB+=	WITH_DOXYGEN="@comment "
+PLIST_SUB+=	WITH_DOT="@comment "
+PLIST_SUB+=	WITHOUT_DOT="@comment "
+NOPORTDOCS=	yes
+
 .endif
 
-.include <bsd.port.mk>
+.include <bsd.port.post.mk>
How-To-Repeat: Install doxygen (and eventually graphviz). The configure test find
the doxygen command (and eventually the dot command) and install
the documentation (with graphics generated by dot).
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2006-08-02 03:10:25 UTC
Responsible Changed
From-To: freebsd-ports-bugs->bms

Over to maintainer
Comment 2 Bruce M Simpson freebsd_committer freebsd_triage 2006-08-02 12:10:35 UTC
State Changed
From-To: open->closed

The fix in this patch isn't 100% good, but a relevant fix 
has been committed to CVS.