Date: Wed, 23 May 2012 19:27:02 +0100 From: Chris Rees <crees@FreeBSD.org> To: Warren Block <wblock@wonkity.com>, perl <perl@freebsd.org> Cc: swills@freebsd.org, wblock@freebsd.org, jgh@freebsd.org Subject: Re: Porter's Handbook Perl section update Message-ID: <CADLo83_2ax9GR7%2Bf7U593B%2BPQxjpFzP_f5_VsjwzRdoZUQ%2BBMQ@mail.gmail.com> In-Reply-To: <alpine.BSF.2.00.1205231101410.48694@wonkity.com> References: <alpine.BSF.2.00.1205231101410.48694@wonkity.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 23 May 2012 18:11, Warren Block <wblock@wonkity.com> wrote:
> The second note in the Using perl section of the Porter's Handbook was
> unclear to me:
>
> =A0Note: It is recommended to avoid using ${SITE_PERL} as a dependency
> =A0listing. This is problematic because it means that bsd.perl.mk must be
> =A0included to satisfy the dependency. Additionally, if the module's file=
s
> =A0change, it would require changing all dependent ports.
>
> After some talking with jgh@, I've suggested this:
>
> =A0Do not use ${SITE_PERL} in dependency declarations. =A0Doing so
> =A0assumes that bsd.perl.mk has been included, which is not always true.
> =A0Ports depending on this port will have incorrect dependencies if this
> =A0port's files move later in an upgrade. =A0The right way to declare Per=
l
> =A0module dependencies is shown in the example below.
>
> The second sentence is a little weak, but overall this makes much more
> sense, to me at least. =A0If this looks okay, I'd like to update the Port=
er's
> Handbook.
[adding perl@]
Please!
Chris
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo83_2ax9GR7%2Bf7U593B%2BPQxjpFzP_f5_VsjwzRdoZUQ%2BBMQ>