[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [p2-dev] committer votes and fumbled fingers.
- From: "Mik Kersten" <mik@xxxxxxxxxxx>
- Date: Tue, 8 Dec 2009 12:24:13 -0800
- Delivered-to: firstname.lastname@example.org
- Organization: Tasktop Technologies
- Thread-index: AQJ3TgGR4GarRQMmbdSjneLag7UwTgFPSRUFArIDwy0=
I see your concern, and was wondering about this myself when we initiated
the move discussions. The oddity being that individuals get commit rights
on a project that they may not have participated in directly to a sufficient
degree. My understanding of it from Wayne's post on bug 295273 is that this
kind of move and commit rights assignment follows the Move Review part of
ve_Review My reading of that is that the P2 project is deciding to give
commit rights to those committers, who have commit rights on another Eclipse
project, because it wants the component in P2, not because those committers
have earned their rights on P2. If that were not the case, I agree that it
would not do right by the others who have earned their commit rights.
> -----Original Message-----
> From: Steffen Pingel [mailto:steffen.pingel@xxxxxxxxxxx]
> Sent: December-08-09 11:05 AM
> To: Mik Kersten; Shawn Minto
> Subject: Fwd: [p2-dev] committer votes and fumbled fingers.
> I agree with Susan's points and don't think it's right that me and
> Shawn are getting commit rights on a project that we have not
> participated in. I think it's worth considering to withdraw and submit
> patches instead of committing directly to the P2 repository. The least
> we should do is to state that we expect the discovery component to be
> part of the incubator and will limit our activity to that one
> particular component.
> ---------- Forwarded message ----------
> From: Susan Franklin McCourt <susan_franklin@xxxxxxxxxx>
> Date: Tue, Dec 8, 2009 at 10:48 AM
> Subject: [p2-dev] committer votes and fumbled fingers.
> To: P2 developer discussions <p2-dev@xxxxxxxxxxx>
> Hi, everyone.
> I felt the need to explain a couple of things with respect to my
> committer votes for Steffen and Shawn.
> I thought I was voting +1 for Daniel as an equinox incubator committer
> (after all, he's already a regular committer), but apparently I
> inadvertently assigned a +1 and "sure" to one of Steffen's
> I was planning a longer explanation for voting +1 for Steffen and Shawn.
> So my votes look a bit confused, one saying "sure" the other with a
> long explanation, and a bunch of +1's after that (after getting
> frustrated with fumbling the portal interface).
> I thought it was important to say that:
> - I've seen patches and bug reports from both Steffen and Shawn in the
> course of my platform UI work, or watching SWT bugs, etc.
> - these guys are talented committers
> - I think it's a bit odd to vote in a committer who hasn't yet
> actively participated in the project itself
> - the Mylyn discovery work fills an important gap in our p2 story and
> has proven itself through adoption by other projects
> - it is best for p2 if we can integrate the work and have a cohesive
> story about how these pieces fit together and why
> - we need those guys to support their code in p2-land, hence they need
> to be committers
> I'm making this lengthy (public) explanation because I don't want to
> "dis" committers like Ian, Thomas, Henrik, Daniel who attended a bunch
> of calls and submitted loads of patches before we even started a
> p2-dev mailing list
> Steffen Pingel
> Committer, http://eclipse.org/mylyn
> Senior Developer, http://tasktop.com