New test-suite functionality

Duncan Coutts duncan.coutts at googlemail.com
Sat Jan 29 19:47:13 CET 2011


On Mon, 2011-01-10 at 13:19 -0600, Thomas Tuegel wrote:
> On Mon, Jan 10, 2011 at 7:34 AM, Duncan Coutts
> <duncan.coutts at 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




More information about the cabal-devel mailing list