From owner-cvs-all@FreeBSD.ORG Sat Jan 21 03:16:50 2012 Return-Path: Delivered-To: cvs-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 408681065689; Sat, 21 Jan 2012 03:16:50 +0000 (UTC) (envelope-from wxs@atarininja.org) Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.49.45]) by mx1.freebsd.org (Postfix) with ESMTP id 187638FC15; Sat, 21 Jan 2012 03:16:49 +0000 (UTC) Received: by syn.atarininja.org (Postfix, from userid 1001) id 4EB565C3A; Fri, 20 Jan 2012 22:16:49 -0500 (EST) Date: Fri, 20 Jan 2012 22:16:49 -0500 From: Wesley Shields To: Michael Scheidell , gabor@FreeBSD.org Message-ID: <20120121031649.GA89853@atarininja.org> References: <201201210047.q0L0lGNH027714@repoman.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201201210047.q0L0lGNH027714@repoman.freebsd.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/astro/merkaartor Makefile X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 21 Jan 2012 03:16:50 -0000 On Sat, Jan 21, 2012 at 12:47:16AM +0000, Michael Scheidell wrote: > scheidell 2012-01-21 00:47:16 UTC > > FreeBSD ports repository > > Modified files: > astro/merkaartor Makefile > Log: > - Don't use INSTALL_ICONS when Gnome not installed > > PR: ports/164344 > Approved by: gabor(mentor, implicit) > > Revision Changes Path > 1.20 +5 -2 ports/astro/merkaartor/Makefile > http://cvsweb.FreeBSD.org/ports/astro/merkaartor/Makefile.diff?r1=1.19&r2=1.20 > | --- ports/astro/merkaartor/Makefile 2011/07/03 15:49:55 1.19 > | +++ ports/astro/merkaartor/Makefile 2012/01/21 00:47:16 1.20 > | @@ -2,7 +2,7 @@ > | # Date created: 9 January 2009 > | # Whom: slaven@rezic.de > | # > | -# $FreeBSD: /usr/local/www/cvsroot/FreeBSD/ports/astro/merkaartor/Makefile,v 1.19 2011/07/03 15:49:55 jlaffaye Exp $ > | +# $FreeBSD: /usr/local/www/cvsroot/FreeBSD/ports/astro/merkaartor/Makefile,v 1.20 2012/01/21 00:47:16 scheidell Exp $ > | # > | > | PORTNAME= merkaartor > | @@ -14,10 +14,10 @@ MASTER_SITES= http://www.merkaartor.be/a > | MAINTAINER= dev2@heesakkers.info > | COMMENT= An openstreetmap mapping program > | > | -BUILD_DEPENDS= ${LOCALBASE}/include/boost/concept_check.hpp:${PORTSDIR}/devel/boost-libs > | - > | LICENSE= GPLv2 > | > | +BUILD_DEPENDS= ${LOCALBASE}/include/boost/concept_check.hpp:${PORTSDIR}/devel/boost-libs > | + > | USE_BZIP2= yes > | USE_LDCONFIG= yes > | > | @@ -25,7 +25,10 @@ USE_QT_VER= 4 > | QT_COMPONENTS= qmake_build uic_build moc_build rcc_build \ > | gui network svg webkit xml imageformats script scripttools > | MAKE_JOBS_SAFE= yes > | + > | +.if exists(${LOCALBASE}/bin/gnome-session) > | INSTALLS_ICONS= yes > | +.endif Isn't there a better way to do this? As it stands now, if I don't have gnome-session then this port will not bring in the gnome stuff for me. What if I want it to bring in gnome? There is no way for this port to conditionally bring in gnome for me. If it sees I already have gnome it will honor that but there is no way for me to say "I don't have gnome, but please bring in the necessary parts and register dependencies for me". Shouldn't we make gnome support an option? Default it to off for the sake of not bloating the dependencies just to get some icons registered. You could even do something like: .if !defined(WITH_GNOME_ICONS) .if exists(${LOCALBASE}/bin/gnome-session) # Gnome is installed, but the user does not have GNOME_ICONS options # turned on, forcibly set this so icons will work. INSTALLS_ICONS= yes .fi .else # User specifically wants gnome icon support. INSTALLS_ICONS=yes .fi