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.

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

The option --pending-deprecation should be added to NODE_OPTIONS whitelist #15484

Closed
tomc974 opened this issue Sep 20, 2017 · 5 comments
Closed
Labels
cli Issues and PRs related to the Node.js command line interface. feature request Issues that request new features to be added to Node.js.

Comments

@tomc974
Copy link
Contributor

tomc974 commented Sep 20, 2017

  • Version: v8.5.0
  • Platform: Microsoft Windows [version 10.0.15063] (64-bit)
  • Subsystem: unknown

I wanted to enable the --pending-deprecation option via the NODE_OPTIONS env variable but received the following message when I tried to run node:

--pending-deprecation is not allowed in NODE_OPTIONS

I think it's safe to allow this option to be set in NODE_OPTIONS.

@jasnell
Copy link
Member

jasnell commented Sep 20, 2017

Would you like to try your hand at opening a PR? :-)

@tomc974
Copy link
Contributor Author

tomc974 commented Sep 20, 2017

@jasnell Well, I'm not really a C++ dev but this one should be easy enough. I'll check if I can build Node.js on my system to implement and test this awesome feature. ;-p

@mscdex mscdex added cli Issues and PRs related to the Node.js command line interface. feature request Issues that request new features to be added to Node.js. labels Sep 20, 2017
@benjamingr
Copy link
Member

Passing NODE_PENDING_DEPRECATION="1" should already work (right?)

@tomc974
Copy link
Contributor Author

tomc974 commented Sep 21, 2017

@benjamingr I didn't knew about this variable since running node --help does not mention its existence.

So what do you think we should do? Reject the PR to support --pending-deprecation option via the NODE_OPTIONS and add documentation for NODE_PENDING_DEPRECATION when running node --help or support both?

@refack
Copy link
Contributor

refack commented Sep 24, 2017

@tomc974 thanks for the report (and the PR 😉). Would you like to try and add NODE_PENDING_DEPRECATION to the help (probably above

node/src/node.cc

Lines 3868 to 3870 in 150b8f7

" prefixed to the module search path\n"
"NODE_REPL_HISTORY path to the persistent REPL history\n"
" file\n"
)

jasnell pushed a commit that referenced this issue Sep 25, 2017
PR-URL: #15494
Fixes: #15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
addaleax pushed a commit to addaleax/ayo that referenced this issue Sep 30, 2017
PR-URL: nodejs/node#15494
Fixes: nodejs/node#15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 14, 2017
PR-URL: #15494
Fixes: #15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 21, 2017
PR-URL: #15494
Fixes: #15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 28, 2017
PR-URL: #15494
Fixes: #15484
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Issues and PRs related to the Node.js command line interface. feature request Issues that request new features to be added to Node.js.
Projects
None yet
Development

No branches or pull requests

5 participants