Moved list (well tuple) of SCM client's out of determine_client() and into a global variable. This variable is now written one SCM per line (with a trailing comma on the last SCM). The aim of this is easier modification of post-review, either from monkey patching externally or with customized copies that are used for internal projects. This also makes it easier for a developer to add support for an SCM, track trunk and once they are ready publish their diffs so the new SCM support can be added easily.

See mailing list discussion thread;

http://groups.google.com/group/reviewboard/t/fcf9bf3e6c87d2f0