Home Original page

Fix documentation of process.argv[0] by tarungarg546 · Pull Request #7449 · nodejs/node

@tarungarg546

@nodejs-github-bot added doc

Issues and PRs related to the documentations.

process

Issues and PRs related to the process subsystem.

labels

Jun 27, 2016

@tarungarg546 

The current documentation states that if run something like node app.js then
in our process.argv array first elements is node,
but actually its process.execPath not node
as documentation currently suggests

This commit fixes this documentation bug.

Fixes : nodejs#7434
PR-URL: nodejs#7449

@Fishrock123 Fishrock123 changed the title Updated ReadMe for #7434 Fix documentation of process.argv[0]

Jun 28, 2016

@tarungarg546 

With this commit everyline of process documentation
is wrapped in 80 characters
and there are some changes for documention
of process.arv[0] and process.argv[1]

@tarungarg546 

With this commit everyline of process documentation
is wrapped in 80 characters
and there are some changes for documention
of process.arv[0] and process.argv[1]

@tarungarg546 

With this commit everyline of process documentation
is wrapped in 80 characters
and there are some changes for documention
of process.arv[0] and process.argv[1]

@tarungarg546 

These changes are just some
grammatical changes like
using `an` not `a` and
space after period etc.

@tarungarg546

addaleax pushed a commit that referenced this pull request

Jul 4, 2016 
The current documentation states that if run something like
`node app.js` then in our process.argv array first elements is `node`,
but actually it's `process.execPath` not `node`
as documentation currently suggests.

Fixes: #7434
PR-URL: #7449
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

Fishrock123 pushed a commit that referenced this pull request

Jul 5, 2016 
The current documentation states that if run something like
`node app.js` then in our process.argv array first elements is `node`,
but actually it's `process.execPath` not `node`
as documentation currently suggests.

Fixes: #7434
PR-URL: #7449
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

ppannuto added a commit to ppannuto/node that referenced this pull request

Aug 1, 2016 
For historical and other reasons, node overwrites `argv[0]` on startup.
See
 - 2c6f79c,
 - nodejs#7434
 - nodejs#7449
 - nodejs#7696

For cases where it may be useful, save the original value of `argv[0]`
in `process.argv0`

addaleax pushed a commit that referenced this pull request

Aug 8, 2016 
For historical and other reasons, node overwrites `argv[0]` on startup.
See
 - 2c6f79c,
 - #7434
 - #7449
 - #7696

For cases where it may be useful, save the original value of `argv[0]`
in `process.argv0`

PR-URL: #7696
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>

cjihrig pushed a commit that referenced this pull request

Aug 10, 2016 
For historical and other reasons, node overwrites `argv[0]` on startup.
See
 - 2c6f79c,
 - #7434
 - #7449
 - #7696

For cases where it may be useful, save the original value of `argv[0]`
in `process.argv0`

PR-URL: #7696
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>