Project

General

Profile

Bug #3177

TestFileUtils#test_cp_r_symlink

Added by shyouhei (Shyouhei Urabe) about 10 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Normal
Target version:
ruby -v:
ruby 1.9.2dev (2010-04-20 trunk 27400) [x86_64-linux]
Backport:
[ruby-dev:41018]

Description

=begin
お気づきのことと思いますが、私のコミット以来trunkで1F増えています。
http://www.rubyist.net/~akr/chkbuild/debian/ruby-trunk/log/20100420T015800.diff.txt.gz

この原因は、FileUtils#cp_rの挙動を1.8と揃えたことにあります。
http://archive.good-day.net/~w3ml/w3ml.cgi/ruby-changes/msg/15498

なぜ1.8のほうに揃えたかというと、1.8のほうの挙動がそもそも意図されたものであるかのように見えるからです。
http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-dev/27149

したがってtrunkのこのテストは不適切ではないでしょうか。いきなりテストごと変えるのはよくないかなと思ったので、一旦チケットにします。
=end

#1

Updated by mame (Yusuke Endoh) about 10 years ago

=begin
遠藤です。

2010年4月20日8:16 Shyouhei Urabe redmine@ruby-lang.org:

お気づきのことと思いますが、私のコミット以来trunkで1F増えています。
http://www.rubyist.net/~akr/chkbuild/debian/ruby-trunk/log/20100420T015800.diff.txt.gz

この原因は、FileUtils#cp_rの挙動を1.8と揃えたことにあります。
http://archive.good-day.net/~w3ml/w3ml.cgi/ruby-changes/msg/15498

なぜ1.8のほうに揃えたかというと、1.8のほうの挙動がそもそも意図されたものであるかのように見えるからです。
http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-dev/27149

したがってtrunkのこのテストは不適切ではないでしょうか。いきなりテストごと変えるのはよくないかなと思ったので、一旦チケットにします。

時系列から見て、テストの直し忘れなんでしょうね。
テストを修正するのに賛成します。

--
Yusuke Endoh mame@tsg.ne.jp

=end

#2

Updated by mame (Yusuke Endoh) about 10 years ago

  • Status changed from Assigned to Closed

=begin
遠藤です。

成瀬さんが r27550 で直してくれましたので close します。

--
Yusuke Endoh mame@tsg.ne.jp
=end

Also available in: Atom PDF