Re: invalid installdirs values

[email protected] (Michael G Schwern)
Newsgroups perl.module.build
Message-ID <[email protected]>
David E. Wheeler wrote:
> I think that what would be needed is a way to validate properties.

Yeah, in this case it's installdirs().

Simplest thing might be to add an optional check/normalizing routine to
add_property().

__PACKAGE__->add_property(installdirs => "site",
	sub {
	    return 1 if /^(core|site|vendor)$/;
	    return shift->property_error("Perhaps you meant 'core'?") if $_ eq 'perl';
	    return 0;
        }
);


-- 
I have a date with some giant cartoon robots and booze.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.