]> granicus.if.org Git - python/commitdiff
After discussion on the PythonMac-SIG it was decided that it is better
authorJack Jansen <jack.jansen@cwi.nl>
Sun, 26 Dec 2004 23:07:48 +0000 (23:07 +0000)
committerJack Jansen <jack.jansen@cwi.nl>
Sun, 26 Dec 2004 23:07:48 +0000 (23:07 +0000)
to make using "-undefined dynamic_lookup" for linking extensions more
automatic on 10.3 and later. So if we're on that platform and
MACOSX_DEPLOYMENT_TARGET is not set we now set it to the current OSX
version during configure. Additionally, distutils will pick up the
configure-time value by default.

Will backport.

Lib/distutils/sysconfig.py
configure
configure.in

index 8986dc9857266394ee008cc085fcacda67c2b8ef..d4eb368e9f379c6e4a321ab3e38b0f7552085106 100644 (file)
@@ -360,11 +360,13 @@ def _init_posix():
     # On MacOSX we need to check the setting of the environment variable
     # MACOSX_DEPLOYMENT_TARGET: configure bases some choices on it so
     # it needs to be compatible.
-    # An alternative would be to force MACOSX_DEPLOYMENT_TARGET to be
-    # the same as during configure.
+    # If it isn't set we set it to the configure-time value
     if sys.platform == 'darwin' and g.has_key('CONFIGURE_MACOSX_DEPLOYMENT_TARGET'):
         cfg_target = g['CONFIGURE_MACOSX_DEPLOYMENT_TARGET']
         cur_target = os.getenv('MACOSX_DEPLOYMENT_TARGET', '')
+        if cur_target == '':
+            cur_target = cfg_target
+            os.putenv('MACOSX_DEPLOYMENT_TARGET', cfg_target)
         if cfg_target != cur_target:
             my_msg = ('$MACOSX_DEPLOYMENT_TARGET mismatch: now "%s" but "%s" during configure'
                 % (cur_target, cfg_target))
index 1d538e45e63c4887bf446753b045337f5acb227d..ca9134fbae2758f923b887ba0ce43450e3b363fe 100755 (executable)
--- a/configure
+++ b/configure
@@ -1,5 +1,5 @@
 #! /bin/sh
-# From configure.in Revision: 1.475 .
+# From configure.in Revision: 1.477 .
 # Guess values for system-dependent variables and create Makefiles.
 # Generated by GNU Autoconf 2.59 for python 2.5.
 #
@@ -10322,11 +10322,12 @@ then
        Darwin/*)
                # Use -undefined dynamic_lookup whenever possible (10.3 and later).
                # This allows an extension to be used in any Python
-               if test ${MACOSX_DEPLOYMENT_TARGET-10.1} '>' 10.2
+               cur_target=`sw_vers -productVersion | sed 's/\(10\.[0-9]*\).*/\1/'`
+               if test ${MACOSX_DEPLOYMENT_TARGET-${cur_target}} '>' 10.2
                then
                        LDSHARED='$(CC) $(LDFLAGS) -bundle -undefined dynamic_lookup'
                        BLDSHARED="$LDSHARED"
-                       CONFIGURE_MACOSX_DEPLOYMENT_TARGET=$MACOSX_DEPLOYMENT_TARGET
+                       CONFIGURE_MACOSX_DEPLOYMENT_TARGET=${MACOSX_DEPLOYMENT_TARGET-${cur_target}}
                else
                        LDSHARED='$(CC) $(LDFLAGS) -bundle'
                        if test "$enable_framework" ; then
index 68720e798c498a47d9d77fbc795420e24513bb87..9371a577c80439965972116d5ac640e61373b44c 100644 (file)
@@ -1324,11 +1324,12 @@ then
        Darwin/*)
                # Use -undefined dynamic_lookup whenever possible (10.3 and later).
                # This allows an extension to be used in any Python
-               if test ${MACOSX_DEPLOYMENT_TARGET-10.1} '>' 10.2
+               cur_target=`sw_vers -productVersion | sed 's/\(10\.[[0-9]]*\).*/\1/'`
+               if test ${MACOSX_DEPLOYMENT_TARGET-${cur_target}} '>' 10.2
                then
                        LDSHARED='$(CC) $(LDFLAGS) -bundle -undefined dynamic_lookup'
                        BLDSHARED="$LDSHARED"
-                       CONFIGURE_MACOSX_DEPLOYMENT_TARGET=$MACOSX_DEPLOYMENT_TARGET
+                       CONFIGURE_MACOSX_DEPLOYMENT_TARGET=${MACOSX_DEPLOYMENT_TARGET-${cur_target}}
                else
                        LDSHARED='$(CC) $(LDFLAGS) -bundle'
                        if test "$enable_framework" ; then