-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
windows server support needed! #2461
Comments
@guzuomuse thank you for reporting. we'd love to support windows, however most of the committers are more familiar with linux/macos. if you can share some of your build errors, we might have some ideas on how to help you better. |
@sijie ,thanks your reply,here is error log when i compile the c++ client with cmake gui:
|
You're missing some of the dependencies. For tests, you can disable by passing For the rest, you need to install:
|
Anyway, the deps for tests are : |
@merlimat ,thank you very much ,i will try and then give my feedback |
A conan.io support would make all this much simpler |
A vcpkg support would make all this much simpler :) |
…able compilation on Windows (#4071) Fixes #2461 ### Motivation My motivation was to be able to use the pulsar cpp client on Windows systems. ### Modifications There are a number of modifications I needed to make to enable Windows compilation * Extensive rework of multiple CMakeLists.txt files * The creation of a PULSAR_PUBLIC define to define symbol visibility in lieue of #pragma GCC visibility push(default). This is because Windows requires specifying __declspec(dllexport) and __declspec(import) explicitly and does not have a #pragma GCC visibility push(default) analogue. * all calls to usleep or sleep have been replaced with calls to boost::this_thread::sleep() ### Verifying this change This change is a trivial rework / code cleanup without any test coverage. However, it does introduce a new platform to test on. It is likely that CI checks will have to be created.
for some reason,we must stay windows platform.
compiling pulsar-cpp always give errors.
any plan offering support for windows? thanks!
The text was updated successfully, but these errors were encountered: