[PATCH 1/3] Remove defaults for Android from cmake
Sebastian Kügler
sebas at kde.org
Wed Dec 2 04:05:05 PST 2015
>From a cursory glance, patches look good to me.
On Tuesday, December 01, 2015 08:46:58 PM Anton Lundin wrote:
> Some of the flags needed to build for android was set in cmake. There
> are many more that needs to be set correctly for things to work, so
> having some in cmake and some in the Android build.sh is just confusing.
>
> This removes the bits from cmake and moves everything into build.sh.
>
> Signed-off-by: Anton Lundin <glance at acc.umu.se>
> ---
> CMakeLists.txt | 5 -----
> packaging/android/build.sh | 1 +
> 2 files changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index dd47be5..7adb2ce 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -158,11 +158,6 @@ if(FTDISUPPORT)
> add_definitions(-DSERIAL_FTDI)
> endif()
>
> -if(ANDROID)
> - set(FBSUPPORT OFF)
> - set(NO_PRINTING ON)
> -endif()
> -
> # setup Grantlee
>
> if(NO_PRINTING)
> diff --git a/packaging/android/build.sh b/packaging/android/build.sh
> index c3efa5b..b03bfef 100644
> --- a/packaging/android/build.sh
> +++ b/packaging/android/build.sh
> @@ -298,6 +298,7 @@ cmake $MOBILE_CMAKE \
> -DNO_MARBLE=ON \
> -DNO_PRINTING=ON \
> -DNO_USERMANUAL=ON \
> + -DFBSUPPORT=OFF \
> -DCMAKE_PREFIX_PATH:UNINITIALIZED=${QT5_ANDROID}/android_${QT_ARCH}/lib/cm
> ake \ -DCMAKE_BUILD_TYPE=Debug \
> -DFTDISUPPORT=${FTDI} \
--
sebas
http://www.kde.org | http://vizZzion.org
More information about the subsurface
mailing list