[Bf-committers] Two issues with VS2013

["Sybren A. Stüvel"]

Hi list,

I know a few Blender devs via the Blender conferences, but since 
unfortunately not everybody is able to go there, let me introduce 
myself. My name is Sybren (also on IRC), and I'm a PhD student at 
Utrecht University; my subject is crowd animation.  If you want to know 
more, check http://wiki.blender.org/index.php/User:Sybren. In short, 
I've used Blender for a long time, and I'd like to become a Blender 
developer.

Due to work requirements I'm currently building Blender using Visual 
Studio 2013 on Windows 7. So I've found two issues with Blender & VS2013:

The first issue: With the latest checkout from Git 
(3361be9b6b00ff26977eef4f52172be65ec32032) VS2013 complains that 
extern/clew/include/clew.h redefines __SSE__ while it's also defined on 
the compiler CLI arguments. The attached patch fixes it, and doesn't 
seem to have any negative side-effects.

The second issue: I can't successfully create a Debug build. I use CMake 
to generate the solution:

cmake \
     -Wdev \
     $TOPDIR/blender \
     -G "Visual Studio 12" \
     -DCMAKE_CONFIGURATION_TYPES="Debug;Release;RelWithDebInfo" \
     -DCUDA_TOOLKIT_ROOT_DIR='"C:/Program Files/NVIDIA GPU Computing 
Toolkit/CUDA/v6.5"' \
     -DWITH_CYCLES=ON \
     -DWITH_CYCLES_CUDA_BINARIES=ON \
     -DCYCLES_CUDA_BINARIES_ARCH:STRING="sm_50;sm_52"

Performing a debug build fails with:

12>------ Build started: Project: blender, Configuration: Debug Win32 ------
12>  buildinfo.c
12>LINK : fatal error LNK1104: cannot open file 
'E:\ssd-workspace\blender-git\blender\..\lib\windows_vc12\opencollada\lib\opencollada\OpenCOLLADASaxFrameworkLoader_d.lib'

The full build log is at http://www.pasteall.org/54956

Hopefully someone can help me with this.

Kind regards,

-- 
Sybren A. Stüvel

http://stuvelfoto.nl/
http://stuvel.eu/

-------------- next part --------------
From 7760fe4556b8a336f7897179152c22f935428355 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?"Sybren=20A.=20St=C3=BCvel"?= <sybren at stuvel.eu>
Date: Wed, 5 Nov 2014 17:09:27 +0100
Subject: [PATCH] Fixed warning about redefining the __SSE__ macro in VS2013

---
 extern/clew/include/clew.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/extern/clew/include/clew.h b/extern/clew/include/clew.h
index f4a50dd..328fd52 100644
--- a/extern/clew/include/clew.h
+++ b/extern/clew/include/clew.h
@@ -350,7 +350,7 @@ typedef unsigned int cl_GLenum;
 
 #ifdef _MSC_VER
 #if defined(_M_IX86)
-#if _M_IX86_FP >= 0
+#if _M_IX86_FP >= 0 && !defined(__SSE__)
 #define __SSE__
 #endif
 #if _M_IX86_FP >= 1
-- 
1.7.9