Harry Henry Gebel: get extra compiler flags from the CFLAGS environment

variable.

(Is this really needed?  Can we drop it when the config file mechanism
allows users to set compiler flags in setup.cfg?)

diff --git a/Lib/distutils/command/build_ext.py b/Lib/distutils/command/build_ext.py
index deb3b130b9871f46d763de68499b167012040339..aa9ea0d034c22356d6e1b3fd45fc8c67a232c1ed 100644 (file)
--- a/Lib/distutils/command/build_ext.py
+++ b/Lib/distutils/command/build_ext.py
@@ -295,6 +295,14 @@ class build_ext (Command):
             macros = build_info.get ('macros')
             include_dirs = build_info.get ('include_dirs')
             extra_args = build_info.get ('extra_compile_args')
+            # honor CFLAGS enviroment variable
+            # XXX do we *really* need this? or is it just a hack until
+            # the user can put compiler flags in setup.cfg?
+            if os.environ.has_key('CFLAGS'):
+                if not extra_args:
+                    extra_args = []
+                extra_args = string.split(os.environ['CFLAGS']) + extra_args
+                
             objects = self.compiler.compile (sources,
                                              output_dir=self.build_temp,
                                              macros=macros,