Backport #5565

STDOUT.puts writes LF (not CR+LF) in textmode

Added by Usaku NAKAMURA over 3 years ago. Updated over 3 years ago.

[ruby-dev:44796]
Status:Closed
Priority:Normal
Assignee:Masaya Tarui

Description

please backport r33627-33629.

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


Related issues

Duplicated by Ruby trunk - Bug #5576: [Universal Newline]改行コードが"\r\n"で出力されない Closed 11/06/2011

Associated revisions

Revision 33649
Added by Usaku NAKAMURA over 3 years ago

  • test/ruby/test_io_m17n.rb (TestIO_M17N#test_default_stdout_stderr_mode): new test for r33627-33629. see [backport #5565]

Revision 34541
Added by Yui NARUSE about 3 years ago

merge revision(s) 33152,33649:

* test/ruby/test_io_m17n.rb (TestIO_M17N#test_{default_mode_on_dosish,
  default_mode_on_unix,text_mode,binary_mode}): sorry for wrong test
  committed in r33144.  I'd misunderstood the spec of ruby's universal
  newline.

* test/ruby/test_io_m17n.rb
  (TestIO_M17N#test_default_stdout_stderr_mode): new test for
  r33627-33629.  see [backport #5565]

History

#1 Updated by Yusuke Endoh over 3 years ago

遠藤です。

2011/11/4 Usaku NAKAMURA redmine@ruby-lang.org:

please backport r33627-33629.

このバックポートがないせいで実際に悲鳴が上がっているようです。
#5576

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

で Yugui さんが新方式を提案しています。

  • trunk のバグを直したコミッタは、active branch にも同じ
    バグがあるかしらべて、あったら backport リクエスト投げよ

    • バックポートしてほしい人が適宜やってくれてもよい
  • 他のコミッタがリクエストを受けて、当該コミットに問題ない
    かレビューして、ブランチにコミットせよ

  • そのバックポートが互換性を崩したと思うコミッタは遠慮なく
    revert せよ

    • 最終的にはブランチメンテナが決定権を持つ

    一言で言うと、「ブランチにだけレビュー制を導入」ということか
    と思いました (遠藤の印象ですが) 。
    まだあくまで提案段階ですが、試しにこの方式でやってみたらどう
    でしょうか。Yugui さんの返事がないとダメかな?

    今回は windows 関係なので、tal さんか arton さんか Luis さん
    などが適任かと思いました。

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

    Yusuke Endoh mame@tsg.ne.jp

#2 Updated by Usaku NAKAMURA over 3 years ago

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

In message " 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:

で Yugui さんが新方式を提案しています。

ふむ。

今回の場合、

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

これはした。

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

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

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

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

頑張って書くか...

それでは。
--
U.Nakamura usa@garbagecollect.jp

#3 Updated by Usaku NAKAMURA over 3 years ago

  • Status changed from Open to Assigned
  • Assignee set to Masaya Tarui

レビュー待ち。

#4 Updated by Akio Tajima over 3 years ago

artonです。

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

では。

--
arton artonx@yahoo.co.jp

#5 Updated by Masaya Tarui over 3 years ago

  • Status changed from Assigned to Closed

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

Also available in: Atom PDF