building libde265 for ubuntu

[nolyn]

added option to cmake file to only build the lib
added install target for the lib, matching ubuntu 16.04

[ChristianFeldmann]

Hmm is this the right way to do it? Is this true for all linux distributions or is there maybe a cmake way to do this?

[nolyn]

I tried to find out how to do that. I don't think there is a cmake way short of testing for the name of the system. Though I did not spend long on this. But since we are going to use snap, which is similar to a docker container, but ubuntu 16.04 based it should be fine.

…

On Wed, Aug 1, 2018 at 7:32 PM, Christian ***@***.***> wrote: ***@***.**** requested changes on this pull request. ------------------------------ In libde265/CMakeLists.txt <#4 (comment)> : > @@ -83,3 +83,12 @@ target_link_libraries(${LIBDE265_LIBRARY_NAME} ${CMAKE_THREAD_LIBS_INIT}) if(CMAKE_SYSTEM_PROCESSOR STREQUAL "x86_64") SET_TARGET_PROPERTIES(${LIBDE265_LIBRARY_NAME} PROPERTIES COMPILE_FLAGS "-fPIC") endif(CMAKE_SYSTEM_PROCESSOR STREQUAL "x86_64") + +if(UNIX) + # could not find out how to determin library install path platform independent. this is for ubuntu (16.04) + INSTALL(TARGETS + #libaries + ${LIBDE265_LIBRARY_NAME} + LIBRARY DESTINATION "/usr/lib/x86_64-linux-gnu/" Hmm is this the right way to do it? Is this true for all linux distributions or is there maybe a cmake way to do this? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#4 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFmzCJ3sMlug-QQDuKuzxbJeddHa8ANRks5uMeZIgaJpZM4Vqsl4> .