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

libsodium: Updated to latest version (1.0.11) #3107

Merged
merged 1 commit into from
Aug 29, 2016

Conversation

black-roland
Copy link
Contributor

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)

@thess
Copy link
Member

thess commented Aug 29, 2016

Thanks

@thess thess merged commit 5958b72 into openwrt:master Aug 29, 2016
@@ -52,6 +52,7 @@ endef

CONFIGURE_ARGS+= \
--disable-ssp \
--without-pthreads \
Copy link
Contributor

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?

Copy link
Contributor Author

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).

Copy link
Contributor

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.

Copy link
Member

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.

Copy link
Contributor Author

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?

Copy link
Contributor

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.

Copy link
Contributor Author

@black-roland black-roland Aug 31, 2016

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.

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.

4 participants