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

A new coverity scan would probaby be a good idea #6992

Closed
cmorty opened this issue May 15, 2019 · 10 comments
Closed

A new coverity scan would probaby be a good idea #6992

cmorty opened this issue May 15, 2019 · 10 comments
Labels
install/build bug in the installation or build scripts solved in the mean time Bug solved in a previous version, to be confirmed

Comments

@cmorty
Copy link
Contributor

cmorty commented May 15, 2019

It's probably I good Idea to run a new coverity scan (The last one was run mid last year). Maybe even set up a travis job (https://scan.coverity.com/travis_ci).
Also there are some old patches (#5627, #5637, #5616 and #5617) that might apply or can be closed. Not sure about their state.

@albert-github
Copy link
Collaborator

@cmorty
Copy link
Contributor Author

cmorty commented May 16, 2019

  * having it with travis-ci would be nice as well, but it should not run with each (proposed) patch or merge

Wouldn't work due to rate limiting. But running it every other day would be a start.

  * who will interpret the results
  * who will fix the results / create (proposed) patches

Currently you need to register to get access. If possible, the results could be public. That could get some drive-by pull-request. I personally would not worry too much. And if one is looking for a bug looking at the list of potential issues is often a good Idea.

OT: Receiving PRs isn't enough. They need to be merged, too. Having looked through the PRs (that's often the first thing I do when finding a bug) I found that there are many open PRs that are pretty old (up to 5 years). I'm not sure whether they merge at all or whether there will ever be someone to rebase them. Maybe adding something like https://probot.github.io/apps/stale/ could help sort out the PRs (and issues) that are still relevant. IM(!)O doxygen is loosing quite a bit o developer power by not merging or rejecting those PRs.

@albert-github
Copy link
Collaborator

A bit off topic but I did register and after login I can see the "View Defects" button but clicking on it results in a 503 error:

Service Unavailable

The server is temporarily unable to service your request due to maintenance downtime or capacity problems. Please try again later.

Apache/2.4.29 (Ubuntu) Server at scan3.coverity.com Port 443

@cmorty
Copy link
Contributor Author

cmorty commented May 16, 2019

I thought that it really was a temporary thing. Maybe @doxygen or Martin Ettl can have a look at it. They are the project admins

@albert-github
Copy link
Collaborator

@cmorty I was under the impression that you were able to access the defects, but apparently you couldn't either.

I took some actions (maybe they will bring something):

@albert-github
Copy link
Collaborator

albert-github commented May 16, 2019

I just got from synopsys an email back that they have resolved the problem and I can see the defects again.

I now looked quickly at the results, would be good to have a new run.

@albert-github albert-github added the install/build bug in the installation or build scripts label May 16, 2019
@cmorty
Copy link
Contributor Author

cmorty commented May 17, 2019

Still not working for me. :( ... It's not a 503 any more but:

It may take a few minutes before you can view your defects, when you change your email or password or sign-in with Github for the first time.

@albert-github
Copy link
Collaborator

I can see you in the list of members and when I understand it well you can login.
My advise write a nice email to scan-admin@coverity.com will, probably, take a little while as they are based in another time zone.

@albert-github albert-github added the solved in the mean time Bug solved in a previous version, to be confirmed label Dec 14, 2021
@albert-github
Copy link
Collaborator

We do a regular run of coverity.
I also have to assume that the access problems should be solved as well (as there are no further messages about it).

I think this problem has been solved in the mean time.

@cmorty cmorty closed this as completed Dec 14, 2021
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 2, 2022
Having Coverity automatically run.
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 2, 2022
Having Coverity automatically run.

Strange error: You have an error in your yaml syntax on line 105
Checking if everything before is correct.
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 2, 2022
Having Coverity automatically run.
Test on getting wget
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 2, 2022
Having Coverity automatically run.

What is wrong with wget ...
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Added on github repository secret
Check version of curl
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.

Correction token part (%% instead of %)
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Do we need to use `%...%` on windows instead of `$...`?
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Replaced token with a Test_value to see what is happening.
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Added token to env
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Now also a test with another env name
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
Coverity was not allowed
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.

Checking all programs present ...
Looks like secrets won't work in a fork :-(
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.

Bison should be a bit earlier ..
albert-github added a commit to albert-github/doxygen that referenced this issue Jan 3, 2022
Having Coverity automatically run.
doxygen added a commit that referenced this issue Jan 7, 2022
@albert-github
Copy link
Collaborator

An automatic github actions procedure has how been installed to run on a regular base.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
install/build bug in the installation or build scripts solved in the mean time Bug solved in a previous version, to be confirmed
Projects
None yet
Development

No branches or pull requests

2 participants