vkrause added a comment.
In D25401#564683 <https://phabricator.kde.org/D25401#564683>, @dfaure wrote: > As a data point: this commit changes things for kimap, which has code saying > > src/imapstreamparser.cpp:493: } else if (KTcpSocket *socket = qobject_cast<KTcpSocket *>(m_socket)) { > src/imapstreamparser.cpp-494- qWarning() << "No incoming packet for" << dt.elapsed()/1000 << "seconds on TCP socket. state=" << socket->state() << "error=" << socket->error() << socket->errorString(); > > > With the class being deprecated, this code now fails to build (because of the "-DKF_DISABLE_DEPRECATED_BEFORE_AND_AT=0x060000" big hammer). > The obvious fix is then to add this to kimap's CMakeLists.txt: > > add_definitions(-DKIOCORE_DISABLE_DEPRECATED_BEFORE_AND_AT=0x054000) # We need KTcpSocket > > > IMHO this is all correct. We *are* using a deprecated class, it's important to know it, even if we didn't instantiate it ourselves. Because this means there will be porting effort when the class is removed. > In some cases one can right away port to a non-deprecated solution, or in this case where we do need to keep support for older KF5 versions, we need to enable the use of the deprecated class for a little while longer. I don't see KTcpSocket in kimap? Or is that some older branch? Then that most definitely should not have the 6.0 deprecation version set. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D25401 To: dfaure, vkrause, kossebau Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
