-
Notifications
You must be signed in to change notification settings - Fork 1
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
rnp reference updated to 0.16.3 #32
Conversation
Curerent failure:
|
80db2dc
to
14f2052
Compare
I believe we should not publish bottle. It will contain rnp binary with direct reference to specific version of botan |
@maxirmx can't we specify a bottle dependency on the particular botan version? The benefit of the bottle is to allow people not to go through compilation, and it is an important benefit. |
My understanding is that botan provides a formula to install only the latest version. Thus, even if we specify a dependency on some other version of botan, it will not be installed. I believe we can
Also I guess we will need several bottles to make it effective. Botan provides seven:
This may require cross-compile options that will also go to core rnp build scripts I suggest we release a simple fix first since keeping the bottle does not look like a one-liner fix |
Agree that a simple fix is desired here. We can fix the bottle after. Thanks. |
@maxirmx Feel free to merge when ready, thanks. |
Needs to be recreated to apply pr-pull workflow |
No description provided.