Re: [Rpm-maint] [rpm-software-management/rpm] Implement %{macrobody:...} built-in for retrieving the literal macro body (#1064)

2020-02-19 Thread Florian Festi
Merged #1064 into master.

-- 
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/1064#event-3051477151___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Implement %{macrobody:...} built-in for retrieving the literal macro body (#1064)

2020-02-19 Thread Panu Matilainen
Yeah it could go to doFoo(), but then I'd actually on a mission to eliminate 
the whole doFoo() (replaced by individual functions to do just one thing), so 
I'm trying to avoid adding more.
The expansion logic around g/gn could and probably should be handled centrally 
in the expandMacros() call-site.

-- 
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/1064#issuecomment-588171214___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Implement %{macrobody:...} built-in for retrieving the literal macro body (#1064)

2020-02-19 Thread Michael Schroeder
As it now expands the argument I think it should be handled in doFoo().

-- 
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/1064#issuecomment-588161577___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint