Closed
Bug 1635112
Opened 5 years ago
Closed 5 years ago
Use `clang-tidy` as warning build error detector for our `static-analysis` targets.
Categories
(Developer Infrastructure :: Source Code Analysis, task)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox78 fixed)
RESOLVED
FIXED
mozilla78
| Tracking | Status | |
|---|---|---|
| firefox78 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
References
Details
Attachments
(4 files)
The purpose of this patch is to add support for build-error reporting through clang-tidy in order to drop the usage of Coverity for build-error reporting during review-phase.
|
Assignee | |
Comment 1•5 years ago
|
||
|
||
Comment 2•5 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Component: General → Source Code Analysis
|
|
Assignee | |
Comment 3•5 years ago
|
||
This was done in preparation to have build errors reporting at review-phase
based on clang-tidy.
|
|
Assignee | |
Comment 4•5 years ago
|
||
Depends on D73692
|
|
Assignee | |
Comment 5•5 years ago
|
||
For clang-tidy static-analysis mark all diagnostic-messages that are clang-diagnostic-error as publishable.
Depends on D74003
|
||
Updated•5 years ago
|
Attachment #9146063 -
Attachment description: Bug 1635112 - Disable the build error reporting at review-phase using `Coverity`. r=glandium → Bug 1635112 - Disable the build error reporting at review-phase using `Coverity`. r=sylvestre,marco
|
|
||
Updated•5 years ago
|
Attachment #9146457 -
Attachment description: Bug 1635112 - Mark `clang-diagnostic-error` as an `publishable` in the `clang-tidy` artifact. r=sylvestre → Bug 1635112 - Mark `clang-diagnostic-error` as `publishable` in the `clang-tidy` artifact. r=sylvestre
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9d015b5e2ef8
remove MSVC support from `WarningsCollector`. r=glandium
https://hg.mozilla.org/integration/autoland/rev/8ade9f1318ba
Use `clang-tidy` as warning build error detector for our `static-analysis` targets. r=glandium
https://hg.mozilla.org/integration/autoland/rev/a9d5c0620a5b
Disable the build error reporting at review-phase using `Coverity`. r=sylvestre,marco
https://hg.mozilla.org/integration/autoland/rev/dfe30a835562
Mark `clang-diagnostic-error` as `publishable` in the `clang-tidy` artifact. r=marco
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/9d015b5e2ef8
https://hg.mozilla.org/mozilla-central/rev/8ade9f1318ba
https://hg.mozilla.org/mozilla-central/rev/a9d5c0620a5b
https://hg.mozilla.org/mozilla-central/rev/dfe30a835562
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox78:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
|
||
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•