Subversionのブランチをソースコードレビューに使う

オンラインで行うソースコードレビューツールには、

• Review Board
• Rietveld

などいろいろある。

が、もっと簡単でお手軽な方法を見つけた。それは、
「Subversionのブランチをソースコードレビューに使う」
という物。

やり方:

1. レビュー対象のソースコードを、ブランチしておく
2. 各自レビューを行い、ソースコードの中にコメントを書き込む。(必ずしも、/* */で括られている必要は無い)
3. コメントをcommitする

これだけ。
コメントには、/* FIXME */とか、/* XXX */とか書いておくと、あとでgrepしやすくて良いかもしれない。

メリット:

• レビュアーが、好きなエディタが使える。
• 導入がカンタン

デメリット:

• レビューの内容をバグ表と関連付けるとか、直ったかどうか確認する、などの高度な事はできない

ReviewboardでリモートSMTPサーバーを使う

setting_local.pyのSEND_REVIEW_MAILをTrueにするだけだと、localhostの25番にSMTP接続しようとする。

リモートのSMTPサーバーを使うには、
EMAIL_HOST="ホスト名"
としてやる。

よくわからないが、Djangoのemailモジュールの仕様のようだ。

ReviewBoardで日本語ファイルのdiffをとる

そのままだと、日本語が化ける。SJIS前程で無理矢理変換。

Index: diffviewer/diffutils.py
===================================================================
--- diffviewer/diffutils.py     (revision 1136)
+++ diffviewer/diffutils.py     (working copy)
@@ -159,7 +159,8 @@
             u = unicode(s, 'utf-8')
             return u
         except UnicodeError:
-            u = unicode(s, 'iso-8859-15')
+            #u = unicode(s, 'iso-8859-15')
+            u = unicode(s, 'sjis')
             return u.encode('utf-8')
     else:

ソースコードレビューツール

フリーで使えてソースが公開されているソースコードレビューツールを探したら、
以下の2つが見つかった。

Review Board
Codestriker

前者はpyton, 後者はperlで書かれている。
とりあえず設置はしてみたので、しばらく使ってみる事にしよう。