Wen, A new version of CGAL is available and it break everything. The biggest change is that it is now "header only"[1], requires c++14 (and sometimes requires gmpxx for example for databases/sfcgal). I've taken the patches when they exist or under discussion. [1] But you can built libraries if you need to. https://doc.cgal.org/latest/Manual/installation.html All ports have been tested with poudriere (10/11/HEAD i386/amd64) I tested databases/sfcgal at works and with our CI.
Created attachment 213688 [details] cgal5 math/cgal: * Update to 5.0.2 Changelog: https://www.cgal.org/2020/02/25/cgal502/
Created attachment 213689 [details] databases/sfcgal databases/sfcgal: - Fix with cgal5 : use c++14 and add gmpxx link [1] [1] https://github.com/Oslandia/SFCGAL/pull/218
Created attachment 213690 [details] databases/postgis* databases/postgis*: - bump portversion needed due to cgal and sfcgal changes
Created attachment 213691 [details] graphics/gdal graphics/gdal: - remove LIB_DEPENDS and add BUILD_DEPENDS
(In reply to Loïc Bartoletti from comment #0) Please go ahead ! wen
Created attachment 213692 [details] graphics/colmap graphics/colmap: - remove LIB_DEPENDS and add BUILD_DEPENDS - Use c++14 and force set(CMAKE_CXX_STANDARD 14) if CGAL option is set on
Created attachment 213693 [details] cad/openscad cad/openscad: - remove LIB_DEPENDS and add BUILD_DEPENDS - Take patches from upstream [1] [1] https://github.com/openscad/openscad/pull/3163
Created attachment 213694 [details] cad/openscad-devel cad/openscad-devel: - remove LIB_DEPENDS and add BUILD_DEPENDS - Use c++14
Created attachment 213695 [details] databases/pgrouting databases/pgrouting: - Update to 3.0.0 rc1 (CGAL is removed upstream)
Created attachment 213696 [details] games/flightgear-terragear games/flightgear-terragear: - remove LIB_DEPENDS and add BUILD_DEPENDS - fix Boost error (ld: error: undefined symbol: boost::thread::hardware_concurrency())
Created attachment 213697 [details] cgal and all ports patched
cc jbeich@ graphics/colmap cc sunpoet@ graphics/gdal cc mr@ cad/openscad and cad/openscad-devel cc tcberner@ and swills@ (mentors) I'm unsure about the "BUILD_DEPENDS= ${LOCALBASE}/lib/cmake/CGAL/UseCGAL.cmake:math/cgal"
(In reply to Loïc Bartoletti from comment #12) Yeah, long paths in the depends also make me uncomfortable... You could switch to depending on the package by version https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/makefile-depend.html#makefile-version-dependency eg: BULD_DEPENDS=cgal>=0:math/cgal mfg Tobias
Created attachment 213762 [details] graphics/colmap (In reply to Loïc Bartoletti from comment #6) Can you try this version instead? > -USES= cmake compiler:${OPENMP}c++11-lib eigen:3 gl localbase qt:5 \ > +USES= cmake compiler:${OPENMP}c++14-lang eigen:3 gl localbase qt:5 \ ${OPENMP} expands to gcc- on i386/aarch64/armv6/armv7 which would break build because USES=compiler:gcc-c++14-lang doesn't exist. > ++ set(CMAKE_CXX_STANDARD 14) Submitted upstream as https://github.com/colmap/colmap/pull/853
(In reply to Jan Beich from comment #14) It works. Thanks!
A commit references this bug: Author: lbartoletti Date: Fri May 8 05:27:26 UTC 2020 New revision: 534324 URL: https://svnweb.freebsd.org/changeset/ports/534324 Log: databases/pgrouting: Update to 3.0.0 - Remove the math/cgal dependency - Use postgis30 PR: 245824 Approved by: tcberner (mentor) Differential Revision: https://reviews.freebsd.org/D24578 Changes: head/databases/pgrouting/Makefile head/databases/pgrouting/distinfo head/databases/pgrouting/pkg-plist
A commit references this bug: Author: lbartoletti Date: Fri May 8 05:35:15 UTC 2020 New revision: 534326 URL: https://svnweb.freebsd.org/changeset/ports/534326 Log: smath/cgal: Update to 5.0.2 The new version of CGAL brings improvements and a lot of changes, including 2 major ones [1](header only[2] and require c++14) requiring to patch all software depending on it. [1] https://www.cgal.org/2019/11/08/cgal50/ [2] https://doc.cgal.org/latest/Manual/installation.html math/cgal: - Update to 5.0.2 databases/sfcgal: - Fix with cgal5 : use c++14 and add gmpxx link databases/postgis*: - bump portversion needed due to cgal and sfcgal changes cad/openscad: - remove LIB_DEPENDS and add BUILD_DEPENDS - Use c++14 cad/openscad-devel: - remove LIB_DEPENDS and add BUILD_DEPENDS - Use c++14 games/flightgear-terragear: - remove LIB_DEPENDS and add BUILD_DEPENDS - fix Boost error (ld: error: undefined symbol: boost::thread::hardware_concurrency()) graphics/colmap: - remove LIB_DEPENDS and add BUILD_DEPENDS - fix CMAKE_CXX_FLAGS with c++14 PR: 245824 Approved by: wen, jbeich, tcberner (mentor) Differential Revision: D24578 Changes: head/cad/openscad/Makefile head/cad/openscad/distinfo head/cad/openscad-devel/Makefile head/databases/postgis23/Makefile head/databases/postgis24/Makefile head/databases/postgis25/Makefile head/databases/postgis30/Makefile head/databases/sfcgal/Makefile head/databases/sfcgal/files/ head/databases/sfcgal/files/patch-CMakeLists.txt head/databases/sfcgal/files/patch-src_CMakeLists.txt head/databases/sfcgal/files/patch-src_config.h.cmake head/games/flightgear-terragear/Makefile head/games/flightgear-terragear/files/patch-src_CMakeLists.txt head/graphics/colmap/Makefile head/graphics/colmap/distinfo head/math/cgal/Makefile head/math/cgal/distinfo head/math/cgal/pkg-plist
Commited, thanks!