Bug #11199: IO.copy_stream degrade that can't call pathname to pathname

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: ruby 2.3.0dev (2015-05-30 trunk 50686) [x86_64-darwin14]
• Backport: 2.0.0: DONTNEED, 2.1: DONTNEED, 2.2: DONTNEED

[ksss (Yuki Kurihara)]

IO.copy_stream について、2.2.2〜2.3.0dev間で動作に差異がありました。

再現スクリプトを添付します。

#! /usr/bin/env ruby

require 'pathname'

IO.write "src", "ok"
src = Pathname.new("src")
dst = Pathname.new("dst")
IO.copy_stream src, dst
puts IO.read("dst")

$ ruby --version
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin14]
$ ruby copy_stream.rb
ok

$ ruby --version
ruby 2.3.0dev (2015-05-30 trunk 50686) [x86_64-darwin14]
$ ruby copy_stream.rb
copy_stream.rb:8:in `read': no implicit conversion from string (TypeError)
	from copy_stream.rb:8:in `read'
	from copy_stream.rb:8:in `copy_stream'
	from copy_stream.rb:8:in `<main>'

仕様変更でなければ、互換性を壊さないために2.2.2の動作を期待します。

[deivid (David Rodríguez)]

Hi!

We run into this here: https://github.com/janko-m/shrine/commit/399adcecad64c66ca4ffb1b1619777ff8551e761. That test passes in 2.2.3 but fails in 2.3.0.preview1, apparently due to this issue. Is fixing this planned for 2.3.0?

Thanks a lot.

[nobu (Nobuyoshi Nakada)]

• Related to Bug #11015: IO.copy_stream does not advance Tempfile destination added

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset r52750.

---

io.c: try to_io first

• io.c (copy_stream_body): try to_io conversion before read,
  readpartial, and write methods. [ruby-dev:49008] [Bug #11199]

[deivid (David Rodríguez)]

That was fast, Nobu! Thanks!!

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN to 2.0.0: DONTNEED, 2.1: DONTNEED, 2.2: DONTNEED

[ksss (Yuki Kurihara)]

Thank you all!