dev-perl/Test-Deep: Weaken dependency on dev-perl/Test-Tester
authorKent Fredric <kentnl@gentoo.org>
Tue, 2 Aug 2016 22:41:41 +0000 (10:41 +1200)
committerKent Fredric <kentnl@gentoo.org>
Tue, 2 Aug 2016 22:43:43 +0000 (10:43 +1200)
Without this fix, Test-Deep effectively needs perl-5.22
stabilized in order to satisfy Test-Tester from Perl-5.22

However, satisfying Test-Tester from a mechanism other than Perl-5.22
confuses portages dependency resolver, and confuses portage to not be able
to upgrade perl, and not being able to upgrade perl means perl can't be
stabilized, invoking a stabilization-needs-stabilization bug.

Hence, the choice here optimises for assuming stabilization will
happen/has happened, and removes any dependency on Test::Tester, assuming
portage will eventually provide it for you.

And just to be sure, we use perl before the test phase executes
to determine if Test::Tester was /actually/ installed and then only run tests
if it was.

This means before stabilization, some people *might* install this and not have
tests run, but everyone *will* get tests working after stabilization, as
opposed to the opposite occurring and stabilization being blocked.

Bug: https://bugs.gentoo.org/584238

Package-Manager: portage-2.3.0
RepoMan-Options: --include-arches="alpha amd64 amd64-fbsd arm arm64 hppa ia64 m68k mips nios2 ppc ppc64 riscv s390 sh sparc sparc-fbsd x86 x86-fbsd"

dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild

index 18203ba23795ce5f4b5dbbead8f14c6d77493e9f..85759f86efa597f2f1f1981c9ed3f7470a6ed600 100644 (file)
@@ -22,9 +22,16 @@ DEPEND="${RDEPEND}
        virtual/perl-ExtUtils-MakeMaker
        test? (
                >=virtual/perl-Test-Simple-0.880.0
-               || (
-                       >=virtual/perl-Test-Simple-1.1.10
-                       ( <virtual/perl-Test-Simple-1.1.10 >=dev-perl/Test-Tester-0.40.0 )
-               )
        )
 "
+
+src_test() {
+       # Bug 584238 Avoidance
+       if perl -e 'exit ( eval { require Test::Tester; Test::Tester->VERSION(0.04); 1 } ? 0 : 1 )'; then
+               perl-module_src_test
+       else
+               einfo "Test phase skipped: Test::Tester required for tests"
+               einfo "Please upgrade to >=dev-lang/perl-5.22.0 or >=virtual/perl-Test-Simple-1.1.10"
+               einfo "if you want this tested"
+       fi
+}