<table><tr><td style="">dfaure 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/D7008" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Why are those executables even installed? For crosscompiling maybe? I'm confused about how that would work. One would build sonnet for host, install it, and then build sonnet for target, importing targets from the host build?<br />
Isn't that "import" what this commit just broke?</p>

<p>Maybe the INSTALL lines should be inside the else, there's no need to install the compiled-for-target executables....</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);">(AFAICS they are internal, not used outside sonnet at all)</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R246 Sonnet</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<br /><strong>Cc: </strong>dfaure<br /></div>