Support -J<jvm-param> and "-Dname=value" in windows runner script

Otherwise, -J-ea has no effect on windows, which breaks assert tests.
Wrap parameters with delimeters in quotes.
 #KT-30211 Fixed

compiler/cli/bin/kotlinc.bat

@@ -27,9 +27,27 @@ if "%_JAVACMD%"=="" set _JAVACMD=java
 rem We use the value of the JAVA_OPTS environment variable if defined
 if "%JAVA_OPTS%"=="" set JAVA_OPTS=-Xmx256M -Xms32M
 
-if not "%_KOTLIN_RUNNER%"=="" (
+rem Iterate through arguments and split them into java and kotlin ones
+:loop
+set _arg=%~1
+if "%_arg%" == "" goto loopend
+
+if "%_arg:~0,2%"=="-J" (
+  set JAVA_OPTS=%JAVA_OPTS% "%_arg:~2%"
+) else (
+  if "%_arg:~0,2%"=="-D" (
+    set JAVA_OPTS=%JAVA_OPTS% "%_arg%"
+  ) else (
+    set KOTLIN_OPTS=%KOTLIN_OPTS% "%_arg%"
+  )
+)
+shift
+goto loop
+:loopend
+
+if "%_KOTLIN_RUNNER%"=="1" (
   "%_JAVACMD%" %JAVA_OPTS% "-Dkotlin.home=%_KOTLIN_HOME%" -cp "%_KOTLIN_HOME%\lib\kotlin-runner.jar" ^
-    org.jetbrains.kotlin.runner.Main %*
+    org.jetbrains.kotlin.runner.Main %KOTLIN_OPTS%
 ) else (
   SET _ADDITIONAL_CLASSPATH=
 
@@ -39,7 +57,7 @@ if not "%_KOTLIN_RUNNER%"=="" (
 
   "%_JAVACMD%" %JAVA_OPTS% -noverify -cp "%_KOTLIN_HOME%\lib\kotlin-preloader.jar" ^
     org.jetbrains.kotlin.preloading.Preloader -cp "%_KOTLIN_HOME%\lib\kotlin-compiler.jar%_ADDITIONAL_CLASSPATH%" ^
-    %_KOTLIN_COMPILER% %*
+    %_KOTLIN_COMPILER% %KOTLIN_OPTS%
 )
 
 exit /b %ERRORLEVEL%

compiler/testData/launcher/property.kt

@@ -0,0 +1,3 @@
+fun main() {
+    println(System.getProperty("result"))
+}

compiler/tests/org/jetbrains/kotlin/cli/LauncherScriptTest.kt

@@ -16,14 +16,14 @@
 
 package org.jetbrains.kotlin.cli
 
-import com.intellij.execution.configurations.GeneralCommandLine
-import com.intellij.execution.util.ExecUtil
 import com.intellij.openapi.util.SystemInfo
 import com.intellij.openapi.util.text.StringUtil
 import org.jetbrains.kotlin.test.KotlinTestUtils
 import org.jetbrains.kotlin.test.TestCaseWithTmpdir
 import org.jetbrains.kotlin.utils.PathUtil
-import java.io.File
+import org.jetbrains.kotlin.utils.addToStdlib.cast
+import java.io.*
+import java.util.concurrent.TimeUnit
 
 class LauncherScriptTest : TestCaseWithTmpdir() {
     private fun runProcess(
@@ -38,19 +38,26 @@ class LauncherScriptTest : TestCaseWithTmpdir() {
         val launcherFile = File(PathUtil.kotlinPathsForDistDirectory.homePath, "bin/$executableFileName")
         assertTrue("Launcher script not found, run dist task: ${launcherFile.absolutePath}", launcherFile.exists())
 
-        val cmd = GeneralCommandLine(launcherFile.absolutePath, *args)
+        // For some reason, IntelliJ's ExecUtil screws quotes up on windows.
-        workDirectory?.let(cmd::withWorkDirectory)
+        // So, use ProcessBuilder instead.
-        val processOutput = ExecUtil.execAndGetOutput(cmd)
+        val pb = ProcessBuilder(
-        val stdout = StringUtil.convertLineSeparators(processOutput.stdout)
+            launcherFile.absolutePath,
-        val stderr = StringUtil.convertLineSeparators(processOutput.stderr).replace("Picked up [_A-Z]+:.*\n".toRegex(), "")
+            // In cmd, `=` is delimeter, so we need to surround parameter with quotes.
-        val exitCode = processOutput.exitCode
+            *quoteIfNeeded(args)
-
+        )
+        pb.directory(workDirectory)
+        pb.environment().remove("_KOTLIN_RUNNER") // FIXME: HACK
+        val process = pb.start()
+        val stdout = StringUtil.convertLineSeparators(process.inputStream.bufferedReader().use { it.readText() })
+        val stderr = StringUtil.convertLineSeparators(process.errorStream.bufferedReader().use { it.readText() })
+            .replace("Picked up [_A-Z]+:.*\n".toRegex(), "")
+        process.waitFor(10, TimeUnit.SECONDS)
+        val exitCode = process.exitValue()
         try {
             assertEquals(expectedStdout, stdout)
             assertEquals(expectedStderr, stderr)
             assertEquals(expectedExitCode, exitCode)
-        }
+        } catch (e: Throwable) {
-        catch (e: Throwable) {
             System.err.println("exit code $exitCode")
             System.err.println("=== STDOUT ===")
             System.err.println(stdout)
@@ -60,6 +67,12 @@ class LauncherScriptTest : TestCaseWithTmpdir() {
         }
     }
 
+    private fun quoteIfNeeded(args: Array<out String>): Array<String> =
+        if (SystemInfo.isWindows) args.map {
+            if (it.contains('=') || it.contains(" ") || it.contains(";") || it.contains(",")) "\"$it\"" else it
+        }.toTypedArray()
+        else args.cast()
+
     private val testDataDirectory: String
         get() = KotlinTestUtils.getTestDataPathBase() + "/launcher"
 
@@ -177,4 +190,13 @@ class LauncherScriptTest : TestCaseWithTmpdir() {
 
         runProcess("kotlin", "LegacyAssertEnabledKt", "-J-ea:kotlin._Assertions", workDirectory = tmpdir)
     }
+
+    fun testProperty() {
+        runProcess("kotlinc", "$testDataDirectory/property.kt", "-d", tmpdir.path)
+
+        runProcess(
+            "kotlin", "PropertyKt", "-Dresult=OK",
+            workDirectory = tmpdir, expectedStdout = "OK\n"
+        )
+    }
 }