kotlin-debug-adapter icon indicating copy to clipboard operation
kotlin-debug-adapter copied to clipboard

feat: support cwd and env variable using sun jdi

Open thunderz99 opened this issue 4 years ago • 2 comments

An enhance for #39

Done

  • cwd

    • Add "cwd" for LaunchConfiguration
      • default to ${projectRoot} if null or blank
    • Add a custom KDACommandLineLauncher which implements LaunchingConnector
      • The implementation is in original code written in Kotlin
    • Add unit test for cwd
  • env variables

    • Add "envs" for LaunchConfiguration
    • Change "envs" config from List<String> to Map<String, String> style for more friendly user experience
    • Add unit test for envs
  • support for jdk1.8

  • support for jdk11

    • SunCommandLineLauncher cannot be extended under jdk11, so KDACommandLineLauncher implements LaunchingConnector directly
    • SocketTransportService cannot be constructed under jdk11, so it is newed by Class.forName

Linked pullreq

  • Add cwd / envs support in vscode-kotlin https://github.com/fwcd/vscode-kotlin/pull/42

thunderz99 avatar Jul 02 '20 07:07 thunderz99

any update on this? what needs to be done to get this merged?

bkleiner avatar Apr 14 '21 12:04 bkleiner

@fwcd Thank you for the advices, I will update this PR

thunderz99 avatar Jul 02 '22 06:07 thunderz99