Skip to content

Comments

Control verbosity of debug output via build config#167

Open
robertbartel wants to merge 4 commits intoNOAA-OWP:masterfrom
robertbartel:f/build_improvements/control_verbosity
Open

Control verbosity of debug output via build config#167
robertbartel wants to merge 4 commits intoNOAA-OWP:masterfrom
robertbartel:f/build_improvements/control_verbosity

Conversation

@robertbartel
Copy link

Control verbosity of debug output using CMake variable DEBUG_VERBOSITY, rather than hard coding.

Removals

  • Hard-coded setting of CFE_DEBUG

Changes

  • CMake build adjusted to accept a DEBUG_VERBOSITY variable and use that to set the CFE_DEBUG macro via compile definition.

Testing

  1. Tested numerous scenarios to ensure build type or explicit DEBUG_VERBOSITY setting results in the desired effect.

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows project standards (link if applicable)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Windows
  • Linux
  • Browser

Accessibility

  • Keyboard friendly
  • Screen reader friendly

Other

  • Is useable without CSS
  • Is useable without JS
  • Flexible from small to large screens
  • No linting errors or warnings
  • JavaScript tests are passing

Set macro CFE_DEBUG via new CMake DEBUG_VERBOSITY variable to support
build-config-level control over how verbosely debugging output is
printed.
@robertbartel robertbartel added the enhancement New feature or request label Feb 18, 2026
Fix integration tests workflow in Mac runner, where Sloth build requires
-DCMAKE_POLICY_VERSION_MINIMUM=3.5 be set.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant