Add a AbstractEbuildProcess class for MiscFunctionsProcess to inherit the

_pipe and _can_log methods that used to be in the EbuildProcess class.

svn path=/main/trunk/; revision=14933

1 files changed, 5 insertions, 18 deletions

diff --git a/pym/_emerge/EbuildProcess.py b/pym/_emerge/EbuildProcess.py
index a6a0b362b..597caf4dd 100644
--- a/pym/_emerge/EbuildProcess.py
+++ b/pym/_emerge/EbuildProcess.py
@@ -2,13 +2,13 @@
 # Distributed under the terms of the GNU General Public License v2
 # $Id$
 
-from _emerge.SpawnProcess import SpawnProcess
+from _emerge.AbstractEbuildProcess import AbstractEbuildProcess
 import portage
 from portage import os
 
-class EbuildProcess(SpawnProcess):
+class EbuildProcess(AbstractEbuildProcess):
 
-	__slots__ = ("phase", "pkg", "settings", "tree")
+	__slots__ = ('tree',)
 
 	def _start(self):
 		# Don't open the log file during the clean phase since the
@@ -16,20 +16,7 @@ class EbuildProcess(SpawnProcess):
 		# prevents the clean phase from removing $T.
 		if self.phase not in ("clean", "cleanrm"):
 			self.logfile = self.settings.get("PORTAGE_LOG_FILE")
-		SpawnProcess._start(self)
-
-	def _pipe(self, fd_pipes):
-		stdout_pipe = fd_pipes.get(1)
-		got_pty, master_fd, slave_fd = \
-			portage._create_pty_or_pipe(copy_term_size=stdout_pipe)
-		return (master_fd, slave_fd)
-
-	def _can_log(self, slave_fd):
-		# With sesandbox, logging works through a pty but not through a
-		# normal pipe. So, disable logging if ptys are broken.
-		# See Bug #162404.
-		return not ('sesandbox' in self.settings.features \
-			and self.settings.selinux_enabled()) or os.isatty(slave_fd)
+		AbstractEbuildProcess._start(self)
 
 	def _spawn(self, args, **kwargs):
 
@@ -47,7 +34,7 @@ class EbuildProcess(SpawnProcess):
 		return rval
 
 	def _set_returncode(self, wait_retval):
-		SpawnProcess._set_returncode(self, wait_retval)
+		AbstractEbuildProcess._set_returncode(self, wait_retval)
 
 		if self.phase not in ("clean", "cleanrm"):
 			self.returncode = portage._doebuild_exit_status_check_and_log(