ports/62398: [PATCH] devel/p5-AppConfig: update to 1.56, take maintainership
Erwin Lansing
erwin at lansing.dk
Fri Feb 6 15:14:33 UTC 2004
On Fri, Feb 06, 2004 at 03:47:04PM +0100, Mathieu Arnold wrote:
> +-le 06/02/2004 14:53 +0100, Lars Thegler écrivait :
> | Erwin Lansing wrote:
> |> Should Test::Simple be added as a dependencies or are you sure
> |> it is only used by make test?
> |
> | Yes, Test::More is only used for testing. But I get your point - to silence
> | the warning, either the dependency should be added, or the Makefile.PL
> | patched. What's best practice in these cases?
>
> As far as I can tell, there are good chances that Test::Simple is already
> installed, but I would not bring a SOMETHING_DEPENDS if it does not really
> depends on it. if a developper wants to make test, he's likely to have it
> already, and if not, he's likely to be smart enough to find out how to add it.
> All this to say that I don't think adding a dependency just to shut up
> Makefile.PL is a good thing.
>
I replied to Lars personally, but let me repeat it here.
I have seen both cases, where the maintainer adds Test::Simple or just
leaves the warning. Personally I add the dependency as I assume (most
likely wrongly in several cases) that if the author tests for it, it
needs it. But I'll leave it up to the maintainers discression to decide.
-erwin
--
_._ _,-'""`-._
Erwin Lansing (,-.`._,'( |\`-/| erwin at lansing.dk
http://droso.org `-.-' \ )-`( , o o) erwin at FreeBSD.org
-bf- `- \`_`"'-
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-ports-bugs/attachments/20040206/27477d43/attachment.sig>
More information about the freebsd-ports-bugs
mailing list