develooper Front page | perl.perl5.porters | Postings from September 2013

Re: intermittent failure of cpan/Pod-Simple/t/search50.t

Thread Previous | Thread Next
From:
David E. Wheeler
Date:
September 19, 2013 16:48
Subject:
Re: intermittent failure of cpan/Pod-Simple/t/search50.t
Message ID:
66946938-C4ED-47D0-8D82-07F7AB5F4C42@kineticode.com
On Sep 19, 2013, at 4:50 AM, Dave Mitchell <davem@iabyn.com> wrote:

> I don't really understand whether:
> 
> 1) search50.t has unreasonable expectations;
> 2) Pod::Simple::Search::survey() should be returning 'Pod::perldoc' as the
>    name rather than 'perldoc';
> 3) perldoc.pod should be installed as lib/perldoc.pod rather than
>    lib/Pod/perldoc.pod;
> 4) something in the content of perldoc.pod is wrong, and that is confusing
>    Pod::Simple::Search.

Neither do I. The search stuff is a bit of a black box to me. That test has failed a bunch of times on case-insensitive file systems, too, which drove me nuts until I figured it out and fixed it.

  https://github.com/theory/pod-simple/commit/2e1cee24c9fe013729880de83bb5765a431886e3

It also used to get confused by .pod files, which I fixed here:

  https://github.com/theory/pod-simple/commit/77391b2f2cf4268608ba9e6a2ccc44f3f9776cdf

Are these the correct fixes? I have no idea. I don’t use the search stuff myself, and have no idea how it’s supposed to work.

Maybe we can ask Sean?

> Independently of that, perhaps that test file should check all pod files
> rather than just one at random, to make errors easier to spot?

That sounds like a good idea. How does this work for you?

  https://github.com/theory/pod-simple/commit/b174dd0927d672ce3a10b1626f4004a8276fa449

Best,

David


Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About