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: Add scan-build #217

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

Conversation

tkc17
Copy link

@tkc17 tkc17 commented Dec 31, 2020

Run scan-build (clang-tools based) static analyzer on all build
variants.

@tkc17 tkc17 force-pushed the add_scan_build_ci branch 2 times, most recently from eae51c1 to b38e2a0 Compare December 31, 2020 19:04
@tkc17
Copy link
Author

tkc17 commented Dec 31, 2020

@Quuxplusone Looks like travis-ci is not integrated in the original repo, it is not triggering any jobs.

@tkc17 tkc17 force-pushed the add_scan_build_ci branch from b38e2a0 to 7923244 Compare December 31, 2020 19:07
@Quuxplusone
Copy link
Collaborator

@chaitu-tk: I see the builds happening in https://travis-ci.org/github/troydhanson/uthash/pull_requests — you might need to log into TravisCI in order to see them, I'm not sure. But it does look like they're failing, by being unable to pull clang-tools and/or unable to find scan-build in the PATH.

@tkc17 tkc17 force-pushed the add_scan_build_ci branch from 7923244 to e923627 Compare December 31, 2020 19:14
@tkc17
Copy link
Author

tkc17 commented Dec 31, 2020

Yes, I have just logged on, by default dist is 16.04, I have modified to use 18.04, if its works I will add 20.04 as well.

@tkc17
Copy link
Author

tkc17 commented Dec 31, 2020

Okay, it seems to work now, but there are a bunch of errors, so, we need to fix them first.

@tkc17 tkc17 force-pushed the add_scan_build_ci branch 14 times, most recently from 5247885 to ed54947 Compare January 3, 2021 10:26
Run scan-build (clang-tools based) static analyzer on all build
variants.

Use specific versions of the OSes as the default ones in Travis are old
and unsuitable to install packages.
@tkc17 tkc17 force-pushed the add_scan_build_ci branch from ed54947 to 6b035de Compare January 3, 2021 17:05
@tkc17
Copy link
Author

tkc17 commented Jan 4, 2021

Finally was able to get MacOS working, Sorry, took many trials as I don't have access to one and never worked on one.

@Quuxplusone
Copy link
Collaborator

It's been a couple of years, but if anyone's interested in picking this up again and trying to get it working in GitHub Actions (instead of TravisCI), that might be nice.
See 1e0baf0, 1a53f30 for the GitHub Actions CI stuff.

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.

2 participants