CODEWITHSUNDEEP
pythonpython-2.7subprocess
Pectus Excavatum
Pectus Excavatum

Reputation: 3783

python subprocess popen synchronous commands

I am trying to use popen to kick off a subprocess that calls two commands (with multiple arguements) one after the other. The second command relies on the first command running, so I was hoping to use a single subprocess to run both rather than spawning two processes and wait on the first.

But I am running into issues because I am not sure how to give two command inputs or to seperate the command as one single object.

Also, I am trying to avoid setting shell to true if possible.

This is essentially, what I am trying to do:

for test in resources:
    command = [
        'pgh',
        'resource',
        'create',
        '--name', test['name'],
        '--description', test['description'],
    ]
    command2 = [
        'pgh',
        'assignment',
        'create',
        '--name', test['name'],
        '--user', test['user'],
    ]


    p = Popen(command, stdout=PIPE, stderr=PIPE)
    stdout, stderr = p.communicate()
    print(stdout)
    print(stderr)

Upvotes: 4

Views: 11264

Answers (2)

shanmuga
shanmuga

Reputation: 4499

You will have to launch command and wait for completion before launching another command. You should do this repeatedly for each command.

This can be done as 

ps = [ Popen(c, stdout=PIPE, stderr=PIPE).communicate() 
       for c in command]

Note that this launches the next command irrespective of weather the first command succeeded or failed. If you want to launch the next command only if the previous command succeds then use

def check_execute(commands):
    return_code = 0
    for c in commands:
        p = Popen(c, stdout=PIPE, stderr=PIPE)
        result = p.communicate()
        yield result
        return_code = p.returncode
        if return_code != 0:
            break

Upvotes: 1

RoshP
RoshP

Reputation: 19

As per my understanding the following should work for you. To chain the execution once the previous completes use.

p1 = subprocess.Popen(command, stdout=subprocess.PIPE)
p2 = subprocess.Popen(command2, stdin=p1.stdout, stdout=subprocess.PIPE)
print p2.communicate()

Upvotes: 1

Related Questions