Move post-review into a new top-level "rbtools" package
Review Request #788 — Created March 25, 2009 and submitted — Latest diff uploaded
Since the dawn of time, post-review has lived in the Review Board tree, despite being a client script and not necessarily being needed on the server. This moves post-review into a new rbtools package, which users will be able to easy_install (easy_install -U RBTools). The simplejson dependency will be fetched, and users won't have to go hunting in SVN for a script. Over time, I'd like to add new scripts into here (such as one to apply a diff from a review request), and further down the road I'd like to create an "rbapi" (or something) package that contains a Python API for talking to a Review Board server, which rbtools (and post-review) will then use. For now, the goal is to just get post-review moved over.
Tested that I could build the various packages we care about, and that the resulting post-review from a package works (I used it to post this review request).
Diff Revision 1 (Latest)
/dev/null Was /trunk/reviewboard/contrib/tools/P4Tool.txt | ||
/dev/null Was /trunk/reviewboard/contrib/tools/README.P4Tool | ||
/dev/null Was /trunk/reviewboard/contrib/tools/post-review | ||
/trunk/rbtools/AUTHORS | ||
/trunk/rbtools/INSTALL | ||
/trunk/rbtools/MANIFEST.in | ||
/trunk/rbtools/README | ||
/trunk/rbtools/ez_setup.py | ||
/trunk/rbtools/setup.cfg | ||
/trunk/rbtools/setup.py | ||
/trunk/rbtools/contrib/P4Tool.txt | ||
/trunk/rbtools/contrib/README.P4Tool | ||
/trunk/rbtools/scripts/post-review |
/trunk/reviewboard/contrib/tools/P4Tool.txt |
---|
/trunk/reviewboard/contrib/tools/README.P4Tool |
---|
/trunk/reviewboard/contrib/tools/post-review |
---|
/trunk/rbtools/AUTHORS |
---|
/trunk/rbtools/INSTALL |
---|
/trunk/rbtools/MANIFEST.in |
---|
/trunk/rbtools/README |
---|
/trunk/rbtools/ez_setup.py |
---|
/trunk/rbtools/setup.cfg |
---|
/trunk/rbtools/setup.py |
---|
/trunk/rbtools/contrib/P4Tool.txt |
---|
/trunk/rbtools/contrib/README.P4Tool |
---|
/trunk/rbtools/scripts/post-review |
---|