Add a P4 trigger script and P4V configurations for RBTools.

Review Request #12977 — Created April 21, 2023 and submitted

Information

RBTools
release-4.x

Reviewers

The new p4-trigger-script interfaces with RBTools and Perforce to
enable administrators to integrate their Perforce servers with Review
Board. It offers options for requiring submitted changes to have review
requests, to require those review requests to be approved, and to
auto-close review requests.

There's also a new set of P4V configurations, which replace the ancient
P4Win configuration (which still mentioned post-review). These are
available for Windows, macOS, and Linux, and allow users to post/update
review requests right from P4V.

Tested all the options in the trigger script locally.

Tested P4V on macOS and on Windows locally. A customer tested it on
Windows as well.

Verified the contents rendered correctly in the README.

Summary ID
Add a P4 trigger script and P4V configurations for RBTools.
The new `p4-trigger-script` interfaces with RBTools and Perforce to enable administrators to integrate their Perforce servers with Review Board. It offers options for requiring submitted changes to have review requests, to require those review requests to be approved, and to auto-close review requests. There's also a new set of P4V configurations, which replace the ancient P4Win configuration (which still mentioned post-review). These are available for Windows, macOS, and Linux, and allow users to post/update review requests right from P4V.
4fe7c36982e4377d5a2ec7a04d6fdee3061a9c68
Description From Last Updated

Should we tell them to talk to their admin rather than us?

daviddavid

This is worded a little bit awkwardly.

daviddavid
david
  1. 
      
  2. contrib/tools/p4-trigger-script (Diff revision 1)
     
     
     
     

    Should we tell them to talk to their admin rather than us?

  3. contrib/tools/p4-trigger-script (Diff revision 1)
     
     
     
     

    This is worded a little bit awkwardly.

  4. 
      
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.x (896f399)
Loading...