<div dir="ltr">None of those includes were needed apparently. I've attached an updated patch. I also looked at the 5.3 branch and it has qstandardpaths_mac.cpp instead of <a href="http://qstandardpaths_mac.mm">qstandardpaths_mac.mm</a> so a similar patch that changes that file would be needed for 5.3.<div><br></div><div>BR,</div><div>Jeremy</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 5, 2015 at 2:46 PM, Jeremy Whiting <span dir="ltr"><<a href="mailto:jpwhiting@kde.org" target="_blank">jpwhiting@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">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.<div><br></div><div>BR,</div><div>Jeremy</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 5, 2015 at 2:41 PM, René J.V. <span dir="ltr"><<a href="mailto:rjvbertin@gmail.com" target="_blank">rjvbertin@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>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>
</span>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>
<span><font color="#888888"><br>
R.<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>