Gentoo Archives: gentoo-portage-dev

From: Zac Medico <zmedico@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCH] SpawnProcess: fix event loop recursion in _pipe_logger_exit (bug 613990)
Date: Mon, 27 Mar 2017 21:42:10
Message-Id: CAMiTYSo9TMcx+ouX=5QHHHXy33J_Qd1JkXujC8Bhmkza3NxE2g@mail.gmail.com
In Reply to: Re: [gentoo-portage-dev] [PATCH] SpawnProcess: fix event loop recursion in _pipe_logger_exit (bug 613990) by Brian Dolbec
1 On Mon, Mar 27, 2017 at 12:09 PM, Brian Dolbec <dolsen@g.o> wrote:
2 > On Sun, 26 Mar 2017 23:54:37 -0700
3 > Zac Medico <zmedico@g.o> wrote:
4 >
5 >> Fix SpawnProcess._pipe_logger_exit to wait for process exit status
6 >> asynchronously, in order to avoid event loop recursion. This is
7 >> required for asyncio compatibility, and also protects emerge from
8 >> exceeding the maximum recursion depth limit like in bug 402335.
9 >>
10 >> X-Gentoo-bug: 613990
11 >> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=613990
12 >> ---
13 >> pym/_emerge/SpawnProcess.py | 3 +--
14 >> pym/_emerge/SubProcess.py | 23 ++++++++++++++++++++++-
15 >> 2 files changed, 23 insertions(+), 3 deletions(-)
16 >
17 > looks fine
18
19 Thanks, pushed:
20
21 https://gitweb.gentoo.org/proj/portage.git/commit/?id=eaf22a6d88ad8e0b7a3a1e21f3234c6b7037018a
22 --
23 Thanks,
24 Zac