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 https://doc.qt.io/qt-5/qtglobal.html#qSetMessagePattern
Hm, I am a bit hazy on the details but I think this changes makes no sense. libssh (upstream) introduced an imported target ssh. 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).
Thank you for reviewing the rev, Hannah! I've merged the parent rev and hence updated libssh to use version 0.9.4 by default in Craft. If we can merge this we would get 🟢 for kio-extras builds. \o/