576355 - Make configure work with clang

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Eli Friedman]

Attachment: Patch

Patch attached.  Two issues here:

1. clang doesn't completely support using -Wp to pass command-line options to the preprocessor; this patch uses the proper gcc frontend options instead.

2. grepping for a specific response to using -fno-handle-exceptions as an argument isn't really a good idea; since Mozilla hasn't supported g++ 2.8 for a *very* long time, just use -fno-exceptions unconditionally.

Comment 1

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/0e6c8cf26d7d

Comment 2

[Nomis101]

Attachment: Same patch for comm-central

This patch works now for a few days without any issues, so I think we can make the same changes to comm-central. So we can also build Thunderbird and SeaMonkey with Clang.
I've made this patch by Copy&Paste and tested it with Clang and GCC (on Mac).
Is it OK like that?

Comment 3

[Nomis101]

Happily comm-central is open now, so can anyone checkin the missing comm-central part, please. Thanks! :-)

Comment 4

[Justin Wood (:Callek)]

http://hg.mozilla.org/comm-central/rev/1dea83e6409f