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

Add Windows and CMake support #54

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

glywk
Copy link

@glywk glywk commented Apr 14, 2023

This update purpose many improvements:

  • add windows support.
  • add cmake support to build serdes and serdes++ static libraries. Option build are detailed in Readme.md.
  • separate tinycthread build in another static library to avoid multiple definition conflict when libserdes is linked with librdkafka.
  • fix memory access violation if custom schema handle is not HandleImpl based.

@glywk glywk changed the title Win support Add Windows and CMake support Apr 14, 2023
@bayueqiankong
Copy link

CMake Error at CMakeLists.txt:19 (find_package):
By not providing "Findlibavrocpp.cmake" in CMAKE_MODULE_PATH this project
has asked CMake to find a package configuration file provided by
"libavrocpp", but CMake did not find one.

Could not find a package configuration file provided by "libavrocpp" with
any of the following names:

libavrocppConfig.cmake
libavrocpp-config.cmake

Add the installation prefix of "libavrocpp" to CMAKE_PREFIX_PATH or set
"libavrocpp_DIR" to a directory containing one of the above files. If
"libavrocpp" provides a separate development package or SDK, be sure it has
been installed.

@glywk
Copy link
Author

glywk commented Jun 6, 2023

Hi,
Thanks for your answer. To install and generate all libserdes CMake dependencies, I have purpose a Conan based solution. You can see the command line example for Windows in the README.md.

cd build
conan install -pr:b win_x86_64_release -pr:h win_x86_64_release ../conan/conanfile.txt
cmake .. --toolchain=conan_toolchain.cmake -DCMAKE_BUILD_TYPE=Release
cmake -P cmake_install.cmake```

Regards

@glywk
Copy link
Author

glywk commented Jul 26, 2023

@edenhill

@glywk
Copy link
Author

glywk commented Aug 12, 2023

@emasab

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