<div dir="ltr">On Wed, Oct 2, 2013 at 10:25 AM, Stephen Kelly <span dir="ltr"><<a href="mailto:steveire@gmail.com" target="_blank">steveire@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">Aurélien Gâteau wrote:<br>
<br>
> On Tuesday 01 October 2013 19:28:04 Aleix Pol wrote:<br>
>> Hi,<br>
>> Since ReviewBoard is not working for me, I decided to send this review as<br>
>> an e-mail. I know it's less practical, but also I think it's important to<br>
>> get it done and I'd rather go reasonably fast with it before we start<br>
>> getting too much conflicts.<br>
<br>
</div>I for one a big reviewboard non-fan. I much prefer reviewing patches in<br>
email or outside git remotes.<br>
<div class="im"><br>
>><br>
>> This patch removes all find_package usages in the root CMakeLists.txt<br>
>> file (not the includes, only find_package) and moves it to every<br>
>> different module or unsplitted folder.<br>
><br>
> ENOPATCH :)<br>
<br>
</div>He pushed a branch to the kdelibs.git repo, which is unfortunate. There's so<br>
many dead branches in there, you'd be forgiven for not noticing. I wish it<br>
only was allowed to contain branches which contain releases and master. All<br>
of these 'personal work branches' should be in other remotes.<br>
<br>
Anyway, Aleix, it looks like a lot of the use of macro_bool_to_01 is<br>
obsolete.<br>
<br>
All configure_file uses of HAVE_X11 seem to already use #cmakedefine01.<br>
<br>
 git grep -e HAVE_X11 --and -e cmakedefine<br>
<br>
Replace the use of macro_bool_to_01 with set() first in a separate patch.<br>
<br>
 set(HAVE_X11 ${X11_FOUND})<br>
<br>
Why did you comment the HAVE_QSSLSOCKET check?<br>
<br>
Thanks,<br>
<br>
Steve.<br>
<div class=""><div class="h5"><br>
<br>
_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div><div class="gmail_extra">I pushed a couple of new commits that remove the usage of macro_bool_to_01 and the HAVE_QSSLSOCKET thing.</div><div class="gmail_extra"><br></div><div class="gmail_extra">

Aleix</div></div>