On Fri, 23 Nov 2018 16:15:23 -0800
Zac Medico <zmed...@gentoo.org> wrote:
> Use portage.process.spawn (with new cwd parameter) and
> self.spawn_kwargs to drop privileges for git gc and merge commands.
>
> Fixes: 3cd8cf93abb6 ("GitSync: abort checkout for signature problem
> (bug 660372)") Fixes: 903c4b1a6768 ("GitSync: support sync-depth (bug
> 552814)") Bug: https://bugs.gentoo.org/669496
> Signed-off-by: Zac Medico <zmed...@gentoo.org>
>
> Zac Medico (2):
> portage.process.spawn: add cwd parameter
> git: drop privileges for gc and merge (bug 669496)
>
> lib/portage/process.py | 13 ++++++++++---
> lib/portage/sync/modules/git/git.py | 10 ++++++----
> lib/portage/tests/sync/test_sync_local.py | 22 ++++++++++++++++++++--
> 3 files changed, 36 insertions(+), 9 deletions(-)
>
This looks good to me :)
