Closed
Bug 308042
Opened 19 years ago
Closed 19 years ago
ld -exported_symbols_list check erroneously fails with Xcode 2
Categories
(Firefox Build System :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mark, Assigned: mark)
Details
Attachments
(1 file)
1.19 KB,
patch
|
bryner
:
review+
dbaron
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Using Xcode 2.1, I'm seeing this: checking for -exported_symbols_list option to ld... ld: -exported_symbols_list: argument missing no Prior versions of ld spit the error to stdout. It's now going to stderr.
Assignee | ||
Comment 1•19 years ago
|
||
Not critical, the build falls back to nmedit anyway.
Attachment #195645 -
Flags: review?(bryner)
Updated•19 years ago
|
Attachment #195645 -
Flags: review?(bryner) → review+
Assignee | ||
Comment 2•19 years ago
|
||
Checked in.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•18 years ago
|
||
Comment on attachment 195645 [details] [diff] [review] Check both stdout and stderr This is bloating the build time and causing warnings like this: nmedit: can't make global coalesced symbols (like __Z10NS_OpenURIP17nsIStreamListenerP11nsISupportsP6nsIURIP12nsIIOServiceP12nsILoadGroupP21nsIInterfaceRequestorj) into static symbols (use ld(1)'s -exported_symbols_list option) in a final linked image: /lizard/1.8/build/firefox-g/layout/build/libgklayout.dylib Baked on the trunk for ages.
Attachment #195645 -
Flags: approval1.8.1?
Comment on attachment 195645 [details] [diff] [review] Check both stdout and stderr a=dbaron on behalf of drivers. Please land on MOZILLA_1_8_BRANCH and add the fixed1.8.1 keyword once you have done so.
Attachment #195645 -
Flags: approval1.8.1? → approval1.8.1+
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•