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

czmq: add new package #8014

Merged
merged 1 commit into from Mar 10, 2019

Conversation

Projects
None yet
3 participants
@ja-pa
Copy link
Contributor

ja-pa commented Jan 22, 2019

Maintainer: me / @ja-pa
Compile tested: Turris Omnia (TOS4), OpenWrt 18.06.1
Compile tested: Turris Omnia (TOS4), OpenWrt 18.06.1

Description:
Signed-off-by: Jan Pavlinec jan.pavlinec@nic.cz
czmq is high-level C binding for ØMQ.

Show resolved Hide resolved libs/czmq/Makefile Outdated
Show resolved Hide resolved libs/czmq/Makefile Outdated
@diizzyy

This comment has been minimized.

Copy link
Contributor

diizzyy commented Jan 23, 2019

Looks good otherwise, however cmake is more "robust" so preferably it should be used over autotools.

@ja-pa ja-pa force-pushed the ja-pa:czmq branch from 65e8de1 to e4540db Jan 23, 2019

@ja-pa ja-pa force-pushed the ja-pa:czmq branch 3 times, most recently from a463c4a to 936a48a Jan 30, 2019

czmq: add new package
Signed-off-by: Jan Pavlinec <jan.pavlinec@nic.cz>
@ja-pa

This comment has been minimized.

Copy link
Contributor Author

ja-pa commented Jan 30, 2019

@diizzyy I changed the source to release tarball and tried to use CMake, but it breaks build so I switch back to Autotools. Let me know if you found any other issues or if it's ready to merge. Thanks.

@diizzyy

This comment has been minimized.

Copy link
Contributor

diizzyy commented Jan 30, 2019

I see, thanks for having a look at it. LGTM

@ja-pa

This comment has been minimized.

Copy link
Contributor Author

ja-pa commented Feb 1, 2019

@hnyman I think it's ready to merge.

@hnyman hnyman merged commit abce403 into openwrt:master Mar 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.