Skip to content

Fix cmake installation - #43

Merged
TinyTinni merged 1 commit into
TinyTinni:masterfrom
Kaedras:master
Apr 20, 2026
Merged

Fix cmake installation#43
TinyTinni merged 1 commit into
TinyTinni:masterfrom
Kaedras:master

Conversation

@Kaedras

@Kaedras Kaedras commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Importing this library in cmake using find_package(ValveFileVDF REQUIRED) fails with the following error:

CMake Error at /usr/cmake/ValveFileVDFConfig.cmake:27 (include):
  include could not find requested file:

    /usr/cmake/ValveFileVDFTargets.cmake

This pull request fixes the issue and also fixes the installation path of the cmake files so they get installed into lib64/cmake/ValveFileVDF instead of cmake.

With these changes find_package(ValveFileVDF) and target_link_libraries(<target> PRIVATE ValveFileVDF::ValveFileVDF) work as expected for both system-wide installation and when using CMAKE_INSTALL_PREFIX / CMAKE_PREFIX_DIR.

@TinyTinni

Copy link
Copy Markdown
Owner

Thank you very much.

@TinyTinni
TinyTinni merged commit 9138f83 into TinyTinni:master Apr 20, 2026
3 checks passed
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.

2 participants