
6 Dec
2010
6 Dec
'10
6:50 p.m.
On Tue, Nov 30, 2010 at 5:12 PM, Duncan Coutts
So we ought to find a better way of representing the situation with configure --disable-tests, so that cabal test fails with a helpful error message to reconfigure with --enable-tests, and rebuild. Currently we cannot distinguish no testsuites from disabled testsuites.
Changing the 'testSuites' field of the PackageDescription to be a 'Maybe [TestSuites]' instead of just '[TestSuites]' would do. ('Nothing' == disabled tests and 'Just []' == no tests.) A patch like that would touch a lot of places in the test suite code, but not in a meaningful way. Should I write a patch to do it this way? -- Thomas Tuegel