buildr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dona...@apache.org
Subject svn commit: r1430017 - in /buildr/trunk: CHANGELOG addon/buildr/findbugs.rb
Date Mon, 07 Jan 2013 21:17:21 GMT
Author: donaldp
Date: Mon Jan  7 21:17:20 2013
New Revision: 1430017

URL: http://svn.apache.org/viewvc?rev=1430017&view=rev
Log:
Findbugs addon correctly handles String dependencies (vs task dependencies)

Modified:
    buildr/trunk/CHANGELOG
    buildr/trunk/addon/buildr/findbugs.rb

Modified: buildr/trunk/CHANGELOG
URL: http://svn.apache.org/viewvc/buildr/trunk/CHANGELOG?rev=1430017&r1=1430016&r2=1430017&view=diff
==============================================================================
--- buildr/trunk/CHANGELOG (original)
+++ buildr/trunk/CHANGELOG Mon Jan  7 21:17:20 2013
@@ -1,4 +1,6 @@
 1.4.10 (Pending)
+* Fixed:  Findbugs addon correctly handles String dependencies
+          (vs task dependencies)
 * Fixed:  Checkstyle addon correctly handles String dependencies
           (vs task dependencies)
 * Added:  Created an addon 'buildr/wsgen' for generating wsdls from java

Modified: buildr/trunk/addon/buildr/findbugs.rb
URL: http://svn.apache.org/viewvc/buildr/trunk/addon/buildr/findbugs.rb?rev=1430017&r1=1430016&r2=1430017&view=diff
==============================================================================
--- buildr/trunk/addon/buildr/findbugs.rb (original)
+++ buildr/trunk/addon/buildr/findbugs.rb Mon Jan  7 21:17:20 2013
@@ -46,7 +46,7 @@ module Buildr
 
       def findbugs(output_file, source_paths, analyze_paths, options = { })
         dependencies = (options[:dependencies] || []) + self.dependencies
-        cp = Buildr.artifacts(dependencies).each(&:invoke).map(&:to_s).join(File::PATH_SEPARATOR)
+        cp = Buildr.artifacts(dependencies).each { |a| a.invoke() if a.respond_to?(:invoke)
}.map(&:to_s).join(File::PATH_SEPARATOR)
 
         args = {
             :output => "xml:withMessages",
@@ -79,7 +79,7 @@ module Buildr
             end
             if options[:extra_dependencies]
               ant.auxClasspath do |aux|
-                Buildr.artifacts(options[:extra_dependencies]).each(&:invoke).each do
|dep|
+                Buildr.artifacts(options[:extra_dependencies]).each { |a| a.invoke() if a.respond_to?(:invoke)
}.each do |dep|
                   aux.pathelement :location => dep.to_s
                 end
               end



Mime
View raw message