On Mon, 2011-01-10 at 13:19 -0600, Thomas Tuegel wrote:
> On Mon, Jan 10, 2011 at 7:34 AM, Duncan Coutts
> <duncan.cou...@googlemail.com> wrote:
> > I'm slightly reticent to add it to the TestSuite bit itself because it's
> > mixing two things: a static description of the package, with status
> > tracking within the build system.
> >
> > That said, there's not an obvious place to put it that exists all the
> > way through the dep resolution stage. If it were just later on then we
> > could add something to the LocalBuildInfo.
> >
> > Give it a go, but add a TODO note suggesting the status info might be
> > better to live somewhere else. We might be able to do it better when we
> > next overhaul the GenericPackageDescription -> PackageDescription
> > resolution stuff.
> 
> The attached bundle of two patches does what I described in my
> previous message. The changes also broke the TestStanza unit test, so
> I fixed that as well.

Finally applied, thanks Thomas. The patch looks fine (given the issues
we discussed previously).

I'll push this to the 1.10 branch too.

Duncan


_______________________________________________
cabal-devel mailing list
cabal-devel@haskell.org
http://www.haskell.org/mailman/listinfo/cabal-devel

Reply via email to