ANTsX / ANTs

Advanced Normalization Tools (ANTs)
Apache License 2.0
1.21k stars 381 forks source link

Consider to ass a SO version to the libraries (at least on linux) #71

Closed gerddie closed 10 years ago

gerddie commented 10 years ago

Please consider adding version information to the shared libraries that get installed into /usr/lib. A patch to add SO version 2 and library version 2.0.0 would be:

--- ants-2.0.0.orig/CMakeLists.txt
+++ ants-2.0.0/CMakeLists.txt
@@ -1,6 +1,8 @@
 cmake_minimum_required(VERSION 2.8.5)
 cmake_policy(VERSION 2.8.5)

+set(LIBRARY_SOVERSION_INFO 2)
+set(LIBRARY_VERSION_INFO 2.0.0)
 set(LOCAL_PROJECT_NAME ANTS)
 INCLUDE(InstallRequiredSystemLibraries)
 ## NOTE THERE SHOULD BE NO PROJECT STATEMENT HERE!
--- ants-2.0.0.orig/Examples/CMakeLists.txt
+++ ants-2.0.0/Examples/CMakeLists.txt
@@ -27,6 +27,10 @@ macro(STANDARD_ANTS_BUILD ANTS_FUNCTION_
   set( ANTS_FUNCTION_NAME ${ANTS_FUNCTION_NAME} )

   add_library(l_${ANTS_FUNCTION_NAME} ${ANTS_FUNCTION_NAME}.cxx ${UI_SOURCES})
+  SET_TARGET_PROPERTIES(l_${ANTS_FUNCTION_NAME} PROPERTIES 
+    SOVERSION ${LIBRARY_SOVERSION_INFO}  VERSION ${LIBRARY_VERSION_INFO})
+
+
   message(STATUS "${ANTS_FUNCTION_NAME} ${EXTRA_LIBS}")
   target_link_libraries(l_${ANTS_FUNCTION_NAME} antsUtilities ${ITK_LIBRARIES} ${EXTRA_LIBS} )
   configure_file( template_for_executables.cxx.in cli_${ANTS_FUNCTION_NAME}.cxx )
stnava commented 10 years ago

added a patch for this in the A.M.

brian

On Fri, Feb 28, 2014 at 9:35 AM, Gert Wollny notifications@github.comwrote:

Please consider adding version information to the shared libraries that get installed into /usr/lib. A patch to add SO version 2 and library version 2.0.0 would be:

--- ants-2.0.0.orig/CMakeLists.txt +++ ants-2.0.0/CMakeLists.txt @@ -1,6 +1,8 @@ cmake_minimum_required(VERSION 2.8.5) cmake_policy(VERSION 2.8.5)

+set(LIBRARY_SOVERSION_INFO 2) +set(LIBRARY_VERSION_INFO 2.0.0) set(LOCAL_PROJECT_NAME ANTS) INCLUDE(InstallRequiredSystemLibraries)

NOTE THERE SHOULD BE NO PROJECT STATEMENT HERE!

--- ants-2.0.0.orig/Examples/CMakeLists.txt +++ ants-2.0.0/Examples/CMakeLists.txt @@ -27,6 +27,10 @@ macro(STANDARD_ANTS_BUILD ANTSFUNCTION set( ANTS_FUNCTION_NAME ${ANTS_FUNCTION_NAME} )

addlibrary(l${ANTS_FUNCTION_NAME} ${ANTS_FUNCTION_NAME}.cxx ${UI_SOURCES})

  • SET_TARGETPROPERTIES(l${ANTS_FUNCTION_NAME} PROPERTIES
  • SOVERSION ${LIBRARY_SOVERSION_INFO} VERSION ${LIBRARY_VERSION_INFO}) + + message(STATUS "${ANTS_FUNCTION_NAME} ${EXTRA_LIBS}") target_linklibraries(l${ANTS_FUNCTION_NAME} antsUtilities ${ITK_LIBRARIES} ${EXTRA_LIBS} ) configure_file( template_forexecutables.cxx.in cli${ANTS_FUNCTION_NAME}.cxx )

Reply to this email directly or view it on GitHubhttps://github.com/stnava/ANTs/issues/71 .