Skip to content

src: fix debugging for multiple categories - #21422

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:debug-multiple
Closed

src: fix debugging for multiple categories#21422
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:debug-multiple

Conversation

@addaleax

Copy link
Copy Markdown
Member

Fix parsing of e.g. NODE_DEBUG_NATIVE=worker,messaging.

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

Fix parsing of e.g. `NODE_DEBUG_NATIVE=worker,messaging`.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jun 20, 2018
@addaleaxaddaleax added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 20, 2018
@addaleax

Copy link
Copy Markdown
MemberAuthor

Full CI: https://ci.nodejs.org/job/node-test-pull-request/15535/

Feel free to 👍 this for fast-tracking.

@BridgeARBridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 20, 2018
@Trott

Copy link
Copy Markdown
Member

CI re-run to see if AIX is clear now: https://ci.nodejs.org/job/node-test-pull-request/15539/

@addaleax

Copy link
Copy Markdown
MemberAuthor

Landed in 7aa9567

@addaleax
addaleax deleted the debug-multiple branch June 20, 2018 20:49
addaleax added a commit that referenced this pull request Jun 20, 2018
Fix parsing of e.g. `NODE_DEBUG_NATIVE=worker,messaging`.
PR-URL: #21422
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
targos pushed a commit that referenced this pull request Jun 22, 2018
Fix parsing of e.g. `NODE_DEBUG_NATIVE=worker,messaging`.
PR-URL: #21422
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@targostargos mentioned this pull request Jul 3, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.c++Issues and PRs that require attention from people who are familiar with C++.fast-trackPRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@addaleax@nodejs-github-bot@Trott@cjihrig@BridgeAR