Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 17 Jun 2015 16:50:31 +0000
From:      Steve Wills <swills@FreeBSD.org>
To:        Adam Weinberger <adamw@adamw.org>
Cc:        Brad Davis <brd@FreeBSD.org>, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r389861 - in head/devel: cmake cmake-gui cmake-modules
Message-ID:  <20150617165030.GA48147@mouf.net>
In-Reply-To: <AD97DB22-EE1C-4779-A29F-EAEDA09E1E73@adamw.org>
References:  <201506162025.t5GKP6lx080770@svn.freebsd.org> <AD97DB22-EE1C-4779-A29F-EAEDA09E1E73@adamw.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jun 17, 2015 at 09:39:11AM -0600, Adam Weinberger wrote:
> > On 16 Jun, 2015, at 14:25, Brad Davis <brd@FreeBSD.org> wrote:
> > 
> > Author: brd
> > Date: Tue Jun 16 20:25:05 2015
> > New Revision: 389861
> > URL: https://svnweb.freebsd.org/changeset/ports/389861
> > 
> > Log:
> >  Update devel/cmake, devel/cmake-gui and devel/cmake-modules to 3.2.3.
> > 
> > Modified: head/devel/cmake/Makefile
> > ==============================================================================
> > 
> > LIB_DEPENDS=	libcurl.so:${PORTSDIR}/ftp/curl \
> > -		libexpat.so:${PORTSDIR}/textproc/expat2
> > +		libexpat.so:${PORTSDIR}/textproc/expat2 \
> > +		libjsoncpp.so:${PORTSDIR}/devel/jsoncpp
> 
> Does this port really need to depend on libjsoncpp? The version bundled with cmake works just fine (--no-system-jsoncpp).
> 
> The only benefit to making everybody install jsoncpp (and scons) would be if the jsoncpp port is updated more often than the cmake port. But the jsoncpp port hasn't had an update since the day it was added 16 months ago.
> 
> Can you please consider dropping the libjsoncpp dependency and just adding --no-system-jsoncpp to CONFIGURE_ARGS?
> 

In general, it is better to not used bundled libs where possible. Let's update
the devel/jsoncpp port.

Steve



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150617165030.GA48147>