Comment #2 on issue 2550 by dikenc...@gmail.com: post-review sounds load wrong client instance and concern PerforceClient does not have diff_changelist method
http://code.google.com/p/reviewboard/issues/detail?id=2550

The root cause is that:
The system path include perforce installation path, remove it solve this issue. Look at rbtools\clients\__init__.py, RBtools run "p4 info" command to determine whether a PerforceClient is needed. PerforceClient is checked prior to SVNClient. Recommend to

Change SCMCLIENTS definition to

    SCMCLIENTS = [
        SVNClient(options=options),
        CVSClient(options=options),
        ClearCaseClient(options=options),
        GitClient(options=options),
        MercurialClient(options=options),
        PerforceClient(options=options),
        PlasticClient(options=options),
    ]
As SVN is currently widely adopted and 'SVN info' return error message if run not from a working copy directory.


--
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en.

Reply via email to