diff options
author | gpotter2 <gabriel@potter.fr> | 2018-01-17 18:02:12 +0100 |
---|---|---|
committer | gpotter2 <gabriel@potter.fr> | 2018-01-19 23:37:14 +0100 |
commit | bae9d29c99cd635eba87d6eb5cde26c88d22493f (patch) | |
tree | ad9cf7f7a0dd5235d53b6721e04e681de0cabbf1 | |
parent | 5bb683c2d97d1b3d3797723132920e56aeea2059 (diff) | |
download | scapy-bae9d29c99cd635eba87d6eb5cde26c88d22493f.tar.gz |
Use conf.interactive
-rw-r--r-- | scapy/utils.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/scapy/utils.py b/scapy/utils.py index c93e28a2..8822e012 100644 --- a/scapy/utils.py +++ b/scapy/utils.py @@ -429,10 +429,9 @@ class ContextManagerSubprocess(object): >>> subprocess.Popen(["unknown_command"]) """ - def __init__(self, name, prog, _raise=True): + def __init__(self, name, prog): self.name = name self.prog = prog - self._raise = _raise def __enter__(self): pass @@ -440,7 +439,7 @@ class ContextManagerSubprocess(object): def __exit__(self, exc_type, exc_value, traceback): if isinstance(exc_value, (OSError, TypeError)): msg = "%s: executing %r failed" % (self.name, self.prog) if self.prog else "Could not execute %s, is it installed ?" % self.name - if self._raise: + if not conf.interactive: raise OSError(msg) else: log_runtime.error(msg, exc_info=True) @@ -505,7 +504,7 @@ def do_graph(graph,prog=None,format=None,target=None,type=None,string=None,optio target = get_temp_file(autoext="."+format) start_viewer = True else: - with ContextManagerSubprocess("do_graph()", conf.prog.display, _raise=False): + with ContextManagerSubprocess("do_graph()", conf.prog.display): target = subprocess.Popen([conf.prog.display], stdin=subprocess.PIPE).stdin if format is not None: |