PythonLauncher universal build fails due to missing -arch and -sysroot

[ned-deily]

BPO	8461
Nosy	@ronaldoussoren, @ned-deily
Files	issue-python-launcher-arch.txt

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/ronaldoussoren'
closed_at = <Date 2010-04-20.20:54:04.354>
created_at = <Date 2010-04-19.13:43:41.424>
labels = ['OS-mac', 'build']
title = 'PythonLauncher universal build fails due to missing -arch and -sysroot'
updated_at = <Date 2010-04-20.20:54:04.352>
user = 'https://github.com/ned-deily'

bugs.python.org fields:

activity = <Date 2010-04-20.20:54:04.352>
actor = 'ronaldoussoren'
assignee = 'ronaldoussoren'
closed = True
closed_date = <Date 2010-04-20.20:54:04.354>
closer = 'ronaldoussoren'
components = ['macOS']
creation = <Date 2010-04-19.13:43:41.424>
creator = 'ned.deily'
dependencies = []
files = ['16987']
hgrepos = []
issue_num = 8461
keywords = []
message_count = 2.0
messages = ['103594', '103755']
nosy_count = 2.0
nosy_names = ['ronaldoussoren', 'ned.deily']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = 'compile error'
url = 'https://bugs.python.org/issue8461'
versions = ['Python 2.6', 'Python 3.1', 'Python 2.7', 'Python 3.2']

[ned-deily]

Fixes for bpo-8366 corrected build failures with universal builds on OS X due to changes in the settings of CFLAGS and BASECFLAGS, which had caused -arch values to be added to both CFLAGS and BASECFLAGS. The Mac Makefile for the PythonLauncher app needs to be changed as well, otherwise universal builds can now fail with errors like:

ld warning: in FileSettings.o, file is not of required architecture
ld warning: in MyAppDelegate.o, file is not of required architecture
ld warning: in MyDocument.o, file is not of required architecture
ld warning: in PreferencesWindowController.o, file is not of required architecture
ld warning: in doscript.o, file is not of required architecture
ld warning: in main.o, file is not of required architecture
Undefined symbols for architecture i386:
"_main", referenced from:
__start in crt1.o
ld: symbol(s) not found for architecture i386

The attached patch corrects the problem.

[ronaldoussoren]

I've fixed this issue in all branches and can now do clean builds.

BTW. Builds with the 10.4u SDK on OSX fail at the moment for 2.7 and 3.2, that's due to bpo-7724, I've attached a patch to that issue that should fix it (for the trunk, porting it to 3.2 should be trivial).