dorsal team mailing list archive

[Harish Narayanan <harish.mlists@xxxxxxxxx>]

On 2/9/11 9:49 AM, Johannes Ring wrote:
> 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.

OK, I will add this after the next release.

Harish