From: cournape <cournape@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Date: Mon, 23 Nov 2009 07:44:43 +0000 (+0000)
Subject: ENH: explicit check for supported arch per version when using batch file.
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=38180a35add4c00ed6432811b007c496d05c1c18;p=scons.git

ENH: explicit check for supported arch per version when using batch file.

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

diff --git a/src/engine/SCons/Tool/MSCommon/vc.py b/src/engine/SCons/Tool/MSCommon/vc.py
index 79567971..b7092387 100644
--- a/src/engine/SCons/Tool/MSCommon/vc.py
+++ b/src/engine/SCons/Tool/MSCommon/vc.py
@@ -50,6 +50,9 @@ class VisualCException(Exception):
 class UnsupportedVersion(VisualCException):
     pass
 
+class UnsupportedArch(VisualCException):
+    pass
+
 class MissingConfiguration(VisualCException):
     pass
 
@@ -119,6 +122,42 @@ _VCVER_TO_PRODUCT_DIR = {
             r'Microsoft\VisualStudio\6.0\Setup\Microsoft Visual C++\ProductDir']
 }
 
+def msvc_version_to_maj_min(msvc_version):
+    t = msvc_version.split(".")
+    if not len(t) == 2:
+        raise ValueError("Unrecognized version %s" % msvc_version)
+    try:
+        maj = int(t[0])
+        min = int(t[1])
+        return maj, min
+    except ValueError, e:
+        raise ValueError("Unrecognized version %s" % msvc_version)
+
+def is_host_target_supported(host_target, msvc_version):
+    """Return True if the given (host, target) tuple is supported given the
+    msvc version.
+
+    Parameters
+    ----------
+    host_target: tuple
+        tuple of (canonalized) host-target, e.g. ("x86", "amd64") for cross
+        compilation from 32 bits windows to 64 bits.
+    msvc_version: str
+        msvc version (major.minor, e.g. 10.0)
+
+    Note
+    ----
+    This only check whether a given version *may* support the given (host,
+    target), not that the toolchain is actually present on the machine.
+    """
+    # We assume that any Visual Studio version supports x86 as a target
+    if host_target != "x86":
+        maj, min = msvc_version_to_maj_min(msvc_version)
+        if maj < 8:
+            return False
+
+    return True
+
 def find_vc_pdir(msvc_version):
     """Try to find the product directory for the given
     version.
@@ -286,6 +325,10 @@ def msvc_setup_env(env):
     elif use_script:
         host_platform, target_platform = get_host_target(env)
         host_target = (host_platform, target_platform)
+        if not is_host_target_supported(host_target, version):
+            raise UnsupportedVersion(
+                    "host, target = %s not supported for MSVC version %s" %
+                    (host_target, version))
         arg = _HOST_TARGET_ARCH_TO_BAT_ARCH[host_target]
         debug('use_script 2 %s, args:%s\n' % (repr(script), arg))
         try: