Re: [Rpm-maint] [rpm-software-management/rpm] Bunch of macro fixes and cleanups (#1062)

2020-02-10 Thread Panu Matilainen
Merged #1062 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/1062#event-3022555693___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Bunch of macro fixes and cleanups (#1062)

2020-02-10 Thread Panu Matilainen
Thanks for the review :)

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


Re: [Rpm-maint] [rpm-software-management/rpm] Bunch of macro fixes and cleanups (#1062)

2020-02-10 Thread Michael Schroeder
mlschroe approved this pull request.

Nice!



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


[Rpm-maint] [rpm-software-management/rpm] Bunch of macro fixes and cleanups (#1062)

2020-02-10 Thread Panu Matilainen
Fix double-expansion of arguments to built-in macros (except for %{expand:..} 
whose sole purpose is to double-expand) refactoring the more special cases out 
of doFoo() and fixing misc other related bugs in the process. Details in the 
individual commit messages.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/1062

-- Commit Summary --

  * Fix buffer overflow in %{S:...} and %{P:...} macro primitives
  * Fix %{uncompress:...} double-expanding arguments + other miscellania
  * Refactor %{expand:...} handling to its own function
  * Dont double-expand arguments to built-in macros (#311)
  * Refactor %{verbose:...} handling out of doFoo()

-- File Changes --

M rpmio/macro.c (195)
M tests/rpmmacro.at (20)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/1062.patch
https://github.com/rpm-software-management/rpm/pull/1062.diff

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