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

JNI wrapper for zyre #331

Merged
merged 3 commits into from
Sep 9, 2015
Merged

JNI wrapper for zyre #331

merged 3 commits into from
Sep 9, 2015

Conversation

awynne
Copy link
Contributor

@awynne awynne commented Sep 8, 2015

Initial attempt at a jni wrapper. recv() not working yet.

hintjens added a commit that referenced this pull request Sep 9, 2015
@hintjens hintjens merged commit 15572f4 into zeromq:master Sep 9, 2015
@hintjens
Copy link
Member

hintjens commented Sep 9, 2015

This is great. I see that Zyre actually has its API defined in api/zyre.xml and api/zyre_event.xml, so the C class header and bindings for other languages come automagically via zproject.

So, we can take your JNI work so far and use it as the basis for a zproject generator (see e.g. https://github.com/zeromq/zproject/blob/master/zproject_bindings_python.gsl)

@jemc
Copy link
Member

jemc commented Sep 9, 2015

+1 to the idea of using this as the basis for making a generic zproject bindings generator for JNI.

@awynne - if you (or anyone else) is interested in doing this as part of zproject and have questions about GSL or zproject please file a ticket in zproject or ask on the mailing list so we can help point you in the right direction. Expanding the languages covered by zproject is a very important goal, but by its nature requires a diverse group of contributors with expertise in different languages, so outreach is important.

@keent
Copy link
Member

keent commented Sep 9, 2015

@jemc @hintjens I can do this - I'm interested as I don't have any experience in gsl and zproject - I filed a ticket in zproject - let me know.

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