<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://git.reviewboard.kde.org/r/104943/">http://git.reviewboard.kde.org/r/104943/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ship It!</pre>
<br />
<p>- Boudewijn</p>
<br />
<p>On May 14th, 2012, 2:59 p.m., Friedrich W. H. Kossebau wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Calligra and Boudewijn Rempt.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated May 14, 2012, 2:59 p.m.</i></p>
<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;">The Applix Word import filter was both in the initial list sent by Boud of things to remove (and noone objected).
And it was also dealt with in the big cleanup commit 4254f88a2a816aa81e2d68e73348354814363960 . But only by having the "add_subdirectory( applixword )" removed, not the subdir filters/words/applixword, that is still present.
And so I found out (while looking at the mimetype handling in general) that the assumption that this filter has not yet been ported is wrong:
David Faure had ported it "to output ODF rather than old-kword format" over a year ago, see
https://projects.kde.org/projects/calligra/repository/revisions/22eb9828bf9f3a142928d157930aeea92d50df68
So this patch reverts the removal of the Applix Word import filter from the build and adds it again.
And without the old "if(NOT WIN32 AND NOT QT_MAC_USE_COCOA AND QT3SUPPORT)" condition as I could not find a reason to have these conditions.
</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;">Loaded a few .aw files found on the interwebs, looked more or less alright imported in Words.</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>filters/words/CMakeLists.txt <span style="color: grey">(4cda84c)</span></li>
<li>filters/words/applixword/CMakeLists.txt <span style="color: grey">(289e97e)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104943/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>