You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by as...@apache.org on 2009/10/05 01:26:18 UTC

svn commit: r821639 - /buildr/trunk/lib/buildr/packaging/package.rb

Author: assaf
Date: Sun Oct  4 23:26:18 2009
New Revision: 821639

URL: http://svn.apache.org/viewvc?rev=821639&view=rev
Log:
Without this fix some packages don't get ActsAsArtifact methods, although we depend on all of them having a spec (specifically lines 151-154). This code passes on 1.8.6 since the type method is an alias for class, breaks on 1.9.1 since the type method no longer exists for Object.

Modified:
    buildr/trunk/lib/buildr/packaging/package.rb

Modified: buildr/trunk/lib/buildr/packaging/package.rb
URL: http://svn.apache.org/viewvc/buildr/trunk/lib/buildr/packaging/package.rb?rev=821639&r1=821638&r2=821639&view=diff
==============================================================================
--- buildr/trunk/lib/buildr/packaging/package.rb (original)
+++ buildr/trunk/lib/buildr/packaging/package.rb Sun Oct  4 23:26:18 2009
@@ -168,7 +168,7 @@
 
         if spec[:file]
           class << package ; self ; end.send(:define_method, :type) { spec[:type] }
-        elsif !package.respond_to?(:install)
+        else
           # Make it an artifact using the specifications, and tell it how to create a POM.
           package.extend ActsAsArtifact
           package.send :apply_spec, spec.only(*Artifact::ARTIFACT_ATTRIBUTES)