[0.9.10] Doesn't compile with CGAL 3.8

Bug #768266 reported by Maik Beckmann
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
DOLFIN
Fix Released
High
Andre Massing

Bug Description

gcc error message: http://codepad.org/1mWy5xvv

Related branches

Changed in dolfin:
status: New → Confirmed
importance: Undecided → High
Changed in dolfin:
assignee: nobody → Andre Massing (massing)
milestone: none → 0.9.11
Revision history for this message
Andre Massing (massing) wrote :

Hi! The "problem" is that CGAL starting with version 3.8 includes a segment-segment intersection in 3D, and therefore conflicts with the DOLFIN variant (which only throws an "non implemented" exception. ) I fixed it here locally, but if backwards compatibility to earlier CGAL version is required, I have to add a proper macro.
I am right now in the process of replacing my Mac OS X with Linux, so give me the weekend :) such that I can properly test the changes.

Revision history for this message
Harish Narayanan (hnarayanan) wrote :

Could you post the patch here so I could use it until it works its way into the main repository?

Revision history for this message
Andre Massing (massing) wrote : Re: [Bug 768266] Re: [0.9.10] Doesn't compile with CGAL 3.8

A rough pach here is attached. Not properly tested and lacks version check
for older CGAL versions.
I won't have time before Sunday to do proper testing, but it should work in
your case. If not,
commenting/macroing out the content og Segment_3_Segment_3_intersection.h in
dolfin/mesh remedied the compile issues in Steffen's case.
Andre

2011/4/29 Harish Narayanan <email address hidden>

> Could you post the patch here so I could use it until it works its way
> into the main repository?
>
> --
> You received this bug notification because you are a bug assignee.
> https://bugs.launchpad.net/bugs/768266
>
> Title:
> [0.9.10] Doesn't compile with CGAL 3.8
>
> Status in DOLFIN:
> Confirmed
>
> Bug description:
> gcc error message: http://codepad.org/1mWy5xvv
>

Revision history for this message
Harish Narayanan (hnarayanan) wrote :

Thanks.

The patch did not work for me, but macroing out the contents of the file worked.

Changed in dolfin:
status: Confirmed → Fix Committed
Changed in dolfin:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.