Dave Mitchell writes:

But I don't think its necessary after all: the new() method in Pod::Man
is only called in three places during the build:
1. Building cpan/podlators.
At this point Encode isn't built, and the warning is emitted
2. Testing cpan/podlators/t/*/*.t
At this point Encode is built and there are no warnings.
3. Installing perl and all its modules
It's called many times here, but again, at this point Encode is built so
there are no warnings
Only during (1) is the environment variable PERL_CORE set.
Unless there's a need for Encode to be present when padlators is made,
I propose that the code in Pod::Man::new() which warns if Encode isn't
present, only does so if !$ENV{PERL_CORE}.
This is a great idea. Thanks! Now implemented, and will be in the next
podlators release (probably later today).

Russ Allbery (eagle@eyrie.org) <http://www.eyrie.org/~eagle/>

Search Discussions

Discussion Posts


Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 10 of 10 | next ›
Discussion Overview
grouppod-people @
postedFeb 1, '16 at 4:15a
activeMar 20, '16 at 11:23p



site design / logo © 2021 Grokbase