On Thu, 2008-05-01 at 01:24 +0200, Thomas Schilling wrote: > On 1 maj 2008, at 00.30, Duncan Coutts wrote:
> > If you use flattenPackageDescription on this description you will > > get a PackageDescription whith buildable: False because flattening > > ignores all conditions and includes *both* sides of conditionals. > > > > The key part here is that 'flattenPackageDescription' (more precisely > 'unionBuildInfo' uses '&&' to join multiple occurences of buildable. > If at any time a branch claims the package is not buildable, we, of > course, want this property to be propagated. Using a '||' only for > within flattenPackageDescription doesn't seem like a good idea to me. Yes. flattenPackageDescription is doing the right thing. It's just not appropriate to use in this particular context. Perhaps we need to document this function more loudly :-). I mean the current documentation is quite accurate but perhaps it needs more big flashing lights. Duncan _______________________________________________ cabal-devel mailing list [email protected] http://www.haskell.org/mailman/listinfo/cabal-devel
