From 0cc735a7b2af86a7dc7ab3b7db9609538fde1f49 Mon Sep 17 00:00:00 2001 From: Darren Schroeder <343840+fdncred@users.noreply.github.com> Date: Fri, 21 Apr 2023 15:30:36 -0500 Subject: [PATCH] use record separator `\x1e` for `include_path` (#8961) # Description This changes how the include paths are joined together so that it uses a character that will work across platforms. `;` didn't work on Mac/Linux. `:` didn't work in Windows. So, the record separator should work, which is \x1e. This goes hand-in-hand with https://github.com/nushell/vscode-nushell-lang/pull/113. # User-Facing Changes # Tests + Formatting # After Submitting --- src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main.rs b/src/main.rs index d0e077753c..4305994834 100644 --- a/src/main.rs +++ b/src/main.rs @@ -141,7 +141,7 @@ fn main() -> Result<()> { let span = include_path.span; let vals: Vec<_> = include_path .item - .split(':') + .split('\x1e') // \x1e is the record separator character (a character that is unlikely to appear in a path) .map(|x| Value::String { val: x.trim().to_string(), span,