<div dir="ltr">Hi<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 22, 2014 at 7:19 PM, John Brooks <span dir="ltr"><<a href="mailto:john.brooks@jolla.com" target="_blank">john.brooks@jolla.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Jan 22, 2014, at 2:48 AM, David Edmundson <<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>> wrote:<br>

> Qt5 support is /meant/ to work, There are code paths that search for<br>
> both and did work at some point. Given the dates this was possibly<br>
> made before 5.0 was officially out, it seems in later Qt/Cmake<br>
> releases have broken this.<br>
<br>
Looks like you’re right. That port must’ve been done very early in Qt5 development, before the major API breaks. I wonder why the ubuntu packaging chose such a difficult method instead.<br></blockquote><div><br></div><div>
David is correct. When we did those changes, we were porting lots of code over to Qt5 (mostly code related to Ubuntu Touch), and the Qt5 support in Telepathy Qt was not working with the versions of Qt5 we were using back then. We fixed the problems in a quick way, and we were to revisit the matter later, but that ended up never happening (you know those temporary fixes that end up lasting forever?).</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> I would really not advise a qmake port. Have a look at the size of<br>
> cmake/modules. It's huge. The equivalent in qmake isn't exactly any<br>
> easier and that's a lot of porting.<br>
<br>
Oh, I agree. That idea just comes from me not knowing how hard it is to do dual-building Qt4/Qt5 in cmake. I’m very happy we don’t need to do it.<br>
<br>
> and done:<br>
><br>
> <a href="http://pastebin.kde.org/pf41bfc3e" target="_blank">http://pastebin.kde.org/pf41bfc3e</a><br>
><br>
> Builds on Qt4.8 and Qt5-master.<br>
> Could you test that it works for you.<br>
<br>
Wow, much easier than I expected. A few more changes were needed to build tests, and we have a few patches to fix test execution under Qt5 as well. I put all of that at:<br>
<br>
<a href="https://github.com/special/telepathy-qt-upstream/commits/qt5" target="_blank">https://github.com/special/telepathy-qt-upstream/commits/qt5</a><br>
<br>
That branch builds for me on Qt4 and Qt5 with tests and optional dependencies. If you can submit your patch plus the tests build fix (feel free to squash), I’ll submit our test fixes separately, and we should have a fully working upstream.<br>
</blockquote><div><br></div><div>Thanks David and John for fixing this.</div><div><br></div><div>Cheers</div><div><br></div><div>Boiko</div></div></div></div>