Skip to content

[vcpkg] Add NVTOOLSEXT_PATH to the env passthrough list #12468

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
Merged
merged 1 commit into from
Jul 21, 2020
Merged

[vcpkg] Add NVTOOLSEXT_PATH to the env passthrough list #12468

merged 1 commit into from
Jul 21, 2020

Conversation

Neumann-A
Copy link
Contributor

(CMake uses it to search for the toolkit)

  if(WIN32)
    # nvtools can be installed outside the CUDA toolkit directory
    # so prefer the NVTOOLSEXT_PATH windows only environment variable
    # In addition on windows the most common name is nvToolsExt64_1
    find_library(CUDA_nvToolsExt_LIBRARY
      NAMES nvToolsExt64_1 nvToolsExt64 nvToolsExt
      PATHS ENV NVTOOLSEXT_PATH
            ENV CUDA_PATH
      PATH_SUFFIXES lib/x64 lib
    )
  endif()

@Neumann-A
Add NVTOOLSEXT_PATH to the env passthrough list
be2f015 
(CMake uses it to search for the toolkit)
@ras0219-msft ras0219-msft merged commit 9220d6f into microsoft:master Jul 21, 2020
@ras0219-msft
Copy link
Collaborator

LGTM, thanks!

@NancyLi1013 NancyLi1013 changed the title Add NVTOOLSEXT_PATH to the env passthrough list [vcpkg] Add NVTOOLSEXT_PATH to the env passthrough list Jul 24, 2020
@Neumann-A Neumann-A deleted the NVTOOLSEXT_PATH branch July 31, 2020 21:05
hellozee pushed a commit to hellozee/vcpkg that referenced this pull request Sep 11, 2020
@Neumann-A @hellozee
[vcpkg] Add NVTOOLSEXT_PATH to the env passthrough list (microsoft#12468
3554af4 
)

(CMake uses it to search for the toolkit)
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
@Neumann-A
[vcpkg] Add NVTOOLSEXT_PATH to the env passthrough list (microsoft#12468
54fe6f3 
)

(CMake uses it to search for the toolkit)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@NancyLi1013 NancyLi1013 NancyLi1013 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@NancyLi1013 NancyLi1013

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Neumann-A @ras0219-msft @NancyLi1013