<table><tr><td style="">ahmadsamir added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24929">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D24929#557643" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D24929#557643</a>, <a href="https://phabricator.kde.org/p/vkrause/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@vkrause</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I think we cannot ifdef the two cases in this way, we need to assume that the 19.08.3 release is not rebuild by distros when updating to the 5.65 KF release.<br />
 That is, if the KF version is higher than 5.65 we can drop the KTcpSocket case, but not the other way around. In that case we need to check which socket type we get at runtime (ie. cast to both types and check which one succeeds).</p></div>
</blockquote>

<p>Right, that makes sense, KIO_VERSION is checked at build time not run time. I'll update the diff.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R44 KDE PIM Runtime</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24929">https://phabricator.kde.org/D24929</a></div></div><br /><div><strong>To: </strong>ahmadsamir, mlaurent<br /><strong>Cc: </strong>vkrause, kde-pim, fbampaloukas, dvasin, rodsevich, winterz, mlaurent, knauss, dvratil<br /></div>