Re: [Module::Build] Re: [Module-build-checkins] [svn:Module-Build] r5869 - Module-Build/trunk/t

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

Ken Williams wrote:
> I'm not sure about this one - it seems weird to set the path with 
> Unix-like syntax but get the result in local syntax.
> 
> Can we take a little pause to review the APIs for this family of 
> methods?  I think there's also some inconsistency between install_base_* 
> and prefix_*, from back when I was a little confused about them.

Yeah, I guess it does seem weird. I wanted to avoid passing in an array 
of path names because it seemed a bit clunky. Since it will be used by 
authors and be expected to work on all platforms, it seemed natural to 
pass in a nutrual, single-style path syntax. I chose unix like paths 
since we do this elsewhere.

That leaves the return value. I figured the return value should be 
immediately usable without the author having to perform the conversion 
from unix to a localized path, so I performed the conversion before 
returning.

The logic made sense to me as I was doing it, but the end result, at 
first glance, does seem unusual.

Randy.