[Rpm-maint] [rpm-software-management/rpm] expression expansion (#834)

Michael Schroeder notifications at github.com
Wed Sep 11 10:04:08 UTC 2019

There's a lot of expansion going on in the %expr macro. Witness this:

./rpm --eval '%{expr: "%%%%%%%%"}'

The string gets expanded
- at the start of doFoo when it expands the argument
- in rdToken when it parses a string
- at the end of doFoo  because of issue #313

I'm somewhat surprised that rdToken expands strings and that this hasn't messed up some `%if` statements yet.
It's also somewhat inconsistent that the integer case does not do expansion.

I kind of like to have the expansion in rdToken instead of doFoo because:

- It makes `"%str"` work when %str contains a `"` character.
- It allows to have short circuit semantics

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190911/ff0cabc7/attachment.html>

More information about the Rpm-maint mailing list