Re: [HACKERS] Commitfest infrastructure

2009-01-29 Thread Stefan Kaltenbrunner

Gregory Stark wrote:

Stefan Kaltenbrunner  writes:


Peter Eisentraut wrote:

On Thursday 29 January 2009 11:40:48 Stefan Kaltenbrunner wrote:

well from a quick glance there is the bugzilla demo install as well as
pieces of reviewboard and patchwork on the trackerdemo jail.

So what's the URL and where can we sign up?

note the "pieces" part of my mail :-) As far as I recall the patchworks install
somehow collided with the reviewboard one so it was disabled because Zdenek was
still actively using reviewboard.


For what it's worth Google seems to have rolled out reviewboard as a Google
App. We could use it hosted on there if we wanted.


well the problem is more that we don't seem to want reviewboard at all - 
getting it hosted as a proper service is not really an issue.




Stefan

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Commitfest infrastructure

2009-01-29 Thread Gregory Stark
Stefan Kaltenbrunner  writes:

> Peter Eisentraut wrote:
>> On Thursday 29 January 2009 11:40:48 Stefan Kaltenbrunner wrote:
>>> well from a quick glance there is the bugzilla demo install as well as
>>> pieces of reviewboard and patchwork on the trackerdemo jail.
>>
>> So what's the URL and where can we sign up?
>
> note the "pieces" part of my mail :-) As far as I recall the patchworks 
> install
> somehow collided with the reviewboard one so it was disabled because Zdenek 
> was
> still actively using reviewboard.

For what it's worth Google seems to have rolled out reviewboard as a Google
App. We could use it hosted on there if we wanted.

-- 
  Gregory Stark
  EnterpriseDB  http://www.enterprisedb.com
  Ask me about EnterpriseDB's On-Demand Production Tuning

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Commitfest infrastructure

2009-01-29 Thread Gregory Stark


I thought reviewboard looked pretty good for code quality patch review. It
would be cool if someone could write a mail filter which automatically added
any patches posted to the list to reviewboard.

Incidentally one issue with reviewboard/patchwork/whatever is that they tend
to encourage the review to do a line-by-line review of the code itself rather
than the overall architecture. That's fine for the kind of code-quality
reviews some authors (like myself I admit :( ) need sometimes. And it would
make it easier to do hit-and-run comments when you see minor issues not worth
pestering authors about on-list. But I don't think it really helps with the
hard reviews.


But that's just a cute tool for one particular part of the work. I don't think
it addresses workflow management like RT or debbugs (or trac?) would.


-- 
  Gregory Stark
  EnterpriseDB  http://www.enterprisedb.com
  Ask me about EnterpriseDB's RemoteDBA services!

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers