JIRA and Review Board are both utilities that the Asterisk development community uses to help track and review code being written for Asterisk. Patches should generally be first attached to an issue in JIRA, as this will ensure that the issue is tracked appropriately and that proper attribution occurs. Review Board is a code review tool to help developers review a patch for submission.
This page provides guidelines for using Review Board.
Review Board Access
Users who have an accepted Digium License Agreement are automatically granted access to Review Board. Your same username/password will work for logging into Review Board as it does for the other Asterisk community sites.
Posting Code to Review Board
The easiest way to post a patch to Review Board is by using the rbt tool. Install it using
Essentially, rbt is a script that will take the output of
svn diff and create a review request out of it for you. Once you have a working copy with the changes you expect in the output of
svn diff, run the following command:
If it complains about not knowing which Review Board server to use, add the server option:
Dealing with New Files
I have one final note about an oddity with using post-review. If you maintain your code in a team branch, and the new code includes new files, there are some additional steps you must take to get post-review to behave properly.
You would start by getting your changes applied to a trunk working copy:
Then, apply the changes from your branch:
Now, the code is merged into your working copy. However, for a new file, subversion treats it as a copy of existing content and not new content, so new files don't show up in
svn diff at this point. To get it to show up in the diff, use the following commands so svn treats it as new content and publishes it in the diff:
Now, it should work, and you can run "rbt" as usual.
Posting a review request manually
On the main Reviewboard page, at the top of the page you'll find a link titled "New Reviewboard Request". Click the link and simply follow the prompts.
One important item to fill out before submitting the request is either the Reviewers field or the Groups field.
You can list specific individuals in the Reviewers field, or preferably you can just add "asterisk-dev" to the Groups field. Doing this will make sure others get notified of the new request.
Updating Patch on Existing Review Request
Most of the time, a patch on Review Board will require multiple iterations before other sign off on it being ready to be merged. To update the diff for an existing review request, you can use post-review and the -r option. Apply the current version of the diff to a working copy as described above, and then run the following command: