test-suite type not detected

[basvandijk]

I just installed the latest cabal:

$ cabal --version
cabal-install version 0.10.2
using version 1.10.1.0 of the Cabal library 

I'm trying to rewrite the custom test-suite of my  threads package to the new Cabal test-suite feature. The original .cabal contains:

...
executable test-threads
  main-is: test.hs
  ...

When I rewrite that to:

...
test-suite test-threads 
  type: exitcode-stdio-1.0

  main-is: test.hs
  ...

And than configure it, I get the following error:

$ cabal configure
cabal: threads.cabal:60: The 'type' field is required for test suites. The available test types are: exitcode-stdio-1.0

I attached the package.

Duncan on #haskell figured out that if a .cabal contains conditionals, like the one I have, the bug appears.

[duncan]

The threads.cabal file in question is attached. If necessary the rest of the repo is at

 http://code.haskell.org/~basvandijk/code/threads

[ttuegel]

Replying to duncan:

The threads.cabal file in question is attached. If necessary the rest of the repo is at  http://code.haskell.org/~basvandijk/code/threads

I'm looking into this now. It definitely seems to be a bug. However, none of the conditionals used in this particular .cabal file seem to be necessary. I'm attaching what I believe to be an equivalent .cabal file to show you what I mean.

[ttuegel]

Fix bug in Cabal HEAD.

[ttuegel]

Fix bug in Cabal-1.10 branch.

[ttuegel]

I uploaded two patches that fix this problem (one for the Cabal-1.10 branch and one for HEAD). I did upload a prior version of these patches this morning, but I realized they introduced a subtle bug in the way BuildInfo? was being handled. Please disregard the old version of these patches.

[duncan]

Applied the version of the patch sent by email

Fri Apr  1 20:21:13 BST 2011  Thomas Tuegel <ttuegel@gmail.com>
  * Fix for test suite stanzas with conditionals.
  Ticket #811. This fixes a problem where Cabal would fail to detect the "type"
  field of a test suite when the test suite contained a conditional. Conditionals
  can now be used, with the restriction that the "type" field and the appropriate
  "main-is" or "test-module" field must be specified together in any conditional
  branch where they occur.

TODO: merge to 1.10 branch.

[ttuegel]

Duncan, I noticed that you did apply the 1.10 branch patch I sent you, so I am closing this bug.

[elga]

*  virgin mobile