Closed Bug 407783 Opened 17 years ago Closed 17 years ago

release automation test target should fail on compile errors

Categories

(Release Engineering :: General, defect, P3)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: rhelmer, Assigned: rhelmer)

References

Details

Attachments

(1 file)

[checked in] let's just exit noisily if there are any compile errors
626 bytes, patch
bhearsum
: review+
Details | Diff | Splinter Review 
The test target in http://mxr.mozilla.org/mozilla/source/tools/release/Makefile does a "perl -c" on each file, but does not exit with a non-0 value (as it should).

more prominent "FAIL" lines would be nice too, it's hard to see these errors in a normal-sized terminal.
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Priority: -- → P3
Attachment #294316 - Flags: review?(bhearsum) → review+
Comment on attachment 294316 [details] [diff] [review]
[checked in] let's just exit noisily if there are any compile errors

Checking in Makefile;
/cvsroot/mozilla/tools/release/Makefile,v  <--  Makefile
new revision: 1.18; previous revision: 1.17
done
Attachment #294316 - Attachment description: let's just exit noisily if there are any compile errors → [checked in] let's just exit noisily if there are any compile errors
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: