Skip to content
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.

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

In some cases, debug longclasspath project failed #541

Closed
sallyur opened this issue Feb 25, 2019 · 2 comments
Closed

In some cases, debug longclasspath project failed #541

sallyur opened this issue Feb 25, 2019 · 2 comments
Assignees

Comments

@sallyur
Copy link

sallyur commented Feb 25, 2019

Environment
  • Operating System: Win & Mac
  • JDK version: 8 & 11
  • Visual Studio Code version: 1.31.1
  • Java extension version: 0.39.0
  • Java Debugger extension version: 0.16.0
Steps To Reproduce
  1. Open the project "longclasspath"
  2. F5 to generate launch.json
  3. Modify the launch.json with the following combinations:
    "shortenCommandLine": ""/"auto"/"jarmanifest"/"argfile", // argfile requires your Java version is 9 and higher.
    "console": ""/"internalConsole"/"integratedTerminal"/"externalTerminal"
  4. Click debug codelens
Current Result

Mac:
It will failed when the combinations are “externalTerminal" + "auto" or "intergratedTerminal”+“auto"
image

Windows:
Sometimes debug also failed in windows, attached the log
log.txt

@sallyur sallyur changed the title Debug longclasspath project failed In some cases, Debug longclasspath project failed Feb 25, 2019
@sallyur sallyur changed the title In some cases, Debug longclasspath project failed In some cases, debug longclasspath project failed Feb 25, 2019
@testforstephen
Copy link
Contributor

When launching the program in the terminal (integrated terminal or external terminal), the terminal will echo the total command line strings in the shell before executing the command. In the case of this bug, the command line strings are long, so the terminal needs more time to print the command line strings. Current timeout for launching in terminal is only 10 seconds. It's not enough for the long command line case. Needs increase the timeout.

@testforstephen
Copy link
Contributor

Fixed with PR #662

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

No branches or pull requests

2 participants