Cleanup jpip cmake compilation

diff --git a/applications/jpip/CMakeLists.txt b/applications/jpip/CMakeLists.txt
index aa8cf7ccf9cfc427bdfdf01971c51c3d52914dda..dec238d248dcfdae0e32fff4d0b6e6d93e0c678e 100644 (file)
--- a/applications/jpip/CMakeLists.txt
+++ b/applications/jpip/CMakeLists.txt
@@ -1,3 +1,3 @@
-
+# lib and utils
 ADD_SUBDIRECTORY(libopenjpip)
 ADD_SUBDIRECTORY(util)

diff --git a/applications/jpip/libopenjpip/CMakeLists.txt b/applications/jpip/libopenjpip/CMakeLists.txt
index d56fba4a4c14ddf88390ed2e69bf1cf76d9b0382..eb29369d77d81e7defec38b173cf0624ee0cc52a 100644 (file)
--- a/applications/jpip/libopenjpip/CMakeLists.txt
+++ b/applications/jpip/libopenjpip/CMakeLists.txt
@@ -47,16 +47,18 @@ TARGET_LINK_LIBRARIES(openjpip_local ${OPENJPEG_LIBRARY_NAME})
 ADD_LIBRARY(openjpip_server STATIC ${OPENJPIP_SRCS})
 TARGET_LINK_LIBRARIES(openjpip_server ${FCGI_LIBNAME})
 SET_TARGET_PROPERTIES(openjpip_server
-  PROPERTIES COMPILE_FLAGS "-DSERVER")
-
+  PROPERTIES COMPILE_FLAGS "-DSERVER"
+  )
 
+# As far as I understand the design openjpip_ libs do not have stable API
+# This is why they are build as static. Thus they should NOT be installed as of yet
 # Install library
- INSTALL(TARGETS openjpip_local
-   EXPORT OpenJPEGTargets
-   DESTINATION ${OPENJPEG_INSTALL_LIB_DIR} COMPONENT Libraries
- )
-
- INSTALL(TARGETS openjpip_server
-   EXPORT OpenJPEGTargets
-   DESTINATION ${OPENJPEG_INSTALL_LIB_DIR} COMPONENT Libraries
- )
+#INSTALL(TARGETS openjpip_local
+#  EXPORT OpenJPEGTargets
+#  DESTINATION ${OPENJPEG_INSTALL_LIB_DIR} COMPONENT Libraries
+#  )
+#
+#INSTALL(TARGETS openjpip_server
+#  EXPORT OpenJPEGTargets
+#  DESTINATION ${OPENJPEG_INSTALL_LIB_DIR} COMPONENT Libraries
+#  )

diff --git a/applications/jpip/util/CMakeLists.txt b/applications/jpip/util/CMakeLists.txt
index eba94e378a410c243b9561a144b0b7b82074872b..52a427ce606240ccd173395fbd41ad7d93f8b4d2 100644 (file)
--- a/applications/jpip/util/CMakeLists.txt
+++ b/applications/jpip/util/CMakeLists.txt
@@ -5,14 +5,15 @@ INCLUDE_DIRECTORIES(
 )
 
 SET(OPJ_SERVER_SRCS
-${CMAKE_CURRENT_SOURCE_DIR}/opj_server.c
-)
+  ${CMAKE_CURRENT_SOURCE_DIR}/opj_server.c
+  )
 
 # Build executable
-ADD_DEFINITIONS(-DSERVER -DQUIT_SIGNAL="quitJPIP")
-#
 ADD_EXECUTABLE(opj_server ${OPJ_SERVER_SRCS})
 TARGET_LINK_LIBRARIES(opj_server openjpip_server ${FCGI_LIBNAME})
+SET_TARGET_PROPERTIES(opj_server
+  PROPERTIES COMPILE_FLAGS "-DSERVER -DQUIT_SIGNAL=\\\"quitJPIP\\\""
+  )
 
 # On unix you need to link to the math library:
 IF(UNIX)
@@ -20,19 +21,22 @@ IF(UNIX)
 ENDIF(UNIX)
 
 # Install exe
- INSTALL(TARGETS opj_server
-   EXPORT OpenJPEGTargets
-   DESTINATION ${OPENJPEG_INSTALL_BIN_DIR} COMPONENT Applications
- )
-#
-REMOVE_DEFINITIONS(-DSERVER -DQUIT_SIGNAL="quitJPIP")
-#
-FOREACH(exe opj_dec_server jpip_to_jp2 jpip_to_j2k test_index)
+INSTALL(TARGETS opj_server
+  EXPORT OpenJPEGTargets
+  DESTINATION ${OPENJPEG_INSTALL_BIN_DIR} COMPONENT Applications
+  )
 #
- ADD_EXECUTABLE(${exe} ${exe}.c)
- TARGET_LINK_LIBRARIES(${exe} openjpip_local)
-  INSTALL(TARGETS ${exe}
-   EXPORT OpenJPEGTargets
-   DESTINATION ${OPENJPEG_INSTALL_BIN_DIR} COMPONENT Applications
+SET(EXES
+  opj_dec_server
+  jpip_to_jp2
+  jpip_to_j2k
+  test_index
   )
+FOREACH(exe ${EXES})
+  ADD_EXECUTABLE(${exe} ${exe}.c)
+  TARGET_LINK_LIBRARIES(${exe} openjpip_local)
+  INSTALL(TARGETS ${exe}
+    EXPORT OpenJPEGTargets
+    DESTINATION ${OPENJPEG_INSTALL_BIN_DIR} COMPONENT Applications
+    )
 ENDFOREACH(exe)