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

Michael Schroeder notifications at github.com
Fri Sep 13 09:37:13 UTC 2019

Here's my proposal:

 1) turn off default string expansion in rdToken
     This is an incompatible change, but it's clearly the right thing and I can't imaging somebody
     actually using this in a %if statement

 2) Add optional expansion for the integer case

 3) Add a new macro that does not expand the macro body in `doFoo`, but instead calls rpmExprStr with a flag that enables expansion in rdToken.
    We could use `%{(  )}` as new macro, as proposed in the thread about the `? :` operator.

4) Add '? : ' support to the expression expansion code

With that changes we can do `%{( %foo ? "%bar" : "%baz" )}`

Does that make sense? Should I create a pull request?

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/20190913/3725c4c1/attachment.html>

More information about the Rpm-maint mailing list