<table><tr><td style="">dfaure accepted this revision.<br />dfaure 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/D3287" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The error message is a bit confusing, it sounds like the right file is ${_tmp_FILE}.<br />
So I would suggest this instead</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);">message(WARNING "${_tmp_FILE}: Broken \"File\" field, make sure it's pointing at the right file")</pre></div>

<p>Also, shouldn't the regexp be File=(.*)\n ? Basically, anything in the File field that is not *.kcfg should lead to "Broken File Field", right?<br />
So in fact this is a bugfix (not "old broken behaviour") because it gives a better error message in case the file says File=foo : it will say "Broken File Field" instead of "Couldn't read the File field" (which would send people on the wrong track ... the field is here, it can be read, it just has the wrong contents).</p></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/D3287" rel="noreferrer">https://phabricator.kde.org/D3287</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, dfaure<br /></div>