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

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.