samepath: functionalize native

Author: scivision

+stdlib/+native/samepath.m

@@ -0,0 +1,15 @@
+%% NATIVE.SAMEPATH are two paths equivalent
+%
+% this canonical string method is less preferred to using device + inode.
+
+function y = samepath(path1, path2)
+
+if stdlib.strempty(path1) || stdlib.strempty(path2)
+  y = false;
+else
+  c1 = stdlib.canonical(path1, true);
+  c2 = stdlib.canonical(path2, true);
+  y = ~stdlib.strempty(c1) && strcmp(c1, c2);
+end
+
+end

+stdlib/+sys/samepath.m

@@ -1,15 +1,10 @@
 function y = samepath(path1, path2)
 
-y = false;
+assert(~ispc(), "Unix-like only")
 
-if stdlib.strempty(path1) || stdlib.strempty(path2)
-  return
-end
+y = false;
 
-if ispc()
-  c1 = stdlib.canonical(path1, true);
-  c2 = stdlib.canonical(path2, true);
-  y = strlength(c1) > 0 && strcmp(c1, c2);
+if ~stdlib.exists(path1) || ~stdlib.exists(path2)
   return
 end
 
@@ -22,7 +17,10 @@
 cmd = "stat " + flag + " %d:%i " + path1 + " && stat " + flag + " %d:%i " + path2;
 
 [s, m] = system(cmd);
-if s ~= 0, return, end
+if s ~= 0
+  warning("stdlib:sys:samepath:RuntimeError", "stdlib.sys.samepath(%s, %s) failed: %s", path1, path2, m)
+  return
+end
 
 m = splitlines(m);
 assert(length(m) >= 2, "samepath(%s, %s) failed: unexpected output", path1, path2);

+stdlib/samepath.m

@@ -18,8 +18,10 @@
 
 if stdlib.has_python()
   y = stdlib.python.samepath(path1, path2);
-else
+elseif isunix()
   y = stdlib.sys.samepath(path1, path2);
+else
+  y = stdlib.native.samepath(path1, path2);
 end
 
 %!assert(samepath(".", "."))

test/TestSame.m

@@ -6,7 +6,7 @@
 {"..", pwd() + "/.."}, ...
 {pwd(), pwd() + "/."}}
 
-same_fun = {@stdlib.samepath, @stdlib.sys.samepath, @stdlib.python.samepath}
+same_fun = {@stdlib.samepath, @stdlib.sys.samepath, @stdlib.python.samepath, @stdlib.native.samepath}
 end
 
 methods(TestClassSetup)

test/is_capable.m

@@ -46,7 +46,10 @@ function is_capable(tc, f)
    end
 
 elseif contains(n, ".sys.")
- % pass
+
+  if endsWith(n, "samepath")
+    tc.assumeTrue(isunix(), "unix only function")
+  end
 
 elseif contains(n, ".native.")