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

[Michael Schroeder]

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:
https://github.com/rpm-software-management/rpm/issues/834#issuecomment-531171559
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20190913/3725c4c1/attachment.html>