Skip to content

lib: remove the invalid command line options - #13764

Closed
JacksonTian wants to merge 1 commit into
nodejs:masterfrom
JacksonTian:debug-agent
Closed

lib: remove the invalid command line options#13764
JacksonTian wants to merge 1 commit into
nodejs:masterfrom
JacksonTian:debug-agent

Conversation

@JacksonTian

@JacksonTianJacksonTian commented Jun 18, 2017

Copy link
Copy Markdown
Contributor

The option --remote_debugging_server and --debug-agent are
not supported now.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

lib/internal/bootstrap_node

@nodejs-github-botnodejs-github-bot added the lib / src Issues and PRs related to general changes in the lib or src directory. label Jun 18, 2017
@addaleax

Copy link
Copy Markdown
Member

LGTM but I’d combine this into #13763.

@cjihrigcjihrig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on its own or as part of #13763

@JacksonTianJacksonTian changed the title lib: remove the useless entry for --debug-agentlib: remove the invalid entriesJun 19, 2017
@JacksonTian

Copy link
Copy Markdown
ContributorAuthor

Hi @addaleax@cjihrig I combined the two commits here.

@cjihrigcjihrig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The commit message title should probably mention command line flags. It's not clear just from the title what "entries" are.

@JacksonTianJacksonTian changed the title lib: remove the invalid entrieslib: remove the invalid command line optionsJun 20, 2017
@JacksonTian

Copy link
Copy Markdown
ContributorAuthor

Updated the commit title.

The option --remote_debugging_server and --debug-agent are
not supported now.
@BridgeAR

Copy link
Copy Markdown
Member

Landed in 9f175d1

BridgeAR pushed a commit that referenced this pull request Aug 30, 2017
The option --remote_debugging_server and --debug-agent are
not supported now.
PR-URL: #13764
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@JacksonTian
JacksonTian deleted the debug-agent branch August 31, 2017 06:57
cjihrig pushed a commit to cjihrig/node that referenced this pull request Aug 31, 2017
The option --remote_debugging_server and --debug-agent are
not supported now.
PR-URL: nodejs#13764
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
addaleax pushed a commit to addaleax/ayo that referenced this pull request Sep 5, 2017
The option --remote_debugging_server and --debug-agent are
not supported now.
PR-URL: nodejs/node#13764
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MylesBorins pushed a commit that referenced this pull request Sep 10, 2017
The option --remote_debugging_server and --debug-agent are
not supported now.
PR-URL: #13764
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@MylesBorinsMylesBorins mentioned this pull request Sep 10, 2017
MylesBorins pushed a commit that referenced this pull request Sep 12, 2017
The option --remote_debugging_server and --debug-agent are
not supported now.
PR-URL: #13764
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@JacksonTian@addaleax@BridgeAR@jasnell@cjihrig@mhdawson@MylesBorins@nodejs-github-bot