Closed Bug 1590414 Opened 5 years ago Closed 4 years ago

For C++ static analysis cleanup the running scripts.

Categories

(Developer Infrastructure :: Source Code Analysis, enhancement)

enhancement
Not set
normal

Tracking

(firefox74 fixed)

RESOLVED FIXED
mozilla74
Tracking Status
firefox74 --- fixed

People

(Reporter: andi, Assigned: andi)

Details

Attachments

(1 file)

The intent of this patch is to cleanup the way how we add the mozconfig file to the scripts that run static-analysis for C/C++. Instead of using echo and redirecting the output we should use cat.

Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f4520f5c3cdb
For C++ static analysis cleanup the running scripts. r=marco
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Backout by dvarga@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/49324ed41cd8
Backed out changeset f4520f5c3cdb on request by ANdi a=backout
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: mozilla72 → ---

We had to backout this, since cov-translate outputs some errors when dealing with debug builds.

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:andi, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(bpostelnicu)

Using the latest version of coverity I can say that the issue that impaired us on having debug builds has been fixed, so re-landing this.

Flags: needinfo?(bpostelnicu)
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/481ab2626f2d
For C++ static analysis cleanup the running scripts. r=marco
Status: REOPENED → RESOLVED
Closed: 5 years ago4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
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: