-
Notifications
You must be signed in to change notification settings - Fork 13
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
cooperate with raco distribute
to include shared libs in distributions
#14
Conversation
Defining a runtime path at this level does cause the `so` to get included in distributions, but it doesn't alter the way the runtime searches for libs so `crypto-lib` can't find it. The proper fix here is to alter `crypto-lib` as in [1]. [1]: rmculpepper/crypto#14
Thanks for the PR! I was ambivalent at first, but now that I have refreshed my understanding of how In private/nettle/ffi.rkt, libhogweed (provides pk support) also needs to be changed. Based on the output of
The test failure is an actual bug introduced by a recent change, so drop that commit and I'll fix the bug. |
Thanks for the review! Re. Let me know if you want me to make a PR to the gmp package or if you want to make the change yourself. |
Also, prefer specific version over versionless library. See rmculpepper/crypto#14 for discussion.
Merged, thanks! |
This makes it so that distros built with
raco distribute
include the shared libs they depend on when those libs are installed specifically for Racket (like in the case of mylibargon2
package).