cvs commit: fptools/libraries/Cabal/Distribution/Simple
Build.hs Configure.hs LocalBuildInfo.hs
simonmar at microsoft.com
Mon Oct 31 05:17:22 EST 2005
On 30 October 2005 17:05, Krasimir Angelov wrote:
> krasimir 2005/10/30 09:05:11 PST
> Modified files:
> libraries/Cabal/Distribution/Simple Build.hs Configure.hs
> The previous implementation for Paths_<pkgid>.hs building was
> broken on Windows. The prefixRel function was expecting that all
> bindir/libdir/datadir/... paths are $prefix relative but the
> corresponding functions (mkBinDir/mkLibDir/...) was returning
> absolute paths with expanded $path variable. This commit fixes the
> bug and also:
> * In LocalBuildInfo are added mkLibDirRel/mkBinDirRel/...
> functions. They return the corresponding but without the $prefix
> part. When the path isn't prefix relative then they return Nothing
> * The restriction that all paths on Windows are $prefix relative
> is removed.
> * The code in Paths_<pkgid>.hs can contain both absolute and
> prefix relative paths. When the package is configured only with
> $prefix relative paths then the generated executable will be prefix
> independent and can be moved from one directory to another.
Thanks - I hadn't got around to testing this on Windows yet.
You changed the behaviour so it's possible to configure a package to
have a fixed install location on Windows. This is fine, but I think if
the user does this then the configure step should emit a warning saying
that the package has a fixed install location. Could you add that?
More information about the Cvs-libraries