<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-24 9:09 GMT+02:00 Kevin Funk <span dir="ltr"><<a href="mailto:kfunk@kde.org" target="_blank">kfunk@kde.org</a>></span>:<br><blockquote style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid" class="gmail_quote">




 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div>
   <table style="border:1px solid rgb(201,195,153)" cellpadding="8" width="100%" bgcolor="#f9f3c9">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/115506/" target="_blank">https://git.reviewboard.kde.org/r/115506/</a>
     </td>
    </tr>
   </tbody></table>
   <br>





</div><blockquote style="padding-left:10px;margin-left:1em;border-left-color:rgb(208,208,208);border-left-width:2px;border-left-style:solid"><div>
 <p style="margin-top:0px">On April 22nd, 2014, 9:55 p.m. UTC, <b>Milian Wolff</b> wrote:</p>
 </div><div><blockquote style="padding-left:10px;margin-left:1em;border-left-color:rgb(208,208,208);border-left-width:2px;border-left-style:solid">
  <pre style="white-space:pre-wrap;word-wrap:break-word">cool - but can you also add a unit test somehow? that would be awesome!</pre>
 </blockquote>




 </div><div><p>On April 24th, 2014, 3:06 a.m. UTC, <b>Todd Nowacki</b> wrote:</p>
 <blockquote style="padding-left:10px;margin-left:1em;border-left-color:rgb(208,208,208);border-left-width:2px;border-left-style:solid">
  <pre style="white-space:pre-wrap;word-wrap:break-word">Do we want to ship it?
>From the other discussion I thought we had decided that this was more of a bug with CMake than anything else. </pre>
 </blockquote>








</div></blockquote>

<pre style="white-space:pre-wrap;word-wrap:break-word">Personally, I'd say abandon it. It's adding too much logic in the filtering strategies. I'd rather try fixing this upstream and patch CMake to print absolute paths.

I've sent another mail to the CMake mailing list regarding the issue a month ago, but apparently there's not much interest from their side wrt fixing this issue (there was no real feedback). I think it'd best to just prepare a patch for CMake and see if it's being accepted.</pre>
</div></div></blockquote><div>+1</div><div> </div></div></div></div>