[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で書くとなぜか読みやすいので、長期的にみてメンテナンスしやすい、
バグの少ないコードになるようです。
ではでは。