<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 3, 2014 at 12:03 AM, Sven Brauch <span dir="ltr"><<a href="mailto:svenbrauch@googlemail.com" target="_blank">svenbrauch@googlemail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div class="">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border:1px #c9c399 solid">
<tbody><tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/113119/" target="_blank">https://git.reviewboard.kde.org/r/113119/</a>
</td>
</tr>
</tbody></table>
<br>
</div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">I pushed this after talking to the patch author in IRC. Since it fixes breakage and was found to be ok, I think it makes sense to not let it rot, even if it is not sufficient to make the plugin usable again.</pre>
<span class="HOEnZb"><font color="#888888">
<br>
<p>- Sven Brauch</p></font></span><div class="">
<br>
<p>On April 2nd, 2014, 10:01 p.m. UTC, Dāvis Mosāns wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image:url('');background-repeat:repeat-x;border:1px black solid">
<tbody><tr>
<td>
<div>Review request for KDevelop and Sandro Andrade.</div>
<div>By Dāvis Mosāns.</div>
<p style="color:grey"><i>Updated April 2, 2014, 10:01 p.m.</i></p>
<div style="margin-top:1.5em">
<b style="color:#575012;font-size:10pt">Repository: </b>
kdev-control-flow-graph
</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">
<tbody><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">libgraph is deprecated in GraphViz 2.30 and above. It's replaced with libcgraph. Fedora 19 ships with GraphViz 2.30.1 and thus doesn't include libgraph so it wasn't possible to compile. Luckily there was required only very little changes.</pre>
</td>
</tr>
</tbody></table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">
<li>CMakeLists.txt <span style="color:grey">(0904ba858da16c58ca637947ba7e698ddaf341bf)</span></li>
<li>cmake/modules/FindGraphViz.cmake <span style="color:grey">(c542b05a06e90068ff8a2812c611f6bce2288913)</span></li>
<li>dotcontrolflowgraph.cpp <span style="color:grey">(7d803a5e2992fbb9def3f7f09a50599fb7c15772)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/113119/diff/" style="margin-left:3em" target="_blank">View Diff</a></p>
</td>
</tr>
</tbody></table>
</div></div>
</div>
<br>_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kde.org">KDevelop-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kdevelop-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kdevelop-devel</a><br>
<br></blockquote></div><br></div><div class="gmail_extra">+1</div></div>