Closed Bug 1878180 Opened 8 months ago Closed 8 months ago

Catch more rust errors in logs

Categories

(Release Engineering :: Applications: MozharnessCore, task)

Tracking

(firefox124 fixed)

RESOLVED FIXED
Tracking Status
firefox124 --- fixed

People

(Reporter: glandium, Assigned: glandium)

Details

Attachments

(1 file)

No description provided.

Not all rust errors start with "error[Exxx]:", as shown by the two other
types of errors we catch that start with error:, but simplifying to
"error:" would be too broad, catching messages about std::error::Error.
"error: " might work, but feels risky (and it would be redundant with
the matchers for other compiler errors).

All the errors from the rust compiler appear at the beginning of the
line, though, so it feels a little safer to match on "error: " being at
the beginning (+ the optional [Exxx]), and it feels rather safe to
assume that if a line begins with error:, it's an error. This obviously
would catch other messages that start with error:, and maybe it should
be used more generally, but I want to see if this causes unexpected
problems first. (which is why I also went with fixing bug 1877891
separately of this)

Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/ba97fc77e526 Catch more rust errors in logs. r=releng-reviewers,aryx,jcristau
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: