Merged: https://github.com/OpenMW/openmw/pull/2029
Want me to back-port to 0.45? Then we have 200 issues
OpenMW 0.45.0
- psi29a
- Posts: 5356
- Joined: 29 Sep 2011, 10:13
- Location: Belgium
- Gitlab profile: https://gitlab.com/psi29a/
- Contact:
Re: OpenMW 0.45.0
Since there are no RC builds for Linux and MacOS yet, I suggest to backport this fix and update Windows builds.psi29a wrote: ↑12 Nov 2018, 11:45 Merged: https://github.com/OpenMW/openmw/pull/2029
Want me to back-port to 0.45? Then we have 200 issues
Also there was a request to update OpenAL-soft version in Windows builds to 1.18 to avoid bug with underwater sounds.
Re: OpenMW 0.45.0
And what's up with Coverity Scan? Can the issue be because of excluded Recast Navigation?
- psi29a
- Posts: 5356
- Joined: 29 Sep 2011, 10:13
- Location: Belgium
- Gitlab profile: https://gitlab.com/psi29a/
- Contact:
Re: OpenMW 0.45.0
I'm not sure what's up there... I'm going try again now that I'm back.
Backporting now... New RCs please.
Backporting now... New RCs please.
- psi29a
- Posts: 5356
- Joined: 29 Sep 2011, 10:13
- Location: Belgium
- Gitlab profile: https://gitlab.com/psi29a/
- Contact:
Re: OpenMW 0.45.0
coverity scan now takes forever on travis (combination of gcc8 and openmw's size).
I'm going to manually do this and upload my results. If that works, then I'll have to re-think how to trigger coverity builds.
I'm going to manually do this and upload my results. If that works, then I'll have to re-think how to trigger coverity builds.
- psi29a
- Posts: 5356
- Joined: 29 Sep 2011, 10:13
- Location: Belgium
- Gitlab profile: https://gitlab.com/psi29a/
- Contact:
Re: OpenMW 0.45.0
Good news and bad news.
Good news:
* Finally managed to get a full build over to Coverity, you can check it out:
https://scan.coverity.com/projects/open ... b=overview
Bad news:
* Latest changes to the coverity tool and our own code has pushed us over the build time limit of 50m. The build, of just openmw, completes at 45 minutes but it takes longer than 5 minutes to upload the result to Coverity which is why we haven't had a successful upload.
Ugly news:
* While 7 old issues were eliminated, 69 new ones have cropped up.
* 5 new issues are marked as 'high', 3 of which is related to our own code and 2 with /extern/recastnavigation itself.
Good news:
* Finally managed to get a full build over to Coverity, you can check it out:
https://scan.coverity.com/projects/open ... b=overview
Bad news:
* Latest changes to the coverity tool and our own code has pushed us over the build time limit of 50m. The build, of just openmw, completes at 45 minutes but it takes longer than 5 minutes to upload the result to Coverity which is why we haven't had a successful upload.
Ugly news:
* While 7 old issues were eliminated, 69 new ones have cropped up.
* 5 new issues are marked as 'high', 3 of which is related to our own code and 2 with /extern/recastnavigation itself.
Re: OpenMW 0.45.0
Another good news:
1. Most of these 69 issues come from autogenerated moc files (code duplication, we just did not scan moc files earlier) and from recast navigation itself.
2. Also there are some issues with editor, since previously we scanned only game itself (probably that's why we did not hit the 50 min tilme limit).
Re: OpenMW 0.45.0
Also I have a guess why filtering does not work:
2. *moc_*.cxx supposed to work with files with .cxx resolution, but now we get (also?) mocs with .cpp resolution.
1. Path /extern/recastnavigation/.* is absolute, but the real path is the /home/psi29a/openmw/extern/recastnavigation/.*
2. *moc_*.cxx supposed to work with files with .cxx resolution, but now we get (also?) mocs with .cpp resolution.
1. Path /extern/recastnavigation/.* is absolute, but the real path is the /home/psi29a/openmw/extern/recastnavigation/.*
- psi29a
- Posts: 5356
- Joined: 29 Sep 2011, 10:13
- Location: Belgium
- Gitlab profile: https://gitlab.com/psi29a/
- Contact:
Re: OpenMW 0.45.0
I'm trying to see if we can exclude those, wondering why they got included as nothing has changed there.
No, that isn't true. Only openmw was being built for coverity on Travis. OpenMW itself has always been 'on the edge', but with recent additions to our code we've hit the timeout. Adding openmw-cs, launcher, importers, etc. would have pushed it up to 2 hours otherwise (only 2 CPUs).
What I uploaded was the almost everything because I was building it locally with coverity.