[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-buildsystem
Subject: Re: Getting rid of the LIB_INSTALL_DIR hack on windows
From: Michael Pyne <mpyne () purinchu ! net>
Date: 2008-01-12 17:46:37
Message-ID: 200801121246.46653.mpyne () purinchu ! net
[Download RAW message or body]
[Attachment #2 (multipart/signed)]
On Saturday 12 January 2008, Christian Ehrlicher wrote:
> Hi,
>
> I know we discussed this already, but I'm very unhappy with the current
> solution.
> The problem is (for all who don't remember) that we want to install the
> shared libs into /bin on windows. When we install the shared libs into
> /lib, we've to add an entry to the PATH env variable. Also it's not the
> normal way to install shared libs in another location than the
> executable - it would just confuse the user.
>
> Because of this a hack for LIB_INSTALL_DIR was added:
> set(LIB_INSTALL_DIR "lib${LIB_SUFFIX}"
> RUNTIME DESTINATION "bin"
> LIBRARY DESTINATION "lib${LIB_SUFFIX}"
> ARCHIVE DESTINATION "lib${LIB_SUFFIX}" )
>
> This works fine until someone uses LIB_INSTALL_DIR in another context -
> e.g. to install additional files, see kdepimlibs/gpgme++:
> <snip>
> Therefore I want to change all CMakeLists.txt from
>
> install(TARGETS foo DESTINATION ${LIB_INSTALL_DIR} )
>
> to
>
> install(TARGETS foo RUNTIME DESTINATION ${BIN_IINSTALL_DIR}
> LIBRARY DESTINATION ${LIB_INSTALL_DIR}
> ARCHIVE DESTINATION ${LIB_INSTALL_DIR}
> )
>
> An additional macro install_kde_library() would work as well.
I like the macro idea a beeellion times better. Especially since it documents
the intent better (i.e. use this for libraries, not for anything else).
Then we could switch libraries to use install_kde4_library() and leave the
rest alone.
> Any objections?
> Christian
["signature.asc" (application/pgp-signature)]
_______________________________________________
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic