refactor(config): adopt unified KCENON_* feature flags#108
Merged
Conversation
- Include feature_flags.h from common_system (guarded with __has_include) - Replace BUILD_WITH_* macros with KCENON_WITH_* equivalents in CMake - Update logging.h to use KCENON_WITH_LOGGER_SYSTEM for integration detection - Maintain backward compatibility with legacy BUILD_WITH_* macros - Update docs/reference/logging.md to reference new macro names Closes #107
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
Headers
logging.h: Include feature_flags.h, use KCENON_WITH_LOGGER_SYSTEM for integration detectionCMake
CMakeLists.txt: Define both legacy and new unified macros (KCENON_WITH_*) for all system integrationsDocumentation
docs/reference/logging.md: Updated to reference KCENON_WITH_LOGGER_SYSTEMRelated Issues
Test plan