Skip to content

Conversation

@ribbons
Copy link

@ribbons ribbons commented May 7, 2018

Under Arch Linux (and Ubuntu 17.10 according to #196), compilation fails with an error in websocketpp: error: no matching function for call to ‘make_shared(std::reference_wrapper<boost::asio::io_context>)’.

To resolve this, update websocketpp to zaphoyd/websocketpp@e036a5a which is the first revision after the current zaphoyd/websocketpp@ac5d7ea5 which compiles without warnings.

Under Arch Linux (and Ubuntu 17.10 according to socketio#196), compilation fails
with an error in websocketpp: "error: no matching function for call to
‘make_shared(std::reference_wrapper<boost::asio::io_context>)’".

To resolve this, update websocketpp to e036a5au which is the first
revision after ac5d7ea5 which compiles without warnings being generated.

Fixes socketio#196.
@ribbons
Copy link
Author

ribbons commented May 7, 2018

Looks like there is an issue with the travis build at the moment which isn't related to this change...

@tobocop2
Copy link

@ribbons Probably makes sense to synchronize both submodules, I opened a similar PR here:

c7cdab2

Compiles for me on centos, arch linux, ubuntu, and macos.

@ribbons
Copy link
Author

ribbons commented Sep 14, 2018

@ribbons Probably makes sense to synchronize both submodules, I opened a similar PR here:

c7cdab2

Compiles for me on centos, arch linux, ubuntu, and macos.

Thanks for the heads-up 👍.

I've subscribed to updates on your PR #214 so if the maintainers decide to merge yours (bring all submodules right up-to-date) in preference to mine (just update websocketpp to the first commit which fixes the build) I can close this PR 😄.

@akontsevich
Copy link

I have this issue in openSUSE Tumbleweed as well. Also solved by

cd socket.io-client-cpp/lib/websocketpp
git pull origin master

Thanks to @syncu2! It is strange why git submodule reference to websocketpp still not updated to latest commit in master which works fine?

@darrachequesne
Copy link
Member

Included in af68bf3.

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.

4 participants