Summary: | net-analyzer/wireshark-1.8.5{,-r1} with <=perl-core/Pod-Simple-3.170.0 - Can't locate object method "recurse" via package "Pod::Simple::Search" | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Chris Mayo <aklhfex> |
Component: | Current packages | Assignee: | Gentoo Netmon project <netmon> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | god, perl, vapier |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | build.log with perl-core/Pod-Simple-3.160.0 |
Description
Chris Mayo
2013-02-02 12:52:50 UTC
Would you mind posting the `emerge -pv <package>` output and upload the full build.log ? I'm unable to reproduce... looks some strange dependency on perl-core/Pod-Simple ? But I don't have that installed and wireshark compiles fine for me. Created attachment 337690 [details]
build.log with perl-core/Pod-Simple-3.160.0
You are right to point at perl-core/Pod-Simple.
The failure is happening with perl-core/Pod-Simple-3.160.0 installed (build.log attached).
After unmerging perl-core/Pod-Simple or upgrading to perl-core/Pod-Simple-3.230.0 wireshark emerges OK.
[ebuild R ] net-analyzer/wireshark-1.8.5:0/1.8.5 USE="-adns caps crypt doc -doc-pdf geoip gtk ipv6 -kerberos -libadns -lua pcap portaudio -profile -python (-selinux) smi ssl zlib" 0 kB
http://cpansearch.perl.org/src/DWHEELER/Pod-Simple-3.26/ChangeLog says: 2011-07-09 David E. Wheeler <david@justatheory.org> * Release 3.17 [...] Added "recurse" option to Pod::Simple::Search. It's enabled by default; disable it to turn off recursion into subdirectories. Marc Green/Google Summer of Code. Looking at make-version.pl, I guess we should perhaps add these to DEPEND: >=perl-core/Pod-Simple-3.170.0 perl-core/Time-Local virtual/perl-Getopt-Long and drop dev-lang/perl since that's implicit. CC'ing perl@ for confirmation. (In reply to comment #3) > Looking at make-version.pl, I guess we should perhaps add these to DEPEND: > >=perl-core/Pod-Simple-3.170.0 > perl-core/Time-Local > virtual/perl-Getopt-Long > > and drop dev-lang/perl since that's implicit. > > CC'ing perl@ for confirmation. I just went and did that anyway. (In reply to comment #3) > Looking at make-version.pl, I guess we should perhaps add these to DEPEND: > >=perl-core/Pod-Simple-3.170.0 > perl-core/Time-Local > virtual/perl-Getopt-Long and removing these "dependencies" net-analyzer/wireshark-1.8.7 builds just fine, so something else is at issue here. # diff wireshark-1.8.7.ebuild ~portdir/net-analyzer/wireshark/wireshark-1.8.7.ebuild 55a56 > >=virtual/perl-Pod-Simple-3.170.0 58a60,61 > virtual/perl-Getopt-Long > virtual/perl-Time-Local # emerge -pv wireshark [ebuild R ] net-analyzer/wireshark-1.8.7:0/1.8.7::aporia-local USE="caps crypt filecaps gtk ipv6 pcap smi ssl zlib -adns -doc -doc-pdf -geoip -kerberos -libadns -lua -portaudio -profile -python (-selinux)" PYTHON_SINGLE_TARGET="python2_7 -python2_5 -python2_6" PYTHON_TARGETS="python2_7 -python2_5 -python2_6" 0 kB sorry, but i think this change is wrong. wireshark doesn't care about these random perl deps, it only cares about `pod2html`. it makes no sense for wireshark to be depending directly on the random modules that `pod2html` itself happens to use. that's a total losing game. so imo, the change should be reverted and wireshark should only depend on dev-lang/perl. if a dep is missing, it's in perl itself. maybe the perl team wants to add a virtual/perl-pod2html package that other packages (like wireshark) can depend on to avoid circular dependencies. If a virtual package is to be created, it should be virtual/perl-Pod-Html to match the name of the pseudo-distribtion in the Perl tree: https://metacpan.org/source/RJBS/perl-5.18.2/ext Though one should certainly _NOT_ be depending on perl-core/* directly, that's what we have virtuals for. Because .... Pod-Simple _is_ shipped with perl itself, and the virtuals are to intelligently ( not really ) determine whether to defer to perl, or to perl-core/* https://metacpan.org/source/RJBS/perl-5.18.2/cpan/Pod-Simple if netmon doesn't mind, i'll revert the perl deps now to dev-lang/perl, and we can debate the pod2html aspect here. i don't think it's a super critical bug considering how few people have reported this problem. (In reply to SpanKY from comment #8) > if netmon doesn't mind, i'll revert the perl deps now to dev-lang/perl, and > we can debate the pod2html aspect here. i don't think it's a super critical > bug considering how few people have reported this problem. Sure. There really aught to be a blocker somewhere so <virtual/perl-Pod-Simple-3.170 and <perl-core/Pod-Simple-3.170 are refused installation in conjunction with Perl >= 5.16, because pod2html is effectively broken in such a combination. That's the real solution we should be looking for here. (In reply to Kent Fredric from comment #10) i don't see the point, but i can live with blockers as they don't bloat the dep graph. committed: http://sources.gentoo.org/net-analyzer/wireshark/wireshark-1.8.14.ebuild?r1=1.1&r2=1.2 Looking at the versions this bug is way obsolete. The real issue is probably that under certain rare circumstances people *might* have perl-core modules installed that are older than the bundled versions. Filed bug 527152 as base for discussion. looks like my changes got lost in the shuffle, so i re-added them: http://sources.gentoo.org/net-analyzer/wireshark/wireshark-1.12.1.ebuild?r1=1.11&r2=1.12 http://sources.gentoo.org/net-analyzer/wireshark/wireshark-1.99.0.ebuild?r1=1.2&r2=1.3 http://sources.gentoo.org/net-analyzer/wireshark/wireshark-1.99.0-r1.ebuild?r1=1.1&r2=1.2 |