Bug 786849 - False results for Test::Pod and Test::Pod::Coverage since F17 mass rebuild
Summary: False results for Test::Pod and Test::Pod::Coverage since F17 mass rebuild
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-Module-ExtractUse
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Daniel Berrangé
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-02 14:55 UTC by Petr Pisar
Modified: 2012-02-06 09:14 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-02-04 10:22:10 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)
Difference in build roots (16.96 KB, text/plain)
2012-02-02 14:59 UTC, Petr Pisar 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
CPAN 74608 0 None None None Never

Internal Links: 786791

Description Petr Pisar 2012-02-02 14:55:47 UTC 
perl-RT-Client-REST-0.37-9.fc17 uses Test::Kwalittee in tests. The tests passed on F17 mass rebuild (https://koji.fedoraproject.org/koji/buildinfo?buildID=289727).

Now, the tests fail:

#   Failed test 'has_test_pod'
#   at /usr/share/perl5/vendor_perl/Test/Kwalitee.pm line 101.
# Add a test using Test::Pod to check for pod correctness.Doesn't include a test for pod correctness (Test::Pod)

#   Failed test 'has_test_pod_coverage'
#   at /usr/share/perl5/vendor_perl/Test/Kwalitee.pm line 101.
# Add a test using Test::Pod::Coverage to check for POD coverage.Doesn't include a test for pod coverage (Test::Pod::Coverage)
# Looks like you failed 2 tests of 13.
t/99-kwalitee.t ......
Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/13 subtests

I discovered it when upgrading perl-RT-CLient-REST, but I can reproduce it with the old srpm (based on perl-RT-CLient-REST commit a69804d94e1c69815f708b1abeb6a5d3525b91cc).

Also it passes in F16.

I cannot say whether this is bug in perl-Test-Kwalitee (versions of Test::Kwalitee and Module::CPANTS::Analyse did not changed:perl-Test-Kwalitee-1.01-11.fc17, perl-Module-CPANTS-Analyse-0.85-10.fc17).

There is suspicious report on upstream site (https://rt.cpan.org/Public/Bug/Display.html?id=24504).
Comment 1 Petr Pisar 2012-02-02 14:59:10 UTC 
Created attachment 559068 [details]
Difference in build roots

The old build root passes, the new one does not. I know the list is long, but I have no better clue now.
Comment 2 Paul Howarth 2012-02-02 18:33:55 UTC 
This is due to an incompatible change in Parse::RecDescent that has broken Module::ExtractUse (https://rt.cpan.org/Public/Bug/Display.html?id=74608).

It can be fixed by recompiling the grammar in %prep:

# Recompile the grammar to work with the new Parse::RecDescent (CPAN RT#74608)
cd lib/Module/ExtractUse
rm Grammar.pm
perl -MParse::RecDescent - grammar Module::ExtractUse::Grammar
cd -

I have verified that this change fixes the build of both perl-Module-ExtractUse and perl-RT-Client-REST.

I am happy to do this change and build it if the perl-Module-ExtractUse maintainer is busy.
Comment 3 Daniel Berrangé 2012-02-04 10:22:10 UTC 
Built perl-Module-ExtractUse-0.23-11.fc17 into F17 rawhide.
Comment 4 Petr Pisar 2012-02-06 09:14:46 UTC 
Thanks. perl-RT-Client-REST has been rebuilt withTest::Kwalitee as 0.43-2.fc17 successfully.
Note You need to log in before you can comment on or make changes to this bug.