With this SCM, is the change identifier a server-stored ID that contains the
description and other information for Review Board to parse? Or is it more
like an atomic ID representing that change that gets pushed to the server
when committed?

If the former, and if you want to support pulling that information from the
server, then we'll need a patch to make Review Board less strict about the
type of input, and move validation of the ID into the SCMTools.

If the latter, then it's not really any different from our current SCMTools.
The changeset field is really for specifying an ID of an uncommitted change
that the SCM repository already knows about and has information useful for
automatically pulling into the review request when created.


Christian Hammond - chip...@chipx86.com
Review Board - http://www.review-board.org
VMware, Inc. - http://www.vmware.com

On Mon, May 11, 2009 at 4:09 AM, Thilo-Alexander Ginkel <
thilo.gin...@gmail.com> wrote:

> Hello everybody,
> while attempting to add some basic support for a proprietary SCM to
> Review Board / post-review I stumbled over one issue that I'm
> currently uncertain how to solve: The SCM I'm integrating uses change
> identifiers, which are not numeric. Instead arbitrary strings are
> used, which (at least with alpha4) leads to the following exception
> when attempting to create the review request:
> - invalid literal for int() with base 10: '<change ID>'
> Are there other SCMs supported by RB that also have to deal with this
> issue? If so, how was the problem solved for these SCMs?
> Any input is highly appreciated!
> Thanks,
> Thilo
> >

You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To post to this group, send email to reviewboard@googlegroups.com
To unsubscribe from this group, send email to 
For more options, visit this group at 

Reply via email to