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

Feature request: add ability to merge .info files together #32

Closed
f18m opened this issue Sep 16, 2019 · 1 comment · Fixed by #40
Closed

Feature request: add ability to merge .info files together #32

f18m opened this issue Sep 16, 2019 · 1 comment · Fixed by #40

Comments

@f18m
Copy link

f18m commented Sep 16, 2019

Hi,
it would be really nice to have in fastcov the same option that lcov provides, i.e. the --add-tracefile option which allows you to merge together .lcov files (or .info files in fastcov case).
This allows to merge contributes from different runs of the same source tree from e.g. different containers...

@RPGillespie6
Copy link
Owner

RPGillespie6 commented Sep 16, 2019

Are you using genhtml? If so, you can just pass them all and it will handle it correctly :

$ genhtml coverage1.info coverage2.info coverage3.info -o html_report

That said, I'll look into seeing how hard it would be to add that feature into fastcov itself.

Would it be acceptable to merge fastcov json files and then spit out lcov format at the very end? Currently fastcov only generates lcov files, so I would have to add the ability to parse lcov files if that is a requirement. However, it would be fairly trivial to be able to merge fastcov json files (since it is just json) and then spit out lcov format for the final report:

$ fastcov ... -o coverage1.json
$ fastcov ... -o coverage2.json
$ fastcov ... -o coverage3.json
...
$ fastcov --add-tracefile coverage1.json coverage2.json coverage3.json ... --lcov -o coverage_final.info

RPGillespie6 added a commit that referenced this issue Mar 18, 2020
RPGillespie6 added a commit that referenced this issue Mar 20, 2020
Description:
- Fix #39
- Fix #32
- Remove support for gcov-json output (nobody uses it)
- Fix example build
- Add additional test to make sure example builds
RPGillespie6 added a commit that referenced this issue Mar 20, 2020
Description:
- Fix #39
- Fix #32
- Remove support for gcov-json output (nobody uses it)
- Fix example build
- Add additional test to make sure example builds
RPGillespie6 added a commit that referenced this issue Mar 28, 2020
Description:
- Fix #39
- Fix #32
- Remove support for gcov-json output (nobody uses it)
- Fix example build
- Add additional test to make sure example builds
RPGillespie6 added a commit that referenced this issue Mar 31, 2020
Description:
- Fix #39
- Fix #32
- Remove support for gcov-json output (nobody uses it)
- Fix example build
- Add additional test to make sure example builds
RPGillespie6 added a commit that referenced this issue Mar 31, 2020
Description:
- Fix #39
- Fix #32
- Remove support for gcov-json output (nobody uses it)
- Fix example build
- Add additional test to make sure example builds
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 a pull request may close this issue.

2 participants