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.

Sign up for GitHub

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

Jump to bottom

Add ARCCON_NO_TBB_CONFIG CMake variable to bypass call to find_package for TBB. #1779

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Add ARCCON_NO_TBB_CONFIG CMake variable to bypass call to find_package for TBB. #1779

merged 1 commit into from
Nov 22, 2024

Conversation

stdcm
Copy link
Member

@stdcm stdcm commented Nov 20, 2024

  • This is needed when using an old gcc and tbb version while a more recent tbb version exists locally.
  • Happens for ifpen_rocky9 and gimkl_2018b toolchain.

@stdcm stdcm self-assigned this Nov 20, 2024
@stdcm stdcm added the arccon Arccon component label Nov 20, 2024
grospelliergilles
grospelliergilles approved these changes Nov 20, 2024
View reviewed changes
Copy link
Member

@grospelliergilles grospelliergilles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is ok for me. We are always using CMake config file for TBB at CEA so it should not have any impact for us

@stdcm stdcm force-pushed the dev/sdc-allow-to-avoid-find-package-for-tbb branch from 34ff71d to 1a7bb62 Compare November 22, 2024 08:22
@stdcm stdcm merged commit 19bb227 into main Nov 22, 2024
23 of 25 checks passed
@stdcm stdcm deleted the dev/sdc-allow-to-avoid-find-package-for-tbb branch November 22, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@grospelliergilles grospelliergilles grospelliergilles approved these changes

Assignees

@stdcm stdcm

Labels
arccon Arccon component
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stdcm @grospelliergilles