<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/D27328">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/D27328#610115" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27328#610115</a>, <a href="https://phabricator.kde.org/p/jgrulich/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@jgrulich</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Looking at some other frameworks, they basically have same code to generate pri files, are they all broken?</p>

<p>For example: <a href="https://cgit.kde.org/bluez-qt.git/tree/src/CMakeLists.txt#n195" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/bluez-qt.git/tree/src/CMakeLists.txt#n195</a></p></div>
</blockquote>

<p>They are only broken if they miss a library they have in their public interface, for all of which there should be a qmake module mentioned in the <tt style="background: #ebebeb; font-size: 13px;">DEPS</tt> argument (unless transitively covered, like <tt style="background: #ebebeb; font-size: 13px;">core</tt> being already brought in by <tt style="background: #ebebeb; font-size: 13px;">gui</tt>).</p>

<p>Bluez-qt looks fine by a quick look at what is <tt style="background: #ebebeb; font-size: 13px;">PUBLIC</tt> argument in the <tt style="background: #ebebeb; font-size: 13px;">target_link_libraries(KF5BluezQt)</tt> call, Seems it does all cals via D-Bus and not have any libbluez in its publc interface.</p>

<p>Yet there surely are more case of broken <tt style="background: #ebebeb; font-size: 13px;">ecm_generate_pri_file</tt> calls. Seems many people just copy-paste-replaced the call, not checking the result :) While looking at this patch here, I e.g. came by prison having an even more broken call, in thei case though can be fixed, as again only other libraries are in the public interface for which there also is a qmake module pri file.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R282 NetworkManagerQt</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27328">https://phabricator.kde.org/D27328</a></div></div><br /><div><strong>To: </strong>kossebau, jgrulich<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>