On Wed, Sep 21, 2011 at 2:27 PM, Thomas DuBuisson
<[email protected]> wrote:
>>  The skein
>> package comes with the "golden" KATs sent by the Skein team to NIST
>
> Great! Care to add that to the crypto-api test code?

I don't really understand how the testing workflow works on the
crypto-api package, but I confess that I didn't try hard enough to
understand.  I don't like the use of the test flag to conditionally
expose Test.* modules.  Given that we can't have a flag constraint in
"build-depends", using those modules would basically break the skein
package test suite by default.  They would break even in my own box,
since crypto-api is installed system-wide without the test flag.

So, could we split those Test.* modules into a new package like
crypto-api-tests?  Then I could have in my .cabal:

Library
  Build-depends: ... crypto-api ...

Test-suite runtests
  Build-depends: ... crypto-api, crypto-api-tests ...

and everything would beautifully work =).

Cheers,

-- 
Felipe.

_______________________________________________
Haskell-Cafe mailing list
[email protected]
http://www.haskell.org/mailman/listinfo/haskell-cafe

Reply via email to