Closed
Bug 681634
Opened 13 years ago
Closed 12 years ago
Mozilla-central fails to build --with-static-checking
Categories
(Developer Infrastructure :: Source Code Analysis, defect)
Developer Infrastructure
Source Code Analysis
Tracking
(Not tracked)
RESOLVED
WONTFIX
mozilla9
People
(Reporter: jcranmer, Assigned: jcranmer)
References
Details
Attachments
(2 files)
(deleted),
patch
|
taras.mozilla
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
cjones
:
review+
|
Details | Diff | Splinter Review |
Notes: 1. Configure documentation suggests using gcc_dehydra.so, which causes things to fail immediately (Detect dehydra v. treehydra or hint to use treehydra). 2. Elfhack throws a JS Exception. --disable-elf-hack works. 3. The frame-verify script uses callable regex. 4. We have two failures for NS_OVERRIDE: + gfx/layers/ipc/ShadowLayersParent (RecvGetParentType) + content/svg/content/src/SVGMotionSMILAnimationFunction (TreatSingleValueAsStatic) 5. Linking errors in libxul.so: ../../js/jsd/jsd_atom.o: In function `MUST_FLOW_THROUGH': /tmp/jcranmer-mozilla/obj-mc/js/jsd/../../dist/include/jsstaticcheck.h:48: multiple definition of `MUST_FLOW_THROUGH' ../../js/jsd/jsdebug.o:/tmp/jcranmer-mozilla/obj-mc/js/jsd/../../dist/include/jsstaticcheck.h:48: first defined here [and more, similar] 6. Two files of the HTML parser (Element and Attribute names) cause out-of-memory errors. 7. I need a small patch to dehydra to add in a TYPE_PACK_EXPANSION to yet another switch statement (c_c).
Assignee | ||
Comment 1•13 years ago
|
||
The gfx/layers/ipc/ShadowLayersParent is a result of patch 3.1 on bug 648484, and appears to be a neglect to remove dead code. The other NS_OVERRIDE I've traced is to bug 614879, and appears to be an actual problem (someone didn't change the later function).
Assignee | ||
Comment 3•13 years ago
|
||
This is the bustage due to NS_OVERRIDE in gfx.
Attachment #556651 -
Flags: review?(jones.chris.g)
Updated•13 years ago
|
Attachment #556618 -
Flags: review?(tglek) → review+
Comment on attachment 556651 [details] [diff] [review] Fix bustage caused by cjones Is this part of a grand scheme to get automated static checking builds again? Oh please oh please oh please ...
Attachment #556651 -
Flags: review?(jones.chris.g) → review+
Comment 6•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/23962193053c http://hg.mozilla.org/mozilla-central/rev/2642c7b0dc60
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla9
Version: unspecified → Trunk
Assignee | ||
Comment 7•13 years ago
|
||
[not quite fixed yet, see blocking bug and notes about the failure in htmlparser]
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Updated•13 years ago
|
Status: REOPENED → ASSIGNED
Assignee | ||
Comment 8•12 years ago
|
||
Dehydra support is deprecated.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago → 12 years ago
Resolution: --- → WONTFIX
Updated•7 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
•