[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