The GitHub Actions job "Fory CI" on fory.git/fix/cpp-msvc-zc-preprocessor has 
succeeded.
Run started by GitHub user truffle-dev (triggered by chaokunyang).

Head commit for run:
961538cb4f87598f0e2bd0f7dcb4f17340326442 / truffle <[email protected]>
fix(c++): propagate /Zc:preprocessor on MSVC for FORY_STRUCT consumers

The FORY_STRUCT macro in cpp/fory/meta/field_info.h forwards __VA_ARGS__
through nested macros, which the legacy MSVC preprocessor expands as a
single token. .bazelrc already adds /Zc:preprocessor for Bazel Windows
builds, but CMake consumers had to remember to add it themselves; PR #3078
papered over this in the docs sample.

Propagate the flag from fory_meta (the library that owns the macro) via
PUBLIC target_compile_options gated on a CXX_COMPILER_ID:MSVC generator
expression. The genex keeps clang-cl unaffected (it warns on the flag and
is already conforming) and keeps the install-export hermetic for non-MSVC
consumers reading the exported target through find_package(fory).

Closes #3693

Report URL: https://github.com/apache/fory/actions/runs/26150473682

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to