Closed
Bug 1405607
Opened 7 years ago
Closed 7 years ago
[Mach][Static-Analysis] Don't pass the list of files to run-clang-tidy
Categories
(Developer Infrastructure :: Source Code Analysis, enhancement)
Developer Infrastructure
Source Code Analysis
Tracking
(firefox58 fixed)
RESOLVED
FIXED
mozilla58
| Tracking | Status | |
|---|---|---|
| firefox58 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
References
Details
Attachments
(1 file)
Since we are using the compile_db to retrieve the number of files that are going to be scanned it's useless to also pass during the execution of run-clang-tidy since the utility will only use files that are in the compile_db.
| Comment hidden (mozreview-request) |
|
||
Comment 2•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8915070 [details] Bug 1405607 - do not pass to run-clang-tidy files that are going to be scanned. https://reviewboard.mozilla.org/r/186328/#review191340 This is fixing https://bugzilla.mozilla.org/show_bug.cgi?id=1405610 we can duplicate it.
Attachment #8915070 -
Flags: review?(sledru) → review+
Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6ad2d1dffb21 do not pass to run-clang-tidy files that are going to be scanned. r=sylvestre
|
||
Comment 5•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/6ad2d1dffb21
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
|
||
Updated•6 years ago
|
Product: Core → Firefox Build System
|
|
||
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
•