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

Add compile def "MSGPACK_DEFAULT_API_VERSION=1" #236

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

CanNuhlar
Copy link

autobahn-cpp only supports msgpack V1. Current libmsgpackc2 & libmsgpack-dev packages on Debian 10 & 11 is using V2 which makes autobahn-cpp incompatible with Debian 10 & 11. Until msgpack V2 support is there, this workaround increases the compatibility on Debian distributions.

@CanNuhlar CanNuhlar mentioned this pull request Oct 18, 2022
@CanNuhlar
Copy link
Author

I don't think the checks are failing because of the changes. Am I missing something here? @oberstet

First time contributing to this project, not familiar with the CI.

@oberstet
Copy link
Contributor

yeah, not sure, it is very likely unrelatedm and it seems like conan>=1.52.0 might solve the issue

conan-io/conan#12076

autobahn-cpp only supports msgpack V1. Current libmsgpackc2 & libmsgpack-dev packages on Debian 10 & 11 is using V2 which makes autobahncpp incompatible with Debian 10 & 11. Until we have moved on to msgpack V2, this workaround increases the compatibility on Debian distributions.
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