[rt.cpan.org #106144] [Patch] Preload dependencies for File::BOM

Thu Jul 30 07:04:34 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-30 05:02:28, SLAFFAN wrote:
> Coming back to File::BOM, do you want a revised patch for it?

Not needed, thanks. I just added

  'File/BOM.pm'      => [qw( Encode/Unicode.pm )],

to %Preload and a separate rule to recognize constructs like 

  open FH, '<:via(Foo)',...

So I'm resolving this ticket.

> The "everything below Foo" option could possibly be added to pp by
> adding wildcard or regexp handling to the -M argument, similar to
> --add in perlapp.
> http://docs.activestate.com/pdk/9.4/PerlApp.html#perlapp_top

Looks like overkill to me. Also, pp allows for both 

  -M Foo::Bar

and

   -M Foo/Bar.pm

so wildcards will be tricky. Perhaps a separate option "--modtree Foo::Bar"
for adding Foo/Bar.pm and anything below Foo/Bar? 

Cheers, Roderich