dolfin team mailing list archive
-
dolfin team
-
Mailing list archive
-
Message #24922
Re: [Branch ~dolfin-core/dolfin/trunk] Rev 6416: Merge BLAS cmake updates.
On 8 November 2011 11:34, Johannes Ring <johannr@xxxxxxxxx> wrote:
> On Tue, Nov 8, 2011 at 11:21 AM, Garth N. Wells <gnw20@xxxxxxxxx> wrote:
>> On 8 November 2011 09:06, Johannes Ring <johannr@xxxxxxxxx> wrote:
>>> Can we remove the call to enable_language(Fortran) in
>>> FindArmadillo.cmake? It makes the configuration fail when there is no
>>> Fortran compiler, like on OS X.
>>
>> The test CMake files don't work without it - they need a Fortran
>> compiler to test that the BLAS lib contains a function.
>>
>> Not sure what's best. Our old BLAS test was poor, and it would be good
>> not to write our own. See
>>
>> http://www.mail-archive.com/cmake@xxxxxxxxx/msg22673.html
>>
>> Maybe we can hack the CMake provided FindBLAS.cmake to skip some checks for OSX?
>
> FindBLAS/FindLAPACK works fine on OS X without calling
> enable_language(Fortran) in FindArmadillo.cmake. I tried to use
> enable_language(Fortran OPTIONAL) but that does not work because of a
> bug in CMake:
>
> http://public.kitware.com/Bug/view.php?id=9220
>
> The workaround listed there could be used to check if it is safe to
> use enable_language(Fortran). Just add language_support.cmake (v2) in
> cmake/modules and add the following in FindArmadillo.cmake (or perhaps
> better in the main CMakeLists.txt file?):
>
I had already started adding something like this to the top-level
CMakeLists.txt file. I'll push it soon.
Garth
> include(language_support)
> workaround_9220(Fortran Fortran_language_works)
> if (Fortran_language_works)
> enable_language(Fortran)
> endif()
>
> Johannes
>
Follow ups
References