dolfin team mailing list archive
-
dolfin team
-
Mailing list archive
-
Message #19255
Re: [Branch ~dolfin-core/dolfin/main] Rev 5046: Add FindCGAL.cmake.dolfin for finding CGAL config. Includes compile/run
On Sat, Aug 21, 2010 at 11:49:06PM +0100, Garth N. Wells wrote:
> On Sat, 2010-08-21 at 20:52 +0000, noreply@xxxxxxxxxxxxx wrote:
> > ------------------------------------------------------------
> > revno: 5046
> > committer: Anders Logg <logg@xxxxxxxxx>
> > branch nick: dolfin-dev
> > timestamp: Sat 2010-08-21 22:45:13 +0200
> > message:
> > Add FindCGAL.cmake.dolfin for finding CGAL config. Includes compile/run
> > test that passes on Lucid.
>
> I don't think that this is the right approach (plus it no longer works
> on my computer).
>
> CGAL uses CMake, and generates a config file with loads of build
> information, like libraries on which it depends. We should use the CMake
> config file.
>
> Garth
It didn't work for me before and I found the CGAL CMake file difficult
to understand. Works fine for me now on Lucid.
Another problem with using the installed CMake file was that if CGAL
was not installed then it would just assume things work.
Anoher improvement with the new config script is that it actually
tries to build a CGAL test program (which uses essentially the stuff
that we need in DOLFIN).
--
Anders
Follow ups
References