<table><tr><td style="">sitter added inline comments.
</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/D10446">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10446#inline-96807">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kf5_entry.desktop:1</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This is going to be a problem, scripty is going to come and whipe these .desktop files out and then make the translators translate them again.</p>

<p style="padding: 0; margin: 8px;">Wonder if we could rename them to .desktop.untransltable or something like that and then use the cmake file copy command to copy them to the build folder and trick the test to find the files there?</p>

<p style="padding: 0; margin: 8px;">Am i making any sense?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Really good point. I've had a look and we only extract src/*. In fact, we only have Messages.sh in src :)</p>

<p style="padding: 0; margin: 8px;">This is in line with other frameworks where we have desktop file fixtures. They all only extract src/ and use regular desktop files for test fixtures (e.g. kservice, kpackage, kparts). So, this should be fine.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D10446#inline-96806">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">klanguagenametest.cpp:32</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">initTestCase is too late and was failing for me</p>

<p style="padding: 0; margin: 8px;"><a href="https://paste.kde.org/p8s9js4r3" class="remarkup-link" target="_blank" rel="noreferrer">https://paste.kde.org/p8s9js4r3</a></p>

<p style="padding: 0; margin: 8px;">makes it work</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">LGTM, I've added the change to the diff. Thanks.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R265 KConfigWidgets</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10446">https://phabricator.kde.org/D10446</a></div></div><br /><div><strong>To: </strong>aacid<br /><strong>Cc: </strong>hein, kde-frameworks-devel, sitter, markg, apol, michaelh, ngraham, bruns<br /></div>