more Windows path fix

[Andriamanitra]

No description provided.

[Andriamanitra]

We should keep this check for Windows, otherwise the gsubs can mess up Linux paths (both /C:/whatever and /C:\\what\ever/file are valid paths that we should not mess with on Linux).

if package.config:sub(1,1) == "\\" then
    -- do windows-specific stuff
else
    -- do the usual stuff
end

@bjornasm

[bjornasm]

if package.config:sub(1,1) == "\\" then

Good point - I think the reason I changed was when testing - there was no package.config set. I this assumed set for everyone that uses the plugin?

Edit I get this error when trying lsp start with if package.config:sub(1,1) == "\\" - it might be something local to me.

Lua API error: main:1645: attempt to index a non-table object(nil) with key 'sub'
stack traceback:
        main:1645: in function 'fileUriFromAbsPath'
        main:288: in function 'initialize'
        main:198: in function 'func'
        main:125: in main chunk
        [G]: ?

However, for me the following alternative works

local runtime = import("runtime")

(...) 
if runtime.GOOS == "windows" then

But this might be just something wrong on my end.

[Andriamanitra]

if package.config:sub(1,1) == "\\" then

Good point - I think the reason I changed was when testing - there was no package.config set. I this assumed set for everyone that uses the plugin?

Edit I get this error when trying lsp start with if package.config:sub(1,1) == "\\" - it might be something local to me.

Lua API error: main:1645: attempt to index a non-table object(nil) with key 'sub'
stack traceback:
        main:1645: in function 'fileUriFromAbsPath'
        main:288: in function 'initialize'
        main:198: in function 'func'
        main:125: in main chunk
        [G]: ?

However, for me the following alternative works

local runtime = import("runtime")

(...) 
if runtime.GOOS == "windows" then

But this might be just something wrong on my end.

Weird, I'm not sure why package.config would be nil on Windows. Your suggested alternative seems fine, let's use that instead.

[bjornasm]

Sorry for the delay @Andriamanitra - I updated the PR now.