Merge pull request #4174 from jcbrill/jbrill-msvc-batchargs

MSVC enhancement to add all remaining msvc batch file command-line options as SCons variables

1 files changed, 54 insertions, 8 deletions

diff --git a/RELEASE.txt b/RELEASE.txt
index b7e5ceb..d3fc5fd 100755
--- a/RELEASE.txt
+++ b/RELEASE.txt
@@ -23,19 +23,34 @@ NEW FUNCTIONALITY
   variables). This allows the user to customize how (for example) PATH is constructed.
   Note that these are called for every build command run by SCons. It could have considerable
   performance impact if not used carefully.
-- Added MSVC_USE_SETTINGS variable to pass a dictionary to configure the msvc compiler
+- Added MSVC_USE_SETTINGS construction variable to pass a dictionary to configure the msvc compiler
   system environment as an alternative to bypassing Visual Studio autodetection entirely.
+- Added MSVC_SDK_VERSION construction variable which allows building with a specific Microsoft
+  SDK version. This variable is used with the msvc batch file determined via autodetection. Refer
+  to the documentation for additional requirements and validation details.
+- Added MSVC_TOOLSET_VERSION construction variable which allows building with a specific toolset
+  version. This variable is used with the msvc batch file determined via autodetection.  This
+  variable does not affect the autodetection and selection of msvc instances. The toolset version
+  is applied after an msvc instance is selected. This could be the default version of msvc. Refer
+  to the documentation for additional requirements and validation details.  Addresses issue #3265,
+  issue #3664, and pull request #4149.
+- Added MSVC_SPECTRE_LIBS construction variable which allows building with spectre-mitigated
+  Visual C++ libraries. This variable is used with the msvc batch file determined via autodetection.
+  Refer to the documentation for additional requirements and validation details.
+- Added MSVC_SCRIPT_ARGS construction variable which specifies command line arguments that are
+  passed to the msvc batch file determined via autodetection.  Refer to the documentation for
+  additional requirements and validation details.  Addresses enhancement issue #4106.
 - Ninja: Added new alias "shutdown-ninja-scons-daemon" to allow ninja to shutdown the daemon.
   Also added cleanup to test framework to kill ninja scons daemons and clean ip daemon logs.
   NOTE: Test for this requires python psutil module. It will be skipped if not present.
 - Ninja: Added command line variable NINJA_CMD_ARGS that allows to pass through ninja command line args.
   This can also be set in your Environment().
-- Added a global policy setting and an environment policy variable for specifying the action to
-  be taken when an msvc request cannot be satisfied. The available options are "error",
+- Added a global policy setting and an environment construction variable for specifying the
+  action to be taken when an msvc request cannot be satisfied. The available options are "error",
   "exception", "warning", "warn", "ignore", and "suppress".  The global policy variable may be
-  set and retrieved via the functions set_msvc_notfound_policy and get_msvc_notfound_policy,
-  respectively.  These two methods may be imported from SCons.Tool.MSCommon.  The environment
-  policy variable introduced is MSVC_NOTFOUND_POLICY.  When defined, the environment policy
+  set and retrieved via the functions msvc_set_notfound_policy and msvc_get_notfound_policy,
+  respectively.  These two methods may be imported from SCons.Tool.MSCommon. The environment
+  construction variable is MSVC_NOTFOUND_POLICY.  When defined, the environment construction
   variable overrides the global policy setting for a given environment. When the active policy
   is "error" or "exception", an MSVCVersionNotFound exception is raised.  When the active policy
   is "warning" or "warn", a VisualCMissingWarning warning is issued and the constructed
@@ -43,6 +58,26 @@ NEW FUNCTIONALITY
   is taken and the constructed environment is likely incomplete.  As implemented, the default
   global policy is "warning".  The ability to set the global policy via an SCons command-line
   option may be added in a future enhancement.
+- Added a global policy setting and an environment construction variable for specifying the
+  action to be taken when msvc script errors are detected. The available options are "error",
+  "exception", "warning", "warn", "ignore", and "suppress".  The global policy variable may be
+  set and retrieved via the functions msvc_set_scripterror_policy and msvc_get_scripterror_policy,
+  respectively.  These two methods may be imported from SCons.Tool.MSCommon. The environment
+  construction variable is MSVC_SCRIPTERROR_POLICY.  When defined, the environment construction
+  variable overrides the global policy setting for a given environment. When the active policy
+  is "error" or "exception", an MSVCScriptExecutionError exception is raised when msvc batch file
+  errors are detected.  When the active policy is "warning" or "warn", an MSVCScriptExecutionWarning
+  warning is issued when msvc batch file errors are detected. When the active policy is "ignore" or
+  "suppress", msvc batch error messages are suppressed.  As implemented, the default global policy
+  is "ignore".  The ability to set the global policy via an SCons command-line option may be added
+  in a future enhancement.
+- Added experimental function msvc_query_version_toolset to SCons.Tool.MSCommon. Given a version
+  specification, this function will return an msvc version and an msvc toolset version.  The msvc
+  toolset version may be None.  The msvc version and msvc toolset version can be used in the
+  environment construction variables MSVC_VERSION and MSVC_TOOLSET_VERSION, respectively.  The
+  version specification may be an msvc version or an msvc toolset version. This is a proxy for
+  using an msvc toolset version to select an msvc instance. This function may be removed when an
+  msvc toolset version is used during msvc instance selection.
 - Fortran: a new construction variable FORTRANCOMMONFLAGS is added which is
   applied to all Fortran dialects, to enable global (all-dialect) settings.
 - lex: two new construction variables are introduced (LEX_HEADER_ILE
@@ -102,8 +137,8 @@ CHANGED/ENHANCED EXISTING FUNCTIONALITY
   automatically executes ninja.
 - Add JavaScanner to include JAVACLASSPATH as a dependency when using the Java tool.
 - The build argument (i.e., x86) is no longer passed to the MSVC 6.0 to 7.1 batch
-  files.  This may improve the effectiveness of the internal msvc cache when using 
-  MSVC detection and when bypassing MSVC detection as the MSVC 6.0 to 7.1 batch files 
+  files.  This may improve the effectiveness of the internal msvc cache when using
+  MSVC detection and when bypassing MSVC detection as the MSVC 6.0 to 7.1 batch files
   do not expect any arguments.
 - Propagate the OS and windir environment variables from the system environment to the msvc
   environment.  The OS and windir environment variables are used in the MSVC 6.0 batch file
@@ -115,6 +150,10 @@ CHANGED/ENHANCED EXISTING FUNCTIONALITY
         require delayed expansion to be enabled which is currently not supported and is
         typically not enabled by default on the host system. The batch files may also require
         environment variables that are not included by default in the msvc environment.
+- An exception is raised when MSVC_UWP_APP is enabled for Visual Studio 2013 and earlier.
+  Previous behavior was to silently ignore MSVC_UWP_APP when enabled for Visual Studio 2013
+  and earlier. Refer to the documentation for additional requirements and validation details.
+  MSVC_UWP_APP was extended to accept True, False, and None in addition to '1' and '0'.
 - Ninja: added option "--skip-ninja-regen" to enable skipping regeneration of the ninja file
   if scons can determine the ninja file doesnot need to be regenerated, which will also
   skip restarting the scons daemon. Note this option is could result in incorrect rebuilds
@@ -171,6 +210,13 @@ FIXES
   error caused when bypassing MSVC detection by specifying the MSVC 7.0 batch file directly.
 - lex: Fixed an issue with the lex tool where file arguments specified to either "--header-file="
   or "--tables-file=" which included a space in the path to the file would be processed incorrectly
+- Modify the MSCommon logger configuration to be independent of the root logger. This fixes an issue
+  when multiple loggers are created and the MSCommon logger added computed fields to the root logger
+  that are not present in other logging instances.
+- Modify the MSVC_USE_SCRIPT_ARGS test fixture to disable the msvc cache. This fixes an issue where
+  the MSVC_USE_SCRIPT_ARGS test for success relied on a debug log message that was not produced when
+  the msvc cache file exists and the test keys are already in the cache as the msvc script invocation
+  was bypassed.
 - Suppress issuing a warning when there are no installed Visual Studio instances for the default
   tools configuration (issue #2813).  When msvc is the default compiler because there are no
   compilers installed, a build may fail due to the cl.exe command not being recognized. At