<table><tr><td style="">meven 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/D7446">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Conditionalize the code here with version ifdefs so that people only get both new entries when using kio-extras from 19.12.0</p></blockquote></blockquote>

<p>It is already the case : users won't get the new entries without recentlyused:/ installed, <tt style="background: #ebebeb; font-size: 13px;">KProtocolInfo::isKnownProtocol(QStringLiteral("recentlyused")</tt> takes care of that without ifdefs (still that is not a perfect solution).<br />
The code in KIO will stay dormant as a sort of runtime dependency, where as long as something is missing it stays almost non-present.</p>

<p>A nice consequence of moving recentlyused:/ to KIO is that it will hit users sooner than Applications 19.12 is released consequently.</p>

<p>But the feature in kactivitymanagerd <a href="https://phabricator.kde.org/D23112" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D23112</a>, its improvement <a href="https://phabricator.kde.org/D23737" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D23737</a> and gwenview <a href="https://phabricator.kde.org/D23741" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D23741</a> will only be part of 19.12, making look recentlyused:/files somewhat unfinished until 19.12 is released...<br />
We could backport the gwenview improvement to 19.08.x though.</p>

<p>I don't mind moving recentlyused:/ to KIO if we feel it necessary, it is not without consequences though.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7446">https://phabricator.kde.org/D7446</a></div></div><br /><div><strong>To: </strong>meven, Dolphin, broulik, elvisangelaccio, VDG, Frameworks, ngraham<br /><strong>Cc: </strong>meven, trickyricky26, andreask, huftis, svenmauch, kde-frameworks-devel, spoorun, andreaska, gregormi, markg, alexeymin, broulik, elvisangelaccio, dfaure, davidedmundson, ltoscano, Konqueror, iasensio, fprice, LeGast00n, MrPepe, fbampaloukas, alexde, GB_2, Codezela, feverfew, michaelh, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov<br /></div>