Re: [Module::Build] ask() and you will receive

[Randy W. S. <ml...@th...>]

Ken Williams wrote:
> 
> On Apr 14, 2006, at 4:44 AM, Randy W. Sims wrote:
> 
>> +
>> +=item allow_nonoption_default
>> +
>> +Normally, the value of the C<default> arguement must exist in the
>> +C<options> array.  Setting this to true allows the C<default>  argument
>> +to be set to an arbitrary value.
> 
> 
> I do still wonder about the need for this parameter.  The author  could 
> just as easily put the value in the options array, and probably  with 
> less typing.
> 
> Not a big deal, though.
> 
> Let's at least update the verbiage to read something like this,  
> indicating that C<options> isn't required:
> 
> +=item allow_nonoption_default
> +
> +Normally, if an C<options> array is present, then the value of the  
> C<default>
> +argument must exist in C<options>.  Setting  C<allow_nonoption_default> 
> to true
> +allows the C<default> argument to be set to an arbitrary value.

I've done this as well as implemented the 'on_validate' callback. The 
callback return value behavior is a little weird, but it's the least 
weird I could think of at the moment. Overall, I think it will work well.

I've gone ahead and comitted what I have as an initial implementation to 
give something concrete to play with, and as a base with which to 
experiment with the other remaining issues.

Randy.