Re: patch: --enable-tests and --only-dependencies

On Thu, Feb 2, 2012 at 3:13 PM, Antoine Latter wrote:

On Sat, Jan 7, 2012 at 12:59 PM, Antoine Latter wrote:

...

Hi Cabal,

When trying to use "cabal install --enable-tests --only-dependencies" it appears to be passing the "--enable-tests" option down to the dependent packages during build. This isn't what I want at all - I only want to use the "--enable-tests" flag for the purpose of building an install plan.

Hello again,

I would create a bug for this, except trac is down after massive spamming.

Can someone review this patch? Sorry to be a bother, but the command 'cabal install --enable-tests --only-dependencies' is actually pretty common for me, since I do a lot of work with 'cabal-dev' (and so am re-installing dependencies all the time).

Thomas Tuegel (CCed) would be the right person to review this. If I do cabal install --enable-tests --only-dependencies with your patch I wil install test-suite dependencies of the current package (as in the current directory) only right? That seems like the intended effect. cabal configure --enable-tests is not affected, right? -- Johan