Backport #5565: STDOUT.puts writes LF (not CR+LF) in textmode

• Status: Closed
• Assignee: tarui (Masaya Tarui)

[usa (Usaku NAKAMURA)]

please backport r33627-33629.

ところで誰が1.9.3のメンテナなんでしょう？
今回もyuguiさん？

[mame (Yusuke Endoh)]

遠藤です。

2011/11/4 Usaku NAKAMURA [e-mail]:

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 [e-mail]

[usa (Usaku NAKAMURA)]

こんにちは、なかむら(う)です。

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, [e-mail] wrote:

[ruby-core:40748] で Yugui さんが新方式を提案しています。

ふむ。

今回の場合、

• trunk のバグを直したコミッタは、active branch にも同じ
  バグがあるかしらべて、あったら backport リクエスト投げよ
  • バックポートしてほしい人が適宜やってくれてもよい

これはした。

• 他のコミッタがリクエストを受けて、当該コミットに問題ない
  かレビューして、ブランチにコミットせよ

talさんartonさんレビューして下さい。
(英語めんどいのでluisは略)

ところで、この regression 対策のテストって、trunk には入って
ます？ あるなら一緒に backport した方がいい気がします。
ないなら、急いで入れるべき？

テスト書くのが面倒だったので省略しました。てへ。
このテストの場合はassert_in_out_errとか使えないのです。

頑張って書くか...

それでは。

U.Nakamura [e-mail]

[usa (Usaku NAKAMURA)]

• Status changed from Open to Assigned
• Assignee set to tarui (Masaya Tarui)

レビュー待ち。

[arton (Akio Tajima)]

artonです。

• 他のコミッタがリクエストを受けて、当該コミットに問題ない
  かレビューして、ブランチにコミットせよ
  これ見落としてました。
  コミットしますよ。

では。

--
arton [e-mail]

[tarui (Masaya Tarui)]

• Status changed from Assigned to Closed

禍根を残しそうなコードだが問題の対処にはなっているという事で、r33660にてバックポートしました。