Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@NeitrinoK
Copy link
Contributor

I try to fix next message

(ERROR) GCOV produced the following errors processing /home/gitlab-runner/disk/builds/HafKdfP1/0/PT/hiper100re/build/contrib/libre2/CMakeFiles/re2.dir/util/strutil.cc.gcda:
	Trouble processing '/home/gitlab-runner/disk/builds/HafKdfP1/0/PT/hiper100re/build/contrib/libre2/CMakeFiles/re2.dir/util/strutil.cc.gcda' with working directory '/home/gitlab-runner/disk/builds/HafKdfP1/0/PT/hiper100re'.
Stdout of gcov was >>None<< End of stdout    (<- Here)
Stderr of gcov was >>None<< End of stderr   (<- Here)
Exception was >>GCOV returncode was 5.<< End of stderr

We lose stdout and stderr when return code of gcov is not 0 or 6, so all useful information lost ( I used strace tool to get messages).

Now, stdout and stderr are saved in object for sure

@Spacetown
Copy link
Member

Spacetown commented Sep 5, 2024

Now we only see the last output. I think a better solution is to keep the local variables and add them to the exception text. In the error output they should not be used if set to None.

@NeitrinoK
Copy link
Contributor Author

Now we only see the last output. I think a better solution is to keep the local variables and add them to the exception text. In the error output they should not be used if set to None.
NP, fixed.

@Spacetown
Copy link
Member

Can you print the output after the error message?

@NeitrinoK
Copy link
Contributor Author

Can you print the output after the error message?

Done.

@codacy-production
Copy link

codacy-production bot commented Sep 5, 2024

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.05% (target: -1.00%) 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1c290d4) 3991 3900 97.72%
Head commit (48beeaf) 4072 (+81) 3981 (+81) 97.77% (+0.05%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#956) 1 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

@Spacetown Spacetown added this to the Upcoming release milestone Sep 5, 2024
Copy link
Member

@Spacetown Spacetown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM. Thanks for fixing this.

Copy link
Member

@Spacetown Spacetown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM. Thanks for fixing this.

@Spacetown Spacetown merged commit 9b60be0 into gcovr:main Sep 5, 2024
@NeitrinoK NeitrinoK deleted the print-stderr-stdout branch September 5, 2024 22:20
@Spacetown Spacetown removed this from the Upcoming release milestone Oct 8, 2024
D-Walther pushed a commit to D-Walther/gcovr that referenced this pull request Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants