Buck9195/Asi_Native_FiJi_launcher
1
0
Fork 0
mirror of https://github.com/ij-plugins/ijp-imagej-launcher.git synced 2024-11-13 16:29:01 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

Update note on relativeTo workaround and the bug in Scala Native

Browse source 
See https://github.com/scala-native/scala-native/issues/3293
This commit is contained in:
Jarek Sacha 2023-05-25 09:22:14 -04:00
parent a91991593f
commit 78448de03d
No known key found for this signature in database
GPG key ID: F29625CE62288163
1 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/main/scala/ij_plugins/imagej_launcher/Updater.scala
View file

@ -48,9 +48,14 @@ object Updater:
private def relativeTo(src: Path, base: Path): RelPath =
// This does what src.relativeTo(base) should do
// Problems is in the native code on Windows, it creates relative path by adding `../` at the beginning of the path
// rather than creating a relative path
// Implementation is very limited and assumes specific relation between src and base that should hold in our case
// Problems is in the native code on Windows,
// os.Path#relativeTo creates relative path by adding `../` at the beginning of the absolute path,
// co you may get `../C:\a\b` which leads to a exception soon after.
// The issue is with Scala Native implementation of java.nio.file.Path#relativize on Windows,
// See https://github.com/scala-native/scala-native/issues/3293
// This implementation is very limited but sufficient for our use.
// It assumes specific relation between src and base.
val srcStr = src.toString
val baseStr = base.toString
@ -60,7 +65,6 @@ object Updater:
val relStr = srcStr.drop(baseStr.length + 1)
RelPath(relStr)
private def deleteEmptyDirs(dir: Path, logger: Logger): Unit =
logger.debug(s"Cleaning directory: $dir")
os.list(dir)