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

fix: update libzmq to the latest version #572

Merged
merged 1 commit into from
Jun 14, 2024
Merged

fix: update libzmq to the latest version #572

merged 1 commit into from
Jun 14, 2024

Conversation

aminya
Copy link
Member

@aminya aminya commented Jun 13, 2023

This fixes the build with newer platforms and compilers. It also includes other fixes from libzmq

@rotu
Copy link
Contributor

rotu commented Jun 11, 2024

What's the deal with this PR and the general CI failures?

@aminya
Copy link
Member Author

aminya commented Jun 11, 2024

Some of the tests have been failing for some time, so they are safe to ignore. However, we should tag them so that they are intentionally ignored.

@rotu
Copy link
Contributor

rotu commented Jun 12, 2024

Some of the tests have been failing for some time, so they are safe to ignore. However, we should tag them so that they are intentionally ignored.

I'll be happy to help fix them rather than ignore test failures.

But that's besides the point. CI doesn't even complete the build! https://github.com/zeromq/zeromq.js/actions/runs/8944056490/job/24570134698

I think the outdated version of libzmq is responsible for most of the failures due to allocator issues fixed in zeromq/libzmq#4480.

@aminya aminya marked this pull request as ready for review June 14, 2024 09:24
@aminya aminya merged commit 6706616 into master Jun 14, 2024
1 of 6 checks passed
@aminya aminya deleted the update-zmq branch June 14, 2024 21:49
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.

2 participants