Closed
Bug 687342
Opened 11 years ago
Closed 11 years ago
arm.obj : warning LNK4221: This object file does not define any previously undefined public symbols
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: emorley, Assigned: emorley)
References
(Blocks 1 open bug)
Details
(Whiteboard: [build_warning])
Attachments
(1 file, 1 obsolete file)
1.00 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
MSVC 2010 using http://hg.mozilla.org/integration/mozilla-inbound/rev/f827e9173fea { arm.obj : warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library } File in question: http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.cpp Included here: http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/objs.mk#72 Only actually does anything #if defined(MOZILLA_ARM_HAVE_CPUID_DETECTION), which is set here: http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.h#110 http://mxr.mozilla.org/mozilla-central/source/xpcom/glue/arm.h#115 ie: if defined(__linux__) || defined(ANDROID) || (defined(_MSC_VER) && defined(_M_ARM))
Flags: in-testsuite-
Assignee | ||
Comment 1•11 years ago
|
||
Adds a |ifeq (arm, $(TARGET_CPU))| around the arm.cpp line. Builds fine locally with MSVC 2010 and the LNK4221 warning in comment 0 no longer appears. Sent to try: https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=2f7ee79fc650
Attachment #560793 -
Flags: review?(khuey)
Assignee | ||
Comment 2•11 years ago
|
||
As before, except this time with += rather than =+ (doh!). https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=2058bc9b70b2
Attachment #560793 -
Attachment is obsolete: true
Attachment #560793 -
Flags: review?(khuey)
Attachment #560794 -
Flags: review?(khuey)
Comment on attachment 560794 [details] [diff] [review] Patch v1.1 >diff --git a/xpcom/glue/objs.mk b/xpcom/glue/objs.mk >+ifeq (arm, $(TARGET_CPU)) Nit: ifeq (arm,$(TARGET_CPU))
Attachment #560794 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 4•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/abfc871cc026
Target Milestone: --- → mozilla9
Comment 5•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/abfc871cc026
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•