<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://svn.reviewboard.kde.org/r/6113/">http://svn.reviewboard.kde.org/r/6113/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdeutils and Raphael Kubo da Costa.</div>
<div>By binro.</div>
<p style="color: grey;"><i>Updated 2010-12-14 15:19:02.896447</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description (updated)</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;">Add ark plugin for dar (Disk Archiver http://dar.linux.free.fr/).
Your suggestion to make isReadOnly() return false did not seem to make any difference. I think the simplest way is check if the AddProgram and DeleteProgram parameters are actually set and issue an error message if not. This check probably should have been done anyway! This way plugins can indicate which operations they support.</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;">Listing archives and extracting files works fine locally using KDE 4.5.4.</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>/trunk/KDE/kdeutils/ark/kerfuffle/cliinterface.h <span style="color: grey">(1206114)</span></li>
<li>/trunk/KDE/kdeutils/ark/kerfuffle/cliinterface.cpp <span style="color: grey">(1206114)</span></li>
<li>/trunk/KDE/kdeutils/ark/plugins/CMakeLists.txt <span style="color: grey">(1206114)</span></li>
<li>/trunk/KDE/kdeutils/ark/plugins/clidarplugin/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/KDE/kdeutils/ark/plugins/clidarplugin/cliplugin.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/KDE/kdeutils/ark/plugins/clidarplugin/cliplugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/KDE/kdeutils/ark/plugins/clidarplugin/kerfuffle_clidar.desktop <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/6113/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://svn.reviewboard.kde.org/r/6113/s/579/"><img src="http://svn.reviewboard.kde.org" style="border: 1px black solid;" alt="Ark showing dar archive" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>