Re: [Module::Build] [PATCH] Write minimal META.yml unless -e YAML.pm

[Randy W. S. <Ra...@Th...>]

On 1/4/2004 12:44 AM, Ken Williams wrote:

> Hi Randy,
> 
> Thanks, I'm working on applying this.
> 
> On Tuesday, December 30, 2003, at 12:37  AM, Randy W. Sims wrote:
> 
>> I'm not sure this patch is clean or complete (I'm up past my bedtime), 
>> but I thought I'd post for comments. This patch:
>>
>>  - Adapts the 'author' parameter to accept either a string or 
>> anonymous array of strings for multiple authors.
>>
>>  - Initialize dist_author and dist_abstract like dist_(name|version).
> 
> 
> I think these methods were a little more complicated than they needed to 
> be, I've simplified 'em (_pod_parse() does its own memoization).
> 
> Do we want $self->{dist_author} to always be an array ref, or always be 
> however the user specified it?  I'm thinking the latter, but I'm not sure.

I think it has to accept either. For backwards compatibility it needs to 
accept a single string, and that's what will probably be used most often 
anyway, so it's convenient. I think it should also accept an array 
reference because many projects are developed by teams, and its easy and 
makes sense to allow for that.

>>  - Adds authored_by field to F<META.yml> and adapts PPMMaker.pm to 
>> write multiple AUTHOR tags if appropriate in PPD files (which is 
>> allowed by the PPD spec).
> 
> 
> Can we change 'authored_by' to just 'author' in the META.yml?

Yeah, the /only/ reason I chose 'authored_by' over 'author' is because 
it makes sense when you read it whether there is one author or a dozen. 
(Ex. "authored by Ken" / "authored by Ken, Micheal, Dave") I think 
either way will be understandable, so I don't have a firm preference.

Randy.