tbb 4.0 doesn't support non-default build_arch

[ryandesign (Ryan Carsten Schmidt)]

Judging by the Portfile, tbb is supposed to support i386. Could you attach a main.log file showing what problem you experienced?

[realmightycat@…]

main.log

[realmightycat@…]

Maybe I do something wrong.

Averich-iMac:Desktop anton$ sudo port install tbb
--->  Fetching archive for tbb
--->  Attempting to fetch tbb-4.0-20120613_0.darwin_12.i386.tbz2 from http://lil.fr.packages.macports.org/tbb
--->  Attempting to fetch tbb-4.0-20120613_0.darwin_12.i386.tbz2 from http://packages.macports.org/tbb
--->  Fetching distfiles for tbb
--->  Verifying checksum(s) for tbb
--->  Extracting tbb
--->  Applying patches to tbb
--->  Configuring tbb
--->  Building tbb
Error: org.macports.build for port tbb returned: command execution failed
Please see the log file for port tbb for details:
    /opt/local/var/macports/logs/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_devel_tbb/tbb/main.log
To report a bug, follow the instructions in the guide:
    http://guide.macports.org/#project.tickets
Error: Processing of port tbb failed

[jmroot (Joshua Root)]

It's certainly not passing any archflags to the compiler.

[ryandesign (Ryan Carsten Schmidt)]

It does add -m64 when building for 64-bit, but does not add -arch i386 or -m32 when building for 32-bit.

[ryandesign (Ryan Carsten Schmidt)]

Reading the file build/macos.inc in the source, it expects the arch variable to be one of "ia32", "intel64", "ppc32" pr "ppc64".

Reading the portfile, it sets the arch_flag variable to "intel32", "intel64", "ppc" or "ppc64".

So it appears the port is doing the right thing on 64-bit Intel and PowerPC systems, but the wrong thing on 32-bit Intel or PowerPC systems.

Ticket #23944 about PowerPC build problems with tbb remains open.

[ryandesign (Ryan Carsten Schmidt)]

I've fixed the arch_flag variable in r100563 and verified it allows me to build for i386 on a 64-bit Intel Mac with Mountain Lion. Haven't tested PowerPC yet but will leave that for the other ticket.