Closed Bug 1635112 Opened 4 years ago Closed 4 years ago

Use `clang-tidy` as warning build error detector for our `static-analysis` targets.

Categories

(Developer Infrastructure :: Source Code Analysis, task)

task
Not set
normal

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.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

Component: General → Source Code Analysis

This was done in preparation to have build errors reporting at review-phase
based on clang-tidy.

For clang-tidy static-analysis mark all diagnostic-messages that are clang-diagnostic-error as publishable.

Depends on D74003

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
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
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: