[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[cvs-ml 454] Re: cvsmailer

中嶋です。

いろいろご指摘ありがとうございます。
まだCVSを骨まで使いこなしてるわけじゃないので、参考になります。
さっそく改造してみようと思います。

> * whoamiはeffective userを返すので、pserver methodをつかっていて
>   $CVSROOT/CVSROOT/passwdでユーザのマッピングをおこなっているとまずい。
>   $CVSROOT/CVSROOT/loginfoの行で$USERはcvsの認証ユーザ名に置換されるので
>   これを利用するとよいと思います。あるいはスクリプトの中で
>   $USER環境変数を参照してもよいかと思います。
> 
> * loginfoで%{sVv}をつかわずにcvs statusを呼んでいるのはなぜでしょう。
> 
> * cvs import時の対策も必要かも。importのときにはcommitinfoは呼ばれない。
> 
> * cvs logの出力の lines: じゃなくて cvs diff -c の出力を元に
>   追加行数/修正行数/削除行数を算出するとおもしろそう。
> 
> Perlで書かれているcommit_prep/log_accumはさっぱり理解できないのですが
> 中嶋さんのコードは読めますね。けっこうたいへんだということが理解できました。
　Rubyで書くとなぜか読みやすいので、長期的にみてメンテナンスしやすい、
バグの少ないコードになるようです。

ではでは。