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

List:       kde-core-devel
Subject:    Re: Re: hard-dep for Qt 4.8
From:       Martin =?ISO-8859-1?Q?Gr=E4=DFlin?= <mgraesslin () kde ! org>
Date:       2012-01-18 5:35:57
Message-ID: 1613718.83VsEKfbQk () martin-desktop
[Download RAW message or body]


On Tuesday 17 January 2012 17:06:26 Allen Winter wrote:
> On Tuesday 17 January 2012 11:34:19 AM Martin Gräßlin wrote:
> > On Friday 13 January 2012 10:06:10 Thiago Macieira wrote:
> > > On Thursday, 12 de January de 2012 22.50.24, Ingo
> > > =?ISO-8859-1?Q?Kl=F6cker?=> > 
> > > wrote:
> > > > If you want developers to switch to Qt 4.8 to get the KDE code better
> > > > tested with this version of Qt then simply ask developers to do so. I
> > > > see no good reason to force developers to do so.
> > > 
> > > You know what happens when people are using a new version of Qt, right?
> > > 
> > > They start using features from it.
> > 
> > and that already happened: https://projects.kde.org/projects/kde/kde-
> > baseapps/repository/revisions/f5eb9ffa53fc96bb1d7b68db304f1ca3816cae6c
> > 
> > Commit makes use of QGraphicsGridLayout::removeItem
> > http://doc.qt.nokia.com/4.8-snapshot/qgraphicsgridlayout.html#removeItem
> 
> It does if you change 1 line.
> Which I already did in my local repo so I don't need 4.8
> 
> A trivial 1 liner
> 
> Which is not a good enough reason.
> Not saying that eventually there won't be a good enough reason, but this
> isn' tit.
I didn't say that this is a reason. I wanted to highlight the problem of not 
depending on 4.8 and everybody using 4.8. I'm not going to start reviewing 
code for "is this a Qt 4.8 change".

With API it's quite simple. With change in behavior it is rather difficult. 
How should you notice that you make use of Qt 4.8 specific behavior?

Our resources are so limited and I don't want to waste them on keeping 
compatibility with a Qt version nobody (of the developers and the distros 
shipping 4.9) is using.

Cheers
Martin

["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