Bug 611015

Summary: perl-Pugs-Compiler-Rule fails to build
Product: [Fedora] Fedora Reporter: Ralf Corsepius <rc040203>
Component: perl-Pugs-Compiler-RuleAssignee: Steven Pritchard <steve>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: low    
Version: 14CC: goeran, iarnell, mmaslano, perl-devel, ppisar, steve
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-11-09 08:03:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 538278, 614982    

Description Ralf Corsepius 2010-07-03 05:50:15 UTC
Description of problem:
perl-Pugs-Compiler-Rule fails to build in rawhide.

Version-Release number of selected component (if applicable):
perl-Pugs-Compiler-Rule-0.37-5.fc14

How reproducible:
Always.

Steps to Reproduce:
1. rebuild the package
  
Actual results:
Fails to build.

Expected results:
Success.

Additional info:
* Apparently this package is incompatible to perl-5.12 and can't be built
against it.
* Its upstream appears dead.

This package needs to be made working or it will have to be removed from 
Fedora >= 14. AFAICT, this package is not used by Fedora itself, so it's
probably safe to remove it.

Comment 1 Iain Arnell 2010-07-25 11:03:59 UTC
This should be simple to fix. It's only t/ast/00-basic.t that is failing due to different indentation of Data::Dumper. And as a bonus, when it fails, it writes out a new t/ast/00-basic.t_ file with the correct indentation.

make test # fails
diff -b t/ast/00-basic.t t/ast/00-basic.t_ # no difference
mv -f t/ast/00-basic.t_ t/ast/00-basic.t
make test # succeeds

Comment 2 Petr Pisar 2010-07-26 10:38:05 UTC
Checking test results on CPAN, results on different perl versions, checking source code show the problem is more mysterious.

More ever adding some debugging prints into the test one can see Data::Dumper treat local hashes and AST hashes in different manner.

In addition if we patch the tests, it will start failing on lower perls.

Comment 3 Ralf Corsepius 2010-07-26 11:43:36 UTC
Openly said, I don't see much sense in trying to "fix" this package.

It's not used by Fedora, its upstream build-matrix 
(http://matrix.cpantesters.org/?dist=Pugs-Compiler-Rule+0.37)
also speaks a very clear language.

My vote goes to "let this package die".

Comment 4 Iain Arnell 2010-07-27 17:24:08 UTC
(In reply to comment #2)
> Checking test results on CPAN, results on different perl versions, checking
> source code show the problem is more mysterious.

Certainly, bleadperl is throwing up more problems.

> More ever adding some debugging prints into the test one can see Data::Dumper
> treat local hashes and AST hashes in different manner.

Just more indentation differences? Or really different?

> In addition if we patch the tests, it will start failing on lower perls.

Not too much of a problem for us. Either patch the tests only in f14+, or better, simply do something like:

make test \
|| diff -qb t/ast/00-basic.t{,_} \
&& mv -f t/ast/00-basic.t{_,} \
&& make test


(In reply to comment #3)
>Openly said, I don't see much sense in trying to "fix" this package.

At the minute, the package itself doesn't seem to be broken - only its tests fail for an understandable reason.

>It's not used by Fedora, 

Think of the DarkPAN!

>its upstream build-matrix 
>(http://matrix.cpantesters.org/?dist=Pugs-Compiler-Rule+0.37)
>also speaks a very clear language.

That its test fail - but we can see why.

>My vote goes to "let this package die".

I don't really mind either way. Just playing devil's advocate. But according to rel-eng, Steve needs to kill it, not you (or us).

Comment 5 Petr Pisar 2010-07-27 17:39:42 UTC
(In reply to comment #4)
> (In reply to comment #2)
> > More ever adding some debugging prints into the test one can see Data::Dumper
> > treat local hashes and AST hashes in different manner.
> 
> Just more indentation differences? Or really different?
> 
Just indentation. Different number of spaces before hash keys and closing bracket. (If I recall properly.).

Comment 6 Bug Zapper 2010-07-30 12:25:14 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 14 development cycle.
Changing version to '14'.

More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping

Comment 7 Marcela Mašláňová 2010-08-10 12:13:59 UTC
I removed the test. Mainly because I haven't permission to remove this package from distro.

Comment 8 Ralf Corsepius 2010-08-10 12:47:41 UTC
(In reply to comment #7)
> I removed the test.

Please no. What are you trying to achieve? Playing down this package's brokenness? This is not helpful.

> Mainly because I haven't permission to remove this package
> from distro.    
Rel-eng made very clear that they want to cope with this package.

REOPENING.

Comment 9 Petr Pisar 2011-07-26 13:30:48 UTC
There is another problem within Perl 5.14.1: https://rt.cpan.org/Public/Bug/Display.html?id=69816

Comment 10 Petr Pisar 2011-09-01 13:40:07 UTC
Steven, please kill (http://fedoraproject.org/wiki/PackageMaintainers/PackageEndOfLife) this package.

Comment 11 Petr Pisar 2011-11-09 08:03:24 UTC
Package blocked in Koji (in F17 only?)
<https://fedorahosted.org/rel-eng/ticket/4976>.

Comment 12 Petr Pisar 2012-01-02 14:43:36 UTC
*** Bug 770960 has been marked as a duplicate of this bug. ***