Thu Jul 30 18:02:11 2015: Request 106144 was acted upon.
Transaction: Correspondence added by shawn_laffan
Queue: Module-ScanDeps
Subject: Re: [rt.cpan.org #106144] [Patch] Preload dependencies for
File::BOM
Broken in: (no value)
Severity: (no value)
Owner: Nobody
Requestors: slaf...@cpan.org
Status: resolved
Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=106144 >
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.
