Hi James -
The contributor should run his own code review. This is outlined here:
as well as here:
You, as the sponsor, need to help identify the appropriate
reviewers. You can do this by looking at previous folks
who had modified the code (particularly signficant modifications).
Looking at this bug, I would also say that security-discuss at opensolaris.org
might be a ogod place to find reviewers.
usually a webrev is sufficient for a code review. Many contributors
(internal & external) put a reasonable timer on the review. Obviously,
you still need to get code review comments - so even if the deadline
passes w/out review, it just means you have to try again.
In addition to security folks, I would want to see a kernel person
review as well.
Hope that helps,
On Wed, 3 Sep 2008, James.Anderson at sun.com wrote:
> Hi Valerie et al -
> I need guidance on the process we should be following for the
> code reviews. I am sponsoring CR 6613349 for Jason King.
> Does the requester drive the code review? Where do they send
> the code for review? What is the format of the code review?
> I can ask the requester to send a webrev out for review, but it
> isn't clear which email alias to send it to since the change is in
> common code and not exactly tied to a particular technology area.
> You have mentioned previously that reviewers need not be Sun
> I just need to know what to ask the requester to do to get the
> code reviewed per the Sun process.
> On 08/21/08 15:29, Valerie Bubb Fenwick wrote:
>> Hi Everyone -
>> We finally have the new best practices for sponsors for
>> Mercurial updated here:
>> There are actually choices for acknowledging the contributor
>> now. Please work with the contributor to determine the best
>> way of handling this for each particular case.
>> This other page has recently been updated as well, please
>> make sure you check it out, even if you have been a sponsor
>> or CRT member for a long time:
>> Many thanks to Mark Nelson, Rich Lowe, Bonnie Corwin & John Beck
>> for their assistance on getting these pages "just right".
>> Thank you,
>> Valerie, ON CRT Chair