Retrive bugs fixed from 'Jobs' in a p4 changelist

Review Request #680 — Created Dec. 25, 2008 and discarded

Information
Owner:	onkarshinde
Repository:	Review Board SVN (deprecated)
Branch:
Bugs:	785
Depends On:
Reviewers
Groups:	reviewboard
People:

Description

Changes to retrieve bugs fixed from jobs specified in a p4 changelist.

Testing Done

Change Summary:

Fixed a stupid mistake.

Diff:

Revision 2 (+40 -4)

Show changes

trunk/reviewboard/contrib/tools/post-review

Thanks for the patch. This is good functionality to have, but there's a better place for it.

We handle filling out these fields automatically in the Perforce SCMTool (scmtools/perforce.py). Basically, a few fields are parsed out of the change description and filled out. You can see the VMware-specific Perforce SCMTool (scmtools/perforce_vmware.py) for an example of adding custom fields to this.

What I'd suggest is modifying parse_change_desc in perforce.py to retrieve the fixes (using the p4 API) and setting changeset.bugs_closed.

This would unify the logic and remove the need for any custom configuration in post-review.

onkarshinde Jan. 19, 2009, 7:13 p.m.

1. I did this change specifically to workaround bug 777. And that is why the change lies in post-review script.
2. If we decide to move the change to scmtools/perforce.py then we will have to add a field in perforce repository configuration where user can specify the regular expression to be used to retrieve the bug numbers from jobs fixed.

You have a pending review.

Review Board 5.0.1 alpha 0 (dev)

Retrive bugs fixed from 'Jobs' in a p4 changelist

Change Summary:

Diff: