hard-dep for Qt 4.8
Martin Gräßlin
mgraesslin at kde.org
Wed Jan 18 05:35:57 GMT 2012
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120118/04502320/attachment.sig>
More information about the kde-core-devel
mailing list