can not see why this sudo woudl be needed, resolves sprinkle-tool#13

crafterm · Sep 16, 2011 · d2f1eab · d2f1eab
1 parent 693657a
commit d2f1eab
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/sprinkle/verifiers/ruby.rb b/lib/sprinkle/verifiers/ruby.rb
@@ -15,10 +15,10 @@ def ruby_can_load(*files)
         @commands << "ruby -e \"#{files.join(';')}\""
       end
 
-      # Checks if a gem exists by calling "sudo gem list" and grepping against it.
+      # Checks if a gem exists by calling "gem list" and grepping against it.
-      def has_gem(name, version=nil)
+      def has_gem(name, version = nil)
-        version = version.nil? ? '' : "--version '#{version}'"
+        version = version ? "--version '#{version}'" : '' 
-        @commands << "sudo gem list '#{name}' --installed #{version} > /dev/null"
+        @commands << "gem list '#{name}' --installed #{version} > /dev/null"
       end
     end
   end

diff --git a/spec/sprinkle/verify_spec.rb b/spec/sprinkle/verify_spec.rb
@@ -90,7 +90,7 @@
     end
 
     it 'should check that a ruby gem is installed' do
-      @verification.commands.should include("sudo gem list 'rails' --installed --version '2.1.0' > /dev/null")
+      @verification.commands.should include("gem list 'rails' --installed --version '2.1.0' > /dev/null")
     end
 
     it 'should check that an RPM is installed' do