[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: [PATCH] Konsole private library
From:       Patrick Spendrin <ps_ml () gmx ! de>
Date:       2009-02-02 12:42:07
Message-ID: 4986EA1F.7060307 () gmx ! de
[Download RAW message or body]

Pau Garcia i Quiles schrieb:
> On Mon, Feb 2, 2009 at 1:20 PM, Patrick Spendrin <ps_ml@gmx.de> wrote:
>> Thiago Macieira schrieb:
>>> Patrick Spendrin wrote:
>>>> Hi,
>>>>
>>>> as I am currently porting Konsole to Windows, I need to change the
>>>> exports a bit. As this is a rather big change, I'd prefer to get some
>>>> more review of the patch here (I already asked in konsole-devel@).
>>>>
>>>> So please take the time to both take a look at the patch itself and try
>>>> to compile & test it.
>>>>
>>>> If nobody objects, I would commit the patch in the coming days.
>>> There's one thing missing in your message and the patch: why?
>>>
>>> Why is it necessary to have a private library?
>> Ah, there are two good reasons for it:
>> 1) For the konsole part all files get recompiled (at least here) as
>> there is a new compile flags set (-DKONSOLE_PART).
>> 2) on windows I can't link to the kdeinit_executable as this is no
>> library. Until now there are simple KDE_EXPORTs done in the code, which
>> do not work here; I want to make the patches work though, so it is
>> easiest to use a "private" library as in kmail and some others for that.
> 
> It is possible to link to an executable:
> http://article.gmane.org/gmane.comp.programming.tools.cmake.user/18735
Yeah but this is slightly different here as konsole is a kdeinit 
executable - meaning on windows you make a static library for it (which 
will not export anything) and then you link that to a stub executable.
Thus it is not possible to link that directly.
> 
> 

regards,
Patrick

-- 
web:                 http://windows.kde.org
mailing list:        kde-windows@kde.org
irc:                 #kde-windows (irc.freenode.net)
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic