-
-
-
/trunk/reviewboard/scmtools/perforce.py (Diff revision 1) I ran into a bug with this on our VMware server. If the changeset does not exist, this throws an exception. I suggest: if changeset: return self.parse_change_desc(...) return None
Update perforce SCMTool to use p4python 08.1 API.
Review Request #460 — Created July 23, 2008 and submitted
Information | |
---|---|
david | |
Review Board SVN (deprecated) | |
trunk | |
Reviewers | |
reviewboard | |
As of the 08.1 API release, the p4python wrapper is now an officially supported product from Perforce. This is awesome, but they changed things up a bit (entirely for the better -- parsing changesets is quite a bit simpler now). This change updates our perforce tool to use the new API.
This code has been running on reviewboard.eng since they upgraded our main perforce server a couple months ago.