Updated some documentation following cmake change 70/212870/3
authorAdeel Kazmi <adeel.kazmi@samsung.com>
Tue, 27 Aug 2019 13:43:08 +0000 (14:43 +0100)
committerAdeel Kazmi <adeel.kazmi@samsung.com>
Tue, 27 Aug 2019 13:58:11 +0000 (14:58 +0100)
Change-Id: Ie192bcbfd8337748483bfaba66a8b1b869aa5b7d

automated-tests/README.md
build/scripts/dali_env
build/tizen/CMakeLists.txt
dali/public-api/common/dali-common.h

index 454a431453c7c5fc4c0dfa60624e0d59512a287e..cfb515f0e59a3c8c9d3e8f45d054cd0514aca5c2 100644 (file)
@@ -46,12 +46,9 @@ Building dali core:
     export CC=gcc
     export CXX=g++
     git clean -fxd . # Only do this in the build folder
-    autoreconf --install
-    CXXFLAGS='-g -O0 --coverage' LDFLAGS='--coverage' ./configure --prefix=$DESKTOP_PREFIX --enable-debug
+    CXXFLAGS='-g -O0 --coverage' LDFLAGS='--coverage' cmake -DCMAKE_INSTALL_PREFIX=$DESKTOP_PREFIX -DCMAKE_BUILD_TYPE=Debug
     make -j8 install
 
-Repeat for dali-adaptor and toolkit.
-
 Note, you __must__ use a local build and not a distributed build, and you __must__ also build with debug enabled to allow *DALI_ASSERT_DEBUG* to trigger on wrong behaviour ( Which should always be a test case failure! )
 
 Building the tests
index 86a15394b0d18db3730647231c1ad7657a9f1f6b..b6b3eb54eba3c5a83576e43adc86e8ef773dbbf7 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/perl
 
-# Copyright (c) 2018 Samsung Electronics Co., Ltd.
+# Copyright (c) 2019 Samsung Electronics Co., Ltd.
 
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -191,8 +191,10 @@ sub create_setenv
 # You can then source this script by using
 # \$ . setenv
 #
-# Use DESKTOP_PREFIX when running configure in dali/build/tizen:
+# Use DESKTOP_PREFIX when running configure or cmake in dali/build/tizen:
 # \$ CXXFLAGS="-g -O0" ./configure --prefix=\$DESKTOP_PREFIX
+# _OR_
+# \$ CXXFLAGS="-g -O0" cmake -DCMAKE_INSTALL_PREFIX=\$DESKTOP_PREFIX
 
 export DESKTOP_PREFIX=$install_path
 export PATH=$install_path/bin:\$PATH
index acece1ee0d89f806e0d13a6ce3fa0f54bccd753a..4fa954676888d34c6aa0f7d663b28d76a2553739 100644 (file)
@@ -27,7 +27,7 @@ OPTION(ENABLE_TRACE          "Enable Trace" OFF)
 
 IF(CMAKE_BUILD_TYPE MATCHES Debug)
   MESSAGE( STATUS "CMAKE_BUILD_TYPE: " Debug )
-  SET(ENABLE_DEBUG ON)
+  SET( ENABLE_DEBUG ON )
 ELSE()
   MESSAGE( STATUS "CMAKE_BUILD_TYPE: " Release )
 ENDIF()
@@ -39,14 +39,15 @@ IF("${ARCH}" STREQUAL "arm")
   ADD_DEFINITIONS("-DTARGET")
 ENDIF("${ARCH}" STREQUAL "arm")
 
-IF( NOT ${ENABLE_EXPORTALL} AND NOT ${ENABLE_DEBUG} )
-  ADD_DEFINITIONS( "-fvisibility=hidden -DHIDE_DALI_INTERNALS" )
-ENDIF( NOT ${ENABLE_EXPORTALL} AND NOT ${ENABLE_DEBUG} )
-
 IF( ENABLE_DEBUG )
   ADD_DEFINITIONS( "-DDEBUG_ENABLED" )
+  SET( ENABLE_EXPORTALL ON )
 ENDIF( ENABLE_DEBUG )
 
+IF( NOT ${ENABLE_EXPORTALL} )
+  ADD_DEFINITIONS( "-fvisibility=hidden -DHIDE_DALI_INTERNALS" )
+ENDIF( NOT ${ENABLE_EXPORTALL} )
+
 IF( ENABLE_BACKTRACE OR ENABLE_LOCK_BACKTRACE )
   ADD_DEFINITIONS( "-DBACKTRACE_ENABLED" )
 ENDIF( ENABLE_BACKTRACE OR ENABLE_LOCK_BACKTRACE )
@@ -248,6 +249,7 @@ MESSAGE( STATUS "Prefix:                " ${PREFIX} )
 MESSAGE( STATUS "Lib Dir:               " ${LIB_DIR} )
 MESSAGE( STATUS "Include Dir:           " ${INCLUDE_DIR} )
 MESSAGE( STATUS "Debug build:           " ${ENABLE_DEBUG} )
+MESSAGE( STATUS "Export all symbols:    " ${ENABLE_EXPORTALL} )
 MESSAGE( STATUS "Backtrace:             " ${ENABLE_BACKTRACE} )
 MESSAGE( STATUS "Scoped Lock backtrace: " ${ENABLE_LOCK_BACKTRACE} )
 MESSAGE( STATUS "Coverage:              " ${ENABLE_COVERAGE} )
index 32711ec4f35adba281dd66a58f614a2beab1ce1d..ba252e8c021b75605fe0109a702562467008f216 100755 (executable)
@@ -23,9 +23,9 @@
 /*
  * Definitions for shared library support.
  *
- * If a library is configured with --enable-exportall or --enable-debug
+ * If a library is built with -DENABLE_EXPORTALL=ON or -DENABLE_DEBUG=ON
  * then HIDE_DALI_INTERNALS is not defined, and nothing is hidden.
- * If it is configured without these options (the default), then HIDE_INTERNALS
+ * If it is built without these options (the default), then HIDE_INTERNALS
  * is defined when building the library, visibility is automatically hidden, and the explicit
  * defines below come into use.
  * When building a library that uses DALI, HIDE_DALI_INTERNALS.