Skip to content
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

Fixing ICPC usage with modern cmake and wrappers #8629

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

Marc-Pierre-Barbier
Copy link

Summary of Changes

Related to conan's conan-io/conan-center-index#25843

replaced 'fp-model strict' by 'fp-model=strict' https://www.intel.com/content/www/us/en/docs/cpp-compiler/developer-guide-reference/2021-8/fp-model-fp.html for more compatibility with nvcc. as 'fp-model=strict' will be directly transmited to the compiler while 'fp-model strict' will have 'strict' being treated as a file input.

Also added a CXX only filter for newer cmake 3.3+

Please use the following template to help us managing pull requests.

Release Management

  • Affected package(s): at least conan

Related to conan's conan-io/conan-center-index#25843

replaced 'fp-model strict' by 'fp-model=strict' https://www.intel.com/content/www/us/en/docs/cpp-compiler/developer-guide-reference/2021-8/fp-model-fp.html
for more compatibility with nvcc. as 'fp-model=strict' will be directly
transmited to the compiler while 'fp-model strict' will have 'strict'
being treated as a file input.

Also added a CXX only filter for newer cmake 3.3+
@sloriot
Copy link
Member

sloriot commented Dec 4, 2024

Successfully tested in CGAL-6.1-Ic-33

@sloriot sloriot added Tested Not yet approved The feature or pull-request has not yet been approved. and removed Under Testing Not yet approved The feature or pull-request has not yet been approved. labels Dec 4, 2024
@sloriot sloriot merged commit 65d4008 into CGAL:6.0.x-branch Dec 4, 2024
8 checks passed
@lrineau lrineau modified the milestones: 6.1-beta, 6.0.2 Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants