relative_to: add native method

Author: scivision

+stdlib/+native/relative_to.m

@@ -0,0 +1,13 @@
+%% NATIVE.RELATIVE_TO find relative path to other from base
+%
+% this only accepts "/" as filesep, hence we use stdlib.posix()
+
+function rel = relative_to(base, other)
+
+if stdlib.strempty(base) || stdlib.strempty(other) 
+  rel = "";
+else
+  rel = matlab.io.internal.filesystem.relativepath(stdlib.posix(base), stdlib.posix(other));
+end
+
+end

+stdlib/+python/relative_to.m

@@ -1,7 +1,7 @@
 function rel = relative_to(base, other)
 
 try
-  rel = string(py.os.path.relpath(other, base));
+  rel = stdlib.posix(string(py.os.path.relpath(other, base)));
 catch e
   if e.identifier == "MATLAB:Python:PyException" && startsWith(e.message, 'Python Error: ValueError')
     rel = "";

+stdlib/+sys/relative_to.m

@@ -1,3 +1,5 @@
+%% SYS.RELATIVE_TO get relative path to other from base
+
 function r = relative_to(base, other)
 
 r = "";
@@ -22,7 +24,7 @@
 r = string(strip(r));
 
 if ispc()
-  r = fullfile(stdlib.normalize(r));
+  r = stdlib.posix(stdlib.normalize(r));
 end
 
 end

+stdlib/posix.m

@@ -4,9 +4,6 @@
 %
 
 function r = posix(p)
-arguments
-  p {mustBeTextScalar}
-end
 
 if ispc
   r = strrep(p, '\', '/');

+stdlib/relative_to.m

@@ -16,16 +16,10 @@
   other {mustBeTextScalar}
 end
 
-if (stdlib.strempty(base) && stdlib.strempty(other)) || ...
-   (stdlib.is_absolute(base) ~= stdlib.is_absolute(other))
-  rel = "";
-  return
-end
-
-if stdlib.has_python()
+if ~isMATLABReleaseOlderThan('R2024a')
+  rel = stdlib.native.relative_to(base, other);
+elseif stdlib.has_python()
   rel = stdlib.python.relative_to(base, other);
-elseif stdlib.dotnet_api() >= 5
-  rel = stdlib.dotnet.relative_to(base, other);
 else
   rel = stdlib.sys.relative_to(base, other);
 end

+stdlib/+dotnet/device.m renamed to example/dotnet/device.m

@@ -1,7 +1,11 @@
 %% DOTNET.RELATIVE_TO find the relative path to other from base
+%
+% not normally used because it's overly complex
 
 function rel = relative_to(base, other)
 
+assert(stdlib.dotnet_api() >= 5)
+
 if stdlib.strempty(other)
   rel = base;
   return

+stdlib/+dotnet/relative_to.m renamed to example/dotnet/relative_to.m

@@ -7,7 +7,7 @@
 properties (TestParameter)
 pr = init_rel()
 pp = init_prox()
-rel_fun = {@stdlib.relative_to, @stdlib.sys.relative_to, @stdlib.dotnet.relative_to, @stdlib.python.relative_to}
+rel_fun = {@stdlib.relative_to, @stdlib.sys.relative_to, @stdlib.native.relative_to, @stdlib.python.relative_to}
 end
 
 methods(TestClassSetup)
@@ -44,8 +44,7 @@ function test_proximate_to(tc, pp)
 
 p = {{"", "", ""}, ...
 {fileparts(pwd()), pwd(), "test"}, ...
-{root, fullfile(root, "test", mfilename() + ".m"), fullfile("test", mfilename + ".m")}, ...
-{"/", "/", "."}, ...
+{root, root + "/test/" + mfilename() + ".m", "test/" + mfilename + ".m"}
 };
 % NOTE: ".." in relative_to(base) is ambiguous including for python.pathlib, C++ <filesystem>, etc.
 
@@ -79,6 +78,7 @@ function test_proximate_to(tc, pp)
 else
 
 p = [p, {
+{"/", "/", "."}, ...
 {"/dev/null", "/dev/null", "."}, ...
 }];
 end

test/TestRelative.m

@@ -16,10 +16,6 @@ function is_capable(tc, f)
     tc.assumeTrue(ispc(), "Windows only function")
   end
 
-  if endsWith(n, "relative_to")
-    tc.assumeGreaterThanOrEqual(dapi, 5)
-  end
-
   if endsWith(n, ["create_symlink", "ram_total", "read_symlink"])
     tc.assumeGreaterThanOrEqual(dapi, 6);
   end
@@ -61,7 +57,7 @@ function is_capable(tc, f)
     tc.assumeFalse(isMATLABReleaseOlderThan('R2025a'))
   end
 
-  if endsWith(n, "canonical")
+  if endsWith(n, ["canonical", "relative_to"])
     tc.assumeFalse(isMATLABReleaseOlderThan('R2024a'))
   end