dorsal team mailing list archive

[Johannes Ring <johannr@xxxxxxxxx>]

On Wed, Feb 9, 2011 at 9:35 AM, Garth N. Wells <gnw20@xxxxxxxxx> wrote:
>
>
> On 09/02/11 08:17, Johannes Ring wrote:
>> On Wed, Feb 9, 2011 at 8:58 AM, Joachim Berdal Haga <jobh@xxxxxxxxx> wrote:
>>>> Is the attached diff an acceptable fix for this?
>>>
>>> I would rather the CONFOPTS were done unconditionally. I believe this option
>>> doesn't take substantial extra time, just writes out the relevant makefiles?
>>
>> It also does some file copying to be able to run the tests from the
>> build directory. However, it is rather that the 'runtests' target Iis
>> added to TARGETS which will make the Dorsal installation time
>> increase.
>>
>
> I don't advocate running 'runtests' automatically. I want to be able to
> run it manually when testing a platform.
>
> My take is that it should as simple as adding cppunit to the package
> lists for systems that have it, and adding
>
>  -DDOLFIN_ENABLE_TESTING=true
>
> to the DOLFIN config. If cppunit it not present, the build system will
> just ignore it.

Well, then it should be as simple as adding this to CONFOPTS.

Johannes