touch: safer approach

Author: scivision

+stdlib/+java/set_modtime.m

@@ -3,7 +3,7 @@
 function ok = set_modtime(file, time)
 
 try
-  utc = posixtime(datetime(time, 'TimeZone', "UTC"));
+  utc = posixtime(datetime(time, 'TimeZone', 'UTC'));
 catch e
   if ~strcmp(e.identifier, 'Octave:undefined-function')
     rethrow(e)

+stdlib/+python/set_modtime.m

@@ -1,6 +1,6 @@
 function ok = set_modtime(file, time)
 
-utc = posixtime(datetime(time, 'TimeZone', "UTC"));
+utc = posixtime(datetime(time, 'TimeZone', 'UTC'));
 
 try
   s = py.os.stat(file);

+stdlib/+sys/set_modtime.m

@@ -2,14 +2,14 @@
 
 ok = false;
 
-tiso = string(datetime(dt), "yyyy-MM-dd HH:mm:ss");
+tiso = string(datetime(dt), 'yyyy-MM-dd HH:mm:ss');
 
 if ispc()
   cmd = sprintf('pwsh -c "(Get-Item ''%s'').LastWriteTime = ''%s''"', file, tiso);
 elseif ismac()
-  cmd = sprintf('touch -mt %s "%s"', string(dt, "yyyyMMddHHmm"), file);
+  cmd = sprintf('touch -mt %s "%s"', string(dt, 'yyyyMMddHHmm'), file);
 else
-  cmd = sprintf('touch -t %s "%s"', string(dt, "yyyyMMddHHmm"), file);
+  cmd = sprintf('touch -t %s "%s"', string(dt, 'yyyyMMddHHmm'), file);
 end
 % https://man7.org/linux/man-pages/man1/touch.1.html
 

+stdlib/touch.m

@@ -2,15 +2,11 @@
 
 function ok = touch(file)
 
-ok = stdlib.exists(file);
-
-if ~ok
-  fid = fopen(file, "w");
-  ok = fid > 0 && fclose(fid) == 0;
-end
+fid = fopen(file, 'a');
+ok = fid > 0 && fclose(fid) == 0;
 
 if ok
-  ok = stdlib.set_modtime(file, datetime("now"));
+  ok = stdlib.set_modtime(file, datetime('now'));
 end
 
 end

test/TestDisk.m

@@ -89,7 +89,7 @@ function test_is_mount(tc, B_ps)
 
 
 function test_hard_link_count(tc, B_jps)
-fn = "test_hard_link_count.txt";
+fn = 'test_hard_link_count.txt';
 tc.assertTrue(stdlib.touch(fn))
 
 [i, b] = stdlib.hard_link_count(fn, B_jps);
@@ -136,7 +136,7 @@ function test_is_dev_drive(tc, B_ps)
 function test_remove_file(tc)
 tc.assumeFalse(stdlib.matlabOlderThan('R2018a'), 'test shaky on Matlab < R2018a')
 
-f = "test_remove.tmp";
+f = 'test_remove.tmp';
 
 tc.verifyFalse(stdlib.remove(f), "should not succeed at removing non-existant path")
 

test/TestHDF5.m

@@ -62,7 +62,7 @@ function setup_file(tc)
 end
 
 
-methods (Test, TestTags = {'R2017b'})
+methods (Test, TestTags = {'R2017a'})
 
 function test_auto_chunk_size(tc)
 

test/TestTime.m

@@ -28,7 +28,7 @@ function test_touch_modtime(tc, B_set_modtime)
 tc.assertTrue(stdlib.touch(fn))
 t0 = stdlib.get_modtime(fn);
 
-ok = stdlib.set_modtime(fn, datetime("tomorrow"), B_set_modtime);
+ok = stdlib.set_modtime(fn, datetime('tomorrow'), B_set_modtime);
 
 if ismember(B_set_modtime, stdlib.Backend().select('set_modtime'))
   tc.assertTrue(ok)