Skip to content

API Changes to Boost and CGAL#4635

Open
seanmccully wants to merge 1 commit intosupermerill:master_27from
seanmccully:archlinux_build
Open

API Changes to Boost and CGAL#4635
seanmccully wants to merge 1 commit intosupermerill:master_27from
seanmccully:archlinux_build

Conversation

@seanmccully
Copy link
Copy Markdown

This pull request fixes Boost and CGAL API changes in the recent builds for both projects. It attempts to add a compatibility layer between the old API and New API to support the minimal difference in supporting both versions.

The goal being to fix the build issues on Linux when using system libraries for distributions with rolling releases like Arch-Linux
.

 - Boost Process V2 API changes
   - Updates CMake for boost linking
 - CGAL property_map accessors
@seanmccully seanmccully requested a review from supermerill as a code owner June 17, 2025 23:30
@supermerill
Copy link
Copy Markdown
Owner

tried to compile on windows but I had an error:
image
can't find file CGAL/AABB_traits_3.h: No such fiel or directory

@seanmccully
Copy link
Copy Markdown
Author

tried to compile on windows but I had an error: image can't find file CGAL/AABB_traits_3.h: No such fiel or directory

Apologies I sort of realized after I opened this I probably should have given this (build) on Windows/Mac. I will look at it again and update you.

jacobcxdev added a commit to jacobcxdev/BambuStudio that referenced this pull request Aug 4, 2025
Updated dependencies, added CGALCompat from supermerill/SuperSlicer#4635.
@RealDeuce
Copy link
Copy Markdown

Trying this PR on FreeBSD, with CGAL 5.6.2, I don't have CGAL/AABB_traits_3.h, only CGAL/AABB_traits.h (and CGAL/Minkowski_sum_2/AABB_traits_2.h and CGAL/Surface_mesh_segmentation/internal/AABB_traits.h) so I think you need to check for the header in CMake rather than assume presence based on version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants