Fix a few small regressions (rev 1)

- honor devel flag in gem dependency checker - flatten targets list in version checker
ManageIQ · Apr 22, 2014 · fd36b69 · fd36b69
1 parent 97f9f48
commit fd36b69
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/bin/gem_dependency_checker.rb b/bin/gem_dependency_checker.rb
@@ -211,13 +211,15 @@ def print_dep(tgt, dep, versions)
 
 if conf[:gemname]
   gem = Polisher::Gem.retrieve(conf[:gemname])
-  gem.versions(:recursive => true, :dev_deps => true) do |tgt, dep, versions|
+  gem.versions(:recursive => true,
+               :dev_deps  => conf[:devel_deps]) do |tgt, dep, versions|
     print_dep(tgt, dep, versions)
   end
 
 elsif conf[:gemspec]
   gem = Polisher::Gem.from_gemspec(conf[:gemspec])
-  gem.versions(:recursive => true, :dev_deps => true) do |tgt, dep, versions|
+  gem.versions(:recursive => true,
+               :dev_deps  => conf[:devel_deps]) do |tgt, dep, versions|
     print_dep(tgt, dep, versions)
   end
 

diff --git a/lib/polisher/version_checker.rb b/lib/polisher/version_checker.rb
@@ -20,7 +20,7 @@ class VersionChecker
     # Enable the specified target(s) in the list of target to check
     def self.check(*target)
       @check_list ||= []
-      target.each { |t| @check_list << t }
+      target.flatten.each { |t| @check_list << t }
     end
 
     def self.should_check?(target)