This change has been marked as completed.

Describe the completed change (optional):

Pushed to rbtools release-0.5.x (3a78ad0). Thanks!

Summary

Support of Perforce Brokers

Review Request #5062 — Created Dec. 6, 2013 and submitted 11 years, 4 months ago

Information

Owner

jmarkham*

Repository

RBTools

Branch

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description*

Modification of Perforce client to support Perforce brokers. First check if the P4 Info has a Broker address defined .. if so use the broker, otherwise (as before) check the Server address.

Testing Done

Been using this mod for over a year at Cadence on RBTools 0.5.1

Issues

Description	From	Last Updated
Please change this to use only one # per line, and start the first setence ("first query") with a capital …	david	11 years, 4 months ago
This can be removed, because it will fall through to line 175 which does this exact same thing.	david	11 years, 4 months ago
There are no open issues

david 11 years, 5 months ago

Can you also cross-link the bug ID in this review request? Thanks!

david 11 years, 4 months ago

We're approaching an rbtools release, so I'm just going to clean up these issues and push it. Thanks!

rbtools/clients/perforce.py (Diff revision 1)

The issue has been resolved. Show all issues

Please change this to use only one # per line, and start the first setence ("first query") with a capital letter.

rbtools/clients/perforce.py (Diff revision 1)

The issue has been resolved. Show all issues

This can be removed, because it will fall through to line 175 which does this exact same thing.

rbtools/clients/perforce.py (Diff revision 1)
```
Will do.
```
rbtools/clients/perforce.py (Diff revision 1)
```
Agreed.
```

Status:: Completed
Change Summary:: Pushed to rbtools release-0.5.x (3a78ad0). Thanks!