<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/120969/">https://git.reviewboard.kde.org/r/120969/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 4th, 2014, 3:55 p.m. UTC, <b>David Faure</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;">What does XDG have to do with data types? Do you mean XDR ?</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;">Ahh, didn't realise I had written XDG. It should be XDR :)</p></pre>
<br />
<p>- Mathias</p>
<br />
<p>On November 4th, 2014, 3:58 p.m. UTC, Mathias Tillman 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 KDE Frameworks and David Faure.</div>
<div>By Mathias Tillman.</div>
<p style="color: grey;"><i>Updated Nov. 4, 2014, 3:58 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kio-extras
</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;">Due to some missing XDR functions on OSX (used to decode unsigned long long), it failed to compile. This fixes that by doing some checks for the existence of the 64-bit datatype functions (for some reason XDR have 5 different functions that do the same thing).
This patch also merges the mount3 rpc code with the nfs3 code as well as remove the unused rpc mount2 code. </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;">I haven't personally tried to compile it on OSX, but Marko Käning (the one who reported it, https://www.mail-archive.com/kde-frameworks-devel@kde.org/msg19770.html) confirmed that it's now compiling fine on OSX. </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>nfs/rpc_mnt3.h <span style="color: grey">(cba4ff656a3d524f39cb7f607b8b4b331b41a0c5)</span></li>
<li>nfs/rpc_mnt2_xdr.c <span style="color: grey">(b380f4f93853ba54112a68ddf851f8f04b251312)</span></li>
<li>nfs/rpc_mnt2.x <span style="color: grey">(4aaf97de9ade2655cd48ae460ba24ee322b57183)</span></li>
<li>nfs/nfsv3.cpp <span style="color: grey">(368a61febc778688507ffb3954caaf69fc18d371)</span></li>
<li>nfs/rpc_nfs2_prot.x <span style="color: grey">(cd21123c7b57040d97245f22038153945abe88ee)</span></li>
<li>nfs/rpc_nfs2_prot.h <span style="color: grey">(62ab3056a51cc0e89536081c9d5741ff136dc804)</span></li>
<li>nfs/rpc_nfs3_prot_xdr.c <span style="color: grey">(57f4a8546ae697f41f98efb24c01b6de739d380f)</span></li>
<li>nfs/rpc_nfs3_prot.x <span style="color: grey">(fba417c8cd6edb955e018a89dd38942f1e788c69)</span></li>
<li>nfs/rpc_nfs3_prot.h <span style="color: grey">(76b2f5becef9cbdc9fd3db08c540c065a43a89a6)</span></li>
<li>nfs/rpc_nfs2_prot_xdr.c <span style="color: grey">(fb63b8ec2329730ef55c8f068e8abf47e865173b)</span></li>
<li>nfs/rpc_mnt3_xdr.c <span style="color: grey">(2016e688770e3753affe20923a8515f2427c5447)</span></li>
<li>nfs/rpc_mnt2.h <span style="color: grey">(ed21256c4754ce861ff1f5ea5ddc4ca224a7da7e)</span></li>
<li>nfs/kio_nfs.h <span style="color: grey">(9943daf670e665121ea51419ea77599ec4d8d4f5)</span></li>
<li>nfs/CMakeLists.txt <span style="color: grey">(2ed2186e113dad9e9c178cbac680d7c96dff789a)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120969/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>