permissions broad compatibility

Author: scivision

+stdlib/+native/set_permissions.m

@@ -1,13 +1,13 @@
 function ok = set_permissions(file, readable, writable, executable)
-arguments
-  file (1,1) string
-  readable (1,1) {mustBeInteger}
-  writable (1,1) {mustBeInteger}
-  executable (1,1) {mustBeInteger}
-end
+
+mustBeInteger(readable)
+mustBeInteger(writable)
+mustBeInteger(executable)
 
 p = filePermissions(file);
 
+assert(isscalar(p), "set_permissions: one file only")
+
 k = string.empty;
 v = logical.empty;
 

+stdlib/private/perm2char.m

@@ -23,7 +23,7 @@
     return
 end
 
-if isa(v, "matlab.io.WindowsPermissions") || ispc()
+if isa(v, 'matlab.io.WindowsPermissions') || ispc()
 
   if p(1) == 'r'
     p(3) = 'x';

example/+javafun/jPosix.m

@@ -4,8 +4,8 @@
 
   if isempty(o)
     s = "";
-  elseif isa(o, "java.io.File") || isa(o, "java.nio.file.Path") || ...
-         isa(o, "sun.nio.fs.UnixPath") || isa(o, "sun.nio.fs.WindowsPath")
+  elseif isa(o, 'java.io.File') || isa(o, 'java.nio.file.Path') || ...
+         isa(o, 'sun.nio.fs.UnixPath') || isa(o, 'sun.nio.fs.WindowsPath')
     s = o.toString();
   end
 

test/TestPermissions.m

@@ -1,5 +1,4 @@
-classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}, ...
-          TestTags = {'impure'}) ...
+classdef (SharedTestFixtures={ matlab.unittest.fixtures.PathFixture(fileparts(fileparts(mfilename('fullpath'))))}) ...
     TestPermissions < matlab.unittest.TestCase
 
 properties (TestParameter)
@@ -9,12 +8,12 @@
 
 methods(TestMethodSetup)
 function w_dirs(tc)
-  tc.applyFixture(matlab.unittest.fixtures.WorkingFolderFixture())
+  tc.applyFixture(matlab.unittest.fixtures.WorkingFolderFixture());
 end
 end
 
 
-methods (Test, TestTags={'R2019b'})
+methods (Test, TestTags={'R2017b'})
 
 function test_get_permissions(tc, Ps)
 import matlab.unittest.constraints.StartsWithSubstring
@@ -32,20 +31,20 @@ function test_get_permissions(tc, Ps)
   end
 
   tc.verifyThat(p, StartsWithSubstring("r"))
-  if isfile(p) && stdlib.suffix(p) == ".m"
+  if isfile(p) && strcmp(stdlib.suffix(p), '.m')
     tc.verifyEqual(p(3), '-')
   end
 end
 end
 
 
 function test_get_permissions_exe(tc)
-matlab_exe = fullfile(matlabroot, "bin/matlab");
+matlab_exe = [matlabroot, '/bin/matlab'];
 if ispc()
-  matlab_exe = matlab_exe + ".exe";
+  matlab_exe = [matlab_exe, '.exe'];
 end
 
-tc.assertThat(matlab_exe, matlab.unittest.constraints.IsFile)
+tc.assertTrue(isfile(matlab_exe))
 p = stdlib.get_permissions(matlab_exe);
 
 tc.assertNotEmpty(p)
@@ -57,7 +56,7 @@ function test_get_permissions_exe(tc)
 function test_set_permissions_nowrite(tc)
 import matlab.unittest.constraints.StartsWithSubstring
 
-nw = fullfile(pwd(), "no-write");
+nw = [pwd(), '/no-write'];
 
 tc.assertTrue(stdlib.touch(nw))
 r = stdlib.set_permissions(nw, 0, -1, 0);
@@ -72,7 +71,7 @@ function test_set_permissions_nowrite(tc)
   tc.assertEqual(b, 'native')
 end
 
-if ~ispc() || b ~= "legacy"
+if ~ispc() || ~strcmp(b, 'legacy')
   tc.verifyThat(p, StartsWithSubstring("r-"), "no-write permission failed to set")
 end
 
@@ -90,7 +89,7 @@ function test_set_permissions_noread(tc)
 % fileattrib can not even set the permissions on Linux.
 tc.assumeFalse(stdlib.matlabOlderThan('R2025a'))
 
-nr = fullfile(pwd(), "no-read");
+nr = [pwd(), '/no-read'];
 
 tc.assertTrue(stdlib.touch(nr))
 tc.assertTrue(stdlib.set_permissions(nr, -1, 0, 0))