diff options
author | drbrain <drbrain@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-13 22:07:59 +0000 |
---|---|---|
committer | drbrain <drbrain@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-02-13 22:07:59 +0000 |
commit | e256dca8b24d225680809c123335c6b9bd69ae05 (patch) | |
tree | 0aa3d365167be1ebda54c942129bdcc1d1844915 | |
parent | 9476346ef5a6d486f8894e45a88ac8e96be14168 (diff) |
* Backport r39213 from trunk [ruby-trunk - Bug #7383]
* lib/rubygems.rb: Return BINARY strings from Gem.gzip and Gem.gunzip.
Fixes intermittent test failures. RubyGems issue #450 by Jeremey
Kemper.
* test/rubygems/test_gem.rb: Test for the above.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_0_0@39229 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | lib/rubygems.rb | 5 | ||||
-rw-r--r-- | test/rubygems/test_gem.rb | 28 |
3 files changed, 41 insertions, 1 deletions
@@ -1,3 +1,12 @@ +Thu Feb 14 07:04:13 2013 Eric Hodel <drbrain@segment7.net> + + * Backport r39213 from trunk [ruby-trunk - Bug #7383] + + * lib/rubygems.rb: Return BINARY strings from Gem.gzip and Gem.gunzip. + Fixes intermittent test failures. RubyGems issue #450 by Jeremey + Kemper. + * test/rubygems/test_gem.rb: Test for the above. + Thu Feb 14 07:01:12 2013 Eric Hodel <drbrain@segment7.net> * Backport part of r39166 from trunk [ruby-trunk - Bug #7809] diff --git a/lib/rubygems.rb b/lib/rubygems.rb index 039d26c1fe..177774376e 100644 --- a/lib/rubygems.rb +++ b/lib/rubygems.rb @@ -475,7 +475,9 @@ module Gem require 'zlib' data = StringIO.new data - Zlib::GzipReader.new(data).read + unzipped = Zlib::GzipReader.new(data).read + unzipped.force_encoding Encoding::BINARY if Object.const_defined? :Encoding + unzipped end ## @@ -486,6 +488,7 @@ module Gem require 'stringio' require 'zlib' zipped = StringIO.new + zipped.set_encoding Encoding::BINARY if Object.const_defined? :Encoding Zlib::GzipWriter.wrap zipped do |io| io.write data end diff --git a/test/rubygems/test_gem.rb b/test/rubygems/test_gem.rb index fdeef699d4..bf77009ca6 100644 --- a/test/rubygems/test_gem.rb +++ b/test/rubygems/test_gem.rb @@ -1,3 +1,4 @@ +# coding: US-ASCII require 'rubygems/test_case' require 'rubygems' require 'rubygems/installer' @@ -1237,6 +1238,33 @@ class TestGem < Gem::TestCase end end + def test_self_gunzip + input = "\x1F\x8B\b\0\xED\xA3\x1AQ\0\x03\xCBH" + + "\xCD\xC9\xC9\a\0\x86\xA6\x106\x05\0\0\0" + + output = Gem.gunzip input + + assert_equal 'hello', output + + return unless Object.const_defined? :Encoding + + assert_equal Encoding::BINARY, output.encoding + end + + def test_self_gzip + input = 'hello' + + output = Gem.gzip input + + zipped = StringIO.new output + + assert_equal 'hello', Zlib::GzipReader.new(zipped).read + + return unless Object.const_defined? :Encoding + + assert_equal Encoding::BINARY, output.encoding + end + if Gem.win_platform? && '1.9' > RUBY_VERSION # Ruby 1.9 properly handles ~ path expansion, so no need to run such tests. def test_self_user_home_userprofile |