Skip to content

[MDAPI-254][C++] Migrate to dxFeed Graal Native SDK 2.5.0 #80

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

Merged
merged 11 commits into from
May 2, 2025

Conversation

AnatolyKalin
Copy link
Collaborator

No description provided.

Reorganized the build system by creating an object library for better modularity and reuse. Replaced redundant target definitions and centralized compilation/linking settings for shared, static, and object libraries. Adjusted visibility of certain link and compile options to be private for cleaner and more encapsulated configuration.
This commit enables `CMAKE_VERBOSE_MAKEFILE` across all `CMakeLists.txt` to improve debug output during builds. It also extends compatibility for Clang in MSVC emulation mode by updating conditions targeting the MSVC runtime configuration logic.
Renamed `DXFCXX_NODEFAULTLIB_LIBCMT` to `DXFCXX_NODEFAULTLIB` for clearer purposes and added support for additional libraries in `/NODEFAULTLIB` configuration. Also commented out an unnecessary `/fsanitize=address` linker option in `LinkAsanUbsan.cmake` to avoid redundancy.
Moved the MSVC static runtime configuration logic into a new reusable function `SetupStaticRuntimeMSVC` in `SetupStaticRuntimeMSVC.cmake`. This simplifies redundant code across multiple targets and makes the configuration clearer and more maintainable.
If the user does not explicitly specify the endpoint name, it will be generated using the template "qdcxx{Id}", where {Id} will be an empty string for the first instance of the endpoint and "-2", "-3", etc. for subsequent instances.
@ttldtor ttldtor merged commit 8aac1d3 into main May 2, 2025
63 checks passed
@ttldtor ttldtor deleted the feature/MDAPI-254 branch May 2, 2025 22:03
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