diff options
| author | David RodrÃguez <deivid.rodriguez@riseup.net> | 2019-05-05 19:00:47 +0200 |
|---|---|---|
| committer | David RodrÃguez <deivid.rodriguez@riseup.net> | 2019-06-11 12:36:19 +0200 |
| commit | 1b7e274cbc9e51892e4efc115a97c728e026d966 (patch) | |
| tree | f10ba90eb7865fa2876129cc17439b7382a55696 | |
| parent | 5946d62ad03c57271ccb037680382b1db9fc2e65 (diff) | |
| download | bundler-1b7e274cbc9e51892e4efc115a97c728e026d966.tar.gz | |
Fix file:// handling under Windows
Windows paths do not start with a slash, so we add an extra slash to
separate the host from the path in file:// urls. Otherwise "D:" is
parsed as the host segment in the URI.
The path for those URLs now starts with "/", so we ignore that leading
character when using the URI's path.
This reduces Windows CI spec failures from 429 to 355.
| -rw-r--r-- | lib/bundler/fetcher.rb | 3 | ||||
| -rw-r--r-- | lib/bundler/fetcher/index.rb | 3 | ||||
| -rw-r--r-- | spec/support/path.rb | 5 |
3 files changed, 7 insertions, 4 deletions
diff --git a/lib/bundler/fetcher.rb b/lib/bundler/fetcher.rb index d21baf659d..8084a530d1 100644 --- a/lib/bundler/fetcher.rb +++ b/lib/bundler/fetcher.rb @@ -99,7 +99,8 @@ module Bundler uri = URI.parse("#{remote_uri}#{Gem::MARSHAL_SPEC_DIR}#{spec_file_name}.rz") if uri.scheme == "file" - Bundler.load_marshal Bundler.rubygems.inflate(Gem.read_binary(uri.path)) + path = Gem.win_platform? ? uri.path[1..-1] : uri.path + Bundler.load_marshal Bundler.rubygems.inflate(Gem.read_binary(path)) elsif cached_spec_path = gemspec_cached_path(spec_file_name) Bundler.load_gemspec(cached_spec_path) else diff --git a/lib/bundler/fetcher/index.rb b/lib/bundler/fetcher/index.rb index 9beb0e27d8..d10c21b041 100644 --- a/lib/bundler/fetcher/index.rb +++ b/lib/bundler/fetcher/index.rb @@ -30,7 +30,8 @@ module Bundler uri = URI.parse("#{remote_uri}#{Gem::MARSHAL_SPEC_DIR}#{spec_file_name}.rz") if uri.scheme == "file" - Bundler.load_marshal Bundler.rubygems.inflate(Gem.read_binary(uri.path)) + path = Gem.win_platform? ? uri.path[1..-1] : uri.path + Bundler.load_marshal Bundler.rubygems.inflate(Gem.read_binary(path)) elsif cached_spec_path = gemspec_cached_path(spec_file_name) Bundler.load_gemspec(cached_spec_path) else diff --git a/spec/support/path.rb b/spec/support/path.rb index 28682a1f25..ca05ef5e3b 100644 --- a/spec/support/path.rb +++ b/spec/support/path.rb @@ -64,10 +64,11 @@ module Spec def file_uri_for(path) protocol = "file://" + root = Gem.win_platform? ? "/" : "" - return protocol + "localhost" + path.to_s if RUBY_VERSION < "2.5" + return protocol + "localhost" + root + path.to_s if RUBY_VERSION < "2.5" - protocol + path.to_s + protocol + root + path.to_s end def gem_repo1(*args) |