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

Do not use CMAKE_INSTALL_BINDIR in EkatSetNvccWrapper.cmake #72

Open
bartgol opened this issue Nov 9, 2020 · 0 comments
Open

Do not use CMAKE_INSTALL_BINDIR in EkatSetNvccWrapper.cmake #72

bartgol opened this issue Nov 9, 2020 · 0 comments
Labels
bug Something isn't working cmake

Comments

@bartgol
Copy link
Contributor

bartgol commented Nov 9, 2020

Describe the bug

The following output shows the problem better than I could.

CMake Warning (dev) at /home/projects/ppc64le/cmake/3.18.0/share/cmake-3.18/Modules/GNUInstallDirs.cmake:225 (message):
  Unable to determine default CMAKE_INSTALL_LIBDIR directory because no
  target architecture is known.  Please enable at least one language before
  including GNUInstallDirs.
Call Stack (most recent call first):
  /home/e3sm-jenkins/weaver/workspace/SCREAM_PullRequest_Autotester_Weaver/621/scream/externals/ekat/cmake/mpi/EkatSetNvccWrapper.cmake:23 (include)
  CMakeLists.txt:39 (EkatSetNvccWrapper)
This warning is for project developers.  Use -Wno-dev to suppress it. 

To Reproduce
Should be reproducible on any GPU build.

Expected behavior
We don't want warnings in our config output.

Possible solution
The bin dir is universally ${CMAKE_INSTALL_PREFIX}/bin, so we can probably hardcode that, and avoid including GNUInstallDirs in the script.

@bartgol bartgol added bug Something isn't working cmake labels Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cmake
Projects
None yet
Development

No branches or pull requests

1 participant