TypeError: 'Popen' object is not callable

Question

I'm trying to query status of a windows service using subprocess module's 'Popen' method. But I'm getting

TypeError: 'Popen' object is not callable

import subprocess, codecs

def serviceStatus(RadiaService):
    status = []
    cmd = 'sc query ' + RadiaService
    pDetails = subprocess.Popen(cmd, shell = True, stdout = subprocess.PIPE)
    for item in pDetails():
        status.append(item)
    finalStatus = b''.join(status).decode('utf-8')
    print(finalStatus)

if __name__ == '__main__':
    serviceStatus('RCA')

Error Trace :

Traceback (most recent call last):
  File "C:\Alen\Cumulative RHF\Radia_Cumulative_Patch\cumulativeHotFixproject\lib\win32.py", line 39, in <module>
    serviceStatus('RCA')
  File "C:\Alen\Cumulative RHF\Radia_Cumulative_Patch\cumulativeHotFixproject\lib\win32.py", line 33, in serviceStatus
    for item in pDetails():
TypeError: 'Popen' object is not callable

try removing the brackets from pDetails() in the line 'for item in pDetails():' so it becomes 'for item in pDetails:' instead — RichSmith
– RichSmith, Commented Jan 18, 2017 at 12:22

Tamas Hegedus · Accepted Answer · 2017-01-18 12:44:36Z

5

It looks like you wish to collect the standard output of the subprocess. You will have to use pDetails.stdout. Here is an example to help you get started:

import subprocess
p = subprocess.Popen("ls -la", shell=True, stdout=subprocess.PIPE)
output = b''.join(p.stdout).decode('utf-8')
print(output)

Based on that this is how your code should look like:

import subprocess, codecs

def serviceStatus(RadiaService):
    cmd = 'sc query ' + RadiaService
    pDetails = subprocess.Popen(cmd, shell = True, stdout = subprocess.PIPE)
    return b''.join(pDetails.stdout).decode('utf-8')

def main():
    print(serviceStatus('RCA'))

if __name__ == '__main__':
    main()

Note: you don't have to collect the output in a list, you can feed the iterable directly to join. If you need a list, you still don't have to use a for loop, you can just write status = list(pDetails.stdout).

edited Jan 18, 2017 at 12:44

answered Jan 18, 2017 at 12:36

Tamas Hegedus

30.1k12 gold badges67 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

TypeError: 'Popen' object is not callable

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related