Skip to content
This repository has been archived by the owner on Sep 27, 2021. It is now read-only.

Support building against nng as well as nanomsg #54

Open
nullr0ute opened this issue Jun 17, 2018 · 5 comments
Open

Support building against nng as well as nanomsg #54

nullr0ute opened this issue Jun 17, 2018 · 5 comments

Comments

@nullr0ute
Copy link

With nng now stable and being encouraged as a protocol/wire compatible with nanomsg it would be useful to be able to build nnpy against nng as well as nanomsg.

@djc
Copy link
Member

djc commented Jun 17, 2018

I wouldn't be opposed to having it, but I'm not sufficiently motivated to do it on my own. So, if you want to contribute a PR, I'd be happy to review it!

Others who were asking about nng support thought that the actual nng API (as opposed to the compatibility API) would be too different to support well in nnpy.

@nullr0ute
Copy link
Author

I think the compat API would be a useful start as it would allow people to migrate without a lot of code change

@ilyaevseev
Copy link

I think the compat API would be a useful start as it would allow people to migrate without a lot of code change

nnpy uses nn_symbol call that is not provided by nng_compat:

  • Runtime symbol information is not implemented.
    Specifically, there is no nn_symbol() function yet.
    (This may be addressed later if there is a need.)

@djc
Copy link
Member

djc commented Jun 26, 2018

Someone could probably find the constants some other way (for example, by parsing headers).

@ilyaevseev
Copy link

Here is my attempt to create quick&dirty python wrapper for libnng:
https://github.com/ilyaevseev/nngpy/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants