From 65509e777eae1185991853bb21edf56b2aa7aaba Mon Sep 17 00:00:00 2001
From: Gabriel Hege <gabriel.hege@hhi.fraunhofer.de>
Date: Tue, 15 Oct 2024 10:28:14 +0200
Subject: [PATCH] cmake: add cache variables for BUILD_SHARED_LIBS &
 VVDEC_LIBRARY_ONLY

closes #202
---
 CMakeLists.txt                     |  4 ++++
 cmake/modules/vvdecInstall.cmake   | 20 ++++++++++++--------
 source/App/vvdecapp/CMakeLists.txt |  3 +++
 3 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index d045019f..084be76b 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -123,6 +123,10 @@ set( VVDEC_ENABLE_LOCAL_BITSTREAM_DOWNLOAD  OFF CACHE BOOL   "Enable or disable
 # also install the test executable vvdecapp instead of the libraries and headers only
 set( VVDEC_INSTALL_VVDECAPP                 OFF CACHE BOOL   "Enable or disable installing the test vvdecapp" )
 
+set( VVDEC_LIBRARY_ONLY                     OFF CACHE BOOL   "Build libvvdec only (no vvdecapp)" )
+
+set( BUILD_SHARED_LIBS                      OFF CACHE BOOL   "Build libvvdec as a shared library" )
+
 # enable postfix
 set( VVDEC_ENABLE_BUILD_TYPE_POSTFIX        OFF CACHE BOOL   "Enable or disable build type postfix for apps and libs" )
 
diff --git a/cmake/modules/vvdecInstall.cmake b/cmake/modules/vvdecInstall.cmake
index 7408c0f0..b138fb0f 100644
--- a/cmake/modules/vvdecInstall.cmake
+++ b/cmake/modules/vvdecInstall.cmake
@@ -58,15 +58,19 @@ target_include_directories( vvdec  SYSTEM INTERFACE $<INSTALL_INTERFACE:${CMAKE_
 install( FILES     ${CMAKE_BINARY_DIR}/vvdec/version.h  DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/vvdec )
 install( DIRECTORY include/vvdec                        DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} )
 
-if( VVDEC_INSTALL_VVDECAPP OR (${CMAKE_SYSTEM_NAME} STREQUAL "Emscripten") )
-    # for Emscripten/WASM builds vvdecapp is always installed, since that is used like a library
-    set( INSTALL_TARGETS "vvdec;vvdecapp" )
-else()
-    set( INSTALL_TARGETS "vvdec" )
+if( VVDEC_INSTALL_VVDECAPP AND VVDEC_LIBRARY_ONLY )
+  message( FATAL_ERROR "VVDEC_INSTALL_VVDECAPP conflicts with VVDEC_LIBRARY_ONLY" )
+endif()
 
-    if ( NOT VVDEC_INSTALL_VVDECAPP )
-      install( CODE "message( NOTICE \"The vvdecapp binary is not installed by default anymore. To also install vvdecapp set '-DVVDEC_INSTALL_VVDECAPP=ON' (with make: 'install-vvdecapp=1')\" )" )
-    endif()
+set( INSTALL_TARGETS vvdec )
+
+if( NOT VVDEC_LIBRARY_ONLY )
+  if( VVDEC_INSTALL_VVDECAPP
+      OR (${CMAKE_SYSTEM_NAME} STREQUAL "Emscripten") ) # for Emscripten/WASM builds vvdecapp is always installed, since that is used like a library
+    list( APPEND INSTALL_TARGETS vvdecapp )
+  else()
+    install( CODE "message( NOTICE \"The vvdecapp binary is not installed by default any more. To also install vvdecapp set '-DVVDEC_INSTALL_VVDECAPP=ON' (with make: 'install-vvdecapp=1')\" )" )
+  endif()
 endif()
 
 
diff --git a/source/App/vvdecapp/CMakeLists.txt b/source/App/vvdecapp/CMakeLists.txt
index 6d7dc1ea..5af5fe39 100644
--- a/source/App/vvdecapp/CMakeLists.txt
+++ b/source/App/vvdecapp/CMakeLists.txt
@@ -25,6 +25,9 @@ set_target_properties( ${EXE_NAME} PROPERTIES MINSIZEREL_POSTFIX     "${CMAKE_MI
 if( VVDEC_INSTALL_RPATH )
   set_target_properties( ${EXE_NAME} PROPERTIES INSTALL_RPATH        "${VVDEC_INSTALL_RPATH}" )
 endif()
+if( VVDEC_LIBRARY_ONLY )
+  set_target_properties( ${EXE_NAME} PROPERTIES EXCLUDE_FROM_ALL TRUE )
+endif()
 
 target_link_libraries( ${EXE_NAME} Threads::Threads vvdec )
 target_include_directories( ${EXE_NAME} PRIVATE  ../../Lib/libmd5 )