<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 6, 2015 at 2:10 AM, Ian Wadham <span dir="ltr"><<a href="mailto:iandw.au@gmail.com" target="_blank">iandw.au@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 06/01/2015, at 8:46 AM, Jeremy Whiting wrote:<br>
> No, kf5 doesn't require Qt 5.4 yet, but some of plasma does iirc. On the other hand, getting this into qt itself is not going to happen on Qt 5.3 branch (it's frozen) or 5.4 (it's also frozen), which is why I created/updated the patch for the 5.4 branch (I guess I really should have used the dev branch instead as that will become 5.5). I'm not sure if the patch will apply to 5.3 or not, the patch was changing qstandardpaths_mac.cpp which doesn't even exist in the 5.4 branch (it's <a href="http://qstandardpaths_mac.mm" target="_blank">qstandardpaths_mac.mm</a> here) If it was .mm in 5.3 it should apply or would be simple to tweak so that it does apply for macports to use I guess.<br>
<br>
</span>Also, I think you will find that there is a change from DataLocation type to<br>
AppDataLocation and AppLocalDataLocation types, with DataLocation<br>
being deprecated, but still supported (== AppDataLocation, IIRC).<br>
<br>
So, separate patches?<br></blockquote><div><br></div><div>Yep, I noticed that in the sources also. Separate patches makes sense to me, but I don't think patches to 5.3 (or 5.4 for that matter) will make it upstream since those are frozen. We can keep those in macports/homebrew/fink packages of qt 5.3 and 5.4 though I suppose.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers, Ian W.<br>
<span class="im HOEnZb"><br>
> On Mon, Jan 5, 2015 at 2:41 PM, René J.V. <<a href="mailto:rjvbertin@gmail.com">rjvbertin@gmail.com</a>> wrote:<br>
> On Monday January 05 2015 13:51:18 Jeremy Whiting wrote:<br>
><br>
> > I've looked at Marko's patch and had to tweak it a bit to get it to apply<br>
> > to qt 5.4 branch. I've attached a first try here while it builds at the<br>
> > same time so I can test it.<br>
><br>
> Could we please make sure the fix applies to Qt 5.3 also? That's what's currently in MacPorts, and unless the maintainer intends to upgrade it to 5.4 soon, that's what I'll be working on to make it co-installable with Qt4 ...<br>
><br>
> KF5 doesn't yet require Qt5.4, or does it??<br>
><br>
> R.<br>
><br>
</span><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> <a href="mailto:kde-mac@kde.org">kde-mac@kde.org</a><br>
> List Information: <a href="https://mail.kde.org/mailman/listinfo/kde-mac" target="_blank">https://mail.kde.org/mailman/listinfo/kde-mac</a><br>
> KDE/Mac Information: <a href="http://community.kde.org/Mac" target="_blank">http://community.kde.org/Mac</a><br>
<br>
_______________________________________________<br>
<a href="mailto:kde-mac@kde.org">kde-mac@kde.org</a><br>
List Information: <a href="https://mail.kde.org/mailman/listinfo/kde-mac
KDE/Mac" target="_blank">https://mail.kde.org/mailman/listinfo/kde-mac<br>
KDE/Mac</a> Information: <a href="http://community.kde.org/Mac" target="_blank">http://community.kde.org/Mac</a><br>
</div></div></blockquote></div><br></div></div>