Skip to content

Commit

Permalink
Merge pull request #12009 from Elinvention/fork/execvp
Browse files Browse the repository at this point in the history
Use execvp instead of execv to allow OS.execute() to search through PATH.
  • Loading branch information
hpvb authored Oct 12, 2017
2 parents b1f2c31 + 600efa0 commit abe47ea
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/unix/os_unix.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ Error OS_Unix::execute(const String &p_path, const List<String> &p_arguments, bo
execvp(getprogname(), &args[0]);
}
#else
execv(p_path.utf8().get_data(), &args[0]);
execvp(p_path.utf8().get_data(), &args[0]);
#endif
// still alive? something failed..
fprintf(stderr, "**ERROR** OS_Unix::execute - Could not create child process while executing: %s\n", p_path.utf8().get_data());
Expand Down

0 comments on commit abe47ea

Please sign in to comment.