<blockquote>
<p>The error messages are simply the same as you get from spec %if conditionals, there are no messages added / changed in this PR. That's not to say the messages couldn't maybe be improved, but any change needs to account for the fact that they're shared between two quite different contexts.</p>
</blockquote>
<p>Not  entirely. If there is a problem in spec %if conditional, then two error messages are emitted e.g.:</p>
<pre><code>error: syntax error while parsing ==
error: /data/SPECS/iftest.spec:22: bad %if condition:  =
</code></pre>
<p>But if the same wrong conditional is used after <em>%{expr</em> (e.g. %{expr:=}) only the first error message is emitted:</p>
<pre><code>error: syntax error while parsing ==
</code></pre>
<blockquote>
<p>There was an actual bug wrt errors in that expression errors did not raise a macro-level error. Fixed now.</p>
</blockquote>
<p>Yes this is corrected now.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/rpm-software-management/rpm/pull/817?email_source=notifications&email_token=ADLPZU2CLQWIAOYIXECQE5LQFTRFDA5CNFSM4IL4A7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4YURHY#issuecomment-523323551">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ADLPZUYI2U5356XNC7QCBDLQFTRFDANCNFSM4IL4A7UQ">mute the thread</a>.<img src="https://github.com/notifications/beacon/ADLPZU4GYPT7357W5GAY6MDQFTRFDA5CNFSM4IL4A7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4YURHY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/rpm-software-management/rpm/pull/817?email_source=notifications\u0026email_token=ADLPZU2CLQWIAOYIXECQE5LQFTRFDA5CNFSM4IL4A7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4YURHY#issuecomment-523323551",
"url": "https://github.com/rpm-software-management/rpm/pull/817?email_source=notifications\u0026email_token=ADLPZU2CLQWIAOYIXECQE5LQFTRFDA5CNFSM4IL4A7U2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4YURHY#issuecomment-523323551",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>