Re: [Module::Build] [PATCH] make test failure report (Cygwin)
Status: Beta
Brought to you by:
kwilliams
|
From: Ken W. <ke...@ma...> - 2003-10-28 03:27:35
|
On Wednesday, October 22, 2003, at 08:15 PM, Randy W. Sims wrote:
>
>
> On 10/15/2003 11:08 PM, Randy W. Sims wrote:
>
>> On 10/15/2003 8:46 AM, Ken Williams wrote:
>>>
>>>> BTW, wouldn't it be better to use Pod::Parser in the dist_*
>>>> routines for more acurate results?
>>>
>>>
>>>
>>> Probably so, yeah. Is that a core module?
>>>
>>> -Ken
>> Yeah, though I'm not sure when it was added. The attached patch
>> should be safe though.
>> Randy.
>
> I forgot to follow up on this. I also forgot how easy it is to find
> out when a module was added to CORE with Module::CoreList:
>
> $ perl -MModule::CoreList -e "print
> Module::CoreList->first_release('Pod::Parser')"
> 5.006
>
> Even though it's kind of new, I'd recommend using it if it's present
> since the current method is flawed.
Okay, I've committed a patch that uses Pod::Parser if available. I
based it on your code, but moved it out into a new
Module::Build::PodParser module that can transmogrify itself based on
whether Pod::Parser is available.
-Ken
|