diff options
author | Tim Harder <radhermit@gmail.com> | 2015-01-26 19:48:11 -0800 |
---|---|---|
committer | Tim Harder <radhermit@gmail.com> | 2015-01-26 22:30:35 -0800 |
commit | 8a3c3cbcc6308dbb4fc464b86e6f5ab20f6f9f72 (patch) | |
tree | 3cbf616eadfa9fbbf4a577d9692ac514825c0db5 | |
parent | pmerge: shorten output formatter names (diff) | |
download | pkgcore-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.py | 13 |
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(): |