Skip to content

Enable running gcov#800

Closed
franz-hoepfinger-4diac wants to merge 4 commits intoeclipse-4diac:developfrom
franz-hoepfinger-4diac:add_more_tests
Closed

Enable running gcov#800
franz-hoepfinger-4diac wants to merge 4 commits intoeclipse-4diac:developfrom
franz-hoepfinger-4diac:add_more_tests

Conversation

@franz-hoepfinger-4diac
Copy link
Contributor

fix 2 small Errors around gcov,
and add a setup Script to call this.

Adds a script to automatically set up the development environment for POSIX platforms.

This script simplifies the configuration process by creating the necessary build directories, setting compiler flags, and generating makefiles for building and testing the fortran environment.

It also enables various features and modules like Ethernet communication, ASN1 encoding, and IEC61131 support.
Corrects the code coverage generation process by adjusting the arguments passed to the test runner during the coverage analysis.
This ensures that the coverage reports are generated correctly, especially when not using the standard 'ctest' runner.
Now it uses the correct arguments according to the test runner being used.

.
@franz-hoepfinger-4diac
Copy link
Contributor Author

steps to get it running:

  1. ./setup_posix-test.sh
  2. cmake --build bin/posix
  3. cmake --build bin/posix --target TestCoverage

Adds compile and link options to enable code coverage
analysis for the Posix architecture during testing.

This ensures proper instrumentation for coverage reporting.
@franz-hoepfinger-4diac
Copy link
Contributor Author

gcov works now, but lcov not.

@franz-hoepfinger-4diac
Copy link
Contributor Author

lcov runs now:

Writing data to coverage.info.cleaned                                                                                                                                                                    
Summary coverage rate:                                                                                                                                                                                      source files: 289                                                                                                                                                                                         lines.......: 92.2% (2670 of 2897 lines)                                                                                                                                                                  functions...: 79.9% (2570 of 3218 functions)                                                                                                                                                             branches....: 47.4% (963 of 2030 branches)                                                                                                                                                             
lcov: ERROR: (unused) 'exclude' pattern '*boost*' is unused.                                                                                                                                             
        (use "lcov --ignore-errors unused ..." to bypass this error)      

try to fix this as well.

Adds the `--ignore-errors inconsistent,mismatch` flag to lcov and genhtml calls. This resolves issues where coverage generation fails due to inconsistent data or mismatches.
@franz-hoepfinger-4diac
Copy link
Contributor Author

lcov now also running, somebody experienced has to review. i let the Commits seperated, for easier check, we can squash later.

@azoitl
Copy link
Contributor

azoitl commented Feb 15, 2026

@franz-hoepfinger-4diac can you please be more considerate when submitting PRs. The idea is that you submit a PR when you are ready or want to interact with others to review and get feedback. The way you are pushing have finished commits is wasting the time of many people who are spammed by your messages. Not only does this waste time I find this extremely impolite and it takes away valuable time from other Eclipse 4diac activities.

I'm therefore closing most of your open PRs. Feel free to resubmit them when they are ready to be reviewed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants