| 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 |
Archives