Re: Patch Review Process

[Steve Lawrence <slawrence@tresys.com>]

On 05/16/2011 12:04 PM, Greg Smith wrote:
> Steve Lawrence wrote:
>> Is commitfest custom built by the postgresql community or is it some
>> open source
>> project? I can't seem to find any information on what the backend is.
>>   
> 
> Yes to both, sort of.  It was built for this purpose, but the result is
> an open-source project available for others.  Source code is
> http://git.postgresql.org/gitweb?p=pgcommitfest.git ; it's a Perl
> application aimed to run inside Apache, with a PostgreSQL back-end as
> you might expect.
> The application was developed after really not finding the workflow of
> existing applications like Reviewboard to mesh well with the
> requirements for PostgrSQL's development process.  The main thing that I
> think may not translate well into some other environments, but might
> actually work here, is its heavy reliance on mailing list integration. 
> The main thing stored when you add a new entry to the CommitFest
> application is a pointer to a mailing list URL.  For the PostgreSQL
> lists, that is driven by the message-id of the e-mail sent.  The main
> data stored is essentially is a list of pointers into the mail archives,
> via message-id, with each tagged for what submission they all belong
> to.  Comments and such actually stored in the web application is
> minimal; the idea is that you're using it to track e-mails sent to the
> mailing list, where the primary review information is sent to.
> 
> Another caveat is that there was already a PostgreSQL single login
> service that this hooks into.  There's no internal support in the
> CommitFest app for account creation/maintenance, it just calls the API
> available on postgresql.org for that.
> 

That looks pretty nice and would probably work well for this community.
I'll definitely take a look at it when I get a chance.

Thanks.

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.