Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Remove usage of find_path to locate cuda/std/detail/__config #291

Merged
merged 1 commit into from
Jul 22, 2022

Conversation

robertmaynard
Copy link
Collaborator

When a consumer of libcudacxx uses the CMake FIND_ROOT_PATH_MODE_INCLUDE option it will cause all find_path searches to only occur under the find root path. Since this normally doesn't include the source files, it means that libcudacxx-header-search.cmake will fail to find the cuda/std/detail/__config file.

This issue was found when using conda-build on a project that includes libcudacxx, since conda-build sets up a cross compilation enviornment including a find root path.

When a consumer of libcudacxx uses the CMake `FIND_ROOT_PATH_MODE_INCLUDE`
option it will cause all find_path searches to only occur under
the find root path. Since this normally doesn't include the
source files, it means that libcudacxx-header-search.cmake will fail
to find the cuda/std/detail/__config file.

This issue was found when using conda-build on a project that
includes libcudacxx, since conda-build sets up a cross compilation
enviornment including a find root path.
@wmaxey wmaxey self-assigned this Jul 20, 2022
@wmaxey wmaxey added the bug: functional Does not work as intended. label Jul 20, 2022
@wmaxey wmaxey added this to the 1.9.0 milestone Jul 20, 2022
@wmaxey wmaxey requested a review from alliepiper July 20, 2022 20:24
@wmaxey wmaxey merged commit f6a2c05 into NVIDIA:main Jul 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug: functional Does not work as intended.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants