427568 - Cross-platform build issues with mac 10.5 and 10.4

Status: VERIFIED FIXED

(Tamarin Graveyard :: Build Config, defect)

Description

[Brent Baker]

Compiling the shell on osx 10.5 using the cross-platform build generates a shell that will not run on osx 10.4. When you run the shell on a 10.4 machine you get the following error: "mach_port_allocate failed bus error".

NOTES:
1) building the shell using the xcode project on 10.5 DOES produce a shell that will execute on 10.4
2) building the shell on 10.4 using the cross-platform build works fine
3) building the shell on 10.4 using the cross-platform build works fine when it is run on 10.5

I have been building using the following basic build:

python ../configure.py --enable-shell
make clean
make

Comment 1

[Brent Baker]

Forgot to mention that this happens in both branches (tamarin-central and tamarin-tracing)

Comment 2

[Benjamin Smedberg]

You need to compile against the 10.4u SDK... I'm looking into exactly the right settings to make that happen correctly.

Comment 3

[Benjamin Smedberg]

If you're using gcc 4.x (the default for x86, but not for PPC), then it should be sufficient to

export CXX="g++ -isysroot /Developer/SDKs/MacOSX10.4u.sdk"

If you want to target PPC, you can either:

* use gcc 4.0, which will get you compatibility with MacOS 10.3.9+
* use gcc 3.3 and lots of hacked rules, see http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/configure.in&rev=1.1973&mark=801-827#774

Comment 4

[Brent Baker]

Looks like the builds are already default compiling against 10.4

This is from the build log:
http://tamarin-builds.mozilla.org/tamarin-central/builders/tamarin-mac/builds/132/steps/Build_Release/logs/stdio

-mmacosx-version-min=10.4 -isysroot /Developer/SDKs/MacOSX10.4u.sdk

Comment 5

[Chris Peyer]

Attachment: Add MACOSX_DEPLOYMENT_TARGET env var to prevent bus error on 10.4 (tamarin-central)

Comment 6

[Chris Peyer]

Attachment: Same patch for tamarin-tracing

Comment 7

[Chris Peyer]

Fixed w/ 519:76d65a6d16b4

Comment 8

[Chris Peyer]

Tamarin-Tracing fixed w/ 340:0cc94211610e

Comment 9

[Chris Peyer]

Resolved fixed engineering / work item that has been pushed.  Setting status to
verified.