From cb03517fd36399d65a773b85ac7c0a5bf73ca8d6 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sun, 9 Oct 2016 13:55:08 -0400 Subject: [PATCH] ci: Force unsigned char usage for the 32bit build Since C leaves whether char is signed or unsigned up to the implementer, there are different defaults on different architectures. Forcing unsigned char for one of our CI builds should help catch these issues moving forward. --- CMakeLists.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/CMakeLists.txt b/CMakeLists.txt index 0a6a9accea..beca0c1d5d 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -253,6 +253,10 @@ option( if(TRAVIS_CI_BUILD) message(STATUS "Travis CI build enabled.") add_definitions(-Werror) + if(DEFINED ENV{BUILD_32BIT}) + # Get some test coverage for unsigned char + add_definitions(-funsigned-char) + endif() endif() if(CMAKE_BUILD_TYPE MATCHES Debug)