Merge pull request ipython#2926 from y-p/GH2925

Don't die if stderr/stdout do not support set_parent() If the user redirects the streams to a file for example, the kernel would raise an exception because file objects do not have set_parent defined, unlike ipython's OutStream. This happens even if the user does this in a spawned thread. closes ipython#2925
mahijar · Feb 13, 2013 · 465f43c · 465f43c
2 parents a03b31a + 9deb416
commit 465f43c
Showing 1 changed file with 17 additions and 5 deletions.
diff --git a/IPython/kernel/zmq/ipkernel.py b/IPython/kernel/zmq/ipkernel.py
@@ -364,8 +364,14 @@ def execute_request(self, stream, ident, parent):
         shell.displayhook.set_parent(parent)
         shell.display_pub.set_parent(parent)
         shell.data_pub.set_parent(parent)
-        sys.stdout.set_parent(parent)
-        sys.stderr.set_parent(parent)
+        try:
+            sys.stdout.set_parent(parent)
+        except AttributeError:
+            pass
+        try:
+            sys.stderr.set_parent(parent)
+        except AttributeError:
+            pass
 
         # Re-broadcast our input for the benefit of listening clients, and
         # start computing output
@@ -567,14 +573,20 @@ def apply_request(self, stream, ident, parent):
             return
 
         self._publish_status(u'busy', parent)
-        
+
         # Set the parent message of the display hook and out streams.
         shell = self.shell
         shell.displayhook.set_parent(parent)
         shell.display_pub.set_parent(parent)
         shell.data_pub.set_parent(parent)
-        sys.stdout.set_parent(parent)
-        sys.stderr.set_parent(parent)
+        try:
+            sys.stdout.set_parent(parent)
+        except AttributeError:
+            pass
+        try:
+            sys.stderr.set_parent(parent)
+        except AttributeError:
+            pass
 
         # pyin_msg = self.session.msg(u'pyin',{u'code':code}, parent=parent)
         # self.iopub_socket.send(pyin_msg)