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

Bug fix: Install yaml.h to INSTALL_INCLUDE_DIR #244

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

Conversation

sloretz
Copy link

@sloretz sloretz commented Mar 2, 2022

The CMake build has a variable INSTALL_INCLUDE_DIR that controls the folder headers get installed to. It appears the intent is to allow installing the header to a different folder using cmake -DINSTALL_INCLUDE_DIR=... , and the include directory on the exported yaml target reflects that.

Unfortunately, the actual install() command ignores it and hard codes include. This PR fixes that bug.

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.

1 participant