← Back to team overview

dolfin team mailing list archive

Re: [Branch ~dolfin-core/dolfin/main] Rev 4406: has_gts -> has_cgal (fixes issue on Mac buildbot that doesn't have CGAL).

 

On Sat, Jan 30, 2010 at 08:24:19PM +0100, Anders Logg wrote:
> On Fri, Jan 29, 2010 at 03:06:01PM +0100, Johannes Ring wrote:
> > On Fri, Jan 29, 2010 at 1:16 PM, Johannes Ring <johannr@xxxxxxxxx> wrote:
> > > On Fri, Jan 29, 2010 at 12:36 PM, Anders Logg <logg@xxxxxxxxx> wrote:
> > >> On Fri, Jan 29, 2010 at 12:08:21PM +0100, Johannes Ring wrote:
> > >>> [resending to include list]
> > >>>
> > >>> On Fri, Jan 29, 2010 at 11:29 AM, Anders Logg <logg@xxxxxxxxx> wrote:
> > >>> > Are there still any HAS_CGAL issues to fix or has this been sorted
> > >>> > out?
> > >>>
> > >>> I'm not sure but the C++ function demo fails on the Mac buildbot with
> > >>> the following message:
> > >>>
> > >>>  $ ./test
> > >>>  Solving linear variational problem
> > >>>    Matrix of size 4913 x 4913 has 126785 nonzero entries.
> > >>>    Solving linear system of size 4913 x 4913 (PETSc LU solver, umfpack).
> > >>>  ##Failure Location unknown## : Error
> > >>>  Test name: Eval::testArbitraryEval
> > >>>  uncaught exception of unknown type
> > >>>
> > >>>  Failures !!!
> > >>>  Run: 1   Failure total: 1   Failures: 0   Errors: 1
> > >>>
> > >>> Could it be because CGAL is missing?
> > >>
> > >> I don't know. I saw some strange errors yesterday in testArbitraryEval
> > >> which went away after I did instant-clean. Is that done on the
> > >> buildbots before they run?
> > >
> > > It is only done nightly when a rebuild of Instant is triggered.
> > > However, one can force instant-clean to run by forcing a rebuild of
> > > Instant. But I don't think that is the problem here since I just
> > > logged into the Mac buildbot and ran instant-clean and then the
> > > function test again and I got the same error.
> >
> > The problem is now gone after I got CGAL working so I guess an #ifdef
> > HAS_CGAL is needed somewhere in the C++ function test. I'm not sure
> > where to add it so I leave that to someone else.
> >
> > BTW, the problem with CGAL was resolved by removing GMP 4.2.4 and
> > build GMP 4.3.1 instead. I also had to apply Harish's dorsal patch and
> > turn off PDB in CGAL.
> >
> > Johannes
>
> ok, I have tried adding one. We currently have double #ifdefs for CGAL
> and GTS. GTS should be removed soon (replaced by CGAL) so then the
> extra #ifdef should disappear.
>
> I'll restart the Windows buildbot and see what happens.

The buildbot seems to be offline.

--
Anders

Attachment: signature.asc
Description: Digital signature


References