Skip to content

Commit 816d721

Browse files
committed
Remove unused (and unsupported) launch args
'env' and 'cwd' were never supported by the Connector, so we might as well remove them. Support for environment variables and CWDs might be added in a future release, by spawning the debuggee externally and attaching to it through a socket.
1 parent b953c64 commit 816d721

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

adapter/src/main/kotlin/org/javacs/ktda/jdi/launch/JDILauncher.kt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,7 @@ import java.util.stream.Collectors
2424
class JDILauncher(
2525
private val attachTimeout: Int = 50,
2626
private val vmArguments: String? = null,
27-
private val modulePaths: String? = null,
28-
private val environmentVariables: Collection<String>? = null
27+
private val modulePaths: String? = null
2928
) : DebugLauncher {
3029
private val vmManager: VirtualMachineManager
3130
get() = Bootstrap.virtualMachineManager()
@@ -58,8 +57,6 @@ class JDILauncher(
5857
args["suspend"]!!.setValue("true")
5958
args["options"]!!.setValue(formatOptions(config))
6059
args["main"]!!.setValue(formatMainClass(config))
61-
args["cwd"]?.setValue(config.projectRoot.toAbsolutePath().toString())
62-
args["env"]?.setValue(urlEncode(environmentVariables) ?: "")
6360
}
6461

6562
private fun createAttachArgs(config: AttachConfiguration, connector: Connector): Map<String, Connector.Argument> = connector.defaultArguments()

0 commit comments

Comments
 (0)