Bug 1186064 - Require Visual Studio 2015 Update 2; r?glandium
Per froydnj in
bug 1186064 comment #23, "it makes sense to proceed with removing
MSVC 2013 support." This commit does that.
We also go a step further and require VS2015 Update 2 instead of just
update 1. This temporarily brings us down to just 1 officially supported
Visual Studio version. However, VS2015u3 was just released and is
unofficially supported.
Since MOZ_CRT is no longer set, references to it have been removed.
MozReview-Commit-ID: 8MUR6qLzQA5
--- a/build/gecko_templates.mozbuild
+++ b/build/gecko_templates.mozbuild
@@ -50,37 +50,29 @@ def GeckoBinary(linkage='dependent', msv
USE_LIBS += [
xpcomglue,
]
elif linkage != None:
error('`linkage` must be "dependent", "standalone" or None')
if mozglue:
- if CONFIG['MOZ_CRT']:
- if msvcrt == 'dynamic':
- USE_LIBS += ['mozcrt']
- elif msvcrt == 'static':
- USE_LIBS += ['mozglue']
- else:
- error('`msvcrt` must be "dynamic" or "static"')
- else:
- LDFLAGS += CONFIG['MOZ_GLUE_WRAP_LDFLAGS']
- if mozglue == 'program':
+ LDFLAGS += CONFIG['MOZ_GLUE_WRAP_LDFLAGS']
+ if mozglue == 'program':
+ USE_LIBS += ['mozglue']
+ if CONFIG['MOZ_GLUE_IN_PROGRAM']:
+ if CONFIG['GNU_CC']:
+ LDFLAGS += ['-rdynamic']
+ if CONFIG['MOZ_MEMORY']:
+ USE_LIBS += ['memory']
+ elif mozglue == 'library':
+ if not CONFIG['MOZ_GLUE_IN_PROGRAM']:
USE_LIBS += ['mozglue']
- if CONFIG['MOZ_GLUE_IN_PROGRAM']:
- if CONFIG['GNU_CC']:
- LDFLAGS += ['-rdynamic']
- if CONFIG['MOZ_MEMORY']:
- USE_LIBS += ['memory']
- elif mozglue == 'library':
- if not CONFIG['MOZ_GLUE_IN_PROGRAM']:
- USE_LIBS += ['mozglue']
- else:
- error('`mozglue` must be "program" or "library"')
+ else:
+ error('`mozglue` must be "program" or "library"')
if not CONFIG['JS_STANDALONE']:
USE_LIBS += [
'fallible',
]
@template
--- a/build/moz.configure/toolchain.configure
+++ b/build/moz.configure/toolchain.configure
@@ -536,23 +536,22 @@ def compiler(language, host_or_target, c
# If you want to bump the version check here search for
# __cpp_static_assert above, and see the associated comment.
if info.type == 'clang' and not info.version:
raise FatalCheckError(
'Only clang/llvm 3.4 or newer is supported.')
if info.type == 'msvc':
- if info.version < '18.00.30723' or (
- '19' < info.version < '19.00.23506'):
+ if info.version < '19.00.23918':
raise FatalCheckError(
'This version (%s) of the MSVC compiler is not '
'supported.\n'
- 'You must install Visual C++ 2013 Update 3, Visual '
- 'C++ 2015 Update 1, or newer in order to build.\n'
+ 'You must install Visual C++ 2015 Update 2 or newer in '
+ 'order to build.\n'
'See https://developer.mozilla.org/en/'
'Windows_Build_Prerequisites' % info.version)
return namespace(
wrapper=wrapper,
compiler=compiler,
flags=flags,
type=info.type,
deleted file mode 100644
--- a/build/win32/mozconfig.vs2013-win64
+++ /dev/null
@@ -1,23 +0,0 @@
-_VSPATH="/c/tools/vs2013"
-export WIN32_REDIST_DIR=${_VSPATH}/VC/redist/x86/Microsoft.VC120.CRT
-
-## includes: win8.1 sdk includes, msvc std library, directx sdk for d3d9 ##
-export INCLUDE=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/shared:/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/um:/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/winrt:${_VSPATH}/vc/include:${_VSPATH}/vc/atlmfc/include:/c/tools/sdks/dx10/include
-
-## libs: win8.1 sdk x86 (32-bit) libs, msvc (32-bit) std library, msvc atl libs, directx sdk (32-bit) for d3d9 ##
-export LIBPATH=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/Lib/winv6.3/um/x86:${_VSPATH}/vc/lib:${_VSPATH}/vc/atlmfc/lib:/c/tools/sdks/dx10/lib
-export LIB=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/Lib/winv6.3/um/x86:${_VSPATH}/vc/lib:${_VSPATH}/vc/atlmfc/lib:/c/tools/sdks/dx10/lib
-
-## paths: win8.1 sdk x86 (32-bit) tools, msvc (64-bit compiling 32-bit) build toolchain, moz tools ##
-export PATH="/c/Program Files (x86)/Windows Kits/8.1/bin/x86:${_VSPATH}/Common7/IDE:${_VSPATH}/VC/BIN/amd64_x86:${_VSPATH}/VC/BIN/amd64:${_VSPATH}/Common7/Tools:${_VSPATH}/VC/VCPackages:/c/mozilla-build/moztools:${PATH}"
-
-## WindowsSDKDir ##
-export WINDOWSSDKDIR="/c/Program Files (x86)/Windows Kits/8.1/"
-
-. $topsrcdir/build/mozconfig.vs-common
-
-mk_export_correct_style LIB
-mk_export_correct_style LIBPATH
-mk_export_correct_style PATH
-mk_export_correct_style INCLUDE
-mk_export_correct_style WIN32_REDIST_DIR
deleted file mode 100644
--- a/build/win64/mozconfig.vs2013
+++ /dev/null
@@ -1,23 +0,0 @@
-_VSPATH="/c/tools/vs2013"
-export WIN32_REDIST_DIR=${_VSPATH}/VC/redist/x64/Microsoft.VC120.CRT
-
-## includes: win8.1 sdk includes, msvc std library, directx sdk for d3d9 ##
-export INCLUDE=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/shared:/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/um:/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/include/winrt:${_VSPATH}/vc/include:${_VSPATH}/vc/atlmfc/include:/c/tools/sdks/dx10/include
-
-## libs: win8.1 sdk x64 (64-bit) libs, msvc (64-bit) std library, msvc atl libs, directx sdk (64-bit) for d3d9 ##
-export LIBPATH=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/Lib/winv6.3/um/x64:${_VSPATH}/vc/lib/amd64:${_VSPATH}/vc/atlmfc/lib/amd64:/c/tools/sdks/dx10/lib/x64
-export LIB=/c/Program\ Files\ \(x86\)/Windows\ Kits/8.1/Lib/winv6.3/um/x64:${_VSPATH}/vc/lib/amd64:${_VSPATH}/vc/atlmfc/lib/amd64:/c/tools/sdks/dx10/lib/x64
-
-## paths: win8.1 sdk x64 (64-bit) tools, msvc (64-bit) build toolchain, moz tools ##
-export PATH="/c/Program Files (x86)/Windows Kits/8.1/bin/x64:${_VSPATH}/Common7/IDE:${_VSPATH}/VC/BIN/amd64:${_VSPATH}/VC/BIN/x86_amd64:${_VSPATH}/VC/BIN:${_VSPATH}/Common7/Tools:${_VSPATH}/VC/VCPackages:${PATH}"
-
-## WindowsSDKDir ##
-export WINDOWSSDKDIR="/c/Program Files (x86)/Windows Kits/8.1/"
-
-. $topsrcdir/build/mozconfig.vs-common
-
-mk_export_correct_style LIB
-mk_export_correct_style LIBPATH
-mk_export_correct_style PATH
-mk_export_correct_style INCLUDE
-mk_export_correct_style WIN32_REDIST_DIR
--- a/config/external/nss/Makefile.in
+++ b/config/external/nss/Makefile.in
@@ -362,22 +362,17 @@ include $(topsrcdir)/config/rules.mk
ifeq (1,$(ALLOW_COMPILER_WARNINGS))
DEFAULT_GMAKE_FLAGS += NSS_ENABLE_WERROR=0
endif
# Can't pass this in DEFAULT_GMAKE_FLAGS because that overrides
# definitions in NSS, so just export it into the sub-make's environment.
ifeq (WINNT_1,$(OS_TARGET)_$(MOZ_MEMORY))
-ifdef MOZ_CRT
-# OS_LIBS comes from having mozcrt as a dependency in moz.build.
-DLLFLAGS := $(OS_LIBS)
-else
DLLFLAGS := -LIBPATH:$(ABS_DIST)/../mozglue/build -DEFAULTLIB:mozglue
-endif
export DLLFLAGS
endif
ifdef MOZ_FOLD_LIBS
# Force the linker to include everything from the static libraries.
EXPAND_LIBS_EXEC += --extract
$(SHARED_LIBRARY): $(addprefix $(DEPTH)/security/,$(NSS_STATIC_LIBS))
--- a/config/recurse.mk
+++ b/config/recurse.mk
@@ -163,19 +163,16 @@ toolkit/library/target: widget/gtk/mozgt
endif
ifdef MOZ_LDAP_XPCOM
ldap/target: config/external/nss/target mozglue/build/target
toolkit/library/target: ldap/target
endif
ifeq ($(MOZ_REPLACE_MALLOC_LINKAGE),dummy library)
mozglue/build/target memory/replace/logalloc/replay/target: memory/replace/dummy/target
endif
-ifdef MOZ_CRT
-mozglue/crt/target: mozglue/build/target
-endif
# js/src/target can end up invoking js/src/host rules (through object files
# depending on jsautokw.h, which depends on host_jskwgen, and that can't
# happen at the same time (bug #1146738)
js/src/target: js/src/host
endif
# Most things are built during compile (target/host), but some things happen during export
# Those need to depend on config/export for system wrappers.
$(addprefix build/unix/stdc++compat/,target host) build/clang-plugin/target: config/export
--- a/js/src/make-source-package.sh
+++ b/js/src/make-source-package.sh
@@ -129,17 +129,16 @@ case $cmd in
${MKDIR} -p ${tgtpath}/toolkit/mozapps/installer
cp -pPR \
${TOPSRCDIR}/toolkit/mozapps/installer/package-name.mk \
${TOPSRCDIR}/toolkit/mozapps/installer/upload-files.mk \
${tgtpath}/toolkit/mozapps/installer
${MKDIR} -p ${tgtpath}/mozglue
cp -pPR \
${TOPSRCDIR}/mozglue/build \
- ${TOPSRCDIR}/mozglue/crt \
${TOPSRCDIR}/mozglue/misc \
${TOPSRCDIR}/mozglue/moz.build \
${tgtpath}/mozglue
${MKDIR} -p ${tgtpath}/memory
cp -pPR \
${TOPSRCDIR}/memory/moz.build \
${TOPSRCDIR}/memory/build \
${TOPSRCDIR}/memory/fallible \
--- a/js/src/old-configure.in
+++ b/js/src/old-configure.in
@@ -210,22 +210,16 @@ case "$target" in
_MSC_VER=`echo ${CC_VERSION} | cut -c 1-2,4-5`
AC_DEFINE(_CRT_SECURE_NO_WARNINGS)
AC_DEFINE(_CRT_NONSTDC_NO_WARNINGS)
AC_DEFINE(_USE_MATH_DEFINES) # Otherwise MSVC's math.h doesn't #define M_PI.
case "$CC_VERSION" in
- 18*)
- _CC_SUITE=12
- MSVS_VERSION=2013
- MSVC_C_RUNTIME_DLL=msvcr120.dll
- MSVC_CXX_RUNTIME_DLL=msvcp120.dll
- ;;
19*)
_CC_SUITE=14
MSVS_VERSION=2015
MSVC_C_RUNTIME_DLL=vcruntime140.dll
MSVC_CXX_RUNTIME_DLL=msvcp140.dll
# C5026: move constructor was implicitly defined as deleted
CXXFLAGS="$CXXFLAGS -wd5026"
@@ -1910,17 +1904,16 @@ if test "$MOZ_MEMORY"; then
dnl complicated. Therefore, simply define special cpp variables for the
dnl platforms we have special knowledge of.
case "${target}" in
*-mingw*)
export MOZ_NO_DEBUG_RTL=1
;;
esac
fi
-AC_SUBST(MOZ_CRT)
AC_SUBST(MOZ_GLUE_IN_PROGRAM)
AC_SUBST_LIST(MOZ_GLUE_WRAP_LDFLAGS)
dnl ========================================================
dnl = Use a smaller chunk size for GC chunks
dnl ========================================================
dnl Use large (1MB) chunks by default. For B2G this option is used to give
dnl smaller (currently 256K) chunks.
--- a/mozglue/build/moz.build
+++ b/mozglue/build/moz.build
@@ -6,18 +6,17 @@
# Build mozglue as a shared lib on Windows, OSX and Android.
# If this is ever changed, update MOZ_SHARED_MOZGLUE in browser/installer/Makefile.in
if CONFIG['OS_TARGET'] in ('WINNT', 'Darwin', 'Android'):
SharedLibrary('mozglue')
else:
Library('mozglue')
-if not CONFIG['MOZ_CRT']:
- SDK_LIBRARY = True
+SDK_LIBRARY = True
if CONFIG['OS_TARGET'] == 'Android':
SOURCES += [
'BionicGlue.cpp',
]
if CONFIG['MOZ_ASAN']:
SOURCES += [
deleted file mode 100644
--- a/mozglue/crt/Makefile.in
+++ /dev/null
@@ -1,89 +0,0 @@
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-# Roll our own custom logic here for the import library
-
-###############################################################################
-#
-# Linking Mozilla itself to jemalloc is not particularly difficult. To do this
-# we avoid linking directly to the Microsoft-provided CRT import libraries.
-# Instead, we link to our own import library which we generate here. To
-# replace the CRT's malloc/free/other memory management symbols we export
-# our own versions out of jemalloc.dll. We then take the import library that
-# the compiler generates for jemalloc.dll and combine it with the MS CRT import
-# libraries. We put our library on the command line first, and the CRT symbols
-# are discarded in favor of our versions!
-#
-# Unfortunately that was too easy. The CRT import library is not a standard
-# import library that contains a list of symbols and whatnot. It also includes
-# object files that are linked into generated programs. One of these,
-# crtdll.obj is (as one might expect) linked into all DLLs that link against
-# the CRT. This file does things like run static C++ constructors when the
-# DLL is attached, call DllMain, etc.
-#
-# In the CRT source all malloc/free calls are made to malloc_crt and free_crt.
-# In debug builds these are both defined to malloc_dbg and free_dbg. In opt
-# builds malloc_crt is an actual function, implemented and exposed from the
-# CRT. free_crt is, however, defined to be just plain old free. This works
-# fine inside the CRT where malloc_crt and free operate on the same heap.
-# Outside the CRT malloc_crt is in the CRT's heap, but free is in jemalloc's
-# heap. This causes much pain at shutdown :-(
-#
-# The obvious solution here is to override malloc_crt too. Unfortunately,
-# that doesn't work because the CRT expects to be able to call msize on this
-# piece of memory deep inside the CRT, which will fail because it'll call the
-# CRT's msize on a pointer in jemalloc's heap.
-#
-# Our solution to this is quite devious. We take apart the CRT's import lib
-# and remove the problematic object file. We then poke at the object file's
-# symbol table and replace '__imp__free' (which means grab free from some
-# other DLL) with '__imp__frex'. Then we define our own dummy no-op function
-# in jemalloc.dll and export it as frex. Then we put the CRT import lib
-# back together with the patched crtdll.obj, glue it to the end of jemalloc's
-# import library and link the rest of Mozilla to that.
-#
-# The result? A binary that uses jemalloc, doesn't crash, and leaks a tiny
-# amount of memory (32 words per DLL in the 2010 CRT) at shutdown.
-#
-###############################################################################
-
-target:: mozcrt.lib
- $(INSTALL) $(IFLAGS2) mozcrt.lib $(DIST)/lib
-
-# And finally combine that with the jemalloc import library to get an import
-# library that has our malloc/free/etc and the CRT's everything else
-mozcrt.lib: ../build/mozglue.lib msvc_modified.lib
- lib -OUT:$@ $^
-
-# Put the fixed object file back in
-msvc_modified.lib: msvc_removed.lib crtdll_fixed.obj
- lib -OUT:$@ $^
-
-# Fix the object file
-crtdll_fixed.obj: crtdll.obj
- $(PYTHON) $(srcdir)/fixcrt.py
-
-# Find the path of crtdll.obj
-CRTDLL_FULLPATH=$(subst \,\\,$(shell lib -list msvc_combined.lib | grep crtdll\\.obj))
-
-# Remove the broken object file, only after we have extracted it
-msvc_removed.lib: msvc_combined.lib crtdll.obj
- lib -OUT:$@ msvc_combined.lib -REMOVE:$(CRTDLL_FULLPATH)
-
-# Extract the broken object file out of the combined library
-crtdll.obj: msvc_combined.lib
- lib -OUT:$@ $^ -EXTRACT:$(CRTDLL_FULLPATH)
-
-# Grab both CRT libraries and combine them into one library to simplify things
-msvc_combined.lib:
- lib -OUT:$@ $(WIN32_CRT_LIBS)
-
-# Normally, we'd use SDK_LIBRARY, but we can't because all the tricks above
-# involve *not* defining the library in moz.build, so SDK_LIBRARY = True would
-# not have the expected outcome.
-SDK_FILES = mozcrt.lib
-SDK_DEST = $(SDK_LIB_DIR)
-SDK_TARGET = target
-INSTALL_TARGETS += SDK
deleted file mode 100644
--- a/mozglue/crt/fixcrt.py
+++ /dev/null
@@ -1,14 +0,0 @@
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-from __future__ import with_statement
-
-with open('crtdll.obj', 'rb') as infile:
- data = infile.read()
- with open('crtdll_fixed.obj', 'wb') as outfile:
- # for Win32
- data = data.replace('__imp__free', '__imp__frex')
- # for Win64
- data = data.replace('__imp_free', '__imp_frex')
- outfile.write(data)
deleted file mode 100644
--- a/mozglue/crt/moz.build
+++ /dev/null
@@ -1,16 +0,0 @@
-# -*- Mode: python; c-basic-offset: 4; indent-tabs-mode: nil; tab-width: 40 -*-
-# vim: set filetype=python:
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-
-Library('mozcrt')
-
-STATIC_LIBRARY_NAME = 'mozcrt_s'
-
-OS_LIBS += [
- '-NODEFAULTLIB:msvcrt',
- '-NODEFAULTLIB:msvcprt',
- '-LIBPATH:%s/mozglue/crt' % TOPOBJDIR,
- '-DEFAULTLIB:mozcrt',
-]
--- a/mozglue/moz.build
+++ b/mozglue/moz.build
@@ -10,13 +10,10 @@ if CONFIG['MOZ_LINKER']:
if CONFIG['MOZ_WIDGET_TOOLKIT'] == 'android':
DIRS += ['android']
DIRS += [
'build',
'misc',
]
-if CONFIG['MOZ_CRT']:
- DIRS += ['crt']
-
if not CONFIG['JS_STANDALONE']:
TEST_DIRS += ['tests']
--- a/old-configure.in
+++ b/old-configure.in
@@ -316,22 +316,16 @@ case "$target" in
_MSC_VER=`echo ${CC_VERSION} | cut -c 1-2,4-5`
AC_DEFINE(_CRT_SECURE_NO_WARNINGS)
AC_DEFINE(_CRT_NONSTDC_NO_WARNINGS)
AC_DEFINE(_USE_MATH_DEFINES) # Otherwise MSVC's math.h doesn't #define M_PI.
case "$CC_VERSION" in
- 18*)
- _CC_SUITE=12
- MSVS_VERSION=2013
- MSVC_C_RUNTIME_DLL=msvcr120.dll
- MSVC_CXX_RUNTIME_DLL=msvcp120.dll
- ;;
19*)
_CC_SUITE=14
MSVS_VERSION=2015
MSVC_C_RUNTIME_DLL=vcruntime140.dll
MSVC_CXX_RUNTIME_DLL=msvcp140.dll
MOZ_CHECK_HEADER(dia2.h, MSVC_HAS_DIA_SDK=1)
if test -n "$MSVC_HAS_DIA_SDK"; then
@@ -5352,34 +5346,21 @@ else
AC_DEFINE(MOZ_MEMORY_DEBUG)
fi
dnl The generic feature tests that determine how to compute ncpus are long and
dnl complicated. Therefore, simply define special cpp variables for the
dnl platforms we have special knowledge of.
case "${target}" in
*-mingw*)
export MOZ_NO_DEBUG_RTL=1
- if test "$MSVS_VERSION" != "2015"; then
- WIN32_CRT_LIBS="msvcrt.lib msvcprt.lib"
- dnl Look for a broken crtdll.obj
- WIN32_CRTDLL_FULLPATH=`lib -nologo -list $WIN32_CRT_LIBS | grep crtdll\\.obj`
- lib -NOLOGO -OUT:crtdll.obj $WIN32_CRT_LIBS -EXTRACT:$WIN32_CRTDLL_FULLPATH
- if grep -q '__imp__\{0,1\}free' crtdll.obj; then
- MOZ_CRT=1
- fi
- rm crtdll.obj
- fi
;;
esac
fi # MOZ_MEMORY
AC_SUBST(MOZ_SYSTEM_JEMALLOC)
-AC_SUBST(MOZ_CRT)
-export MOZ_CRT
AC_SUBST(MOZ_GLUE_IN_PROGRAM)
-AC_SUBST_LIST(WIN32_CRT_LIBS)
# Allow the application to provide a subconfigure script.
# This should be after 'export MOZ_NO_DEBUG_RTL=1' since
# ldap/c-sdk/configure refers to the enviroment value.
if test -f "${srcdir}/${MOZ_BUILD_APP}/configure.in" ; then
do_output_subdirs() {
if test -n "$_subconfigure_subdirs"; then
AC_MSG_ERROR([Cannot specify more than one sub-sub-configure])
--- a/python/mozbuild/mozbuild/backend/visualstudio.py
+++ b/python/mozbuild/mozbuild/backend/visualstudio.py
@@ -33,27 +33,23 @@ from mozbuild.base import ExecutionSumma
MSBUILD_NAMESPACE = 'http://schemas.microsoft.com/developer/msbuild/2003'
def get_id(name):
return str(uuid.uuid5(uuid.NAMESPACE_URL, name)).upper()
def visual_studio_product_to_solution_version(version):
- if version == '2013':
- return '12.00', '12'
- elif version == '2015':
+ if version == '2015':
return '12.00', '14'
else:
raise Exception('Unknown version seen: %s' % version)
def visual_studio_product_to_platform_toolset_version(version):
- if version == '2013':
- return 'v120'
- elif version == '2015':
+ if version == '2015':
return 'v140'
else:
raise Exception('Unknown version seen: %s' % version)
class VisualStudioBackend(CommonBackend):
"""Generate Visual Studio project files.
This backend is used to produce Visual Studio projects and a solution
--- a/python/mozbuild/mozbuild/test/configure/test_toolchain_configure.py
+++ b/python/mozbuild/mozbuild/test/configure/test_toolchain_configure.py
@@ -119,16 +119,17 @@ def VS(version):
'*.cpp': DEFAULT_CXX_97,
})
VS_2013u2 = VS('18.00.30501')
VS_2013u3 = VS('18.00.30723')
VS_2015 = VS('19.00.23026')
VS_2015u1 = VS('19.00.23506')
+VS_2015u2 = VS('19.00.23918')
# Note: In reality, the -std=gnu* options are only supported when preceded by
# -Xclang.
CLANG_CL_3_9 = (CLANG_BASE('3.9.0') + VS('18.00.00000') + DEFAULT_C11 +
SUPPORTS_GNU99 + SUPPORTS_GNUXX11) + {
'*.cpp': {
'__STDC_VERSION__': False,
'__cplusplus': '201103L',
@@ -527,40 +528,41 @@ class WindowsToolchainTest(BaseToolchain
HOST = 'i686-pc-mingw32'
# For the purpose of this test, it doesn't matter that the paths are not
# real Windows paths.
PATHS = {
'/opt/VS_2013u2/bin/cl': VS_2013u2,
'/opt/VS_2013u3/bin/cl': VS_2013u3,
'/opt/VS_2015/bin/cl': VS_2015,
- '/usr/bin/cl': VS_2015u1,
+ '/opt/VS_2015u1/bin/cl': VS_2015u1,
+ '/usr/bin/cl': VS_2015u2,
'/usr/bin/clang-cl': CLANG_CL_3_9,
}
PATHS.update(LinuxToolchainTest.PATHS)
VS_2013u2_RESULT = (
'This version (18.00.30501) of the MSVC compiler is not supported.\n'
- 'You must install Visual C++ 2013 Update 3, Visual C++ 2015 Update 1, '
- 'or newer in order to build.\n'
+ 'You must install Visual C++ 2015 Update 2 or newer in order to build.\n'
'See https://developer.mozilla.org/en/Windows_Build_Prerequisites')
- VS_2013u3_RESULT = CompilerResult(
- flags=[],
- version='18.00.30723',
- type='msvc',
- compiler='/opt/VS_2013u3/bin/cl',
- )
+ VS_2013u3_RESULT = (
+ 'This version (18.00.30723) of the MSVC compiler is not supported.\n'
+ 'You must install Visual C++ 2015 Update 2 or newer in order to build.\n'
+ 'See https://developer.mozilla.org/en/Windows_Build_Prerequisites')
VS_2015_RESULT = (
'This version (19.00.23026) of the MSVC compiler is not supported.\n'
- 'You must install Visual C++ 2013 Update 3, Visual C++ 2015 Update 1, '
- 'or newer in order to build.\n'
+ 'You must install Visual C++ 2015 Update 2 or newer in order to build.\n'
'See https://developer.mozilla.org/en/Windows_Build_Prerequisites')
- VS_2015u1_RESULT = CompilerResult(
+ VS_2015u1_RESULT = (
+ 'This version (19.00.23506) of the MSVC compiler is not supported.\n'
+ 'You must install Visual C++ 2015 Update 2 or newer in order to build.\n'
+ 'See https://developer.mozilla.org/en/Windows_Build_Prerequisites')
+ VS_2015u2_RESULT = CompilerResult(
flags=[],
- version='19.00.23506',
+ version='19.00.23918',
type='msvc',
compiler='/usr/bin/cl',
)
CLANG_CL_3_9_RESULT = CompilerResult(
flags=['-Xclang', '-std=gnu99',
'-fms-compatibility-version=18.00.30723', '-fallback'],
version='18.00.30723',
type='clang-cl',
@@ -577,45 +579,48 @@ class WindowsToolchainTest(BaseToolchain
CLANG_3_6_RESULT = LinuxToolchainTest.CLANG_3_6_RESULT
CLANGXX_3_6_RESULT = LinuxToolchainTest.CLANGXX_3_6_RESULT
GCC_4_7_RESULT = LinuxToolchainTest.GCC_4_7_RESULT
GCC_4_9_RESULT = LinuxToolchainTest.GCC_4_9_RESULT
GXX_4_9_RESULT = LinuxToolchainTest.GXX_4_9_RESULT
GCC_5_RESULT = LinuxToolchainTest.GCC_5_RESULT
GXX_5_RESULT = LinuxToolchainTest.GXX_5_RESULT
+ # VS2015u2 or greater is required.
def test_msvc(self):
self.do_toolchain_test(self.PATHS, {
+ 'c_compiler': self.VS_2015u2_RESULT,
+ 'cxx_compiler': self.VS_2015u2_RESULT,
+ })
+
+ def test_unsupported_msvc(self):
+ self.do_toolchain_test(self.PATHS, {
'c_compiler': self.VS_2015u1_RESULT,
- 'cxx_compiler': self.VS_2015u1_RESULT,
+ }, environ={
+ 'CC': '/opt/VS_2015u1/bin/cl',
})
- def test_msvc_2013(self):
+ self.do_toolchain_test(self.PATHS, {
+ 'c_compiler': self.VS_2015_RESULT,
+ }, environ={
+ 'CC': '/opt/VS_2015/bin/cl',
+ })
+
self.do_toolchain_test(self.PATHS, {
'c_compiler': self.VS_2013u3_RESULT,
- 'cxx_compiler': self.VS_2013u3_RESULT,
}, environ={
'CC': '/opt/VS_2013u3/bin/cl',
})
- def test_unsupported_msvc(self):
- # While 2013 is supported, update 3 or higher is needed.
self.do_toolchain_test(self.PATHS, {
'c_compiler': self.VS_2013u2_RESULT,
}, environ={
'CC': '/opt/VS_2013u2/bin/cl',
})
- # Likewise with 2015, update 1 or higher is needed.
- self.do_toolchain_test(self.PATHS, {
- 'c_compiler': self.VS_2015_RESULT,
- }, environ={
- 'CC': '/opt/VS_2015/bin/cl',
- })
-
def test_clang_cl(self):
# We'll pick clang-cl if msvc can't be found.
paths = {
k: v for k, v in self.PATHS.iteritems()
if os.path.basename(k) != 'cl'
}
self.do_toolchain_test(paths, {
'c_compiler': self.CLANG_CL_3_9_RESULT,