[2.x] fix: Fixes AccessDeniedException issue on Windows (#9538)

**Problem**
When test classloader holds on to the JAR file, Windows gets
AccessDefinedException on packageBin.

**Solution**
Flip the default to close the test class loader.

main/src/main/scala/sbt/internal/ClassLoaders.scala

@@ -224,11 +224,11 @@ private[sbt] object ClassLoaders {
               scalaReflectLayer,
               () =>
                 new ReverseLookupClassLoaderHolder(
-                  allDependencies,
-                  scalaReflectLayer,
-                  close,
-                  allowZombies,
-                  logger
+                  classpath = allDependencies,
+                  parent = scalaReflectLayer,
+                  closeThis = close,
+                  allowZombies = allowZombies,
+                  logger = logger,
                 )
             )
           } else scalaReflectLayer

main/src/main/scala/sbt/internal/SysProp.scala

@@ -132,7 +132,7 @@ object SysProp:
    */
   lazy val color: Boolean = ITerminal.isColorEnabled
 
-  def closeClassLoaders: Boolean = getOrFalse("sbt.classloader.close")
+  def closeClassLoaders: Boolean = getOrTrue("sbt.classloader.close")
 
   def fileCacheSize: Long =
     SizeParser(System.getProperty("sbt.file.cache.size", "128M")).getOrElse(128L * 1024 * 1024)

sbt-app/src/sbt-test/classloader-cache/close-run/src/main/scala/Main.scala

@@ -1,13 +0,0 @@
-object Main {
-  class Foo
-
-  def main(args: Array[String]): Unit = {
-    new Thread {
-      override def run(): Unit = {
-        Thread.sleep(500)
-        try new Foo
-        catch { case t: Throwable => sys.exit(1) }
-      }
-    }.start()
-  }
-}

sbt-app/src/sbt-test/classloader-cache/close-run/test

@@ -1 +0,0 @@
-> run