Backport #5565
closedSTDOUT.puts writes LF (not CR+LF) in textmode
Description
please backport r33627-33629.
ところで誰が1.9.3のメンテナなんでしょう?
今回もyuguiさん?
Updated by mame (Yusuke Endoh) about 13 years ago
遠藤です。
2011/11/4 Usaku NAKAMURA redmine@ruby-lang.org:
please backport r33627-33629.
このバックポートがないせいで実際に悲鳴が上がっているようです。
[ruby-list:48514] #5576
ところで誰が1.9.3のメンテナなんでしょう?
今回もyuguiさん?
[ruby-core:40748] で Yugui さんが新方式を提案しています。
-
trunk のバグを直したコミッタは、active branch にも同じ
バグがあるかしらべて、あったら backport リクエスト投げよ- バックポートしてほしい人が適宜やってくれてもよい
-
他のコミッタがリクエストを受けて、当該コミットに問題ない
かレビューして、ブランチにコミットせよ -
そのバックポートが互換性を崩したと思うコミッタは遠慮なく
revert せよ- 最終的にはブランチメンテナが決定権を持つ
一言で言うと、「ブランチにだけレビュー制を導入」ということか
と思いました (遠藤の印象ですが) 。
まだあくまで提案段階ですが、試しにこの方式でやってみたらどう
でしょうか。Yugui さんの返事がないとダメかな?
今回は windows 関係なので、tal さんか arton さんか Luis さん
などが適任かと思いました。
ところで、この regression 対策のテストって、trunk には入って
ます? あるなら一緒に backport した方がいい気がします。
ないなら、急いで入れるべき?
--
Yusuke Endoh mame@tsg.ne.jp
Updated by usa (Usaku NAKAMURA) about 13 years ago
こんにちは、なかむら(う)です。
In message "[ruby-dev:44816] Re: [Backport93 - Backport #5565][Open] STDOUT.puts writes LF (not CR+LF) in textmode"
on Nov.07,2011 12:52:22, mame@tsg.ne.jp wrote:
[ruby-core:40748] で Yugui さんが新方式を提案しています。
ふむ。
今回の場合、
- trunk のバグを直したコミッタは、active branch にも同じ
バグがあるかしらべて、あったら backport リクエスト投げよ
- バックポートしてほしい人が適宜やってくれてもよい
これはした。
- 他のコミッタがリクエストを受けて、当該コミットに問題ない
かレビューして、ブランチにコミットせよ
talさんartonさんレビューして下さい。
(英語めんどいのでluisは略)
ところで、この regression 対策のテストって、trunk には入って
ます? あるなら一緒に backport した方がいい気がします。
ないなら、急いで入れるべき?
テスト書くのが面倒だったので省略しました。てへ。
このテストの場合はassert_in_out_errとか使えないのです。
頑張って書くか...
それでは。¶
U.Nakamura usa@garbagecollect.jp
Updated by usa (Usaku NAKAMURA) about 13 years ago
- Status changed from Open to Assigned
- Assignee set to tarui (Masaya Tarui)
レビュー待ち。
Updated by arton (Akio Tajima) about 13 years ago
artonです。
- 他のコミッタがリクエストを受けて、当該コミットに問題ない
かレビューして、ブランチにコミットせよ
これ見落としてました。
コミットしますよ。
では。
--
arton artonx@yahoo.co.jp
Updated by tarui (Masaya Tarui) about 13 years ago
- Status changed from Assigned to Closed
禍根を残しそうなコードだが問題の対処にはなっているという事で、r33660にてバックポートしました。