Commit Graph

19 Commits (a1b0408f5b52fbf4b16fcf19fe0c82eebe97dab3)

Author SHA1 Message Date
Jing Dong a1b0408f5b fix CMAKE_BUILD_TYPE var to cache varible in default 2016-05-13 12:18:18 -04:00
Frank Dellaert 5a9ba414f6 Removed obsolete comment 2016-02-26 04:46:21 +00:00
Frank e7ba551523 I believe that this is the right way: CMAKE_CXX_FLAGS are set to -std=c++11 -Wall, and the different build types *append* to it. E.g., for Debug, the CMAKE_CXX_FLAGS_DEBUG (set from GTSAM_CMAKE_CXX_FLAGS_DEBUG) will be appended automatically. 2016-02-25 17:06:31 -08:00
Frank 8edf3042d8 Merge remote-tracking branch 'bitbucket/develop' into feature/improvements_and_bugfixes
Conflicts:
	cmake/GtsamBuildTypes.cmake
2016-02-24 17:36:02 -08:00
Enrique Fernandez 443bb0776b Set FLAGS for CMAKE_BUILD_TYPE None
Set the FLAGS to the ones from RelWithDebInfo CMAKE_BUILD_TYPE

See https://cmake.org/Wiki/CMake_Useful_Variables#Compilers_and_Tools

Without this, the C and CXX FLAGS are this (wrong):

--   C compilation flags            : -g -O2 -fstack-protector
--param=ssp-buffer-size=4 -Wformat -Werror=format-security
-D_FORTIFY_SOURCE=2
--   C++ compilation flags          : -g -O2 -fstack-protector
--param=ssp-buffer-size=4 -Wformat -Werror=format-security
-D_FORTIFY_SOURCE=2

and the compilation fails with this and similar errors (because C++11 is
needed):

gtsam/navigation/ImuFactor.cpp:144:15: error: ‘nullptr’ was not declared
in this scope
  return e != nullptr && base && pim;

With this changes, the C and CXX FLAGS are this (good):

--   C compilation flags            : -std=c11 -g -O3 -Wall -DNDEBUG
--   C++ compilation flags          : -std=c++11 -g -O3 -Wall -DNDEBUG

and everything compiles.
2016-02-24 17:05:12 -05:00
Enrique Fernandez 3226ec1d6a 💄 2016-02-24 17:05:07 -05:00
Frank 0372a959ee Many small improvements, bug-fixes, and tests 2016-02-24 11:01:19 -08:00
cbeall3 55ed99141e Add C++11 compiler flag to GtsamBuildTypes.cmake 2015-10-20 13:41:31 -04:00
Abe 13a4da21b2 misc bugfixes and cleanup from skydio 2015-05-13 22:26:24 -07:00
Thomas Schneider 279751c7a2 Remove -ftemplate-depth for apple with clang < 5.0. 2015-04-09 14:01:16 +02:00
Thomas Schneider 1ea5ae2253 Set -ftemplate-depth=1024 for all Clang compilers. Allows to compile
e.g. under Ubuntu with clang.
2015-04-09 13:41:14 +02:00
Chris Beall 7cdbac4b5c Fix warnings by latest CMake 3.1. Variables should be unquoted: Policy CMP0054 is not set: Only interpret if() arguments as variables or keywords when unquoted. 2015-01-16 09:33:02 -05:00
cbeall3 53a24ed93a Add EIGEN_INITIALIZE_MATRICES_BY_NAN to Debug mode to resolve issue #184. 2014-12-04 12:40:11 -05:00
jing 020d2e43f8 change cmake option into correct way: use set CMAKE_CXX_FLAGS replace add_definition 2014-05-25 20:09:49 -04:00
Richard Roberts ac14ddcb74 Reverted reduced debug info in Profiling mode (seems to prevent inspecting source code). Fixed Profiling build not receiving build type postfixes. 2014-02-21 17:18:36 -05:00
Richard Roberts b4d7ede123 Better debug flag for Profiling build type 2014-02-19 15:43:16 -05:00
Richard Roberts 3e3f31c4d1 Made FIRST_PASS_DONE variable internal 2014-02-13 10:56:20 -05:00
Richard Roberts 4b8c17ec77 Changed target name to 'gtsam' instead of 'gtsam-shared' and 'gtsam-static'. Modified CMake options to only allow building either static or shared, but not both. Enabled GTSAM_BUILD_TYPE_POSTFIXES by default on all platforms. 2014-01-30 14:41:25 -05:00
Richard Roberts 395b8a2f48 Added cmake subdirectory without history (at SVN r20364) 2013-12-22 13:04:47 -05:00