764469 - Make c-c build system compatible with OS X 10.8

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Nomis101]

Attachment: WIP

This currently only ports Bug 750574. I will tweak the other stuff in the next few days.

Comment 1

[Nomis101]

Attachment: Fix build issues on 10.8

OK, this patch now fixes every issue in MailnewsCore I had while building TB on 10.8. First, because of the egrep issue, I had to port Bug 750574. The second problem is, that per default it trys to build with llvm-gcc, but this isn't supported from Mozilla and it fails with an error. So I also had to port the "select clang over llvm-gcc" patch from Bug 555727. Because Apple ships in 10.8 an enhanced version of Clang, I had to include MOZ_COMPILER_OPTS from Bug 727145. In Bug 762071 the position of this MOZ_COMPILER_OPTS has changed and the rtti part has moved to compiler-opts.m4, so I better also ported Bug 762071 (to be on the safe side).

With this patch now I can build TB on 10.8 DP4 until I run into Bug 747621 (which I try to fix there).

Comment 2

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/2b69382950af

Thanks for the patch! In the future, please follow the directions below for patches you submit. It makes life much easier for those checking them in on your behalf. Thanks!
https://developer.mozilla.org/en/Creating_a_patch_that_can_be_checked_in

Comment 3

[Nomis101]

(In reply to Ryan VanderMeulen from comment #2)
> https://hg.mozilla.org/comm-central/rev/2b69382950af
> 
> Thanks for the patch! In the future, please follow the directions below for
> patches you submit. It makes life much easier for those checking them in on
> your behalf. Thanks!
> https://developer.mozilla.org/en/Creating_a_patch_that_can_be_checked_in
Yes I know, sorry for that, 'hg qnew' doesn't work for me currently on 10.8. :-(
Thanks for checkin in.