-
Notifications
You must be signed in to change notification settings - Fork 12
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
Check if using "internal ZMQ" before messing with RPATH #54
base: master
Are you sure you want to change the base?
Conversation
We check this via the Autotool's determined ENABLE_INTERNAL_ZMQ variable which requires AC_SUBSTing which requires generating src/install.libs.R from src/install.libs.R.in
submitted upstream at: snoweye/pbdZMQ#54
Seems I may have broken Windows here, suggestions welcome. |
No, you should not do that. If It will be helpful to have more information about |
The issue here is the opposite of what you say. I do not set and do not want |
You can read more about The Anaconda Distribution here: https://www.anaconda.com |
I am not sure what the opposite means.
Which one is your case? If you don't have an external ZMQ library and "do not set and do not want", then I don't know a way to have |
We have an external libzmq.dylib but it is not where this code is looking for it and we do not need this code to run install_name_tool since our build system does that correctly for our required layout. |
What is your case? |
I don't think this is needed, so I close this one. Let me know if there are other problems. |
OK thanks. I tried your suggestion and other than needing to replace |
Either one is fine. I will alter any of them if the changes don't upset CRAN. |
We check this via the Autotool's determined ENABLE_INTERNAL_ZMQ variable
which requires AC_SUBSTing which requires generating src/install.libs.R
from src/install.libs.R.in
Without this, when building this package for the
Anaconda Distribution
we run into: