<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/107536/">http://git.reviewboard.kde.org/r/107536/</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;">Tjanks a lot Pino for your latest great Marble patches. This patch looks like a good idea to solve the QExtSerialDevice issue (And lazlo is right that in the future we should move to or offer parallel support for QSerialPort :-)). Hm, actually we should enable this aprs plugin by default :)</pre>
<br />
<p>- Torsten</p>
<br />
<p>On November 30th, 2012, 8:24 p.m., Pino Toscano 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 kdewin, Marble, Release Team, and Wes Hardaker.</div>
<div>By Pino Toscano.</div>
<p style="color: grey;"><i>Updated Nov. 30, 2012, 8:24 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;">Instead of embedding an (old) copy of the QextSerialPort library, find for an external one; only if found enable the reading from TTY, which is otherwise disabled (leaving its configuration tab disabled).
The drop of the internal QextSerialPort should also fix all the portability issues, since the plugin itself does not use any OS-dependent API, and it is then reenabled unconditionally.
Hence, bug 241125 should now be fixed, and bug 237931 and bug 242039 should not happen anymore.
@release-team: yes, I know this would introduce a new optional dependency, but on the other hand a copy of a 3rd party library would go away. Would this be acceptable at this point?</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;">The Aprs plugin compiles fine with and without an external QextSerialPort library.</pre>
</td>
</tr>
</table>
<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=241125">241125</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>cmake/modules/FindQextSerialPort.cmake <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugins/render/CMakeLists.txt <span style="color: grey">(d82293ee782e735ff1c90e6e13d330fb7cf8563c)</span></li>
<li>src/plugins/render/aprs/AprsPlugin.cpp <span style="color: grey">(f406cec2ad665977830416aa7f5df59851a5e430)</span></li>
<li>src/plugins/render/aprs/AprsTTY.cpp <span style="color: grey">(c65ac38b24269b608c8f3ea1452b670f9422174d)</span></li>
<li>src/plugins/render/aprs/CMakeLists.txt <span style="color: grey">(fb6ef13c80568a72a5bfcf8a2e675b969238b9f6)</span></li>
<li>src/plugins/render/aprs/aprsconfig.h.in <span style="color: grey">(d0e6b5c4ce36080dc0e59422529c55728ff04b3a)</span></li>
<li>src/plugins/render/aprs/posix_qextserialport.cpp <span style="color: grey">(118843f02a5c62fd708b9157e59a039dff06e238)</span></li>
<li>src/plugins/render/aprs/qextserialport.h <span style="color: grey">(457d831cffc4ae8c43ac7db2d85a20546eb65044)</span></li>
<li>src/plugins/render/aprs/qextserialport.cpp <span style="color: grey">(790e5a2701ba1291a645c4fd4b09a8a1c55d7541)</span></li>
<li>src/plugins/render/aprs/qextserialport_global.h <span style="color: grey">(013a6dcd4ecab97425b1286139af4f0e911c38c9)</span></li>
<li>src/plugins/render/aprs/win_qextserialport.cpp <span style="color: grey">(5f21d7302e61b50825f79a68b352d5b9544b3fa3)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107536/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>