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 -- Christian Hammond - [email protected] 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 < [email protected]> 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 [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~----------~----~----~----~------~----~------~--~---
