[5191] コミットルールを見直したいげんた 2007年11月26日 22:26

半年ほど前から気にはなっていたのになかなか言い出せなかったのですが...
現状の誰かがレビューした後でcommitというルールが開発の速度を落とすだけで効果に乏しいのではと感じています．

レビューという行為に意味がないというつもりはありませんが，レビューでバグはなくせないというのが実情ではないかと思います．特に状態遷移を伴う物やGUIに関わるものなどコードの目視だけで判断するのは無理(少なくとも非効率)でしょう．
そこで動作試験となりますが，十分な試験を行ったかが重要です．しかし，他の人の作った新機能とその影響範囲を理解して試験をするのが結構大変です．

UNICODE版の掲示板で様々な方が活発に意見を述べているのを見て，これまでの運営が能力ある方々の意欲をそいでいたのではと反省しつつ，何かよい方法がないかを模索したいと思います．レビュー，試験を見落とし無く完璧に行わずとも，容易にできる部分とコアとなる部分に作業を分けて，能力に応じて分担することで品質，速度，そして参加してくださる方の満足度とも高められるような形に持って行けないでしょうか．

皆様のご意見をお待ちしています．

[5192] Re:コミットルールを見直したい kobake 2007年11月27日 00:04

自分的には、ピアレビュー式の運用には不満は無いです。

＃現在のUNICODEパッチのレビューをすると言いつつ遅れてしまってますが、
　今回の場合はレビュー自体が手間というわけではなく、たまたま私生活の用事が重なって
　しまって作業が滞ってました。すみません。今レビュー作業を行ってます。

＞レビューという行為に意味がないというつもりはありませんが，
＞レビューでバグはなくせないというのが実情ではないかと思います．
＞特に状態遷移を伴う物やGUIに関わるものなどコードの目視だけで
＞判断するのは無理(少なくとも非効率)でしょう．

たしかに、これは自分も思いました。
「木を見て森を見ず」の言い回しで例えるならば、
「木」に関するミスは発見しやすいですけど
「森」に関するミスの発見や整合性の確認には骨が折れます。

ただ、とりあえずは「木」のミスだけでも発見できることに
意味があるんじゃないかなぁという思いです。
[5198] RE: コミットルールを見直したいげんた 2007年12月01日 10:43

多少ヒントになりそうな記事

Multiple Stable 2.6.23 Releases
http://kerneltrap.org/Linux/Multiple_Stable_2.6.23_Releases

As there's no real way that everyone can review all of these patches, I've decided to split them up into 6 different categories, and will be sending patches out in these categories for review.

これはrepository上では分岐しているのかな？
- [5241] Re2: コミットルールを見直したい kobake 2008年01月06日 01:56
  
  1つのパッチに対するレビューの労力 (作業量、知識量ともに) が少なく済むように、
  できるだけパッチは小さくするように務めてはいます。
  
  が、それでも大きくなってしまうパッチはあって、
  やはりレビューが付きにくくなりますね。。
  レビュー人口の少なさがネックなのでしょうか。
  
  未コミットのパッチを保留したまま新しいパッチを量産することはできますが、
  そうすると、保留されたパッチがそのうちコンフリクトしまくりの「使えない」パッチと化することが不安です。
  
  ＃そんなわけで、今のところ UNICODE版のコーディング作業は、
  　現存のパッチをコミットし終えるまで控えています。

[5191] コミットルールを見直したい げんた 2007年11月26日 22:26

[5192] Re:コミットルールを見直したい kobake 2007年11月27日 00:04

[5198] RE: コミットルールを見直したい げんた 2007年12月01日 10:43

[5241] Re2: コミットルールを見直したい kobake 2008年01月06日 01:56

[5191] コミットルールを見直したいげんた 2007年11月26日 22:26

[5198] RE: コミットルールを見直したいげんた 2007年12月01日 10:43