commit:     200d876b01dd11521cdcd9bfa07abdca165d24e8
Author:     Zac Medico <zmedico <AT> gentoo <DOT> org>
AuthorDate: Wed Sep 23 16:16:45 2015 +0000
Commit:     Zac Medico <zmedico <AT> gentoo <DOT> org>
CommitDate: Wed Sep 23 22:31:36 2015 +0000
URL:        https://gitweb.gentoo.org/proj/portage.git/commit/?id=200d876b

SyncManager.async: initialize attributes before fork (bug 561234)

Since commit 496ff326dc18890889d1ea5d2aec590394635960, the pre_sync
method executes in a forked child process. Therefore, initialize
attributes before the fork (in the async method).

Fixes: 496ff326dc18 ("sync repositories in parallel (bug 557426)")
X-Gentoo-Bug: 561234
X-Gentoo-Bug-url: https://bugs.gentoo.org/show_bug.cgi?id=561234
Acked-by: Brian Dolbec <dolsen <AT> gentoo.org>

 pym/portage/sync/controller.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/pym/portage/sync/controller.py b/pym/portage/sync/controller.py
index e992cc4..28dbc57 100644
--- a/pym/portage/sync/controller.py
+++ b/pym/portage/sync/controller.py
@@ -115,13 +115,16 @@ class SyncManager(object):
                return []
 
        def async(self, emerge_config=None, repo=None):
+               self.emerge_config = emerge_config
+               self.settings, self.trees, self.mtimedb = emerge_config
+               self.xterm_titles = "notitles" not in self.settings.features
+               self.portdb = 
self.trees[self.settings['EROOT']]['porttree'].dbapi
                proc = AsyncFunction(target=self.sync,
                        kwargs=dict(emerge_config=emerge_config, repo=repo))
                proc.addExitListener(self._sync_callback)
                return proc
 
        def sync(self, emerge_config=None, repo=None):
-               self.emerge_config = emerge_config
                self.callback = None
                self.repo = repo
                self.exitcode = 1
@@ -199,13 +202,10 @@ class SyncManager(object):
 
 
        def pre_sync(self, repo):
-               self.settings, self.trees, self.mtimedb = self.emerge_config
-               self.xterm_titles = "notitles" not in self.settings.features
                msg = ">>> Syncing repository '%s' into '%s'..." \
                        % (repo.name, repo.location)
                self.logger(self.xterm_titles, msg)
                writemsg_level(msg + "\n")
-               self.portdb = 
self.trees[self.settings['EROOT']]['porttree'].dbapi
                try:
                        st = os.stat(repo.location)
                except OSError:

Reply via email to