From owner-svn-ports-head@FreeBSD.ORG Thu Sep 5 04:30:36 2013 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 1ED95B6C for ; Thu, 5 Sep 2013 04:30:36 +0000 (UTC) (envelope-from bsd-src@helfman.org) Received: from mail-pa0-f46.google.com (mail-pa0-f46.google.com [209.85.220.46]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id E6309277E for ; Thu, 5 Sep 2013 04:30:35 +0000 (UTC) Received: by mail-pa0-f46.google.com with SMTP id fa1so1370071pad.19 for ; Wed, 04 Sep 2013 21:30:35 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=8iCIitEveqEYE4rN3fn1jAtIa7nsnf2gExbfS9fxnkA=; b=QxlwZrhUmhdxx+MlA7rLuu3f8vavh/O9x/dt4IdHyHLH/GyJe1+ufedwZ+CsuM6HUC 37V2wa+Ppu+yrVBrgzoq79WlWUzHERfzz0bRWHx8D+leERNau2oMfD4hovVCkVzG5vIY tVE8UJAk8Qot3eHq6QY7ySUuPlB1rGPFx+k+BYf81/u1vn/DXoTrO4MkN0pWjgYrtqTt IUJd6WF8EjY8LyGWdeBVRLbK1tyE24F6O/K9rI3Sg1REzjrbY6hB8NAWdvz8P1ww5C0i oT0ZPv3cBj3KC7kS0bxfAC/YLcRzx4sLxkJhjZY78ualLXKZU4imGHDSQ2vOS/3QStyN 1yEQ== X-Gm-Message-State: ALoCoQnvYyGL9dEiBSRZMxTTcegFiojUdskSNlu31DmP4SFA5285A0nEKQbKk8dlyW4yQkRNSMvm MIME-Version: 1.0 X-Received: by 10.66.119.202 with SMTP id kw10mr7083100pab.118.1378355435369; Wed, 04 Sep 2013 21:30:35 -0700 (PDT) Sender: bsd-src@helfman.org Received: by 10.70.100.165 with HTTP; Wed, 4 Sep 2013 21:30:35 -0700 (PDT) In-Reply-To: References: <201309040646.r846kOiI053502@svn.freebsd.org> Date: Wed, 4 Sep 2013 21:30:35 -0700 X-Google-Sender-Auth: EsPTbOUBfAsGRSNgcKsoDoN8SKs Message-ID: Subject: Re: svn commit: r326253 - head/games/p5-Games-Dice From: Jason Helfman To: Sunpoet Po-Chuan Hsieh Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.14 Cc: "svn-ports-head@freebsd.org" , "svn-ports-all@freebsd.org" , Jason Helfman , Andrej Zverev , "ports-committers@freebsd.org" X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 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: Thu, 05 Sep 2013 04:30:36 -0000 On Wed, Sep 4, 2013 at 8:51 PM, Sunpoet Po-Chuan Hsieh wrote: > On Thu, Sep 5, 2013 at 11:47 AM, Jason Helfman wrote: > >> On Wed, Sep 4, 2013 at 8:29 AM, Sunpoet Po-Chuan Hsieh >> wrote: >> >> > On Wed, Sep 4, 2013 at 2:59 PM, Andrej Zverev wrote: >> > >> >> On Wed, Sep 4, 2013 at 10:46 AM, Jason Helfman >> wrote: >> >> > Author: jgh >> >> > Date: Wed Sep 4 06:46:23 2013 >> >> > New Revision: 326253 >> >> > URL: http://svnweb.freebsd.org/changeset/ports/326253 >> >> > >> >> > Log: >> >> > - update to 0.043 >> >> > - adoption of perl USES framework >> >> > - adjust dependencies >> >> > >> >> > PR: 181770 >> >> > Submitted by: danilogondolfo@gmail.com ( maintainer ) >> >> > >> >> >> >> > >> >> > +BUILD_DEPENDS= p5-Capture-Tiny>=0:${PORTSDIR}/devel/p5-Capture-Tiny >> \ >> >> > + >> p5-Test-Script>=1.06:${PORTSDIR}/devel/p5-Test-Script \ >> >> > + p5-Test-Simple>=0.96:${PORTSDIR}/devel/p5-Test-Simple >> >> >> >> We try not to add to the Test:: namespace into (RUN|BUILD)_DEPENDS for >> >> p5- and what you need only for the tests (t/*). In this case, I do not >> >> see why it's done. >> >> >> > >> > These three perl modules are all test-only dependencies. We have >> > TEST_DEPENDS [1] for this purpose. At least tinderbox recognizes it. >> > >> > [1] https://wiki.freebsd.org/Perl#Test_Dependencies >> > >> >> Originally, I had thought about adding these in as TEST_DEPENDS, but found >> it wasn't documented anywhere in ports framework, or the handbook. >> That being said, buildlogs for the port, itself, also showed failure on an >> incomplete kit using pourdiere. I don't know if this is expected, or if >> this is an issue with configuration of the software for a build mode vs. a >> test mode. >> >> How does this diff look? >> >> -jgh >> >> Index: games/p5-Games-Dice/Makefile >> =================================================================== >> --- games/p5-Games-Dice/Makefile (revision 326340) >> +++ games/p5-Games-Dice/Makefile (working copy) >> @@ -3,6 +3,7 @@ >> >> PORTNAME= Games-Dice >> PORTVERSION= 0.043 >> +PORTREVISION= 1 >> CATEGORIES= games perl5 >> MASTER_SITES= CPAN >> PKGNAMEPREFIX= p5- >> @@ -12,7 +13,7 @@ >> >> LICENSE= MIT >> >> -BUILD_DEPENDS= p5-Capture-Tiny>=0:${PORTSDIR}/devel/p5-Capture-Tiny \ >> +TEST_DEPENDS= p5-Capture-Tiny>=0:${PORTSDIR}/devel/p5-Capture-Tiny \ >> p5-Test-Script>=1.06:${PORTSDIR}/devel/p5-Test-Script \ >> p5-Test-Simple>=0.96:${PORTSDIR}/devel/p5-Test-Simple >> >> > It's ok. BTW, you don't need to bump PORTREVISION if there's only > BUILD_DEPENDS changes. > > Regards, > sunpoet > > I thought with the removal of the dependencies that it would warrant a bump, however in reading over the handbook, it seems unnecessary. Thanks! -jgh