On Thu, Mar 10, 2011 at 11:00 PM, Eric Niebler <eric@boostpro.com> wrote:
On 3/11/2011 3:39 AM, Rene Rivera wrote:
On 3/10/2011 1:14 PM, Henrik Sundberg wrote:
I've proposed this earlier: Add a precommit check, when the release branch is locked, that checks that any commit in the release path has a commit comment that is prefixed with e.g. "Authorized: ". If not, return error and write a description of how to proceed to stderr.
The result is that no one commits by mistake, and that everyone can understand what has happened and how to proceed.
I like it.
Seems easy and reasonable. I'd likely make the comment more specific, like the trac comments. Something like: (authorized by beman). That way there's an audit trail.
Great. Rene, can I convince you to implement this hook? Beman, once it's impemented, can you add it to your release procedures?
Sounds good to me! --Beman