Project

General

Profile

Actions

Backport #5565

closed

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

Added by usa (Usaku NAKAMURA) about 13 years ago. Updated about 13 years ago.

Status:
Closed
[ruby-dev:44796]

Description

please backport r33627-33629.

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


Related issues 1 (0 open1 closed)

Has duplicate Ruby master - Bug #5576: [Universal Newline]改行コードが"\r\n"で出力されないClosed11/06/2011Actions

Updated by mame (Yusuke Endoh) about 13 years ago

遠藤です。

2011/11/4 Usaku NAKAMURA :

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

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, wrote:

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

ふむ。

今回の場合、

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

これはした。

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

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

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

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

頑張って書くか...

それでは。

U.Nakamura

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

Updated by tarui (Masaya Tarui) about 13 years ago

  • Status changed from Assigned to Closed

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

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0