git-fastclone 1.4.3 → 1.4.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4b755aef58d17ded702b989b16fb92cf93908413403bb99f92b542319c59fdbc
-  data.tar.gz: c7ad9ab44516bea864027aa562af274bd49999aea62eccbfc3df9421cc6674d6
+  metadata.gz: f23af05975dc472665784e91a3e69112694837cbd2a873076a7fb3197828970f
+  data.tar.gz: b3e2a26cbf1aff738c647eb37aa6b661975a396b680352288f68c0ba20da429d
 SHA512:
-  metadata.gz: 8f6cff1907474ac42d8a4936dba9e5e684440b72f9995ce80bdb0bc79bc169d79969386b32eb3c8e8cc64358fc1872b0cbb07011d2bebfd753b04423fc1198be
-  data.tar.gz: 889d56227ae2aff1aba3d9dfc33fdd7ab554b9f6ae77f21c036c66af5c2eb47756d49e62a2244ca922855442856caaeb772baace5bc1c9f093d1eae0a3043b02
+  metadata.gz: b9f239ae1a6cdf02d8aef5944f4d704bffaeb0d410191d7f5b7c759ba1d525dca27dd8efcec2e19ac4ffee9787ff3706b16860d0e16ff493136bcb2378b70a7c
+  data.tar.gz: fb954090b896bf41a396c956847133fc7d367fc564fbc8e3973ed61f2738b16f3926397b4526994d6c6d1781439970055f67e5c7d8137a0174a4f614b69f7a60

data/README.md

@@ -35,7 +35,7 @@ It aggressively updates these mirrors from origin and then clones from the mirro
 directory of your choosing. It always works recursively and multithreaded to get your checkout up as
 fast as possible.
 
-Detailed explanation [here][1].
+Detailed explanation [here](https://developer.squareup.com/blog/introducing-git-fastclone/).
 
 
 Usage
@@ -71,7 +71,7 @@ requests.
 When submitting code, please make every effort to follow existing conventions and style in order to
 keep the code as readable as possible.
 
-Before accepting any pull requests, we need you to sign an [Individual Contributor Agreement][2]
+Before accepting any pull requests, we need you to sign an [Individual Contributor Agreement](https://docs.google.com/a/squareup.com/forms/d/13WR8m5uZ2nAkJH41k7GdVBXAAbzDk00vxtEYjd6Imzg/viewform?formkey=dDViT2xzUHAwRkI3X3k5Z0lQM091OGc6MQ&ndplr=1)
 (Google form).
 
 Once landed, please reach out to any owner listed in https://rubygems.org/gems/git-fastclone and ask them to help publish the new version.
@@ -79,15 +79,10 @@ Once landed, please reach out to any owner listed in https://rubygems.org/gems/g
 
 Acknowledgements
 ----------------
-[thoughtbot/terrapin][3] - jyurek and collaborators
-
-[robolson][4]
-
-[ianchesal][5]
-
-[mtauraso][6]
-
-[chriseckhardt][7]
+* [robolson](https://github.com/robolson)
+* [ianchesal](https://github.com/ianchesal)
+* [mtauraso](https://github.com/mtauraso)
+* [chriseckhardt](https://github.com/chriseckhardt)
 
 
 License
@@ -105,11 +100,3 @@ License
     WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
     See the License for the specific language governing permissions and
     limitations under the License.
-
-[1]: https://corner.squareup.com/2015/11/fastclone.html
-[2]: https://docs.google.com/a/squareup.com/forms/d/13WR8m5uZ2nAkJH41k7GdVBXAAbzDk00vxtEYjd6Imzg/viewform?formkey=dDViT2xzUHAwRkI3X3k5Z0lQM091OGc6MQ&ndplr=1
-[3]: https://github.com/thoughtbot/terrapin
-[4]: https://github.com/robolson
-[5]: https://github.com/ianchesal
-[6]: https://github.com/mtauraso
-[7]: https://github.com/chriseckhardt

data/lib/git-fastclone/version.rb

@@ -2,5 +2,5 @@
 
 # Version string for git-fastclone
 module GitFastCloneVersion
-  VERSION = '1.4.3'
+  VERSION = '1.4.5'
 end

data/lib/git-fastclone.rb

@@ -47,13 +47,23 @@ module GitFastClone
     end
     module_function :reference_repo_dir
 
+    def reference_filename(filename)
+      separator = if RbConfig::CONFIG['host_os'] =~ /mswin|msys|mingw|cygwin|bccwin|wince|emc/
+                    '__'
+                  else
+                    ':'
+                  end
+      "#{separator}#{filename}"
+    end
+    module_function :reference_filename
+
     def reference_repo_submodule_file(url, reference_dir, using_local_repo)
-      "#{reference_repo_dir(url, reference_dir, using_local_repo)}:submodules.txt"
+      "#{reference_repo_dir(url, reference_dir, using_local_repo)}#{reference_filename('submodules.txt')}"
     end
     module_function :reference_repo_submodule_file
 
     def reference_repo_lock_file(url, reference_dir, using_local_repo)
-      lock_file_name = "#{reference_repo_dir(url, reference_dir, using_local_repo)}:lock"
+      lock_file_name = "#{reference_repo_dir(url, reference_dir, using_local_repo)}#{reference_filename('lock')}"
       File.open(lock_file_name, File::RDWR | File::CREAT, 0o644)
     end
     module_function :reference_repo_lock_file
@@ -110,7 +120,7 @@ module GitFastClone
     def run
       url, path, options = parse_inputs
 
-      require_relative './git-fastclone/version'
+      require_relative 'git-fastclone/version'
       msg = "git-fastclone #{GitFastCloneVersion::VERSION}"
       if color
         puts msg.yellow
@@ -354,10 +364,17 @@ module GitFastClone
       # To avoid corruption of the cache, if we failed to update or check out we remove
       # the cache directory entirely. This may cause the current clone to fail, but if the
       # underlying error from git is transient it will not affect future clones.
-      clear_cache(mirror, url)
+      #
+      # The only exception to this is authentication failures, because they are transient,
+      # usually due to either a remote server outage or a local credentials config problem.
+      clear_cache(mirror, url) unless auth_error?(e.output)
       raise e if fail_hard
     end
 
+    def auth_error?(error)
+      error.to_s =~ /.*^fatal: Authentication failed/m
+    end
+
     def retriable_error?(error)
       error_strings = [
         /^fatal: missing blob object/,
@@ -366,7 +383,8 @@ module GitFastClone
         /^fatal: pack has \d+ unresolved delta/,
         /^error: unable to read sha1 file of /,
         /^fatal: did not receive expected object/,
-        /^fatal: unable to read tree [a-z0-9]+\n^warning: Clone succeeded, but checkout failed/
+        /^fatal: unable to read tree [a-z0-9]+\n^warning: Clone succeeded, but checkout failed/,
+        /^fatal: Authentication failed/
       ]
       error.to_s =~ /.*#{Regexp.union(error_strings)}/m
     end

data/spec/git_fastclone_runner_spec.rb

@@ -308,7 +308,7 @@ describe GitFastClone::Runner do
 
   describe '.store_updated_repo' do
     context 'when fail_hard is true' do
-      it 'should raise a Runtime error and clear cache' do
+      it 'should raise a Runtime error and clear cache if there were no authentication errors' do
         status = double('status')
         allow(status).to receive(:exitstatus).and_return(1)
         ex = RunnerExecution::RunnerExecutionRuntimeError.new(status, 'cmd')
@@ -318,6 +318,18 @@ describe GitFastClone::Runner do
           subject.store_updated_repo(placeholder_arg, placeholder_arg, placeholder_arg, true)
         end.to raise_error(ex)
       end
+
+      it 'should raise a Runtime error and skip clearing the cache if there were authentication errors' do
+        status = double('status')
+        allow(status).to receive(:exitstatus).and_return(1)
+        ex = RunnerExecution::RunnerExecutionRuntimeError.new(status, 'cmd')
+        allow(ex).to receive(:output).and_return('fatal: Authentication failed')
+        allow(subject).to receive(:fail_on_error) { raise ex }
+        expect(FileUtils).to_not receive(:remove_entry_secure).with(placeholder_arg, force: true)
+        expect do
+          subject.store_updated_repo(placeholder_arg, placeholder_arg, placeholder_arg, true)
+        end.to raise_error(ex)
+      end
     end
 
     context 'when fail_hard is false' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: git-fastclone
 version: !ruby/object:Gem::Version
-  version: 1.4.3
+  version: 1.4.5
 platform: ruby
 authors:
 - Michael Tauraso
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-01 00:00:00.000000000 Z
+date: 2023-12-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -66,7 +66,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: git-clone --recursive on steroids!