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

List:       kde-core-devel
Subject:    Re: qt-copy patch 0180
From:       Thiago Macieira <thiago () kde ! org>
Date:       2008-01-30 13:40:07
Message-ID: 200801301440.07439.thiago () kde ! org
[Download RAW message or body]


On Wednesday 30 January 2008 14:29:12 Lubos Lunak wrote:
> On Wednesday 30 of January 2008, Thiago Macieira wrote:
> > On Wednesday 30 January 2008 10:26:18 Marijn Kruisselbrink wrote:
> > > Thiago Macieira wrote:
> > > > Lubos Lunak wrote:
> > > >>> Exactly. QtUiTools cannot use the private inheritance tree for that
> > > >>> reason.
> > > >>
> > > >>  Given that QUiLoader is a QObject, that will be quite hard to avoid
> > > >> without breaking compatibility somehow.
> > > >
> > > > It's linked statically. There is no binary compatibility to maintain.
> > >
> > > Isn't there a potential problem that the part of it that is linked
> > > statically into an executable is not binary compatible with the part
> > > that is linked dynamically when running the executable?
> >
> > You did not understand: it's a static library. It cannot be loaded
> > dynamically.
>
>  But Qt itself can, and that's the problem.
>
>  I've committed qt-copy patch #0208 that removes the QUiTools dependency on
> Qt internals, so changing Qt versions or using patch #0180 should no longer
> be a problem (you obviously first need to recompile libplasma once with
> such patched Qt). There's also new patch #0209 that should help detect any
> similar possible problems caused by using qt-copy.
>
>  Is somebody from TT already looking into the QUiTools problem or should I
> submit #0208 to qt-bugs?

We're looking into the issue. I'll post the patch as soon as I have it.

-- 
   Thiago Macieira   -   thiago (AT) macieira.info - thiago (AT) kde.org
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C   966C 33F5 F005 6EF4 5358

["signature.asc" (application/pgp-signature)]

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

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