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

rapids-cmake can now be correctly used by multiple adjacent directories #33

Conversation

robertmaynard
Copy link
Contributor

@robertmaynard robertmaynard commented Jun 24, 2021

rapids-cmake can now be correctly used by multiple adjacent directories
When in a situation where multiple projects are nested together in the following configuration:

- root
 - sub_project_1
 - sub_project_2    

If sub_project_1 doesn't import all the rapid-cmake components thatsub_project_2 requires it will cause weird runtime errors

When in a situation where multiple projects are nested together in the following configuration

- root
  - sub_project_1
  - sub_project_2

If sub_project_1 doesn't import all the rapid-cmake components that sub_project_2 requires it will cause bad errors

This is fixed by making sure that cmake module paths are always cached
@robertmaynard
Copy link
Contributor Author

@trxcllnt This should fix the issue you are seeing when multiple projects require rapids-cmake.

@robertmaynard robertmaynard merged commit 6347519 into rapidsai:branch-21.08 Jun 28, 2021
@robertmaynard robertmaynard deleted the bug/support_multiple_ways_to_include_rapids_cmake branch June 28, 2021 15:52
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