Two more win32 fixes: don't have SConfTests.py look for sin(), since it's available...

git-svn-id: http://scons.tigris.org/svn/scons/trunk@998 fdb21ef1-2011-0410-befe-b5e4ea1792b1

diff --git a/src/engine/SCons/SConfTests.py b/src/engine/SCons/SConfTests.py
index 39af3a18d12537612e640a6d40be45bc2a9198a3..9b3f2a6246e05d31c3b84f81ddcf16d22a4b7c14 100644 (file)
--- a/src/engine/SCons/SConfTests.py
+++ b/src/engine/SCons/SConfTests.py
@@ -342,8 +342,16 @@ int main() {
             assert r, "did not find %s" % existing_lib
             r = sconf.CheckLib( ["hopefullynolib"], "main", autoadd=0 )
             assert not r, "unexpectedly found hopefullynolib"
-            r = sconf.CheckLib( [], "sin", autoadd=0 )
-            assert not r, "unexpectedly found nonexistent library"
+            # This is a check that a null list doesn't find functions
+            # that are in libraries that must be explicitly named.
+            # This works on POSIX systems where you have to -lm to
+            # get the math functions, but it fails on Visual Studio
+            # where you apparently get all those functions for free.
+            # Comment out this check until someone who understands
+            # Visual Studio better can come up with a corresponding
+            # test (if that ever really becomes necessary).
+            #r = sconf.CheckLib( [], "sin", autoadd=0 )
+            #assert not r, "unexpectedly found nonexistent library"
             r = sconf.CheckLib( [existing_lib,"hopefullynolib"], "main", autoadd=0 )
             assert r, "did not find %s,%s " % (existing_lib,r)
             r = sconf.CheckLib( ["hopefullynolib",existing_lib], "main", autoadd=0 )
@@ -396,8 +404,16 @@ int main() {
             assert r, "did not find %s" % existing_lib
             r = sconf.CheckLibWithHeader( [existing_lib], ["stdio.h", "math.h"], "C", autoadd=0 )
             assert r, "did not find %s, #include stdio.h first" % existing_lib
-            r = sconf.CheckLibWithHeader( [], "math.h", "C", call="sin(3);", autoadd=0 )
-            assert not r, "unexpectedly found non-existent library"
+            # This is a check that a null list doesn't find functions
+            # that are in libraries that must be explicitly named.
+            # This works on POSIX systems where you have to -lm to
+            # get the math functions, but it fails on Visual Studio
+            # where you apparently get all those functions for free.
+            # Comment out this check until someone who understands
+            # Visual Studio better can come up with a corresponding
+            # test (if that ever really becomes necessary).
+            #r = sconf.CheckLibWithHeader( [], "math.h", "C", call="sin(3);", autoadd=0 )
+            #assert not r, "unexpectedly found non-existent library"
             r = sconf.CheckLibWithHeader( ["hopefullynolib"], "math.h", "C", autoadd=0 )
             assert not r, "unexpectedly found hopefullynolib"
             r = sconf.CheckLibWithHeader( ["hopefullynolib",existing_lib], ["stdio.h", "math.h"], "C", autoadd=0 )

diff --git a/src/engine/SCons/Tool/mingw.py b/src/engine/SCons/Tool/mingw.py
index 22df3d4ad52776a80b3090c5806b07963d1604ec..debb873d5fd64c0796f6da2492976dd22608948e 100644 (file)
--- a/src/engine/SCons/Tool/mingw.py
+++ b/src/engine/SCons/Tool/mingw.py
@@ -94,7 +94,9 @@ def shlib_emitter(target, source, env):
 
 shlib_action = SCons.Action.CommandGenerator(shlib_generator)
 
-res_builder = SCons.Builder.Builder(action='$RCCOM', suffix='.o')
+res_builder = SCons.Builder.Builder(action='$RCCOM', suffix='.o',
+                                    source_scanner=SCons.Defaults.ObjSourceScan)
+SCons.Defaults.ObjSourceScan.add_scanner('.rc', SCons.Defaults.CScan)
 
 def generate(env):
     mingw = find(env)
@@ -137,7 +139,6 @@ def generate(env):
     env['RCINCPREFIX'] = '--include-dir '
     env['RCINCSUFFIX'] = ''
     env['RCCOM'] = '$RC $RCINCFLAGS $RCFLAGS -i $SOURCE -o $TARGET'
-    SCons.Defaults.ObjSourceScan.add_scanner('.rc', SCons.Defaults.CScan)
     env['BUILDERS']['RES'] = res_builder
     
     # Some setting from the platform also have to be overridden:

diff --git a/src/engine/SCons/Tool/msvc.py b/src/engine/SCons/Tool/msvc.py
index 3280eb5d5578f4323e2a97afe81a9f73764744ff..a32c503731d8eb1cb9f824637f1bc6bc75ed12b0 100644 (file)
--- a/src/engine/SCons/Tool/msvc.py
+++ b/src/engine/SCons/Tool/msvc.py
@@ -398,7 +398,9 @@ def shared_object_emitter(target, source, env):
                           SCons.Defaults.SharedObjectEmitter)
 
 pch_builder = SCons.Builder.Builder(action='$PCHCOM', suffix='.pch', emitter=pch_emitter)
-res_builder = SCons.Builder.Builder(action='$RCCOM', suffix='.res')
+res_builder = SCons.Builder.Builder(action='$RCCOM', suffix='.res',
+                                    source_scanner=SCons.Defaults.ObjSourceScan)
+SCons.Defaults.ObjSourceScan.add_scanner('.rc', SCons.Defaults.CScan)
 
 def generate(env):
     """Add Builders and construction variables for MSVC++ to an Environment."""
@@ -438,7 +440,6 @@ def generate(env):
     env['RC'] = 'rc'
     env['RCFLAGS'] = SCons.Util.CLVar('')
     env['RCCOM'] = '$RC $_CPPDEFFLAGS $_CPPINCFLAGS $RCFLAGS /fo$TARGET $SOURCES'
-    SCons.Defaults.ObjSourceScan.add_scanner('.rc', SCons.Defaults.CScan)
     env['BUILDERS']['RES'] = res_builder
 
     try: