[Rpm-maint] [rpm-software-management/rpm] Implement support for literal and one-shot macro modifiers (PR #4129)

[Panu Matilainen]

@pmatilai commented on this pull request.



> +],
+[0],
+[123
+],
+[])
+
+RPMTEST_CHECK([
+rpm --define "aa 123" --define "mymod()<l> %aa" --eval "%mymod"
+],
+[1],
+[],
+[error: Macro %mymod has incompatible modifiers
+])
+
+RPMTEST_CHECK([
+rpm --define "aa 123" --define "mymod<lo> %aa" --eval "%mymod"

Mm, it does say the modifiers are incompatible, I'm not sure what it should say instead.
The modifier explanations could use improvement for sure - a better explanation would make it obvious that literal and one-shot are an incompatible combo without even explicitly stating that: one-shot macros expand their body once, literal macros never expand their body.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/4129#discussion_r2882787838
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/4129/review/3888267295 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20260304/f3e6aa09/attachment.htm>