Re: [Module-build-general] [PATCH] manifypods

[Michael G S. <sc...@po...>]

On Fri, Jul 18, 2003 at 07:24:45PM +0100, Steve Purkis wrote:
> >However, there's one problem with this patch:
> >
> >	% ./Build fakeinstall
> >	...
> >	Can't figure out where to install things of type 'bindoc' at
> >	/.../lib/Module/Build/Base.pm line 1551.
> >
> >I think it's a problem with which Config.pm vars we're using, but I'm 
> >not sure.  I'll post a follow-up on this in a sec.
> 
> This also happens when you set 'install_dirs=vendor', but not 'core'.
> 
> Config.pm is not setting 'installsiteman*dir' - at least not on my 
> machines.  I don't know if this is something I've done wrong when 
> installing perl, but TBH I have fairly standard installs.

installsiteman*dir and installvendorman*dir have only been introduced as
Config variables in 5.8.1.  Currently, 5.8.1 RC2 gets them slightly wrong.


> More likely, we should be using one of the following for both 'site' 
> and 'vendor':
> 
> 	installman*dir
> 	man*dir
> 	man*direxp

Ideally, you'd want to follow the Config variables if present.  If they're not
present, what do you do?  That's where things become a little fuzzy.
Previous to the introduction of install(site|vendor)man*dir to Config, the
installman*dir vars were used for everything by MakeMaker.

Recently, I had it start guessing the value of installsiteman*dir and
installvendorman*dir from siteprefix and vendorprefix.  I'm somewhat
convinced this is a good heuristic.  I like the idea of core, site and
vendor installations being totally seperate.

Otherwise you could just fall back to the old behavior and use 
installman*dir if install(site|vendor)man*dir don't exist.


-- 
It's Absinthe time!