Patch 983206: distutils obeys LDSHARED env var. Removed the code in

anthonybaxter · anthonybaxter · commit 22dcf66f849c · 2004-10-13T15:54:17.000Z
Python's own setup.py that did the same thing (and tested on Solaris,
where LDSHARED is needed...)
diff --git a/Lib/distutils/sysconfig.py b/Lib/distutils/sysconfig.py
@@ -153,6 +153,8 @@ def customize_compiler(compiler):
             cc = os.environ['CC']
         if os.environ.has_key('CXX'):
             cxx = os.environ['CXX']
+        if os.environ.has_key('LDSHARED'):
+            ldshared = os.environ['LDSHARED']
         if os.environ.has_key('CPP'):
             cpp = os.environ['CPP']
         else:
diff --git a/Misc/ACKS b/Misc/ACKS
@@ -499,6 +499,7 @@ Just van Rossum
 Hugo van Rossum
 Saskia van Rossum
 Donald Wallace Rouse II
+Liam Routt
 Audun S. Runde
 Jeff Rush
 Sam Rushing
diff --git a/Misc/NEWS b/Misc/NEWS
@@ -47,6 +47,9 @@ Extension modules
 Library
 -------
 
+- Patch 983206: distutils now obeys environment variable LDSHARED, if
+  it is set.
+
 - Added Peter Astrand's subprocess.py module.  See PEP 324 for details.
 
 - time.strptime() now properly escapes timezones and all other locale-specific
diff --git a/setup.py b/setup.py
@@ -167,15 +167,12 @@ def build_extensions(self):
         # those environment variables passed into the setup.py phase.  Here's
         # a small set of useful ones.
         compiler = os.environ.get('CC')
-        linker_so = os.environ.get('LDSHARED')
         args = {}
         # unfortunately, distutils doesn't let us provide separate C and C++
         # compilers
         if compiler is not None:
             (ccshared,opt,base) = sysconfig.get_config_vars('CCSHARED','OPT','BASECFLAGS')
             args['compiler_so'] = compiler + ' ' + opt + ' ' + ccshared + ' ' + base
-        if linker_so is not None:
-            args['linker_so'] = linker_so
         self.compiler.set_executables(**args)
 
         build_ext.build_extensions(self)
