Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix "include" directory installation. #842

Closed
wants to merge 1 commit into from

Conversation

LocutusOfBorg
Copy link
Contributor

the variable INSTALL_INCLUDE_DIR already exists, and defaults to include if not specificied otherwise.
Using it allows people to customize the installation from outside, fixing issues with other OS like Haiku

Signed-off-by: Gianfranco Costamagna costamagnagianfranco@yahoo.it
Signed-off-by: Gianfranco Costamagna locutusofborg@debian.org

the variable INSTALL_INCLUDE_DIR already exists, and defaults to include if not specificied otherwise.
Using it allows people to customize the installation from outside, fixing issues with other OS like Haiku

Signed-off-by: Gianfranco Costamagna <costamagnagianfranco@yahoo.it>
Signed-off-by: Gianfranco Costamagna <locutusofborg@debian.org>
@LocutusOfBorg
Copy link
Contributor Author

@Begasus thanks for the nice contribution, I'm uploading the patch in Debian too!

@LocutusOfBorg
Copy link
Contributor Author

So it turned out, my previous pr "Install websocketpp includes in the right cmake way" had an even righter way :D

@Begasus
Copy link

Begasus commented Sep 18, 2019

Thank you for merging! 👍

@zaphoyd
Copy link
Owner

zaphoyd commented Sep 26, 2019

This change has been merged into the develop branch

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants