ports/94288: [PATCH] misc/tinderbox: OPTIONaly depend on c[v]sup

[Ion-Mihai Tetcu]

On Sat, 18 Mar 2006 22:44:24 +0200
Ion-Mihai Tetcu <itetcu at people.tecnik93.com> wrote:

> On Sat, 18 Mar 2006 22:38:44 +0200
> Ion-Mihai Tetcu <itetcu at people.tecnik93.com> wrote:
> 
> > On Sat, 18 Mar 2006 20:13:14 +0000
> > Shaun Amott <shaun at inerd.com> wrote:
> > 
> > > On Sat, Mar 18, 2006 at 12:00:41PM +0000, Andreas Kohn wrote:
> > > > 
> > > >  I like the patch to enable cvsup/csup selection.
> > > >  I think the unconditional RUN_DEPENDS on cvsup needs to get removed in
> > > >  that case as well.
> > > >  The patch for MySQL is incomplete, as setup-mysql.sh would still barf
> > > >  about missing MySQL-ports.
> > > >  The attached patch also fixes that, and sets USE_APACHE=3D1.3+ to stop
> > > >  annoying apache2 users that apache13 is needed :)
> > > >  
> > > 
> > > Can we remove the apache dependency completely? Or, at least make it
> > > optional.
> > 
> > Hard, since we use databases/pear-DB which pulls in php mod which pulls
> > in apache.
> 
> ^^^ I didn't check this so I might be wrong, I don't remember what type
> of php pears pulls in.
> 
> Well, we could add an OPTION:
> 
> -		${LOCALBASE}/share/pear/DB.php:${PORTSDIR}/databases/pear-DB \
> ...
> + WEB_INTERFACE "Install experimental web interface" on
> ...
> - USE_APACHE=	1.3+
> ...
> .if defined(WITH_WEB_INTERFACE)
> USE_APACHE=	1.3+
> RUN_DEPENDS+= ${LOCALBASE}/share/pear/DB.php:${PORTSDIR}/databases/pear-DB
> .endif

and wrap the post-patch in .if defined(WITH_WEB_INTERFACE) too


-- 
IOnut - Unregistered ;) FreeBSD "user"
  "Intellectual Property" is   nowhere near as valuable   as "Intellect"

There is an order of things in this universe.
		-- Apollo, "Who Mourns for Adonais?" stardate 3468.1