Skip to content

Rebrand Diagnostics Server to DiagnosticPort #40499

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.

Sign up for GitHub

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

Merged
josalem merged 19 commits into from
Aug 14, 2020
Merged

Rebrand Diagnostics Server to DiagnosticPort #40499

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
b4e32e1
Rebrand Diagnostic Server to Diagnostic Port
Aug 6, 2020
000142d
Update tests to use new config switches
Aug 7, 2020
af9b4cc
Update diagnosticserver.cpp
Aug 7, 2020
5fafe42
Appease gcc warning
Aug 7, 2020
4144d1a
Merge branch 'dev/josalem/diagnostic-port' of github.com:josalem/runt…
Aug 7, 2020
b76cf6c
Const shenanigans
Aug 7, 2020
51c7377
Add tests for DiagnosticPort functionality
Aug 7, 2020
e3cb293
Attempting to appease GCC
Aug 7, 2020
335f959
Defer disable till after streaming starts
Aug 8, 2020
693d91a
Rename ConnectionMode enum values
Aug 11, 2020
c3e5b84
PR feedback
Aug 11, 2020
e5c8a06
Merge branch 'dev/josalem/diagnostic-port' of github.com:josalem/runt…
Aug 11, 2020
9efac34
Merge branch 'master' into dev/josalem/diagnostic-port
Aug 11, 2020
bb70ca7
Fix #40696 and add test to cover that
Aug 12, 2020
6a84b6f
Merge branch 'dev/josalem/diagnostic-port' of github.com:josalem/runt…
Aug 12, 2020
2c4f0de
Fix scope of runtime cookie
Aug 12, 2020
c5134d0
Ignore listen ports for now
Aug 13, 2020
bcf4506
default port type is connect
Aug 13, 2020
f97f77a
PR feedback
Aug 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Appease gcc warning
  • Loading branch information
John Salem committed Aug 7, 2020
commit 5fafe42bea6663dceb6230da17dd7a35cfc8c1eb
2 changes: 1 addition & 1 deletion src/coreclr/src/vm/diagnosticserver.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ void DiagnosticServer::PauseForDiagnosticsMonitor()
if (dwFiveSecondWait == WAIT_TIMEOUT)
{
CLRConfigStringHolder dotnetDiagnosticPortString = CLRConfig::GetConfigValue(CLRConfig::EXTERNAL_DOTNET_DiagnosticPorts);
LPCWSTR dotnetDiagnosticPortPrintable = dotnetDiagnosticPortString == nullptr ? W("") : dotnetDiagnosticPortString;
LPWSTR dotnetDiagnosticPortPrintable = dotnetDiagnosticPortString == nullptr ? W("") : dotnetDiagnosticPortString;
wprintf(W("The runtime has been configured to pause during startup and is awaiting a Diagnostics IPC ResumeStartup command from a server in the following list:\n"));
wprintf(W("DOTNET_DiagnosticPorts=\"%s\"\n"), dotnetDiagnosticPortPrintable);
fflush(stdout);
Expand Down