<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/120839/">https://git.reviewboard.kde.org/r/120839/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 28th, 2014, 8:09 p.m. UTC, <b>Frank Schütte</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;">I'm not familiar with the policies. I resolved the raised issues, so should I review my own request or is this something stupid?</p></pre>
 </blockquote>




 <p>On October 28th, 2014, 8:59 p.m. UTC, <b>Thomas Lübking</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;">You're not supposed to approve your own patches (but if nobody ever feels in charge to approve it, "no veto" is good enough as well)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch seems to partially revert </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">commit c0681052c7d026507f799ea287fcaa3b4b9187ae
Author: Helio Chissini de Castro
Date:   Wed Sep 25 11:59:45 2013 -0300</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%"><span style="color: #666666">-</span> Automount Unknown device option has wrong logic, and the only way to
  make automount is by having a known device, turning the
  AutomountUnknownDevices option useless. Detected by
  george <span style="border: 1px solid #FF0000">'</span>d<span style="border: 1px solid #FF0000">ö</span>t<span style="border: 1px solid #FF0000">'</span> tavares <span style="border: 1px solid #FF0000">'ä</span>t<span style="border: 1px solid #FF0000">'</span> sabiaexperience <span style="border: 1px solid #FF0000">'</span>d<span style="border: 1px solid #FF0000">ø</span>t<span style="border: 1px solid #FF0000">'</span> com <span style="border: 1px solid #FF0000">'</span>d<span style="border: 1px solid #FF0000">ö</span>t<span style="border: 1px solid #FF0000">'</span> br
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">so I'd suggest ot attach at least Helio to get a comment on the intention.</p></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;">The problem is that review board now lists this as "approved", so nobody will look at it.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I added Helio to list of reviewers, but I haven't seen anything of him lately.</p></pre>
<br />










<p>- Christoph</p>


<br />
<p>On October 28th, 2014, 7:50 p.m. UTC, Frank Schütte 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 kdelibs, Solid and Christoph Feck.</div>
<div>By Frank Schütte.</div>


<p style="color: grey;"><i>Updated Oct. 28, 2014, 7:50 p.m.</i></p>







<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=243046">243046</a>, 

 <a href="http://bugs.kde.org/show_bug.cgi?id=261376">261376</a>


</div>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-runtime
</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;">Hi Christoph,
solid-device-automounter fails to automount unknown devices, even if it is checked,
because in kded/DeviceAutomounter.cpp it fails to evaluate shouldAutomountDevice.
So this patch corrects the logic to at first evaluate shouldAutomountDevice.
Inside this evaluation automountUnknownDevices is evaluated correctly.
Please review this patch. Christoph Feck urged me to submit this patch.
It took me quite a while to figure it out, though.
Bye,
Frank</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>solid-device-automounter/kded/DeviceAutomounter.cpp <span style="color: grey">(14b4e87)</span></li>

 <li>solid-device-automounter/lib/AutomounterSettings.cpp <span style="color: grey">(2b3e6be)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/120839/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>








  </div>
 </body>
</html>