Date: Sat, 24 Sep 2011 05:00:37 -0700 (PDT) From: Jeremy Chadwick <freebsd@jdc.parodius.com> To: FreeBSD-gnats-submit@FreeBSD.org Subject: ports/160972: devel/p5-Config-Record -- make Test::Pod and friends optional Message-ID: <20110924120037.AE01F102C31@icarus.home.lan> Resent-Message-ID: <201109241210.p8OCA3Fq011124@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 160972 >Category: ports >Synopsis: devel/p5-Config-Record -- make Test::Pod and friends optional >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: change-request >Submitter-Id: current-users >Arrival-Date: Sat Sep 24 12:10:03 UTC 2011 >Closed-Date: >Last-Modified: >Originator: Jeremy Chadwick >Release: FreeBSD 8.2-STABLE amd64 >Organization: >Environment: System: FreeBSD icarus.home.lan 8.2-STABLE FreeBSD 8.2-STABLE #0: Mon Sep 19 17:34:57 PDT 2011 root@icarus.home.lan:/usr/obj/usr/src/sys/X7SBA_RELENG_8_amd64 amd64 >Description: devel/p5-Config-Record has an unconditional dependency on numerous perl Test::* modules which are absolutely not required for the Config::Record module to be built, installed, or run. This patch adds conditionals for the dependency list, while retaining existing compatibility. That way, those of us who do not want 14 other perl modules installed on their system (trickle-down dependencies from Test::Pod and Test::Pod::Coverage) can continue to have a "minimal" system. Note that WITHOUT_TESTS was chosen because of numerous existing ports use the same name; grep -r WITHOUT_TESTS /usr/ports to verify. >How-To-Repeat: See above. >Fix: Apply below patch. Patch will also be available at the below URL once a PR number is provided: http://jdc.parodius.com/freebsd/{prnumber} --- devel/p5-Config-Record/Makefile.orig 2010-01-28 19:01:28.000000000 -0800 +++ devel/p5-Config-Record/Makefile 2011-09-24 04:51:06.000000000 -0700 @@ -14,8 +14,10 @@ MAINTAINER= cm@therek.net COMMENT= Configuration file access +.if !defined(WITHOUT_TESTS) BUILD_DEPENDS+= ${SITE_PERL}/Test/Pod.pm:${PORTSDIR}/devel/p5-Test-Pod \ ${SITE_PERL}/Test/Pod/Coverage.pm:${PORTSDIR}/devel/p5-Test-Pod-Coverage +.endif MAN3= Config::Record.3 >Release-Note: >Audit-Trail: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110924120037.AE01F102C31>