[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-buildsystem
Subject: Re: macro kde4_install_library
From: Alexander Neundorf <neundorf () kde ! org>
Date: 2007-07-21 2:56:06
Message-ID: 200707202256.06727.neundorf () kde ! org
[Download RAW message or body]
Hi,
On Friday 20 July 2007 04:11, Ralf Habacker wrote:
> Alexander Neundorf schrieb:
> > Hi PutHuhn,
> >
> > On Monday 16 July 2007 11:49, Nhuh Put wrote:
> > > Hello
> > > With default calls to install, shared libraries are installed into
> > > LIB_INSTALL_DIR. This is fine for linux, but on windows, they should go
> > > into BIN_INSTALL_DIR. The install function of cmake supports this with
> > > the RUNTIME DESTINATION and LIBRARY DESTINATION options.
> > > To make things simpler, I would like to add a macro called
> > > kde4_install_library for this.
> > > Usage would be kde4_install_library( kdecore ) for example.
> >
> > thanks for the patch.
> > I'm not sure what to do about this.
> > Actually I like the previous (long) version better, where the full
> > INSTALL() syntax is used everywhere.
> > With the current patch we introduce another macro which hides a bit what
> > is actually going on.
>
> Which has the advantage that special library install handling would be
> possible on a common place.
That's also true.
Still I wouldn't do it.
Then you will have a CMakeLists.txt full of KDE4_do_this() KDE4_do_that() and
wonder what it all does. I'd like to avoid macros which just save some typing
but then hide what's happening.
If there's INSTALL(), just have a look at the cmake man page.
If there's KDE4_INSTALL_LIBS() you have to dig out the FindKDE4Internal.cmake
file and check the docs at the top. IMO in this case that's not worth it.
Anyway, email sent to k-c-d.
> > Maybe we should ask on kde-core-devel if there are objections against
> > using the full INSTALL() syntax.
>
> In cmake 2.4.7 changelog i found:
>
> * Allow for platform choice of executable shared libs install
>
> I understand that there is now an option to set something like this,
> unfortunally I haven't found any documentation about this option.
I think it's something different, you can now select whether you want the
executable bit set on an installed library:
http://www.cmake.org/cgi-bin/viewcvs.cgi/Source/cmFileCommand.cxx?r1=1.56.2.6&root=CMake&sortby=date&r2=1.56.2.7&only_with_tag=CMake-2-4
Bye
Alex
_______________________________________________
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