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
depends: build zeromq with CMake #29723
base: master
Are you sure you want to change the base?
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsNo conflicts as of last run. |
Concept ACK. |
From zeromq/libzmq#4667 (comment):
|
Can you elaborate / suggest something concrete? As far as I can see, the cross-compiling build of Windows for this branch currently works fine, we already link again |
328da34
to
eac93d3
Compare
eac93d3
to
0dd69d4
Compare
Given both changes have landed, I've reordered the commits, and undrafted. Will followup with the Windows issues. |
IPC build issue should be fixed in zeromq/libzmq#4672 |
0dd69d4
to
8ebe5d3
Compare
8ebe5d3
to
2a7b137
Compare
2a7b137
to
37da2ae
Compare
This was resolved using a different change. Have pulled in that patch, rebased and updated the PR description. |
37da2ae
to
796a271
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested 796a271 on Ubuntu 23.10 using a patch from #29960.
There are a few differences between Autotools and CMake builds:
- In CMake, the resulted archive lacks object files from the following sources:
gssapi_client.cpp
gssapi_mechanism_base.cpp
gssapi_server.cpp
vmci_address.cpp
vmci_connecter.cpp
vmci_listener.cpp
vmci.cpp
-
CMake build is effectively compiled with
-O3
flag. -
CMake adds
-DZMQ_CUSTOM_PLATFORM_HPP
.
796a271
to
02a95f6
Compare
|
02a95f6
to
111ca59
Compare
Fixed the Windows build error, but drafted while based on #30078. |
Some of these have been upstreamed. Some, i.e changing minimum versions are unlikely to be accepted, so have not and exist here to suppress output.
Currently one Windows related fixup: * The CMake WIN32_WINNT autodetection is broken, and must be set manually. We may want to set is explicitly in any case, but the brokenness should also be fixed upstream Co-authored-by: fanquake <fanquake@gmail.com>
111ca59
to
7dfe36e
Compare
Based on #30078.
This picks up a change, which is a switch to building zeromq with CMake. It includes a patch with a couple changes I've upstreamed:
and another Windows-related change:
I also noticed the CMake Windows version autodetction was broken with mingw-w64 (zeromq/libzmq#4669), so we set the version explicitly.