<table><tr><td style="">joselema created this revision.<br />joselema added reviewers: drosca, mweichselbaumer.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />joselema requested review of this revision.
</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/D22346">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Hi,</p>

<p>while doing packaging work on bluez-qt for Kubuntu, we detected the following situation:</p>

<p>Since the addition of this code:<br />
<a href="https://cgit.kde.org/bluez-qt.git/commit/?id=1a5660ab1547236c7bc0f883bbbb039889489a22" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/bluez-qt.git/commit/?id=1a5660ab1547236c7bc0f883bbbb039889489a22</a><br />
<a href="https://phabricator.kde.org/D21584" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D21584</a></p>

<p>there's a number of <strong>installed</strong> c++ headers which have the following include:</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>#include "bluezqt_dbustypes.h"</p></blockquote>

<p>however  bluezqt_dbustypes.h is NOT installed along with the headers which are suposed need it</p>

<p>An example of this is src/gattapplication.h:<br />
<a href="https://cgit.kde.org/bluez-qt.git/diff/src/gattapplication.h?id=1a5660ab1547236c7bc0f883bbbb039889489a22" class="remarkup-link" target="_blank" rel="noreferrer">https://cgit.kde.org/bluez-qt.git/diff/src/gattapplication.h?id=1a5660ab1547236c7bc0f883bbbb039889489a22</a></p>

<p>So, even while I don't understand the code involved, I have the impression that bluezqt_dbustypes.h must be installed along with the other headers. If that's the case I can commit and push the patch proposed.</p>

<p>Thank you for your time.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>We detected this thing because we have a test in our packaging which checks if the installed headers can be built and fails if they are missing bits. I ran the test in question with the attached patch and it doesn't fail.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R269 BluezQt</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22346">https://phabricator.kde.org/D22346</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>bluez-qt-5.60.0/src/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>joselema, drosca, mweichselbaumer<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns<br /></div>