-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
[Bug]: CMake Quickstart: policy CMP0135 is not set. #4289
Comments
Nickrader
changed the title
[Bug]: Please title this bug report
CMake Quickstart: policy CMP0135
Jun 19, 2023
Nickrader
changed the title
CMake Quickstart: policy CMP0135
CMake Quickstart: policy CMP0135 is not set.
Jun 19, 2023
Nickrader
changed the title
CMake Quickstart: policy CMP0135 is not set.
[Bug]: CMake Quickstart: policy CMP0135 is not set.
Jun 20, 2023
Steps to ReproduceUsing the given repo can't reproduce the problem.
|
I probably shouldn't have the repository head point to my workaround. |
I opened Pull Request #4630 to address this issue |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the issue
CMake Quickstart:
When building per instructions, after exectuting: cmake -S . -B build
"CMake Warning (dev) at /usr/share/cmake-3.24/Modules/FetchContent.cmake:1267 (message):
The DOWNLOAD_EXTRACT_TIMESTAMP option was not given and policy CMP0135 is
not set. "
Steps to reproduce the problem
https://github.com/Nickrader/GTestCMakeQuickstart
What version of GoogleTest are you using?
Latest to fix error.
What operating system and version are you using?
Lubuntu 22.10
What compiler and version are you using?
gcc version 12.2.0 (Ubuntu 12.2.0-3ubuntu1)
What build system are you using?
cmake version 3.24.2
Additional context
Unsure about implementation of fix.
Simple enough to use latest commit and GIT_REPOSITORY, GIT_TAG.
To use URL it seems setting CMP0135 to NEW would be solution, based on other similar problems/solutions. Unsure where to set, as error didn't point to source code in Googletest.
The text was updated successfully, but these errors were encountered: