[Rpm-maint] [rpm-software-management/rpm] Support new %[ ] expression expansion syntax (#846)
Michael Schroeder
notifications at github.com
Fri Sep 20 14:27:10 UTC 2019
Ah, I get what you mean. I think we should print a bare word error if the expanded string does not start with a digit. This makes it similar to what `%{expr:...}` and `%if` does:
```
$ ./rpm --define "aaa a" --define "bbb 123b" --eval '%{expr: %aaa }'
error: bare words are no longer supported, please use "...": a
error: ^
$ ./rpm --define "aaa a" --define "bbb 123b" --eval '%[ %aaa ]'
error: macro expansion returned a bare word, please use "...": %aaa
error: ^
error: expanded string: a
$ ./rpm --define "aaa a" --define "bbb 123b" --eval '%[ %bbb ]'
error: macro expansion did not return an integer: %bbb
error: ^
error: expanded string: 123b
```
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/846#issuecomment-533576274
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190920/91638ddc/attachment.html>
More information about the Rpm-maint
mailing list