monzero-core.git/src/CMakeLists.txt, branch v0.10.0

cmake: support BUILD_SHARED_LIBS built-in option

2016-09-18T06:56:26Z

Support building internal libraries as shared. This reduces development time by eliminating the need to re-link all binaries every time non-interface code in the library changes. Instead, can hack on libxyz, then `make libxyz`, and re-run monerod. By default BUILD_SHARED_LIBS is OFF in release build type, and ON in debug build type, but can be overriden with -D.

cmake: transitive deps and remove deprecated LINK_*

2016-09-18T06:56:26Z

Keep the immediate direct deps at the library that depends on them, declare deps as PUBLIC so that targets that link against that library get the library's deps as transitive deps. Break dep cycle between blockchain_db <-> crytonote_core. No code refactoring, just hide cycle from cmake so that it doesn't complain (cycles are allowed only between static libs, not shared libs). This is in preparation for supproting BUILD_SHARED_LIBS cmake built-in option for building internal libs as shared.

cmake: fix def propagation, fixes 32-bit build

2016-09-03T21:44:28Z

32-bit build would fail to link with 'mdb_env_create undefined' (because for 32-bit build, mdb_env_create_vl32 is defined instead). This bug was introduced with the recent change to virtual object libraries. The problem is that the COMPILE_DEFINITIONS property was not propagated from dependee target (lmdb) to depedent target's (blockchain_db) virtual object lib (obj_blockchain_db). This patch makes that happen. I chose to include INTERFACE_COMPILE_DEFINITIONS because there should not be a need to propagate private defs, but it doesn't make a difference in this case.

fix leftover add_directory

2016-09-03T11:51:07Z

cmake: merge libs via virtual object libraries

2016-08-31T03:01:48Z

CMake issued a warming about policy CMP0026: access of LOCATION target property at config time was disallowed. Offending code was the code that merged static libraries to generate libwallet_merged.a. This patch does that same merge task in a much simpler way. And, since it doesn't violate the policy, the warning went away.

ringct: import of Shen Noether's ring confidential transactions

2016-08-28T20:26:54Z

cmake: fix typo in handling of STACK_TRACE

2016-08-20T03:38:44Z

This fixes the log output not getting redirected to log file.

cmake: cleanup logic that sets flags per target/subdir

2016-07-29T01:20:50Z

The previous logic that used a COMMON_*_FLAGS intermediate variable and then re-assigned CMAKE_*_FLAGS before including each subdirectory was confusing and ugly. This PR is the right way to do it. This commit is purely refactoring: built binaries unchanged.

remove connectivity tool, comment it out from munin plugins too

2016-03-25T12:52:19Z

cmake: Fix overly broad use of dynamic link settings

2016-02-12T22:49:43Z

Remove LINK_SEARCH_START_STATIC and LINK_SEARCH_END_STATIC. This is more appropriate when the compiler flag -static is used. This had been causing CMake to omit the linker flags necesssary to distinguish static and dynamic library linking. CMake had assumed static linking for the target, causing it to omit explicit static link flags. That is problematic without the -static compile flag being set. With a library located in system directories, like libboost_date_time, the full static path (.a), though found correctly by CMake, was treated by the linker as a dynamic library. This is because target_link_libraries() transforms the full path to -l if it's in a system directory. Without -static or explicit linker flags, the dynamic library (.so) is linked. Removing the above two properties removes the assumption of static. So -Wl;-Bstatic is inserted where needed. This causes -l to properly refer to the static library instead of dynamic.