<table><tr><td style="">palokisa 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/D5521" rel="noreferrer">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;"><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>What is <tt style="background: #ebebeb; font-size: 13px;">KF5::Service</tt> used for? You don't seem to be <tt style="background: #ebebeb; font-size: 13px;">#ifdef</tt> it. Maybe it's not needed at all?</p></blockquote>

<p>I don't know. I didn't investigate that.....I thought, it's some kind of "self-initializing" component and it is enough to link the library.</p></blockquote>

<p>I don't think it's the case for KService (it is for KCrash though). Please investigate (or wait for feedback).</p></blockquote>

<p>Then I believe, it's only some remnant, because it's not used anywhere in the code.</p></blockquote>



<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>... but looking at the fingerprints of packages (in debian), the libkf5crash5 is very light. So not real disadvantage to pull this dependency. So the KCoreAddons can be seen as more problematic dependency (from size point of view).</p></blockquote>

<p>Should I close this request and create a new one for the KF5::Service removal?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R268 KGlobalAccel</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5521" rel="noreferrer">https://phabricator.kde.org/D5521</a></div></div><br /><div><strong>To: </strong>palokisa, graesslin, mck182<br /><strong>Cc: </strong>apol, Frameworks<br /></div>