On Thu Dec 02 18:10:28 2010, MichaelRWolf@att.net wrote:
Show quoted text> > I'd suggest documenting both, and also providing a separate
> > helper function (install_module_share) to to the later. This would make
> > the API more usable (IMHO).
>
> This is definately uuugly:
> install_module_share 'lib/Dist/Name/Mod/Sub1/Sub2/share', 'module',
> 'Dist::Name::Mod::Sub1::Sub2';
>
> These would be "pretteir"
>
> install_module_share 'lib/Dist/Name/Mod/Sub1/Sub2/share';
> or
> install_module_share 'lib::Dist::Name::Mod::Sub1::Sub2';
>
Oops. I misread the code, expecting a shift on the args, when, in fact,
it was a pop. Therefore, this isn't as uuugly as I had thought.
install_module_share(
'module',
'Dist::Name::Mod::Sub1::Sub2',
'lib/Dist/Name/Mod/Sub1/Sub2/share');
But it could still be nicer...
... and documented.
Show quoted text>
> And... just for completeness, and self-documentation, an alias from
> install_share to install_dist_share would be great.
>
