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

Exposing functions related to CurveZMQ authentication. #35

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

Conversation

dentonbriandavid
Copy link

This pull request exposes several additional functions in a more recent version of ZeroMQ (specifically 4.2.2).

Additionally, the user can specify the number of io_threads to use for server and client sockets.

I have not created new documentation for the additional functions so the following warning appears in R CMD check:

Undocumented code objects:
‘close.socket’ ‘get.curve.server’ ‘get.io_threads’ ‘get.keypair’
‘get.public.key’ ‘get.secret.key’ ‘get.server.key’ ‘set.curve.server’
‘set.public.key’ ‘set.secret.key’ ‘set.server.key’

A warning will also appear in R CMD check because the function signature of close.socket does not follow the signature of the S3 generic base::close.

@jeroen
Copy link
Member

jeroen commented Jun 13, 2017

This breaks the installation on most distributions that ship with an older version of libzmq. Can you please #if new stuff conditional on the version of libzmq and raise an error if something is called which requires a more recent version of libzmq than that is available?

Here is an example of a similar trick in the magick package.

@jeroen
Copy link
Member

jeroen commented Jun 26, 2017

I think you forgot to remove some debugging? It complains that the rzmq2 package is missing.

@jeroen jeroen force-pushed the master branch 2 times, most recently from 8ff7ca8 to 9795f9b Compare May 8, 2023 10:09
@jeroen jeroen force-pushed the master branch 3 times, most recently from de16540 to 0711fad Compare September 25, 2023 21:18
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.

3 participants