What I'm trying to do is simply have the output of some terminal commands print out to a wx.TextCtrl widget. I figured the easiest way to accomplish this is to create a custom stdout class and overload the write function to that of the widget.
stdout class:
class StdOut(sys.stdout):
def __init__(self,txtctrl):
sys.stdout.__init__(self)
self.txtctrl = txtctrl
def write(self,string):
self.txtctrl.write(string)
And then I would do something such as:
sys.stdout = StdOut(createdTxtCtrl)
subprocess.Popen('echo "Hello World!"',stdout=sys.stdout,shell=True)
What results is the following error:
Traceback (most recent call last):
File "mainwindow.py", line 12, in <module>
from systemconsole import SystemConsole
File "systemconsole.py", line 4, in <module>
class StdOut(sys.stdout):
TypeError: Error when calling the metaclass bases
file() argument 2 must be string, not tuple
Any ideas to fix this would be appreciated.
sys.stdout
is not a class
, it's an instance (of type file
).
So, just do:
class StdOut(object):
def __init__(self,txtctrl):
self.txtctrl = txtctrl
def write(self,string):
self.txtctrl.write(string)
sys.stdout = StdOut(the_text_ctrl)
No need to inherit from file
, just make a simple file-like object like this! Duck typing is your friend...
(Note that in Python, like most other OO languages but differently from Javascript, you only ever inherit from classes AKA types, never from instances of classes/types;-).
If all you need to implement is writing, there is no need to define a new class at all. Simply use createdTxtCtrl
instead of StdOut(createdTxtCtrl)
, because the former already supports the operation you need.
If all you need to do with stdout is direct some programs' output there, not direct all kinds of stuff there, don't change sys.stdout
, just instantiate the subprocess.Popen
with your own file-like object (createdTxtCtrl
) instead of sys.stdout
.
Wouldn't sys.stdout = StdOut(createdTxtCtrl)
create cyclical dependency? Try not reassigning sys.stdout
to the class derived from sys.stdout
.