1 |
commit: 4126f7aa2ec132ce4237e4f631ccd067349913ba |
2 |
Author: Kent Fredric <kentnl <AT> gentoo <DOT> org> |
3 |
AuthorDate: Tue Aug 2 22:41:41 2016 +0000 |
4 |
Commit: Kent Fredric <kentnl <AT> gentoo <DOT> org> |
5 |
CommitDate: Tue Aug 2 22:43:43 2016 +0000 |
6 |
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4126f7aa |
7 |
|
8 |
dev-perl/Test-Deep: Weaken dependency on dev-perl/Test-Tester |
9 |
|
10 |
Without this fix, Test-Deep effectively needs perl-5.22 |
11 |
stabilized in order to satisfy Test-Tester from Perl-5.22 |
12 |
|
13 |
However, satisfying Test-Tester from a mechanism other than Perl-5.22 |
14 |
confuses portages dependency resolver, and confuses portage to not be able |
15 |
to upgrade perl, and not being able to upgrade perl means perl can't be |
16 |
stabilized, invoking a stabilization-needs-stabilization bug. |
17 |
|
18 |
Hence, the choice here optimises for assuming stabilization will |
19 |
happen/has happened, and removes any dependency on Test::Tester, assuming |
20 |
portage will eventually provide it for you. |
21 |
|
22 |
And just to be sure, we use perl before the test phase executes |
23 |
to determine if Test::Tester was /actually/ installed and then only run tests |
24 |
if it was. |
25 |
|
26 |
This means before stabilization, some people *might* install this and not have |
27 |
tests run, but everyone *will* get tests working after stabilization, as |
28 |
opposed to the opposite occurring and stabilization being blocked. |
29 |
|
30 |
Bug: https://bugs.gentoo.org/584238 |
31 |
|
32 |
Package-Manager: portage-2.3.0 |
33 |
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" |
34 |
|
35 |
dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild | 15 +++++++++++---- |
36 |
1 file changed, 11 insertions(+), 4 deletions(-) |
37 |
|
38 |
diff --git a/dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild b/dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild |
39 |
index 18203ba..85759f8 100644 |
40 |
--- a/dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild |
41 |
+++ b/dev-perl/Test-Deep/Test-Deep-1.120.0.ebuild |
42 |
@@ -22,9 +22,16 @@ DEPEND="${RDEPEND} |
43 |
virtual/perl-ExtUtils-MakeMaker |
44 |
test? ( |
45 |
>=virtual/perl-Test-Simple-0.880.0 |
46 |
- || ( |
47 |
- >=virtual/perl-Test-Simple-1.1.10 |
48 |
- ( <virtual/perl-Test-Simple-1.1.10 >=dev-perl/Test-Tester-0.40.0 ) |
49 |
- ) |
50 |
) |
51 |
" |
52 |
+ |
53 |
+src_test() { |
54 |
+ # Bug 584238 Avoidance |
55 |
+ if perl -e 'exit ( eval { require Test::Tester; Test::Tester->VERSION(0.04); 1 } ? 0 : 1 )'; then |
56 |
+ perl-module_src_test |
57 |
+ else |
58 |
+ einfo "Test phase skipped: Test::Tester required for tests" |
59 |
+ einfo "Please upgrade to >=dev-lang/perl-5.22.0 or >=virtual/perl-Test-Simple-1.1.10" |
60 |
+ einfo "if you want this tested" |
61 |
+ fi |
62 |
+} |