zchuango commented on PR #3428: URL: https://github.com/apache/brpc/pull/3428#issuecomment-5570561459
@wwbmmm Thanks for pointing this out. Addressed in commit 07c4661d. The Bazel source selection has been restructured: - All src/brpc/urma/*.cpp files are excluded from the base source glob. - URMA_SRCS explicitly excludes mock_urma.cpp. - brpc_with_urma_use_real includes only the non-mock URMA sources and links liburma. - The regular brpc_with_urma configuration appends mock_urma.cpp exactly once. - URMA implementation sources are no longer compiled when URMA is disabled. This now matches the CMake and Make behavior and prevents mock urma_* symbols from conflicting with the real liburma. bazel query successfully loaded and expanded the updated source selection. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
