1 |
On 11/25/18 9:54 AM, Brian Dolbec wrote: |
2 |
> On Fri, 23 Nov 2018 16:15:23 -0800 |
3 |
> Zac Medico <zmedico@g.o> wrote: |
4 |
> |
5 |
>> Use portage.process.spawn (with new cwd parameter) and |
6 |
>> self.spawn_kwargs to drop privileges for git gc and merge commands. |
7 |
>> |
8 |
>> Fixes: 3cd8cf93abb6 ("GitSync: abort checkout for signature problem |
9 |
>> (bug 660372)") Fixes: 903c4b1a6768 ("GitSync: support sync-depth (bug |
10 |
>> 552814)") Bug: https://bugs.gentoo.org/669496 |
11 |
>> Signed-off-by: Zac Medico <zmedico@g.o> |
12 |
>> |
13 |
>> Zac Medico (2): |
14 |
>> portage.process.spawn: add cwd parameter |
15 |
>> git: drop privileges for gc and merge (bug 669496) |
16 |
>> |
17 |
>> lib/portage/process.py | 13 ++++++++++--- |
18 |
>> lib/portage/sync/modules/git/git.py | 10 ++++++---- |
19 |
>> lib/portage/tests/sync/test_sync_local.py | 22 ++++++++++++++++++++-- |
20 |
>> 3 files changed, 36 insertions(+), 9 deletions(-) |
21 |
>> |
22 |
> |
23 |
> This looks good to me :) |
24 |
|
25 |
Thanks, merged: |
26 |
|
27 |
https://gitweb.gentoo.org/proj/portage.git/commit/?id=0490cfa00afcf1347e4e72528b20c93648d6871c |
28 |
-- |
29 |
Thanks, |
30 |
Zac |