Merge lp:~ebichete/olive/bugfix_lp374036 into lp:olive
Proposed by
Edward Ari Bichetero
Status: | Merged |
---|---|
Merged at revision: | 695 |
Proposed branch: | lp:~ebichete/olive/bugfix_lp374036 |
Merge into: | lp:olive |
Diff against target: |
18 lines (+2/-2) 1 file modified
olive/launch.py (+2/-2) |
To merge this branch: | bzr merge lp:~ebichete/olive/bugfix_lp374036 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jelmer Vernooij (community) | code | Approve | |
Olive Team | Pending | ||
Review via email: mp+39271@code.launchpad.net |
Description of the change
Switch the file/directory "launch" (actually open) code to use os.spawn() instead of os.system(). By explicitly filling the launched process' argument list we sidestep the shell's command splitting and correctly handle paths that contain spaces.
To post a comment you must log in.
Looks good to me.