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

zerotier: update to 1.4.4 #9903

Merged
merged 1 commit into from
Sep 3, 2019
Merged

zerotier: update to 1.4.4 #9903

merged 1 commit into from
Sep 3, 2019

Conversation

mwarning
Copy link
Contributor

@mwarning mwarning commented Sep 2, 2019

Maintainer: me
Compile tested: openwrt master
Run tested: rampis, nexx wt3020, program starts, soft link to config file is created

@mwarning
Copy link
Contributor Author

mwarning commented Sep 2, 2019

@ian-droid local.conf is usually too big to put it into one line. I added a file link for local.conf now.

@ian-droid
Copy link

@ian-droid local.conf is usually to big to put it into one line. I added a file link now for local.conf now.

That's great, thanks!

@diizzyy
Copy link
Contributor

diizzyy commented Sep 3, 2019

Please squash PR. Regarding binary size, you might want to try LTO (-flto).

Also allow path to local.conf to be set and enable linker optimisations
to save a few bytes.

Signed-off-by: Moritz Warning <[email protected]>
@mwarning
Copy link
Contributor Author

mwarning commented Sep 3, 2019

@diizzyy -flto did not have any effect. Commits are squashed now.

@neheb neheb merged commit 8c6373f into openwrt:master Sep 3, 2019
@mwarning mwarning deleted the zerotier branch September 3, 2019 22:05
@mwarning
Copy link
Contributor Author

mwarning commented Sep 3, 2019

I just noticed that 1.4.4 changed the license to a commercial license (https://github.com/zerotier/ZeroTierOne/releases/tag/1.4.4).

Does that mean that the package has to be pulled? I have opened a ticket and hope that things become clearer: zerotier/ZeroTierOne#1027

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