Closed
Bug 690682
Opened 13 years ago
Closed 13 years ago
error: couldn't compile a simple C file with some versions of binutils
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox8 affected, firefox9 affected)
RESOLVED
FIXED
mozilla10
People
(Reporter: glandium, Assigned: glandium)
References
Details
(Whiteboard: [inbound])
Attachments
(1 file)
|
(deleted),
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
The error looks like this:
checking whether removing dead symbols breaks debugging... configure: error: couldn't compile a simple C file
The problem comes from an LD bug: http://sourceware.org/bugzilla/show_bug.cgi?id=13195
Instead of failing, we should just disable dead symbol removal.
|
Assignee | |
Updated•13 years ago
|
status-firefox8:
--- → affected
status-firefox9:
--- → affected
|
|
Assignee | |
Updated•13 years ago
|
Assignee: nobody → mh+mozilla
|
|
Assignee | |
Comment 1•13 years ago
|
||
Attachment #563685 -
Flags: review?(ted.mielczarek)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 563685 [details] [diff] [review]
Disable dead symbol removal when failing to test for bug 670659
Review of attachment 563685 [details] [diff] [review]:
-----------------------------------------------------------------
::: configure.in
@@ +6889,5 @@
> GC_SECTIONS_BREAKS_DEBUG_RANGES=yes
> fi
> else
> dnl We really don't expect to get here, but just in case
> + GC_SECTIONS_BREAKS_DEBUG_RANGES="no, but it's broken in some other way"
Seems a little weird to stick an arbitrary string in here. Maybe you should just use "yes"?
Attachment #563685 -
Flags: review?(ted.mielczarek) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
(In reply to Ted Mielczarek [:ted, :luser] from comment #2)
> Seems a little weird to stick an arbitrary string in here. Maybe you should
> just use "yes"?
"yes" would be lying, since the issue is entirely different ; and an arbitrary string is supported just fine.
|
|
Assignee | |
Comment 4•13 years ago
|
||
Whiteboard: [inbound]
|
||
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
|
||
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
•