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