Skip to content

Fix installation with absolute paths in CMAKE_INSTALL_*DIR - #769

Closed
jansol wants to merge 1 commit into
google:masterfrom
jansol:master
Closed

jansol wants to merge 1 commit into
google:masterfrom
jansol:master

Conversation

@jansol

@jansol jansol commented Nov 15, 2021

Copy link
Copy Markdown

See discussion in #764. See also libexpat/libexpat#459 for a different approach.

This produces correct results when CMAKE_INSTALL_LIBDIR and CMAKE_INSTALL_INCLUDEDIR are absolute paths.

@tomfinegan

Copy link
Copy Markdown
Contributor

Closing this: obsoleted by #775

Please let me know if I missed something, or if you're still running into issues similar to what you reported in #764

@tomfinegan tomfinegan closed this Dec 6, 2021
@jansol

jansol commented Dec 6, 2021

Copy link
Copy Markdown
Author

With that change the build is indeed fine, thanks!

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