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