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: slaf...@cpan.org
      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

 


Reply via email to