[rubygems/rubygems] Refactor: Move filesystem access into Definition …

…class rubygems/rubygems@2720da2659
ruby · Dec 12, 2023 · 0653fb0 · 0653fb0
1 parent 0ec88b5
commit 0653fb0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/bundler/cli/lock.rb b/lib/bundler/cli/lock.rb
@@ -64,7 +64,6 @@ def run
           file = file ? File.expand_path(file) : Bundler.default_lockfile
           puts "Writing lockfile to #{file}"
           definition.lock(file)
-          FileUtils.touch(file)
         end
       end
 

diff --git a/lib/bundler/definition.rb b/lib/bundler/definition.rb
@@ -342,7 +342,10 @@ def lock(file, preserve_unknown_sections = false)
 
       preserve_unknown_sections ||= !updating_major && (Bundler.frozen_bundle? || !(unlocking? || @unlocking_bundler))
 
-      return if file && File.exist?(file) && lockfiles_equal?(@lockfile_contents, contents, preserve_unknown_sections)
+      if file && File.exist?(file) && lockfiles_equal?(@lockfile_contents, contents, preserve_unknown_sections)
+        FileUtils.touch(file)
+        return
+      end
 
       if Bundler.frozen_bundle?
         Bundler.ui.error "Cannot write a changed lockfile while frozen."