<table><tr><td style="">sitter 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/D29461">View Revision</a></tr></table><br /><div><div><p>Pid changes look fine, though perhaps we should just throw those two lines away? With Qt5 logging the pid is fairly pointless because one can simply set QT_MESSAGE_PATTERN to include the pid when necessary <a href="https://doc.qt.io/qt-5/qtglobal.html#qSetMessagePattern" class="remarkup-link" target="_blank" rel="noreferrer">https://doc.qt.io/qt-5/qtglobal.html#qSetMessagePattern</a></p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29461#inline-168739">View Inline</a><span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:26</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; ">   Qt5::Network
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">   <span class="bright">ssh</span>)
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">   <span class="bright">${LIBSSH_LIBRARIES}</span>)
</div><div style="padding: 0 8px; margin: 0 4px; ">set_target_properties(kio_sftp PROPERTIES OUTPUT_NAME "sftp")
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Hm, I am a bit hazy on the details but I think this changes makes no sense. libssh (upstream) introduced an imported target <tt style="background: #ebebeb; font-size: 13px;">ssh</tt>. For backwards compatibility we also inject this target when building with older libssh's than the latest (to be honest though, with libssh you basically always want the latest or you'll have an incredibly subpar experience).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R320 KIO Extras</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29461">https://phabricator.kde.org/D29461</a></div></div><br /><div><strong>To: </strong>brute4s99, vonreth, meven<br /><strong>Cc: </strong>sitter, meven, kde-frameworks-devel, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, rdieter, mikesomov<br /></div>