From owner-svn-ports-head@freebsd.org Fri Jul 10 14:02:57 2015 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 89A343684; Fri, 10 Jul 2015 14:02:57 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from vps.amdmi3.ru (vps.amdmi3.ru [109.234.38.216]) by mx1.freebsd.org (Postfix) with ESMTP id 483BC1CA1; Fri, 10 Jul 2015 14:02:56 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from hive.panopticon (unknown [78.153.152.119]) by vps.amdmi3.ru (Postfix) with ESMTPS id 1535BB0614; Fri, 10 Jul 2015 17:02:56 +0300 (MSK) Received: from hades.panopticon (hades.panopticon [192.168.0.32]) by hive.panopticon (Postfix) with ESMTP id C9439644; Fri, 10 Jul 2015 17:01:03 +0300 (MSK) Received: by hades.panopticon (Postfix, from userid 1000) id D2F61369E5; Fri, 10 Jul 2015 16:59:40 +0300 (MSK) Date: Fri, 10 Jul 2015 16:59:40 +0300 From: Dmitry Marakasov To: marino@freebsd.org Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r391626 - head/games/warzone2100 Message-ID: <20150710135940.GB9933@hades.panopticon> References: <201507090944.t699iqBR039850@repo.freebsd.org> <559E431B.6030708@marino.st> <20150709171246.GB46547@hades.panopticon> <559EAD6F.3050409@marino.st> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <559EAD6F.3050409@marino.st> User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 10 Jul 2015 14:02:57 -0000 * John Marino (freebsd.contact@marino.st) wrote: > >>> Author: amdmi3 > >>> Date: Thu Jul 9 09:44:51 2015 > >>> New Revision: 391626 > >>> URL: https://svnweb.freebsd.org/changeset/ports/391626 > >>> > >>> Log: > >>> - Remove unncecessary full path to binary from *_DEPENDS > >>> > >>> Modified: > >>> head/games/warzone2100/Makefile > >>> > >>> Modified: head/games/warzone2100/Makefile > >>> ============================================================================== > >>> --- head/games/warzone2100/Makefile Thu Jul 9 09:44:33 2015 (r391625) > >>> +++ head/games/warzone2100/Makefile Thu Jul 9 09:44:51 2015 (r391626) > >>> @@ -13,7 +13,7 @@ LICENSE_FILE= ${WRKSRC}/COPYING > >>> > >>> BUILD_DEPENDS= zip:${PORTSDIR}/archivers/zip \ > >>> unzip:${PORTSDIR}/archivers/unzip \ > >>> - ${LOCALBASE}/bin/flex:${PORTSDIR}/textproc/flex > >>> + flex:${PORTSDIR}/textproc/flex > >>> LIB_DEPENDS= libphysfs.so:${PORTSDIR}/devel/physfs \ > >>> libpng.so:${PORTSDIR}/graphics/png \ > >>> libvorbis.so:${PORTSDIR}/audio/libvorbis \ > >>> > >> > >> This could be a mistake. > >> This changes depends from /usr/local/bin/flex to /usr/bin/flex. > >> Are you sure that's what you want to do? > > > > As I understand, that's why binary names are preferred: as there's > > /usr/bin/flex, unneeded dependency won't be installed. However, that > > seem to be an artifact from 6.x days, and this dependency is not needed > > at all, all supported FreeBSD releases have flex in base system. > > > > My understanding is that you created a dependency specification that is > only accidently satisfied. You asked textproc/flex to be installed, but > then check /usr/bin/flex from that package (which is not inside). > > Flex is notorious. If flex is specified in BUILD_DEPENDS, then 100% of > the time, you really, really want /usr/local/bin/flex. If /usr/bin/flex > works, then you would not put it in BUILD_DEPENDS. /usr/bin/flex is > available on all releases so if it works as you say, then textproc/flex > will never get pulled in. > > My guess is that if textproc/flex is listed here, then it's needed over > the base flex. Thus, I think the change is an error. Like I've already said, the dependency is absolutely correct and does what it should. It's a no-op though on all supported FreeBSD versions, so I'm removing it. -- Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D amdmi3@amdmi3.ru ..: jabber: amdmi3@jabber.ru http://amdmi3.ru