fixed use wsl feature (issue#509) #510
Conversation
convert path wsl after full path constructed leetCodeRootPathInWsl no longer used getLeetCodeRootPath function removed since it has only one reference after the bug fixed
add missing signature
|
This pull request introduces 1 alert when merging f3ad33f into 1c4a39e - view on LGTM.com new alerts:
|
src/leetCodeExecutor.ts
Outdated
| this.leetCodeRootPathInWsl = `${await wsl.toWslPath(this.leetCodeRootPath)}`; | ||
| } | ||
| return `${this.leetCodeRootPathInWsl}`; | ||
| return `${yield wsl.toWslPath(`"${path.join(yield`"${this.leetCodeRootPath}"`, "bin", "leetcode")}"`)}`; |
src/leetCodeExecutor.ts
Outdated
|
|
||
| public async getLeetCodeBinaryPath(): Promise<string> { // wrapped by "" | ||
| return `"${path.join(await this.getLeetCodeRootPath(), "bin", "leetcode")}"`; | ||
| return `"${path.join(yield`"${this.leetCodeRootPath}"`, "bin", "leetcode")}"`; |
There was a problem hiding this comment.
Orz
i'm not too much familiar with ts, maybe you should take over this fix if i keep failed the build
patch to solve wslpath fail when path not exists
|
when open new problem, wslpath conversion to windows fails as the target path under wsl doesn't exists yet, I tried fixed this and append to this same PR |
|
Thank you @LinkeyLeo, Sorry I have no Windows machine right now (it's in my office, and given the current situation, It's not easy to get it). Could you please help me to understand the change? You have mentioned:
What's the path you mentioned here? |
the workflow for open problem for coding is looks like following: in step 3, we need to create a new source file for writing default codes, due to using node under wsl, the target file path reported by node is a wsl path, but the extension host is running under windows and need a windows path for creating and open the file, thus we need wslpath to convert the path, then the problem appeared. wslpath will finish the path conversion only if the path is valid and the path is exists, or it rise a path doesn't exist error and won't do the path conversion. considering other path may need to be conversion and have the same problem in the future, I fixed this problem by modified toWinPath function, making it only process the prefix of whole path and manually reconstruct the complete path |
There was a problem hiding this comment.
@LinkeyLeo My apologize for the late reply, sorry!
The PR looks good to me overall. Just one question, when will the path does not start with "\\mnt\\". we have this: return (await executeCommand("wsl", ["wslpath", "-w", "/"])).trim() + path;
Could you explain more about this?
Thanks.
|
i divide wsl paths into two parts, mounted drive root from windows which located in /mnt/ and other path located in / except /mnt/ for the after ones, we need to map /{WSL PATH} to \wsl${distor name}{WSL PATH} |
convert path wsl after full path constructed
resolve problem cased by wslpath conversion
leetCodeRootPathInWsl no longer used
getLeetCodeRootPath function removed since it has only one reference after the bug fixed
#509