On 2/9/11 9:49 AM, Johannes Ring wrote:
> On Wed, Feb 9, 2011 at 9:35 AM, Garth N. Wells <[email protected]> wrote:
>>
>>
>> On 09/02/11 08:17, Johannes Ring wrote:
>>> On Wed, Feb 9, 2011 at 8:58 AM, Joachim Berdal Haga <[email protected]> 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

_______________________________________________
Mailing list: https://launchpad.net/~dorsal
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~dorsal
More help   : https://help.launchpad.net/ListHelp

Reply via email to