Skip to content
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

testsuite: coverage: application unable to include coverage.h #80723

Closed
ycsin opened this issue Nov 1, 2024 · 0 comments · Fixed by #80638
Closed

testsuite: coverage: application unable to include coverage.h #80723

ycsin opened this issue Nov 1, 2024 · 0 comments · Fixed by #80638
Assignees
Labels
area: Code Coverage area: Testsuite Testsuite bug The issue is a bug, or the PR is fixing a bug

Comments

@ycsin
Copy link
Member

ycsin commented Nov 1, 2024

Describe the bug

Application should be able to include the private header coverage.h that is conditionally exposed when CONFIG_COVERAGE_GCOV=y by:

#include "coverage"

but since there is a typo in the CMakeLists.txt, it is not able to do so:

zephyr_include_directories_ifdef(CONFIG_COVERAGE_GCOV ${zephyr_BASE}/subsys/testsuite/coverage)

Specifically, the zephyr_BASE variable should be ZEPHYR_BASE instead.

To Reproduce
Steps to reproduce the behavior:

  1. Enable CONFIG_COVERAGE_GCOV and try to include the coverage.h in an application
  2. west build ...
  3. See error

Expected behavior
Compilation should pass

Impact
Application can't include the coverage.h, otherwise the compilation fails.

Environment (please complete the following information):

  • OS: Linux
  • Toolchain: Zephyr SDK
  • Commit SHA or Version used: 193eeae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Code Coverage area: Testsuite Testsuite bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants