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

cmake: Add options. Revise directory tree. #6

Merged
merged 3 commits into from
Jun 17, 2024

Conversation

maartenarnst
Copy link
Contributor

@maartenarnst maartenarnst commented Jun 12, 2024

Summary

This PR starts to fill our Kokkos-utils library with a header file, a test, and doc.

Description

  • options to enable tests and doc
  • find Kokkos and GTest if the needed targets are not yet defined
  • set C++ standard on the target, i.e. for the library that we build
  • add add_one_test helper function
  • add tests.dox with a Doxygen group unittests to collect a list of unit tests

The second and third changes are needed to allow kokkos-utils builds as part of another code. This other code may provide the targets for the needed dependencies. And we don't want to interfere with how it sets the CXX standard.

Related to

Extends:

@maartenarnst maartenarnst force-pushed the install branch 3 times, most recently from 9fa42c0 to ca662e2 Compare June 13, 2024 05:42
@maartenarnst maartenarnst force-pushed the install branch 3 times, most recently from d8cc3aa to f578714 Compare June 13, 2024 09:31
@maartenarnst maartenarnst force-pushed the install branch 2 times, most recently from dd105a9 to bf660eb Compare June 14, 2024 13:05
@maartenarnst maartenarnst force-pushed the install branch 4 times, most recently from ed4c111 to 0e75055 Compare June 14, 2024 15:51
@maartenarnst maartenarnst self-assigned this Jun 14, 2024
@maartenarnst maartenarnst force-pushed the install branch 3 times, most recently from 7f2139b to 05a069b Compare June 17, 2024 12:28
Move dockerfile to dedicated docker directory
@romintomasetti romintomasetti merged commit 365df5b into uliegecsm:develop Jun 17, 2024
2 checks passed
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.

2 participants