[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [jdt-dev] Bugzilla vs. Gerrit
|
Thanks Stephan, I also answered in the Gerrit to Eric. Thanks for the
notification!
Best regards, Lars
On Sun, Feb 23, 2020 at 2:44 PM Stephan Herrmann
<stephan.herrmann@xxxxxxxxx> wrote:
>
> Thanks Andrey, Lars, for clarification.
>
> See in particular https://git.eclipse.org/r/158132
>
> An example for a mixed change is https://git.eclipse.org/r/158128/
>
> His other recent gerrits you can see under Related Changes (7)
>
> Regarding groovy tooling: yes, contributions from s.o. who intimately knows the
> code from maintaining a fork is a well-tried model :)
> I'm even willing to review a few changes for the explicit benefit of the groovy
> tooling, provided I'm able to understand what is the advantage, and that the
> risk of changing JDT for all doesn't outweigh the benefit for groovy.
>
> Stephan
>
>
> On 23.02.20 13:09, Lars Vogel wrote:
> > Hi Stephan,
> >
> > The PMC decision was that we do not necessary require bug reports for cleanup or
> > code improvement patches to simply the contribution process.
> >
> > Also the PMC decided to encourage such patches.
> >
> > Contributions which changes something or fix something should have a bug report
> > associated with it.
> >
> > Also we dislike patches which mix cleanups and real changes
> >
> > Please share link to your discussion with Eric do that I can add that
> > clarification there also.
> >
> > Btw. it is IMHO awesome that Eric who IIRC works on the Groovy tooling
> > contributed to JDT and I definitely encourage this.
> >
> > Best regards, Lars
> >
> > Stephan Herrmann <stephan.herrmann@xxxxxxxxx
> > <mailto:stephan.herrmann@xxxxxxxxx>> schrieb am So., 23. Feb. 2020, 12:59:
> >
> > Hi Team,
> >
> > I wondered about a batch of gerrit changes recently created by Eric Milles,
> > some
> > of which gave no obvious reasons why a fix was needed in the first place. So on
> > one gerrit I asked him "why?" and he responded:
> > "Andrey Lusktov and Lars Vogel have repeatedly asked me to submit gerrit
> > changes instead of opening bugs with proposed fixes/patches."
> >
> > I haven't asked Andrey and Lars whether indeed they said "instead of", and I
> > haven't asked him if they spoke about Platform specifically or all of Eclipse,
> > but I think the JDT team should speak in one voice in such matters.
> >
> > I personally prefer to use bugzilla first and work with gerrits only for
> > working
> > out the details towards an agreed-upon goal. And in fact I believe the
> > option to
> > use gerrit without bugzilla was intended only for trivial changes obviously not
> > needing a discussion, wasn't it?
> >
> > I would even vote to keep the gerrit-only approach only for committers and
> > route
> > all incoming JDT contributions through bugzilla.
> >
> > Finally, I think it's important that every committer can easily see, whenever
> > communication already happened between a committer and a contributor, to the
> > end
> > that we don't annoy contributors with contradictory requests.
> >
> > What do others think?
> >
> > best,
> > Stephan
> >
> > PS: There's also the issue of separating real fixes from "style cleanup", but I
> > think we already agree that both kinds of changes should never be mixed in a
> > single contribution.
> > _______________________________________________
> > jdt-dev mailing list
> > jdt-dev@xxxxxxxxxxx <mailto:jdt-dev@xxxxxxxxxxx>
> > To change your delivery options, retrieve your password, or unsubscribe from
> > this list, visit
> > https://www.eclipse.org/mailman/listinfo/jdt-dev
> >
> >
> > _______________________________________________
> > jdt-dev mailing list
> > jdt-dev@xxxxxxxxxxx
> > To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> > https://www.eclipse.org/mailman/listinfo/jdt-dev
> >
>
> _______________________________________________
> jdt-dev mailing list
> jdt-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://www.eclipse.org/mailman/listinfo/jdt-dev
--
Eclipse Platform project co-lead
CEO vogella GmbH
Haindaalwisch 17a, 22395 Hamburg
Amtsgericht Hamburg: HRB 127058
Geschäftsführer: Lars Vogel, Jennifer Nerlich de Vogel
USt-IdNr.: DE284122352
Fax (040) 5247 6322, Email: lars.vogel@xxxxxxxxxxx, Web: http://www.vogella.com