ReviewBoard+SVN配置强制code review

本文需要在svn与reviewboard安装成功之后开始进行配置，详情参考如下两篇文章根据实际情况操作：

Ubuntu安装Apache+Svn+Submin
https://blog.csdn.net/qq_26709459/article/details/80378373
Ubuntu下安装ReviewBoard
https://blog.csdn.net/qq_26709459/article/details/79247525

使用须知：

1、使用ReviewBoard作为code review工具，通过SVN hooks强制每一次提交文件都需要经过review；
2、必须获得"ship it"后才能提交；
3、"ship ip"成员以及数量可以自己进行配置

1、首先安装"reviewboard-svn-hooks",下载地址如下：
http://pypi.python.org/pypi/reviewboard-svn-hooks

下载之后解压并进入解压后的目录进行安装：

python setup.py install 

安装成功之后修改如下文件:

/etc/reviewboard-svn-hooks/conf.ini

文件内容如下：

[common]  
# 是否记录debugging输出，0为不输出，1为输出  
debug = 0  
  
[reviewboard]  
url=http://192.168.2.107/review/   # reviewboard访问地址
# reviewboard 的用户名密码
username=admin 
password=admin
  
[rule]  
# 最少需要有几个ship it  
min_ship_it_count = 2 
# 最少需要有几个专家 ship it  
min_expert_ship_it_count = 2
# 专家的reviewboard用户名，使用半角逗号分格  
experts = admin,button,test
 
# 指定必须review 的目录,默认为空，即表示强制 review 所有提交请求  
# eg： review_path = src  
review_path =   
# 配置的路径则不会触发检测。  
ignore_path =  

2、SVN hooks配置
进入svn仓库的hooks目录，将pre-commit.tmpl更名为pre-commit，删除所有内容，修改如下：

#!/bin/sh
REPOS="$1"  
TXN="$2"  
strict_review $REPOS $TXN  
exit $?  

注：pre-commit必须赋予权限，否则会出现如下错误:

pre-commit hook (exit code 255) with output:

3、eclipse配置Tao-ReviewBoard插件并配置
Tao-ReviewBoard离线下载地址：http://code.taobao.org/p/tao-reviewboard/file/2852/tao_reviewboard.zip
eclipse添加Tao-ReviewBoard插件可自行baidu添加，配置如下：

4、使用eclipse对修改文件进行review
选中需要review的修改文件->Pre-Commit Request to ReviewBoard


如下则提交成功：

5、查看review文件
登录ReviewBoard便可以查看review的文件：

点击对应文件：

6、提交文件
当获得正确数量的"ship it"之后就有了提交的权限
提交注释"review:60"，即review:id，id即为当前的review的id，“:”必须为英文状态

注：
①提交注释没有对应review的id

②提交的文件没有满足配置的"ship it"

③每个review的id只能使用一次，第二次使用则会出现如下错误

注：提交时遇到如下错误：

解决方案：对于"/etc/reviewboard-svn-hooks"目录必须要有权限，否则就会出现如上所示的问题，加上权限重新提交便可解决。