Thu Jul 30 03:31:22 2015: Request 106144 was acted upon.
Transaction: Correspondence added by RSCHUPP
Queue: Module-ScanDeps
Subject: [Patch] Preload dependencies for File::BOM
Broken in: (no value)
Severity: (no value)
Owner: Nobody
Requestors: [email protected]
Status: open
Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=106144 >
On 2015-07-29 19:42:33, SLAFFAN wrote:
> Actually, if you agree that the Encode::* modules do need to be
> preloaded then it should be under Encode or Encode::Config, not
> File::BOM.
Well, adding
'Encode.pm' => 'sub',
to %Preload will pack ALL of them for a whopping 21 MB of stuff for anyone
who just wants to
use Encode qw(encode decode);
...
$string = decode("utf8", $bytes);
So I'm hesitant to do that. If someone knows in advance which encodings they
need
they can add an appropriate -M option to their pp command. If they want to be
flexible (e.g. program takes the encoding as input), this could get cumbersome,
though.
There was once a suggestion on the list to add an option to pp that would mean
"include Foo.pm and everything below Foo/". Implementation would be almost
trivial
(just call _glob_in_inc("Foo.pm")), the problem is to come up with a good name
for this option.
Cheers, Roderich
