-
Notifications
You must be signed in to change notification settings - Fork 701
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
{cae}[foss/2021a] OpenFOAM v2106 #13266
{cae}[foss/2021a] OpenFOAM v2106 #13266
Conversation
Test report by @boegel |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 898899889 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
First compiler error from the build log of the failing build on
This looks a lot like CGAL/cgal#4974, but that was fixed in CGAL/cgal#4981, which is included in CGAL 5.2.2. I'm puzzled why this problem only pops up on @olesenm Is OpenFOAM v2106 supposed to be compatible yet with CGAL 5.x? For v2012, we were still using CGAL 4.14.3... |
@boegelbot please test @ generoso |
I've swapped to CGAL 4.14.3 (latest 4.x) as dependency for OpenFOAM v2106, since that seems to work just fine... I'll also use that same CGAL version for OpenFOAM 9 (even though we used CGAL 5.x there for OpenFOAM 8) |
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... - notification for comment with ID 900880206 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Test report by @branfosj |
Test report by @boegel |
Test report by @boegel |
Going in, thanks @robert-mijakovic! |
(created using
eb --new-pr
)Depends on:
edit (by @boegel): fixes #13276