traxys/tree-sitter
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(wasm): use / paths for workdir

Browse source 
Reimplemented the fix from #2183 to fix building WASM files with Docker
on Windows again. The --workdir argument gives a path inside the Docker
container, so it must use forward slashes regardless of the default path
separator on the host OS.
This commit is contained in:
Joel Spadin 2024-09-21 14:34:30 -05:00 • committed by Amaan Qureshi
parent 4f0d463d49
commit 755e49e212
4 changed files with 11 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
cli/loader/src/lib.rs
View file

@ -23,6 +23,7 @@ use fs4::FileExt;
use indoc::indoc;
use libloading::{Library, Symbol};
use once_cell::unsync::OnceCell;
use path_slash::PathBufExt as _;
use regex::{Regex, RegexBuilder};
use serde::{Deserialize, Deserializer, Serialize};
use tree_sitter::Language;
@ -818,7 +819,7 @@ impl Loader {
path.push(src_path.strip_prefix(root_path).unwrap());
path
};
command.args(["--workdir", &workdir.to_string_lossy()]);
command.args(["--workdir", &workdir.to_slash_lossy()]);
// Mount the root directory as a volume, which is the repo root
let mut volume_string = OsString::from(&root_path);