Using Python subprocess rather than os.system

Question

Following the advice of Ignacio ( Python remove spaces and append ) I am trying to convert the below command to use subprocess rather than os.system.

The script: os.system("/usr/local/bin/growlnotify -n emesene -a emesene -t """+title+""" -m """+text+"""""")

I would of thought: subprocess.call(['/usr/local/bin/growlnotify', '-n emesene', '-a emesene', '-t ""+title+"""', '-m """+text+"""""'], shell=True)

But it doesnt work. Any ideas how to get this working? I've looked at the Python docs and looked here, but i cannot figure it out!

BTW the title and text are variables from emesene messengers notification system

consider to accept Ignacio's answer in the linked question.

Sven Marnach
– Sven Marnach

2011-06-10 17:55:50 +00:00
Commented Jun 10, 2011 at 17:55 — Sven Marnach
– Sven Marnach, Commented Jun 10, 2011 at 17:55

user1114 · Accepted Answer · 2011-06-10 17:44:34Z

5

Instead of

subprocess.call(['/usr/local/bin/growlnotify', '-n emesene', '-a emesene', '-t ""+title+"""', '-m """+text+"""""'], shell=True)

use

subprocess.call(['/usr/local/bin/growlnotify', '-n',  'emesene', '-a', 'emesene', '-t', title, '-m', text], shell=True)

Everywhere you have an unquoted space in the shell command, split another list item.

The shlex.split function can be used to convert from a shell-style command to a list of the sort Subproces expects.

answered Jun 10, 2011 at 17:44

user1114

1

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

1 Comment

Josh Fradley Over a year ago

Doesn't work. Gives a blank notification, so i guess its not taking the variables.

Prashant Kumar · Accepted Answer · 2011-06-10 18:14:50Z

4

You don't want "shell=True" when you use a list to pass the command. I would have merely commented as such on Jeremy Banks's post (since his statement on shlex.split is most notable), but I don't currently have the rep to do so :/

subprocess.call(['/usr/local/bin/growlnotify', '-n',  'emesene', '-a', 'emesene', '-t', title, '-m', text])

answered Jun 10, 2011 at 18:14

Prashant Kumar

23.2k14 gold badges52 silver badges64 bronze badges

Collectives™ on Stack Overflow

Using Python subprocess rather than os.system

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related