[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
RE: [cdt-dev] CDT 7.0 RC4 Candidate Available
|
> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Alex Blewitt
> Sent: Friday, June 04, 2010 3:08 PM
> To: CDT General developers list.
> Subject: Re: [cdt-dev] CDT 7.0 RC4 Candidate Available
>
> On 4 Jun 2010, at 19:49, James Blackburn wrote:
>
> > The egit guys are using gerrit and just clicking around their gerrit
> > instance it looks really really cool. The ability to comment in line
> > on patches would make the process of reviewing could be a killer
> > feature IMHO:
> > http://egit.eclipse.org/
>
> Using Gerrit is definitely a killer feature, and that
> (combined with GitHub) is one of the best reasons for using Git.
>
> One of the ways it helps is ensuring that patches never
> really go out of date. I've had several patches in Eclipse go
> stale because someone didn't get around to looking at them in
> time, and then they became unapplyable. Or, head had moved on
> slightly and they ask for a new patch they can apply against HEAD.
This would still be necessary for testing the patch, no?
> With a push to the Git review, the diff stays in diffable
> format even if the head (master) moves on.
Bugzilla diff gives you this.
Which is one of the reasons why attaching patches to a bug is so useful
in my opinion.
Marc