Re: New test-suite functionality

On Mon, 2011-01-10 at 13:19 -0600, Thomas Tuegel wrote:

On Mon, Jan 10, 2011 at 7:34 AM, Duncan Coutts 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