stevedlawrence commented on code in PR #1452:
URL: https://github.com/apache/daffodil-vscode/pull/1452#discussion_r2414278826


##########
vite/package.vite.config.mjs:
##########
@@ -109,23 +121,38 @@ function copyToPkgDirPlugin() {
     { from: 'src/styles/styles.css', to: `${pkg_dir}/src/styles/styles.css` },
     { from: 'src/tdmlEditor/', to: `${pkg_dir}/src/tdmlEditor` },
   ]
-  const serverPackageFolder = path.join(
-    path.resolve('dist/package'),
-    serverPackage
-  )
-
-  console.debug(`== [Vite] | serverPackageFolder: ${serverPackageFolder}`)
-  // remove debugger package folder if exists
-  if (fs.existsSync(serverPackageFolder)) {
-    fs.rmSync(serverPackageFolder, { recursive: true })
+
+  const serverPackageFolders = {
+    scala2_12: path.join(
+      path.resolve('dist/package'),
+      `daffodil-debugger-${daffodilVersions['scala2.12']}-${pkg_version}`
+    ),
+    scala2_13: path.join(
+      path.resolve('dist/package'),
+      `daffodil-debugger-${daffodilVersions['scala2.13']}-${pkg_version}`
+    ),
+    scala3: path.join(
+      path.resolve('dist/package'),
+      `daffodil-debugger-${daffodilVersions['scala3']}-${pkg_version}`
+    ),

Review Comment:
   > That question was for like when we run say sbt test
   
   Running "sbt test" will still have daffodil jars on the classpath, and will 
test with whatever versions are defined in libraryDependencies, so that should 
all be unchanged. And it'll will just test with three daffodil versions. All 
the "provided" thing does is makes it so those jars don't end up in the vsix 
file and must be provided some other way.
   
   > do we want to use the temp directory?
   
   I would recommend something other than /tmp since that is non-persistent, 
and is often cleaned up on reboot. Which means users would have to download 
versions every time they reboot. It doesn't necessarily have to be ~/.cache 
though. Maybe VS Code has recommendations or someking of global storage where 
these kinds of things should be stored?
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to