aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Harder <radhermit@gmail.com>2015-01-26 19:48:11 -0800
committerTim Harder <radhermit@gmail.com>2015-01-26 22:30:35 -0800
commit8a3c3cbcc6308dbb4fc464b86e6f5ab20f6f9f72 (patch)
tree3cbf616eadfa9fbbf4a577d9692ac514825c0db5
parentpmerge: shorten output formatter names (diff)
downloadpkgcore-8a3c3cbcc6308dbb4fc464b86e6f5ab20f6f9f72.tar.gz
pkgcore-8a3c3cbcc6308dbb4fc464b86e6f5ab20f6f9f72.tar.bz2
pkgcore-8a3c3cbcc6308dbb4fc464b86e6f5ab20f6f9f72.zip
ebuild/processor: remove unnecessary set_sandbox_state method
I don't think any external user of EbuildProcessor will need access to it since we expose run_phase() that allows for a sandbox parameter. So instead we just write the sandbox state directly to the daemon without the extra function call.
-rw-r--r--pkgcore/ebuild/processor.py13
1 files changed, 1 insertions, 12 deletions
diff --git a/pkgcore/ebuild/processor.py b/pkgcore/ebuild/processor.py
index e7999c15..1557ecb3 100644
--- a/pkgcore/ebuild/processor.py
+++ b/pkgcore/ebuild/processor.py
@@ -381,8 +381,7 @@ class EbuildProcessor(object):
self.write("process_ebuild %s" % phase)
if not self.send_env(env, tmpdir=tmpdir):
return False
- if sandbox:
- self.set_sandbox_state(sandbox)
+ self.write("set_sandbox_state %i" % sandbox)
if logging:
if not self.set_logfile(logging):
return False
@@ -634,16 +633,6 @@ class EbuildProcessor(object):
# which isn't always true.
self.pid = None
- def set_sandbox_state(self, state):
- """
- tell the daemon whether to enable the sandbox, or disable it
- :param state: boolean, if True enable sandbox
- """
- if state:
- self.write("set_sandbox_state 1")
- else:
- self.write("set_sandbox_state 0")
-
def _generate_env_str(self, env_dict):
data = []
for key, val in env_dict.iteritems():