Commit a3c564b
doc,child_process: pid can be undefined when ENOENT
From the code `nodejs@8` and up should behave the same:
github.com/nodejs/node/blame/v8.17.0/lib/internal/child_process.js#L290
And a short test snippet:
```js
const { spawn } = require('child_process')
const subProcess = spawn('non-exist-command')
subProcess.on('error', (error) =>
console.warn('mute Unhandled "error" event:', error))
console.log('- pid:', subProcess.pid)
process.nextTick(() => console.log('- after error emit'))
console.log('== end of test ==')
```
Note: the sync spawn result `pid` currently do not follow this pattern.
Co-authored-by: Rich Trott <rtrott@gmail.com>
PR-URL: #37014
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>1 parent 605cb4c commit a3c564b
1 file changed
Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1243,9 +1243,11 @@ does not indicate that the child process has been terminated. | |||
| 1243 | 1243 | added: v0.1.90 | |
| 1244 | 1244 | --> | |
| 1245 | 1245 | ||
| 1246 | - * {integer} | ||
| 1246 | + * {integer|undefined} | ||
| 1247 | 1247 | ||
| 1248 | - Returns the process identifier (PID) of the child process. | ||
| 1248 | + Returns the process identifier (PID) of the child process. If the child process | ||
| 1249 | + fails to spawn due to errors, then the value is `undefined` and `error` is | ||
| 1250 | + emitted. | ||
| 1249 | 1251 | ||
| 1250 | 1252 | ```js | |
| 1251 | 1253 | const { spawn } = require('child_process'); | |
0 commit comments