[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [p2-dev] committer votes and fumbled fingers.

Mik, Susan, Thanks for taking the time to consider this issue.  I personally don't feel "dis'd", but I can see how that could happen.

In this case there is absolutely no problem as both Shawn and Steffen are very strong developers and established members of the Eclipse community (and we would be lucky to have them help out with p2).  However, I think this particular policy could be gamed.  Do we see fundamental difference between someone committing 6 months worth of patches (both bug fixes and new features) and someone else creating a new enhancement in a separate project and "moving" it?  It would seem that a project *could* manipulate this policy to fast track commit status. (Give a developer commit rights on an incubator, have them fix one bug there, move the code).

It's an interesting discussion, clearly not relevant in this case, but interesting none-the-less.

I'm looking forward to the code in the p2 repository :-).

cheers,
ian

On Tue, Dec 8, 2009 at 1:33 PM, Susan Franklin McCourt <susan_franklin@xxxxxxxxxx> wrote:

Hi, Mik...
Even though I was aware of the move and the commit vote (having participated in calls about it and following the bug report), I must say I hadn't read the specific move review link before casting my vote. It does explain quite clearly that the commit rights are part of the move.

The most important thing to me was that other committers understand the nature of this nomination so it didn't seem like we were subjectively "fast tracking" folks.

susan

Inactive hide details for "Mik Kersten" ---12/08/2009 12:24:48 PM---Hi Susan,"Mik Kersten" ---12/08/2009 12:24:48 PM---Hi Susan,





To: <p2-dev@xxxxxxxxxxx>
cc:
Subject: RE: [p2-dev] committer votes and fumbled fingers.


Hi Susan,

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
the EDP:
http://www.eclipse.org/projects/dev_process/development_process.php#6_3_7_Mo
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.  

Mik

> -----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.
>
> Steffen
>
>
> ---------- 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
> nominations.
>
> 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
> nomination.
>
> susan
>
> _______________________________________________
> p2-dev mailing list
> p2-dev@xxxxxxxxxxx
>
https://dev.eclipse.org/mailman/listinfo/p2-dev
>
>
>
>
> --
> Steffen Pingel
> Committer,
http://eclipse.org/mylyn
> Senior Developer,
http://tasktop.com

_______________________________________________
p2-dev mailing list
p2-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/p2-dev


_______________________________________________
p2-dev mailing list
p2-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/p2-dev




--
R. Ian Bull | EclipseSource Victoria | +1 250 477 7484
http://eclipsesource.com | http://twitter.com/eclipsesource

GIF image

GIF image