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

libnopoll: noPoll C WebSocket library 0.4.2 #3286

Merged
merged 2 commits into from
Oct 4, 2016
Merged

libnopoll: noPoll C WebSocket library 0.4.2 #3286

merged 2 commits into from
Oct 4, 2016

Conversation

inindev
Copy link
Contributor

@inindev inindev commented Oct 2, 2016

Maintainer: John Clark inindev@gmail.com / @inindev
Compile tested: ar71xx, gl-inet-6416A-v1, LEDE trunk
Run tested: ar71xx, gl-inet-6416A-v1, library integration test

Description: noPoll C WebSocket library 0.4.2
http://www.aspl.es/nopoll

Signed-off-by: John Clark inindev@gmail.com
Tested-by: John Clark inindev@gmail.com

Signed-off-by: John Clark <inindev@gmail.com>
Tested-by: John Clark <inindev@gmail.com>
@diizzyy
Copy link
Contributor

diizzyy commented Oct 2, 2016

Use release tarballs instead of git if available (which is available in this case)

Looks good in general otherwise

Best regards, Daniel

@inindev
Copy link
Contributor Author

inindev commented Oct 2, 2016

The reason I didn't use the release tarball is that it is named 0.4.2.tar.gz and wondered about a conflict in the dl directory. When pulling from git I am able to specify that it be named --.tar.gz. Should I not worry about this?
(tag 0.4.2 is commit 60a81fbd199551ac5dd017b9a44fb46e8530680b)

tarball: lede-project/source/dl/0.4.2.tar.gz

git: lede-project/source/dl/nopoll-0.4.2-60a81fbd199551ac5dd017b9a44fb46e8530680b.tar.gz

@diizzyy
Copy link
Contributor

diizzyy commented Oct 2, 2016

Ahh, nevermind... It also relies on Githubs "on-the-fly" compression system so tarball checksum will change. Sorry for not noticing and sorry for the noise.

It looks fine otherwise, please change gz to xz however :-)

Signed-off-by: John Clark inindev@gmail.com
Tested-by: John Clark inindev@gmail.com
@inindev
Copy link
Contributor Author

inindev commented Oct 2, 2016

good call on the xz compression

@karlp
Copy link
Contributor

karlp commented Oct 3, 2016

@diizzyy where does the guidelines on "not using git" come from? I've not seen that anywhere recommended as a formal guideline.

@diizzyy
Copy link
Contributor

diizzyy commented Oct 3, 2016

It isn't however it's a reasonable suggestion since git (checkout) packages for now aren't reproducible which is bad for availability since you cannot mirror those.

@inindev
Copy link
Contributor Author

inindev commented Oct 3, 2016

If the PR is looks reasonable, who does the actual commit?

@hnyman hnyman merged commit b8c6018 into openwrt:master Oct 4, 2016
@inindev inindev deleted the libnopoll branch October 5, 2016 01:37
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