-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add Coverity scan #1455
Merged
Merged
Add Coverity scan #1455
Changes from 37 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
8829458
Add Coverity scan workflow
omalyshe 047a415
Fix indentation
omalyshe 0f10ea0
Fix indentation -2
omalyshe e8c5bb6
Fix steps
omalyshe ae56588
Fix typo
omalyshe 6d5b4b3
Replace slash / with the code %2F
omalyshe 9b5df6d
Typo-2
omalyshe 2e07c2a
Show download progress
omalyshe 7798ca7
Remove the comment
omalyshe af044fb
Fix curl
omalyshe 87aa7be
Try wget instead of curl to download
omalyshe 134c9b2
Fix syntax
omalyshe 32f8a81
Actions checkout v3
omalyshe add08ba
Remove comment
omalyshe cdd2df5
Use curl again
omalyshe cb9f737
Remove timeout
omalyshe d878f30
Cleanup
omalyshe 8341edf
Try to download to /tmp
omalyshe 1a95713
Check the file in tmp
omalyshe 4f45942
Change runner
omalyshe ce8ffe3
Use workspace dir
omalyshe 1460f60
Job permissions write
omalyshe c7ba3e4
Set write permission for workflow
omalyshe 6feda75
Fix typos
omalyshe 734b62a
Save artifact
omalyshe 6500e72
Fix file name
omalyshe cc4d025
Fix github_sha usage
omalyshe bc1a777
Add status badge and clean-up the code
omalyshe 291139a
Add back saving artifact
omalyshe b2fe19e
Exclude tests
omalyshe a257b5b
Don't create build directory
omalyshe 7133067
Typo fix
omalyshe 207ac14
Create build dir and cd to it to archive coverity results
omalyshe 28568e1
Fix indentation
omalyshe d7628b4
Enable test again; fix saving artifacts
omalyshe 018678f
Remove saving artifacts.
omalyshe 9f2b62a
Run coverity only on merge to master
omalyshe c68c54b
Remove debug print
omalyshe aa140ec
Check if the old project works
omalyshe 546173f
Merge branch 'master' into dev/omalyshe/coverity
omalyshe a364b87
Remove temporary added condition
omalyshe 35b1ac3
Test comment
omalyshe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
# Copyright (c) 2024 Intel Corporation | ||
# | ||
# Licensed under the Apache License, Version 2.0 (the "License"); | ||
# you may not use this file except in compliance with the License. | ||
# You may obtain a copy of the License at | ||
# | ||
# http://www.apache.org/licenses/LICENSE-2.0 | ||
# | ||
# Unless required by applicable law or agreed to in writing, software | ||
# distributed under the License is distributed on an "AS IS" BASIS, | ||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
# See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
|
||
name: Coverity Scan | ||
|
||
on: | ||
push: | ||
branches: [master] | ||
|
||
permissions: read-all | ||
|
||
env: | ||
BUILD_CONCURRENCY: 4 | ||
COVERITY_PROJECT: oneapi-src%2FoneTBB | ||
|
||
jobs: | ||
coverity_linux: | ||
name: Coverity Linux | ||
runs-on: [ubuntu-latest] | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- name: Download Linux 64 Coverity Tool | ||
run: | | ||
curl https://scan.coverity.com/download/cxx/linux64 --output ${GITHUB_WORKSPACE}/cov-linux64-tool.tar.gz \ | ||
--data "token=${{secrets.COVERITY_TOKEN}}&project=${{env.COVERITY_PROJECT}}" | ||
ls -l ${GITHUB_WORKSPACE} | ||
isaevil marked this conversation as resolved.
Show resolved
Hide resolved
|
||
mkdir cov-linux64-tool | ||
tar -xzf cov-linux64-tool.tar.gz --strip 1 -C cov-linux64-tool | ||
- name: Build with cov-build | ||
run: | | ||
export PATH="${PWD}/cov-linux64-tool/bin:${PATH}" | ||
mkdir build && cd build | ||
cmake -DCMAKE_CXX_STANDARD=20 -DCMAKE_BUILD_TYPE=relwithdebinfo \ | ||
-DCMAKE_CXX_COMPILER=g++ -DCMAKE_C_COMPILER=gcc -DTBB_CPF=ON .. | ||
cov-build --dir cov-int make VERBOSE=1 -j${{env.BUILD_CONCURRENCY}} | ||
- name: Archive Coverity build results | ||
run: | | ||
cd build | ||
tar -czvf cov-int.tgz cov-int | ||
- name: Submit Coverity results for analysis | ||
run: | | ||
cd build | ||
curl \ | ||
--form token="${{ secrets.COVERITY_TOKEN }}" \ | ||
--form email="${{ secrets.COVERITY_EMAIL }}" \ | ||
--form [email protected] \ | ||
--form version="${GITHUB_SHA}" \ | ||
--form description="" \ | ||
"https://scan.coverity.com/builds?project=${{env.COVERITY_PROJECT}}" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to change coverity project name from
oneapi-src
touxlfoundation
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I found in the documentation - no, it is not possible to change the project name. Only create the new one.