Appveyor CI build and MSVC tweaks

* appveyor build script
* Minor cmake cleanups
* Tweak tests to appease broken Visual Studio 12 warning C4313

Author: c42f

CMakeLists.txt

@@ -1,6 +1,6 @@
 # CMake build for tests.
 #
-# See also the Makefile, which is currently more fully featured.
+# See also the Makefile, which is currently more fully featured on unix.
 
 
 # Set cmake builtin variables before calling project(), otherwise the
@@ -15,24 +15,27 @@ set(CXX_STD "c++11" CACHE STRING "Version of C++ standard in use")
 # be errors on all compilers, unless explicitly silenced.
 if(NOT WIN32)
     set(CMAKE_CXX_FLAGS "-Wall -Werror -std=${CXX_STD}" CACHE STRING "Flags used by the compiler during all build types.")
-else()
-    set(CMAKE_CXX_FLAGS "-W3 -WX")
 endif()
 
 project(tinyformat)
 cmake_minimum_required(VERSION 2.8)
-option(COMPILE_SPEED_TEST FALSE)
-
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
 
+if(WIN32)
+    # Treat warnings as errors.  Would set this above, but need the default
+    # flags too, and `project()` behaves is differently on different systems.
+    set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /W3 /WX")
+endif()
+
 add_executable(tinyformat_test tinyformat_test.cpp)
+enable_testing()
+if(CMAKE_CONFIGURATION_TYPES)
+    set(ctest_config_opt -C ${CMAKE_BUILD_TYPE})
+endif()
+add_test(NAME test COMMAND tinyformat_test)
+add_custom_target(testall COMMAND ${CMAKE_CTEST_COMMAND} -V ${ctest_config_opt} DEPENDS tinyformat_test)
 
+option(COMPILE_SPEED_TEST FALSE)
 if (COMPILE_SPEED_TEST)
     add_executable(tinyformat_speed_test tinyformat_speed_test.cpp)
 endif ()
-
-enable_testing()
-add_test(NAME test COMMAND tinyformat_test)
-
-add_custom_target(testall COMMAND ${CMAKE_CTEST_COMMAND} DEPENDS tinyformat_test)
-

appveyor.yml

@@ -0,0 +1,51 @@
+# Windows CI build via the Appveyor service
+
+version: '{build}'
+
+# branches to build
+branches:
+  only:
+    - master
+
+# scripts that are called at very beginning, before repo cloning
+init:
+  - git config --global core.autocrlf input
+
+# Build matrix variables -
+#   * environment
+#   * platform
+#   * configuration
+environment:
+  matrix:
+    - COMPILER: "Visual Studio 12 Win64"
+    - COMPILER: "Visual Studio 14 Win64"
+    #- COMPILER: "Visual Studio 15 Win64"  # Not in appveyor standard image yet
+
+# build platform, i.e. x86, x64, Any CPU.
+platform:
+  - x86
+  - x64
+
+# build Configuration, i.e. Debug, Release, etc.
+configuration:
+  - Debug
+  - Release
+
+before_build:
+  - echo --------------------------------------------------------------------------------
+  - echo Appveyor environment info:
+  - echo COMPILER = %COMPILER%, CONFIGURATION = %CONFIGURATION%
+  - cmake -h
+
+build_script:
+  - echo --------------------------------------------------------------------------------
+  - echo Build tinyformat
+  - mkdir build
+  - cd build
+  - cmake -G "%COMPILER%" ..
+  - cmake --build . --config %CONFIGURATION%
+
+test_script:
+  # cmake testall target has problems finding the correct configuration, so use ctest directly.
+  - ctest -C %CONFIGURATION%
+

tinyformat_test.cpp

@@ -4,6 +4,11 @@
 namespace std { class type_info; }
 #endif
 
+#if defined(_MSC_VER) && _MSC_VER == 1800
+    // Disable spurious warning about printf format string mismatch in VS 2012
+#   pragma warning(disable:4313)
+#endif
+
 #include <stdexcept>
 #include <climits>
 #include <cfloat>
@@ -255,8 +260,8 @@ int unitTests()
     // which would noramlly go to the stdout
     std::ostringstream coutCapture;
     std::streambuf* coutBuf = std::cout.rdbuf(coutCapture.rdbuf());
-    tfm::printf("%s %s %d\n", "printf", "test", "1");
-    tfm::printfln("%s %s %d", "printfln", "test", "1");
+    tfm::printf("%s %s %d\n", "printf", "test", 1);
+    tfm::printfln("%s %s %d", "printfln", "test", 1);
     std::cout.rdbuf(coutBuf); // restore buffer
     CHECK_EQUAL(coutCapture.str(), "printf test 1\nprintfln test 1\n");