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

CI #66

Open
Lewiscowles1986 opened this issue Jun 11, 2018 · 3 comments
Open

CI #66

Lewiscowles1986 opened this issue Jun 11, 2018 · 3 comments

Comments

@Lewiscowles1986
Copy link

Hello. Thanks for taking the time to write this and make it public. I came in via #42 but can see you use a unit testing framework.

I've been trying to generate coverage, both by forking your repo and running your travis.yml there are errors. I wasn't sure if you knew about them, including but not limited to commands for executable files that are not installed, passing a token that can just be set via travis settings.

I'm currently trying to fix these problems, but it seems C++ coverage is harder to generate than the same in Java, C# or scripting language.

@Lewiscowles1986
Copy link
Author

https://github.com/CODESIGN2/silicon/tree/coverage-works/ has the dockerfile build tests that generate coverage using lcov. There still seems to be some problem (it's reporting 0.0%) but it does so based upon the silicon source tree so as soon as I've worked out what has gone wrong there I have the coveralls system receiving that data.

@matt-42
Copy link
Owner

matt-42 commented Jun 12, 2018

Just for info I did some work some time ago to make the travis.yml work again: https://travis-ci.org/matt-42/silicon
I had to disable all compilers but clang-5.0 and also the optional test.

@Lewiscowles1986
Copy link
Author

That is the vein of work I've picked up. Thanks for that. So-far the largest problem seems to be differing CLI API's between versions of needed software. I think once it's built successfully matrix, tests only built for clang builds and sending coverage, then the rest should be easy to turn back on.

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

No branches or pull requests

2 participants