Closed Bug 933047 Opened 9 years ago Closed 9 years ago
configure should fail if CXX can't link C++ executables
While trying to setup a cross compile, I inadvertently set CXX to be 'clang' instead of 'clang++' in my mozconfig. The configure step still passed, but the build failed later when trying to link a C++ executable because it wasn't using the right libraries. It would be nice if configure would check to make sure that CXX is actually capable of linking properly.
Does this look somewhat appropriate? It seems to correctly flag CXX=clang in the mozconfig as invalid, though I have to rm -rf objdir when I change CXX.
Assignee: nobody → mh+mozilla
Status: NEW → ASSIGNED
Comment on attachment 824986 [details] [diff] [review] 0001-Bug-933047-WIP.patch This is roughly what i did, at a different level with a different test code.
Attachment #824999 - Flags: review?(ted)
Attachment #824999 - Flags: review?(ted) → review+
sigh. the test doesn't work with MSVC. OTOH, we do have a specific similar test for MSVC already in configure.in.
This fixes it. The fixup is trivial enough to not rerequest review.
Attachment #824999 - Attachment is obsolete: true
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
You need to log in before you can comment on or make changes to this bug.