Skip to content

Show warning message when debuggee and debugger run at different jvms#92

Merged
testforstephen merged 5 commits into
masterfrom
jinbo_fix
Nov 7, 2017
Merged

Show warning message when debuggee and debugger run at different jvms#92
testforstephen merged 5 commits into
masterfrom
jinbo_fix

Conversation

@testforstephen

Copy link
Copy Markdown
Contributor

Signed-off-by: Jinbo Wang jinbwan@microsoft.com

Fix the issue : microsoft/vscode-java-debug#30

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
akaroml
akaroml previously approved these changes Nov 6, 2017
String debuggeeVersion = debugSession.getVM().version();
String debuggerVersion = System.getProperty("java.version");
if (!debuggerVersion.equals(debuggeeVersion)) {
String warnMessage = String.format("[Warn] Remote debuggee JVM: %s, but debugger JVM: %s. The source mapping for jre system libraries "

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The debugger and the debuggee are running in different versions of JVMs. You could see wrong source mapping results.
Debugger JVM version: ***
Debuggee JVM version: ***

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed.

Signed-off-by: Jinbo Wang <jinbwan@microsoft.com>
@testforstephen
testforstephen merged commit f2d20c4 into master Nov 7, 2017
@testforstephen
testforstephen deleted the jinbo_fix branch November 7, 2017 01:46
@vitaliis

Copy link
Copy Markdown

Is "Debuggee JVM" the one bundled together with vscode? And in case of a Linux snap package, the Java distributed together with vscode? Because I keep seeing this warning and don't know how can I change the "Debuggee JVM version". This "Debuggee JVM version" is different from JVM I have installed locally.

Can I change the "Debuggee JVM version" somehow (to match the java version I have locally)?

@DovieW

DovieW commented Feb 20, 2025

Copy link
Copy Markdown

@vitaliis

Copy link
Copy Markdown

@vitaliis yo you ever figured this out?

I do not remember it. It was long time ago. Sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants