<table><tr><td style="">apol accepted this revision.<br />apol added a comment.<br />This revision is now accepted and ready to land.
</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/D7008" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>This patch is correct, EXPORT is only useful if the target is used within cmake, AFAIU.</p>

<p>Furthermore, we should modify sonnet to install these applications into LIBEXEC rather than bin, otherwise we can't have both architectures installed along with the library that is going to use it (at least on some linux distros, e.g. debian based), but this we can do in a separate patch.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R246 Sonnet</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7008" rel="noreferrer">https://phabricator.kde.org/D7008</a></div></div><br /><div><strong>To: </strong>vkrause, Frameworks, cordlandwehr, apol<br /><strong>Cc: </strong>apol, sitter, sandsmark, habacker, dfaure<br /></div>