Possible misuse of ioutil.TempDir

[objectx]

Don't hardcode /tmp. This will cause unexpected test failure on Windows.

# I'll check it on windows later. (at this time, no Windows station near me 😢)

# Appended

Under windows environment ioutil.TempDir("/tmp", ... cause error

[marc0der]

To be fair, this will only be used for testing and needs to be in a place where we can easily examine the outcome in case of failure. Alternatively we could place this in the project folder (pwd) as a hidden .sdkman directory. wdyt?

[objectx]

I've created the #3 to run tests successfully on windows.

needs to be in a place where we can easily examine the outcome in case of failure.

System's default temporary directory (used by ioutil.TempDir) is determined by os.TempDir(). It not so difficult place to investigate.

we could place this in the project folder (pwd) as a hidden .sdkman directory.

I think writing something in the source tree is a bad thing.

[marc0der]

Sure, but I also think that this is a lot of extra boilerplate, complexity and duplication for something that can be solved in a far simpler way. Shall we move these discussions to our Gitter channel before continuing?

[objectx]

@marc0der OK. Continuing in the Gitter channel…