Ken Williams wrote: > > On Oct 15, 2007, at 7:14 PM, Michael G Schwern wrote: > >> Eric Wilhelm wrote: >>> Hi all, >>> >>> Steve is proposing that we add the following parameter to Build.PL >>> >>> installdirs => ($] >= 5.009004 ? 'core' : 'site'), >>> >>> Is that the preferred way of handling this? >> >> Yes. > > I recently had half-a-go at adding a fourth alternative besides 'core', > 'site', and 'vendor', called 'auto'. It would sense whether the given > module was currently installed in a core lib, and if so would choose > 'core'. Otherwise 'auto'. > > That would eliminate the need to hard-code the core version number.
Is it worth the bother? Sometimes it is a good idea to hard code something when that thing is, in fact, hard coded. The point when Module::Build got added to the core isn't going to change, and not many modules have to deal with this problem. Heuristics make me nervous, they can guess wrong. And if you don't have to guess, don't. Of course, the real problem is our default @INC order is backwards. It should be site, vendor, core not core, site, vendor. -- The interface should be as clean as newly fallen snow and its behavior as explicit as Japanese eel porn.
