Closed
Bug 654080
Opened 14 years ago
Closed 13 years ago
Display which GLIBCXX symbols cause an error
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla6
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files)
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
Details | Diff | Splinter Review |
In bug 643690, I added an error displaying GLIBCXX versions we don't want to link against. It would be more useful to also list which symbols correspond to these versions.
(Note that I used > /dev/null instead of grep -q because I don't know if we can rely on grep providing the -q option)
Attachment #529446 -
Flags: review?(ted.mielczarek)
Assignee | ||
Comment 1•14 years ago
|
||
While at it, "Error" in the message doesn't seem to trigger anything on tinderbox logs :-/
http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304659221.1304663419.6828.gz
Does it need to be on stderr or something?
Assignee | ||
Comment 3•14 years ago
|
||
Does that mean we need TinderboxPrint *and* Error to be displayed on tbpl?
Comment 4•14 years ago
|
||
No. TinderboxPrint is what makes things show up in the build column on tinderbox.mozilla.org. The error parser controls what shows up in the summary of the short log:
http://mxr.mozilla.org/mozilla/source/webtools/tinderbox/ep_unittest.pl
Assignee | ||
Comment 5•14 years ago
|
||
So, we need to abuse one of these:
http://mxr.mozilla.org/mozilla/source/webtools/tinderbox/ep_unittest.pl#27
Or should we add a generic type of error there ?
Comment 6•14 years ago
|
||
I'd just print TEST-UNEXPECTED-FAIL, honestly, since this is in "make check". I think we don't match "Error:" because we get too many false positives from tests that have expected JS Errors or print that string in the test text, etc.
Assignee | ||
Comment 7•14 years ago
|
||
fair enough, though it's not in make check, but in plain make.
Updated•14 years ago
|
Assignee: nobody → mh+mozilla
Comment 8•14 years ago
|
||
Comment on attachment 529446 [details] [diff] [review]
Display which GLIBCXX symbols cause an error
Review of attachment 529446 [details] [diff] [review]:
-----------------------------------------------------------------
r=me with the output fixed to match the tinderbox error parser.
Attachment #529446 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 9•13 years ago
|
||
Assignee | ||
Comment 10•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/d21dae3474b3
(the new attachment was the patch as landed)
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•