<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/119830/">https://git.reviewboard.kde.org/r/119830/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On marts 24th, 2015, 12:18 p.m. EET, <b>Kevin Funk</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Would be nice if you could go through the other KDevelop-related repos and check if they still compile with this change.</p></pre>
</blockquote>
<p>On marts 24th, 2015, 2:06 p.m. EET, <b>Dāvis Mosāns</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Any others should I check?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Theese build fine:</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdevelop-pg-qt</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-clang</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-php</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-php-docs</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-python</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-qmljs</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-upload</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-valgrind</li>
</ul>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">These doesn't build
because of this patch, I'll fix them:</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-cppcheck</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-qmake</li>
</ul>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">These doesn't build for me
because of some other issue not related to this patch:</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-executebrowser</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-perforce</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-ruby</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">kdev-xdebug</li>
</ul></pre>
</blockquote>
<p>On marts 24th, 2015, 3:38 p.m. EET, <b>Miquel Sabaté Solà</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">@D?vis kdev-ruby builds just fine for me with both gcc 4.9.2 and clang 3.6 (with and without your patch applied). What issues are you having ?</p></pre>
</blockquote>
<p>On marts 24th, 2015, 3:48 p.m. EET, <b>Miquel Sabaté Solà</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">aaaand I was wrong :_) It should be ok now.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I had different issue, this is error I got</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><div class="codehilite" style="background: #f8f8f8"><pre style="line-height: 125%">[ 5%] Building CXX object parser/CMakeFiles/kdevrubyparser.dir/parser.cpp.o
cc1plus: error: /home/build/KDE/playground/devtools/plugins/kdev-ruby/include/kdevplatform: No such file or directory [-Werror]
cc1plus: error: /home/build/KDE/playground/devtools/plugins/kdev-ruby/libs: No such file or directory [-Werror]
cc1plus: all warnings being treated as errors
parser/CMakeFiles/kdevrubyparser.dir/build.make:108: recipe for target 'parser/CMakeFiles/kdevrubyparser.dir/parser.cpp.o' failed
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I created those folders and it does compile now, but idk why it looks for them...</p></pre>
<br />
<p>- Dāvis</p>
<br />
<p>On marts 21st, 2015, 6:19 a.m. EET, Dāvis Mosāns wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDevelop.</div>
<div>By Dāvis Mosāns.</div>
<p style="color: grey;"><i>Updated marts 21, 2015, 6:19 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdevplatform
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Remove deprecated IProject methods. No need to keep them, when KUrl will be changed to QUrl signatures would be different anyway, so just remove now.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Tests does pass :)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">but note that KDevelop must also be updated and currently it won't compile against this because it still uses these deprecated methods.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>interfaces/iproject.h <span style="color: grey">(bdfd66557dc9d9eebe4f8773e0428a8183ec1825)</span></li>
<li>plugins/execute/nativeappconfig.cpp <span style="color: grey">(9c83db9e021201b0732ce186423a3d922458262d)</span></li>
<li>plugins/executescript/scriptappconfig.cpp <span style="color: grey">(947cc464a42e3c18c38034287c93bdcb4ae8726c)</span></li>
<li>plugins/externalscript/externalscriptjob.cpp <span style="color: grey">(e65516d65e9315129bb8a9efc6d8897b4413cde6)</span></li>
<li>plugins/filetemplates/templateclassassistant.cpp <span style="color: grey">(8681590b8e15ed1f3ceb15cba118733eb67c7e06)</span></li>
<li>plugins/grepview/grepdialog.cpp <span style="color: grey">(4671c30bcdb75699d0cd474da78cfbf15eadf535)</span></li>
<li>plugins/projectmanagerview/projectmanagerview.cpp <span style="color: grey">(95ae9a90273ec4f4be57fdc5e526e4416440788f)</span></li>
<li>plugins/projectmanagerview/vcsoverlayproxymodel.cpp <span style="color: grey">(62e632c9c6e243d4a34626837f9232b3a3cc3f2a)</span></li>
<li>project/projectmodel.h <span style="color: grey">(e7513ff99cdb8f1724828e7e4d338c0fdcf1a7fa)</span></li>
<li>project/projectmodel.cpp <span style="color: grey">(49cd984bdcbba661fedd8fbf42912b43fa28b733)</span></li>
<li>shell/project.h <span style="color: grey">(aec5be208c058a17540b3646dd4a8a0a12a29750)</span></li>
<li>shell/project.cpp <span style="color: grey">(e7e6efa462e8000c566af265d441464b5ab5cdce)</span></li>
<li>shell/projectcontroller.cpp <span style="color: grey">(50bdce4b47c16a4e62c8b7014897a6ae12d6bba6)</span></li>
<li>tests/testproject.h <span style="color: grey">(06757db24be0a356f4a0aee27f07899b6ea906b7)</span></li>
<li>tests/testproject.cpp <span style="color: grey">(39a6d0be6505fcb7f5e130f27481fe02b89a7446)</span></li>
<li>vcs/models/brancheslistmodel.cpp <span style="color: grey">(0737f8bcb8f855ac0901be6885428721218acefb)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119830/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>