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)

All
Linux
defect
Not set
normal

Tracking

(firefox8 affected, firefox9 affected)

RESOLVED FIXED
mozilla10
Tracking Status
firefox8 --- affected
firefox9 --- affected

People

(Reporter: glandium, Assigned: glandium)

References

Details

(Whiteboard: [inbound])

Attachments

(1 file)

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: nobody → mh+mozilla
Blocks: 670659
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+
(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.
https://hg.mozilla.org/mozilla-central/rev/0151c893093e
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: