Skip to content

Allow build with CMake 4.0.0 #256

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

heitbaum
Copy link

fixes deprecated cmake option and uses cmake 2.8.0 TARGET_FILE update cmake supported minimum versions to 2.8.0...3.10

ref:

CMake Error at .../CMakeLists.txt: (GET_TARGET_PROPERTY):
The LOCATION property may not be read from target "json_...". Use the
target name directly with add_custom_command, or use the generator
expression $<TARGET_FILE>, as appropriate.

fixes #254

fixes deprecated cmake option and uses cmake 2.8.0 TARGET_FILE
update cmake supported minimum versions to 2.8.0...3.10

ref:
- https://cmake.org/cmake/help/latest/policy/CMP0026.html

CMake Error at .../CMakeLists.txt: (GET_TARGET_PROPERTY):
  The LOCATION property may not be read from target "json_...".  Use the
  target name directly with add_custom_command, or use the generator
  expression $<TARGET_FILE>, as appropriate.

Signed-off-by: Rudi Heitbaum <[email protected]>
@jrmarino
Copy link

jrmarino commented Apr 8, 2025

Thank you! These patches worked great on Ravenports.

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.

update cmake version
2 participants