[ruby-dev:45193] Re: Plan to 1.9.3 1st patch release

From: "NARUSE, Yui" <naruse@...>
Date: 2012-02-05 13:45:06 UTC
List: ruby-dev #45193
(2012/02/05 7:31), KOSAKI Motohiro wrote:
>>>> = backport 時のレビューの廃止
>>>> めんどくさいんでやめませんかこれ。
>>>> メンテナ的視点でチケットを眺めると、誰か欲しい人がいるかも的にチケットを作られると、
>>>> どの程度切実に backport して欲しいのかわかりづらいし。
>>>> https://bugs.ruby-lang.org/projects/ruby-193/issues
>>>
>>> やめるのは労力が減る方向なのでまったく反対しませんが、いままで私の1.9.3へのbackport contribution は
>>> チケットドリブンだったので私のcontributionは継続が難しくなります
>>
>> コミット権がない人はチケットを作ってコミットできる人に backport してもらわないと
>> いけませんので、それを見てあげてくださるといいんじゃないでしょうか。
>> あとは、コミットされたものが問題ないか見て頂くとか。
> 
> うーん、想定している運用がよく分からないので教えて貰えますか。
> コミット権がない人のbackport request
> に関しては玉石混淆なので第一ステップはrejectするかどうかの判断だと思うのですがこれは誰がやる想定ですか。パッチの元々の作者はしない?

誰がやってもいいんじゃないでしょうか。backport request を見た人は、
(1) backport
(2) reject
(3) なんかコメント
(4) 放置
というような選択肢がありますね。

> あとリリースブランチに対してコミット後にレビューを開始するというのもピンと来ません。リリースブランチのコミットを監視してる人間なんて現状二桁いないので何人かが忙しかったら0レビューですり抜けていくような気がします。

結局品質を担保するのは chkbuild だろうってのと、
きちんとやってもどうせ実際のアプリで使ってみると問題出るだろうなと思っているので、
そこのレビューで手間を増やしても、その手間に見合わないのではないか、と言うのが
本質的な疑問なのですよ。
なので、「0レビューですり抜ける可能性がある」という指摘に対しては、
「その可能性は許容する」というのが答えです。

-- 
NARUSE, Yui  <naruse@airemix.jp>

In This Thread