223323 - Fix building under 10.3 with Xcode

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[Simon Fraser [no longer active]]

Building Camino on 10.3 breaks because pbxbuild is no longer present; it's
xcodebuild now.

Comment 1

[Simon Fraser [no longer active]]

Attachment: Patch to use xcodebuild when present

Comment 2

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 133899 [details] [diff] [review]
Patch to use xcodebuild when present

r=pink

Comment 3

[Simon Fraser [no longer active]]

Attachment: Patch to make Rendezvous stuff build

Comment 4

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 133951 [details] [diff] [review]
Patch to make Rendezvous stuff build

landed

Comment 5

[Mike Calmus]

*** Bug 223778 has been marked as a duplicate of this bug. ***

Comment 6

[Mike Pinkerton (not reading bugmail)]

eric albert gave us this help on the xcode mailing list:


Cool bug!  This appears to be running afoul of the compiler's
implicit wrapping of all /usr/include headers with 'extern "C"'.
Prior to Panther, system headers didn't always do a great job of
exporting their APIs as C.  GCC implicitly exports anything in
/usr/include as a C API, but CodeWarrior doesn't.  If you look
through various list and newsgroup archives, you'll see a lot of
messages from CodeWarrior users who had trouble with the Mach headers
from C++ code on Jaguar.  It seems that GCC isn't recognizing the SDK
headers as system headers, so you're running into the same problem
there.  The compiler thinks the functions are C++ APIs, so it links
against the C++-name-mangled versions, but those don't exist at link
time.

Until this is fixed, the workaround is to wrap all includes of Mach
headers like this:
#ifdef __cplusplus
extern "C" {
#endif
#include <mach/mach.h>
#ifdef __cpluplus
}
#endif

This works for me with a test program in Xcode that calls
mach_host_self() and uses the 10.2.7 SDK.

Hope this helps,
Eric 

Comment 7

[Mike Pinkerton (not reading bugmail)]

this patch we were waiting on has already landed, but other issues are still in
bug 224161. closing this one out.

Comment 8

[Andreas Kunz]

Comment on attachment 133899 [details] [diff] [review]
Patch to use xcodebuild when present

Removing obsolete approval request.