<table><tr><td style="">kossebau 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/D28360">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28360#636615" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D28360#636615</a>, <a href="https://phabricator.kde.org/p/eszlari/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@eszlari</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
color: #6B748C;
font-style: italic;
margin: 4px 0 12px 0;
padding: 8px 12px;
background-color: #F8F9FC;">
<div style="font-style: normal;
padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D28360#636610" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D28360#636610</a>, <a href="https://phabricator.kde.org/p/kossebau/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@kossebau</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Can you tell which packagers missed that this is a required runtime dependency?</p></div>
</blockquote>
<p><a href="https://packages.ubuntu.com/focal/kde-config-gtk-style" class="remarkup-link" target="_blank" rel="noreferrer">https://packages.ubuntu.com/focal/kde-config-gtk-style</a><br />
<a href="https://src.fedoraproject.org/rpms/kde-gtk-config/blob/master/f/kde-gtk-config.spec" class="remarkup-link" target="_blank" rel="noreferrer">https://src.fedoraproject.org/rpms/kde-gtk-config/blob/master/f/kde-gtk-config.spec</a><br />
<a href="https://gitweb.gentoo.org/repo/gentoo.git/tree/kde-plasma/kde-gtk-config/kde-gtk-config-5.18.3.ebuild" class="remarkup-link" target="_blank" rel="noreferrer">https://gitweb.gentoo.org/repo/gentoo.git/tree/kde-plasma/kde-gtk-config/kde-gtk-config-5.18.3.ebuild</a></p></div>
</blockquote>
<p>Okay, that is quite a few. Though, I wonder why they missed it. The current addition to RUNTIME dependency was only added post 5.18, right? And no-one told the packagers explicitly otherwise by the usual ways (like in release announcement). They do read the cmake log though, and usually ask for that listing as RUNTIME dependency.</p>
<p>So the current state with those packages might not be a reason for the proposed noisy approach.</p>
<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>Edit: The proposed solution to make it simply required only forces packagers to add that and all what it pulls in as build-time dependency to their package creation, even if not needed for the package creation itself at all, which is not something packagers like (wastes resources on package build servers).</p></blockquote>
<p>But they don't have to! They can just compile with</p>
<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">cmake -DENABLE_XSETTINGSD=OFF ...</pre></div></blockquote>
<p>They can, but <tt style="background: #ebebeb; font-size: 13px;">ENABLE_XSETTINGSD</tt> needs extra work, makes things more complex, and the term is also misleading (does it enable support for the demon?). One has to know detais.</p>
<p>All in all, this is an unusual approach. Best talk to packagers what they prefer and expect, instead of more complex code based on assumption of how packagers operate. Done that before myself, and pure listing as RUNTIME dep always worked out :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R99 KDE Gtk Configuration Tool</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28360">https://phabricator.kde.org/D28360</a></div></div><br /><div><strong>To: </strong>eszlari, gikari<br /><strong>Cc: </strong>kossebau, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>