Ensure any file object in a tar archive has an mtime (#18113)

* Ensure any file object in a tar archive has an mtime

Following up on #18091, when minitar
writes a directory or symlink it also needs explicit mtime. After inspecting
artifacts built from #18019 we see some other missing mtimes. This commit
ensures that information is explicitly passed to the minitar writer.

* Update rakelib/artifacts.rake

Co-authored-by: João Duarte <[email protected]>

---------

Co-authored-by: João Duarte <[email protected]>

Author: donoghuc

rakelib/artifacts.rake

@@ -620,12 +620,14 @@ namespace "artifact" do
 
     def write_to_tar(tar, path, path_in_tar)
       stat = File.lstat(path)
+      # in the off-chance that mtime returns nil we don't want nil to be interpreted as epoch, so fall back to Time.now
+      mtime = (stat.mtime || Time.now).to_i
       if stat.directory?
-        tar.mkdir(path_in_tar, :mode => stat.mode)
+        tar.mkdir(path_in_tar, :mode => stat.mode, :mtime => mtime)
       elsif stat.symlink?
-        tar.symlink(path_in_tar, File.readlink(path), :mode => stat.mode)
+        tar.symlink(path_in_tar, File.readlink(path), :mode => stat.mode, :mtime => mtime)
       else
-        tar.add_file_simple(path_in_tar, :mode => stat.mode, :size => stat.size, :mtime => (stat.mtime || Time.now).to_i) do |io|
+        tar.add_file_simple(path_in_tar, :mode => stat.mode, :size => stat.size, :mtime => mtime) do |io|
           File.open(path, 'rb') do |fd|
             chunk = nil
             size = 0