Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

doc: clarify child_process.execFile{,Sync} file argument #5310

Closed
wants to merge 1 commit into from
Closed

doc: clarify child_process.execFile{,Sync} file argument #5310

wants to merge 1 commit into from

Commits on Feb 18, 2016

  1. doc: clarify child_process.execFile{,Sync} file arg 

    The changes to the file argument of execFile in nodejs#4504 make it appear
that execFile requires an absolute or relative path to the executable
file, when it also supports a filename which will be resolved using
$PATH.  Although the example makes this clear, assuming there isn't a
node binary in $CWD, it's easy to overlook.  This commit clarifies that
point.

It also updates the argument description for execFileSync to match,
since it was overlooked in nodejs#4504 and behaves identically.

Signed-off-by: Kevin Locke <[email protected]>
    @kevinoid
    kevinoid committed Feb 18, 2016
    Configuration menu
    Copy the full SHA
    aa62969 View commit details
    Browse the repository at this point in the history