Skip to content

Conversation

@testforstephen
Copy link
Contributor

Fixes microsoft/vscode-java-pack#720
Fixes microsoft/vscode-java-pack#756
Fixes #622
Fixes #623
Fixes #646
Fixes #813

This PR made such changes:

  • Remove the launcher.bat from the generated command line, since it brings some side effects on Windows.
  • If launching application in terminals (e.g. cmd/ps), then make Java application and terminal to use the platform default encoding.
  • If launching application in VS Code Internal Console (aka. Debug Console), then use UTF-8 in Java application because VS Code Debug Console use UTF-8 to display output.

For Windows user, the recommended approach to solve encoding issue is to change the system locale to the target language. Learn more from the Encoding Troubleshooting Guide. https://github.com/microsoft/vscode-java-debug/blob/main/Troubleshooting_encoding.md
image

@Eskibear
Copy link
Member

remember to remove launcher script related code from 1. utility , 2. .bat script from code base

@testforstephen testforstephen added this to the 0.37.0 milestone Nov 19, 2021
@testforstephen testforstephen merged commit 0289e29 into microsoft:main Nov 19, 2021
@testforstephen testforstephen deleted the jinbo_encoding branch November 19, 2021 07:32
@imba-tjd
Copy link

This should be VSC's files.encoding in case of messing up with java's -Dfile.encoding.

图片

@testforstephen
Copy link
Contributor Author

@imba-tjd Nice finding, yes, it's files.encoding.

@testforstephen
Copy link
Contributor Author

The new diagram is as follows.

image

@imba-tjd
Copy link

The hint message needs to be changed.

图片

另外在Changelog里要一句thanks不过分吧

@testforstephen
Copy link
Contributor Author

欢迎提PR来fix这个问题

@imba-tjd
Copy link

搜了一下没搜到
图片

不过我的本意是在已有的0.37的changelog里加一句thanks我for helping addressing the issue的,如果你们觉得不能修改已发布的就算了吧。这个提示信息我就不pr了,我不会ts也没开发过vsc的扩展,还是你们来修

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

Labels

None yet

Projects

None yet

4 participants