[rubygems/rubygems] Don't pass regexp to Gem::Dependeny.new
from list, search, and query commands
It's deprecated functionality. https://github.com/rubygems/rubygems/commit/13d3eb6cb0
This commit is contained in:
parent
8b6a02de2f
commit
0350c179ea
@ -10,7 +10,7 @@ class Gem::Commands::ListCommand < Gem::Command
|
|||||||
|
|
||||||
def initialize
|
def initialize
|
||||||
super 'list', 'Display local gems whose name matches REGEXP',
|
super 'list', 'Display local gems whose name matches REGEXP',
|
||||||
:name => //, :domain => :local, :details => false, :versions => true,
|
:domain => :local, :details => false, :versions => true,
|
||||||
:installed => nil, :version => Gem::Requirement.default
|
:installed => nil, :version => Gem::Requirement.default
|
||||||
|
|
||||||
add_query_options
|
add_query_options
|
||||||
|
@ -20,7 +20,7 @@ class Gem::Commands::QueryCommand < Gem::Command
|
|||||||
def initialize(name = 'query',
|
def initialize(name = 'query',
|
||||||
summary = 'Query gem information in local or remote repositories')
|
summary = 'Query gem information in local or remote repositories')
|
||||||
super name, summary,
|
super name, summary,
|
||||||
:name => //, :domain => :local, :details => false, :versions => true,
|
:domain => :local, :details => false, :versions => true,
|
||||||
:installed => nil, :version => Gem::Requirement.default
|
:installed => nil, :version => Gem::Requirement.default
|
||||||
|
|
||||||
add_option('-n', '--name-matches REGEXP',
|
add_option('-n', '--name-matches REGEXP',
|
||||||
|
@ -7,7 +7,7 @@ class Gem::Commands::SearchCommand < Gem::Command
|
|||||||
|
|
||||||
def initialize
|
def initialize
|
||||||
super 'search', 'Display remote gems whose name matches REGEXP',
|
super 'search', 'Display remote gems whose name matches REGEXP',
|
||||||
:name => //, :domain => :remote, :details => false, :versions => true,
|
:domain => :remote, :details => false, :versions => true,
|
||||||
:installed => nil, :version => Gem::Requirement.default
|
:installed => nil, :version => Gem::Requirement.default
|
||||||
|
|
||||||
add_query_options
|
add_query_options
|
||||||
|
@ -54,12 +54,12 @@ module Gem::QueryUtils
|
|||||||
end
|
end
|
||||||
|
|
||||||
def defaults_str # :nodoc:
|
def defaults_str # :nodoc:
|
||||||
"--local --name-matches // --no-details --versions --no-installed"
|
"--local --no-details --versions --no-installed"
|
||||||
end
|
end
|
||||||
|
|
||||||
def execute
|
def execute
|
||||||
gem_names = if args.empty?
|
gem_names = if args.empty?
|
||||||
Array(options[:name])
|
[options[:name]]
|
||||||
else
|
else
|
||||||
options[:exact] ? args.map{|arg| /\A#{Regexp.escape(arg)}\Z/ } : args.map{|arg| /#{arg}/i }
|
options[:exact] ? args.map{|arg| /\A#{Regexp.escape(arg)}\Z/ } : args.map{|arg| /#{arg}/i }
|
||||||
end
|
end
|
||||||
@ -96,7 +96,7 @@ module Gem::QueryUtils
|
|||||||
end
|
end
|
||||||
|
|
||||||
def gem_name?
|
def gem_name?
|
||||||
!options[:name].source.empty?
|
!options[:name].nil?
|
||||||
end
|
end
|
||||||
|
|
||||||
def prerelease
|
def prerelease
|
||||||
@ -129,12 +129,10 @@ module Gem::QueryUtils
|
|||||||
display_header("LOCAL")
|
display_header("LOCAL")
|
||||||
|
|
||||||
specs = Gem::Specification.find_all do |s|
|
specs = Gem::Specification.find_all do |s|
|
||||||
s.name =~ name and req =~ s.version
|
name_matches = name ? s.name =~ name : true
|
||||||
end
|
version_matches = show_prereleases? || !s.version.prerelease?
|
||||||
|
|
||||||
dep = Gem::Deprecate.skip_during { Gem::Dependency.new name, req }
|
name_matches and version_matches
|
||||||
specs.select! do |s|
|
|
||||||
dep.match?(s.name, s.version, show_prereleases?)
|
|
||||||
end
|
end
|
||||||
|
|
||||||
spec_tuples = specs.map do |spec|
|
spec_tuples = specs.map do |spec|
|
||||||
@ -149,7 +147,7 @@ module Gem::QueryUtils
|
|||||||
|
|
||||||
fetcher = Gem::SpecFetcher.fetcher
|
fetcher = Gem::SpecFetcher.fetcher
|
||||||
|
|
||||||
spec_tuples = if name.respond_to?(:source) && name.source.empty?
|
spec_tuples = if name.nil?
|
||||||
fetcher.detect(specs_type) { true }
|
fetcher.detect(specs_type) { true }
|
||||||
else
|
else
|
||||||
fetcher.detect(specs_type) do |name_tuple|
|
fetcher.detect(specs_type) do |name_tuple|
|
||||||
|
@ -252,7 +252,7 @@ class TestGemCommandManager < Gem::TestCase
|
|||||||
Gem::Deprecate.skip_during do
|
Gem::Deprecate.skip_during do
|
||||||
@command_manager.process_args %w[query]
|
@command_manager.process_args %w[query]
|
||||||
end
|
end
|
||||||
assert_equal(//, check_options[:name])
|
assert_nil(check_options[:name])
|
||||||
assert_equal :local, check_options[:domain]
|
assert_equal :local, check_options[:domain]
|
||||||
assert_equal false, check_options[:details]
|
assert_equal false, check_options[:details]
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user