Author: boisvert
Date: Fri Sep 17 17:10:30 2010
New Revision: 998209

URL: http://svn.apache.org/viewvc?rev=998209&view=rev
Log:
BUILDR-506 Gem packaging does not work under windows (Peter Donald)

Modified:
    buildr/trunk/CHANGELOG
    buildr/trunk/lib/buildr/packaging/gems.rb

Modified: buildr/trunk/CHANGELOG
URL: 
http://svn.apache.org/viewvc/buildr/trunk/CHANGELOG?rev=998209&r1=998208&r2=998209&view=diff
==============================================================================
--- buildr/trunk/CHANGELOG (original)
+++ buildr/trunk/CHANGELOG Fri Sep 17 17:10:30 2010
@@ -40,6 +40,7 @@
           e.g. (Java.java.lang.String cached as Java::Lang::String) 
           can shadow Ruby modules
 * Fixed:  BUILDR-501 Fix buildr label when listing tasks (Peter Donald)
+* Fixed:  BUILDR-506 Gem packaging does not work under windows (Peter Donald)
 * Fixed:  BUILDR-508 Remove unnecessary use of Java.classpath in OpenJPA
           extension (Peter Donald)
 

Modified: buildr/trunk/lib/buildr/packaging/gems.rb
URL: 
http://svn.apache.org/viewvc/buildr/trunk/lib/buildr/packaging/gems.rb?rev=998209&r1=998208&r2=998209&view=diff
==============================================================================
--- buildr/trunk/lib/buildr/packaging/gems.rb (original)
+++ buildr/trunk/lib/buildr/packaging/gems.rb Fri Sep 17 17:10:30 2010
@@ -60,11 +60,10 @@ module Buildr
       spec.mark_version
       spec.validate
 
-      File.open(name, 'w') do |io|
+      File.open(name, 'wb') do |io|
         Gem::Package.open(io, 'w', nil) do |pkg|
           pkg.metadata = spec.to_yaml
           file_map.each do |path, content|
-            p "path #{path} content #{content}"
             next if content.nil? || File.directory?(content.to_s)
             pkg.add_file_simple(path, File.stat(content.to_s).mode & 0777, 
File.size(content.to_s)) do |os|
               File.open(content.to_s, "rb") do |file|


Reply via email to