[Mesa-dev] [PATCH] appveyor: Build with MSVC 2015.

[Brian Paul]

On 02/22/2018 09:29 AM, Jose Fonseca wrote:
> The MSVC version we (at VMware) primarily care about from now on is
> 2015.
> 
> We can drop support for building with 2013 in a future commit.  I'm not
> aware of significant changes in C99/C11 support from MSVC 2013 to 2015,
> but there's no point in continuing supporting old MSVC versions when
> nobody cares.
> 
> Testing done: https://ci.appveyor.com/project/jrfonseca/mesa/build/46
> ---
>   appveyor.yml | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/appveyor.yml b/appveyor.yml
> index 96eb1a67b3..bd33e2e955 100644
> --- a/appveyor.yml
> +++ b/appveyor.yml
> @@ -35,13 +35,13 @@ clone_depth: 100
>   
>   cache:
>   - win_flex_bison-2.5.9.zip
> -- llvm-3.3.1-msvc2013-mtd.7z
> +- llvm-3.3.1-msvc2015-mtd.7z
>   
> -os: Visual Studio 2013
> +os: Visual Studio 2015
>   
>   environment:
>     WINFLEXBISON_ARCHIVE: win_flex_bison-2.5.9.zip
> -  LLVM_ARCHIVE: llvm-3.3.1-msvc2013-mtd.7z
> +  LLVM_ARCHIVE: llvm-3.3.1-msvc2015-mtd.7z
>   
>   install:
>   # Check pip
> @@ -69,10 +69,10 @@ install:
>   - set LLVM=%CD%\llvm
>   
>   build_script:
> -- scons -j%NUMBER_OF_PROCESSORS% MSVC_VERSION=12.0 llvm=1
> +- scons -j%NUMBER_OF_PROCESSORS% MSVC_VERSION=14.0 llvm=1
>   
>   after_build:
> -- scons -j%NUMBER_OF_PROCESSORS% MSVC_VERSION=12.0 llvm=1 check
> +- scons -j%NUMBER_OF_PROCESSORS% MSVC_VERSION=14.0 llvm=1 check
>   
>   
>   # It's possible to setup notification here, as described in
> 

Looks OK to me.

Reviewed-by: Brian Paul <brianp at vmware.com>