Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 455122 - 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"
Summary: net-analyzer/wireshark-1.8.5{,-r1} with <=perl-core/Pod-Simple-3.170.0 - Can'...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Netmon project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-02 12:52 UTC by Chris Mayo
Modified: 2014-10-27 22:34 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log with perl-core/Pod-Simple-3.160.0 (build.log.bz2,75.49 KB, application/x-bzip2)
2013-02-02 16:14 UTC, Chris Mayo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Mayo 2013-02-02 12:52:50 UTC
/usr/bin/pod2html							\
--title="capinfos - The Wireshark Network Analyzer 1.8.5"	\
--css=../docbook/ws.css				\
--noindex							\
./capinfos.pod > capinfos.html
Can't locate object method "recurse" via package "Pod::Simple::Search" at /usr/lib64/perl5/5.16.2/Pod/Html.pm line 317.
make[2]: *** [idl2wrs.html] Error 2
make[2]: *** Waiting for unfinished jobs....
Can't locate object method "recurse" via package "Pod::Simple::Search" at /usr/lib64/perl5/5.16.2/Pod/Html.pm line 317.
make[2]: *** [tshark.html] Error 2
Can't locate object method "recurse" via package "Pod::Simple::Search" at /usr/lib64/perl5/5.16.2/Pod/Html.pm line 317.
make[2]: *** [wireshark-filter.html] Error 2
Can't locate object method "recurse" via package "Pod::Simple::Search" at /usr/lib64/perl5/5.16.2/Pod/Html.pm line 317.
make[2]: *** [capinfos.html] Error 2
make[2]: Leaving directory `/var/tmp/portage/net-analyzer/wireshark-1.8.5/work/wireshark-1.8.5/doc'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/var/tmp/portage/net-analyzer/wireshark-1.8.5/work/wireshark-1.8.5'
make: *** [all] Error 2
 [31;01m*[0m ERROR: net-analyzer/wireshark-1.8.5 failed (compile phase):
 [31;01m*[0m   emake failed


Same with USE="doc" or "-doc"
dev-lang/perl-5.16.{1,2}

and net-analyzer/wireshark-1.8.5-r1
Comment 1 Denis M. (Phr33d0m) 2013-02-02 14:44:15 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.
Comment 2 Chris Mayo 2013-02-02 16:14:54 UTC
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
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2013-03-29 16:47:02 UTC
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.
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2013-05-08 02:29:26 UTC
(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.
Comment 5 Khayyam 2013-05-20 20:32:17 UTC
(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
Comment 6 SpanKY gentoo-dev 2014-03-21 03:15:40 UTC
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.
Comment 7 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2014-03-21 15:38:00 UTC
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
Comment 8 SpanKY gentoo-dev 2014-03-21 19:03:01 UTC
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.
Comment 9 Jeroen Roovers (RETIRED) gentoo-dev 2014-03-21 21:28:21 UTC
(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.
Comment 10 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2014-03-22 03:43:28 UTC
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.
Comment 11 SpanKY gentoo-dev 2014-04-29 22:11:25 UTC
(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
Comment 12 Andreas K. Hüttel archtester gentoo-dev 2014-10-27 21:06:51 UTC
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.