<table><tr><td style="">heikobecker 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/D29136">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/D29136#660270" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D29136#660270</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>using kdeinstalldirs variables needs to ensure that KDEInstallDirs has been included before, also introduces ahard dependency on ECM for any users of KI18n. While 99% of apps using KI18n might do this, by design idea of KDE Frameworks KI18n as tier1 should not pull in another dependency, even ECM (so someone using plain cmake & GnuInstallDirs should be still able to use tier1 stuff). This needs some more pondering then...</p></div>
</blockquote>
<p>Isn't that a mere theoretical concern? At least when building ki18n yourself, you need ECM anyway, which is easy to install and almost infinitesimal small in size compared to Qt. And btw, where is the "should not pull in another dependency, even ECM" documentend? I only know of "Tier 1 Frameworks can depend only on Qt official frameworks or other system libraries" [1], which admittedly already creates a discrepancy with the ECM (build) requirement of ki18n itself.</p>
<p>I'd also note that the macro already used KDEInstallDirs before, apparently without anybody complaining about it, even though I don't want to cargo cult this. Furthermore I'm not sure how to solve the bug I encountered differently, other than making marble stop using KDE_INSTALL_DIRS_NO_DEPRECATED (or using GnuInstallDirs, which possibly might break existing things).</p>
<p>[1] <a href="https://community.kde.org/Frameworks/Policies" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Frameworks/Policies</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R249 KI18n</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29136">https://phabricator.kde.org/D29136</a></div></div><br /><div><strong>To: </strong>heikobecker<br /><strong>Cc: </strong>kossebau, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>