Re: [PATCH 1/4] gdb: make discrete_position return optional

Maciej W. Rozycki
2020-12-08T11:41:46Z

On Tue, 8 Dec 2020, Joel Brobecker wrote:

> > > ... that you revision logs have the Change-Id: field, which I assume
> > > was for when we were trying Gerrit out. Since it's no longer the case,
> > > perhaps you might want to disconnect the hook that adds them
> > > automatically?
> >
> > I use Gerrit myself for tracking the patches I send to GDB. That's the only
> > reliable way I found to keep track of patches I've sent that I haven't
> > merged yet, so they don't just get lost and forgotten on the list. And I
> > presume it doesn't really cause an inconvenience to anybody.
>
> I can only speak for myself, but personally I don't mind. The only
> reason I highlighted them is because I thought they were an oversight.

FWIW at <https://patchwork.sourceware.org/project/gdb/> we have a working
instance of patchwork too that can be used to track own and other people's
submissions, which also has been recently brought up to date as far as the
engine is concerned, thanks to people using it actively for the glibc
project. They've even set up a weekly meeting recently to review patches:
<https://sourceware.org/glibc/wiki/PatchworkReviewMeetings>. That might
be something to consider I believe for patch management.

Maciej