-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
libsodium: Updated to latest version (1.0.11) #3107
Conversation
Signed-off-by: Black Roland <[email protected]>
Thanks |
@@ -52,6 +52,7 @@ endef | |||
|
|||
CONFIGURE_ARGS+= \ | |||
--disable-ssp \ | |||
--without-pthreads \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you explain why disable this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because pthreads does not work with uClibc (OpenWrt CC and low).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not report bugs to uClibc? I don't have such issue using LEDE trunk with musl-libc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A pull request for master should not disable features based on limitations in CC. It's meant to be used with LEDE/OpenWrt master.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nbd168 What are my next steps? How I can split master and CC versions? Should I create PR for for-15.05
branch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can select target branch when creating pull requests, and check the diff shown below before sending it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I understand CC support limited to bug-fixes and security fixes only. So I made new pull request without CC support.
Maintainer: @damianorenfer
Compile tested: ar71xx OpenWTR DD/CC/BB
Run tested: ar71xx OpenWTR DD (r48648)/CC/BB
Description:
Updated to latest version (1.0.11)