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

Jump to bottom

cmake: Install websocketpp includes in the right cmake way, leaving o… #732

Merged
merged 1 commit into from
Jul 16, 2018
Merged

cmake: Install websocketpp includes in the right cmake way, leaving o… #732

merged 1 commit into from
Jul 16, 2018

Conversation

LocutusOfBorg
Copy link
Contributor

…ut the examples and other stuff.

I had previously code to remove examples and other binaries from polluting /usr/include, better fix this in the right way

list of unwanted files, because they contained hpp files:

usr/include/echo_server/
usr/include/external_io_service 
usr/include/test_connection

@zaphoyd zaphoyd merged commit e99aef0 into zaphoyd:master Jul 16, 2018
@zaphoyd
Copy link
Owner

zaphoyd commented Jul 16, 2018

Agreed. Merged and released with 0.8.1

@LocutusOfBorg LocutusOfBorg deleted the do-not-install-examples branch July 17, 2018 09:34
@LocutusOfBorg
Copy link
Contributor Author

thanks! this is a problem I had a lot of time ago, but I never bothered to fix it :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Begasus Begasus Begasus left review comments

Assignees
No one assigned
Labels
Build Issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LocutusOfBorg @zaphoyd @Begasus