Configure should test that the C and C++ compiler versions are the same. At a minimum, this is needed to make our compiler version checks sane; we do tests on only one of the versions. It's probably also good hygiene, since I suspect most cases of people trying to build with different versions are mistakes, and ones that might not be caught quickly. (I've certainly done it; see bug 227867.)
Created attachment 8576781 [details] [diff] [review] Require that the same compiler version be used for C and C++ (at the very least, so that our version checks are valid for both) I tested locally that both checks give the expected error if I temporarily change the != to an =.
Attachment #8576781 - Flags: review?(mh+mozilla)
Attachment #8576781 - Flags: review?(mh+mozilla) → review+
sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=e76c33c4d452 since the other patch caused a bustage and this 2 csets were tight togehter
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.