On 30/07/2015 21:04, Roderich Schupp via RT wrote:
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




Thanks again Roderich,

How about --moduletree for the option name? --modtree makes me think of modifying a tree.

The wildcard approach would be tricky but could use some sort of file find approach. If I get a chance I'll look into it.

Regards,
Shawn.


Reply via email to