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

Integrate Coverity Scan tool into superbuild #104

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nocnokneo
Copy link
Contributor

Associated bug report: http://bugs.mitk.org/show_bug.cgi?id=19104

Download and install the Coverity build capture tool and add a
"coverity_scan_submission" build target. To use this feature the following
CMake variables must be provided:

-DMITK_COVERITY_SCAN_TOKEN:STRING=XXX
-DMITK_COVERITY_SCAN_EMAIL:STRING=XXX

Alternatively, Coverity can be manually installed and the intermediate build
target "coverity_data_archive" can be used to generate cov-int.tar.bz2 which
can be manually uploaded to Coverity Scan for analysis.

Implemented only for 64-bit Linux for now.

Download and install the Coverity build capture tool and add a
"coverity_scan_submission" build target. To use this feature the following
CMake variables must be provided:

-DMITK_COVERITY_SCAN_TOKEN:STRING=XXX
-DMITK_COVERITY_SCAN_EMAIL:STRING=XXX

Alternatively, Coverity can be manually installed and the intermediate build
target "coverity_data_archive" can be used to generate cov-int.tar.bz2 which
can be manually uploaded to Coverity Scan for analysis.

Implemented only for 64-bit Linux for now.
@kislinsk
Copy link
Contributor

kislinsk commented Mar 6, 2018

Closing old pull requests. Please leave a comment if you think that this pull request still really needs to be addressed , thank you.

@kislinsk kislinsk closed this Mar 6, 2018
@nocnokneo
Copy link
Contributor Author

Seem like the MITK code base could benefit from something like this. And the free service that Coverity provides for open source projects is a valuable benefit that's hard to pass up. Have you tried using this branch to run Coverity analysis?

@kislinsk
Copy link
Contributor

kislinsk commented Mar 6, 2018

I agree, but we didn't try yet. We have some major work ahead regarding our CI, and this is something we will keep in mind for it. I re-open this PR.

@kislinsk kislinsk reopened this Mar 6, 2018
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.

None yet

2 participants