From 93cded47a2732cadddca657ae04721f38e86bbf0 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Sun, 15 Aug 2010 20:59:26 -0700 Subject: Enable EbuildIpcDaemon support by default. --- pym/_emerge/AbstractEbuildProcess.py | 9 +++++---- pym/portage/tests/ebuild/test_doebuild_spawn.py | 15 +++++---------- 2 files changed, 10 insertions(+), 14 deletions(-) diff --git a/pym/_emerge/AbstractEbuildProcess.py b/pym/_emerge/AbstractEbuildProcess.py index e2be8b856..aca254191 100644 --- a/pym/_emerge/AbstractEbuildProcess.py +++ b/pym/_emerge/AbstractEbuildProcess.py @@ -25,6 +25,10 @@ class AbstractEbuildProcess(SpawnProcess): ('_ipc_daemon', '_exit_command',) _phases_without_builddir = ('clean', 'cleanrm', 'depend', 'help',) + # The EbuildIpcDaemon support is well tested, but this variable + # is left so we can temporarily disable it if any issues arise. + _enable_ipc_daemon = True + def __init__(self, **kwargs): SpawnProcess.__init__(self, **kwargs) if self.phase is None: @@ -40,10 +44,7 @@ class AbstractEbuildProcess(SpawnProcess): # since we're not displaying to a terminal anyway. self.settings['NOCOLOR'] = 'true' - enable_ipc_daemon = \ - self.settings.get('PORTAGE_IPC_DAEMON_ENABLE') == '1' - - if enable_ipc_daemon: + if self._enable_ipc_daemon: self.settings.pop('PORTAGE_EBUILD_EXIT_FILE', None) if self.phase not in self._phases_without_builddir: self.settings['PORTAGE_IPC_DAEMON'] = "1" diff --git a/pym/portage/tests/ebuild/test_doebuild_spawn.py b/pym/portage/tests/ebuild/test_doebuild_spawn.py index 615062709..842bb45f0 100644 --- a/pym/portage/tests/ebuild/test_doebuild_spawn.py +++ b/pym/portage/tests/ebuild/test_doebuild_spawn.py @@ -48,15 +48,10 @@ class DoebuildSpawnTestCase(TestCase): # Create a fake environment, to pretend as if the ebuild # has been sourced already. open(os.path.join(settings['T'], 'environment'), 'wb') - for enable_ipc in (False, True): - if enable_ipc: - settings['PORTAGE_IPC_DAEMON_ENABLE'] = '1' - else: - settings.pop('PORTAGE_IPC_DAEMON_ENABLE', None) - for phase in ('_internal_test',): - rval = doebuild_spawn( - "%s %s" % (_shell_quote(EBUILD_SH_BINARY), phase), - settings, free=1) - self.assertEqual(rval, os.EX_OK) + for phase in ('_internal_test',): + rval = doebuild_spawn( + "%s %s" % (_shell_quote(EBUILD_SH_BINARY), phase), + settings, free=1) + self.assertEqual(rval, os.EX_OK) finally: playground.cleanup() -- cgit v1.2.3-1-g7c22