This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: Clean up state on patchwork
- From: Joseph Myers <joseph at codesourcery dot com>
- To: Mike Frysinger <vapier at gentoo dot org>
- Cc: Carlos O'Donell <carlos at redhat dot com>, Siddhesh Poyarekar <sid at reserved-bit dot com>, <libc-alpha at sourceware dot org>, Florian Weimer <fweimer at redhat dot com>
- Date: Mon, 28 Sep 2015 15:53:01 +0000
- Subject: Re: Clean up state on patchwork
- Authentication-results: sourceware.org; auth=none
- References: <1442410615 dot 2348 dot 19 dot camel at reserved-bit dot com> <alpine dot DEB dot 2 dot 10 dot 1509172302350 dot 2455 at digraph dot polyomino dot org dot uk> <alpine dot DEB dot 2 dot 10 dot 1509252025170 dot 1123 at digraph dot polyomino dot org dot uk> <20150925205526 dot GA1535 at vapier dot lan> <56094872 dot 80102 at redhat dot com> <20150928153301 dot GL5140 at vapier dot lan>
On Mon, 28 Sep 2015, Mike Frysinger wrote:
> > What if you're on vacation? What if you're busy? The patchwork queue
> > is still really helpful to keep your list of patches you need to go
> > back to and resubmit.
>
> my proposal doesn't prevent that. perform a search where the owner is you and
> the state is set to dropped.
Dropped is for patches the submitter decided to drop. It's meant to be an
explicit submitter decision that a patch is no longer of relevance, not
someone else's decision.
I don't think we should consider the AArch64 ILP32 patches, for example,
as being dropped (currently at 11 months since submission) simply because
of how long it takes to get the ILP32 support into the Linux kernel (maybe
we should use the Deferred state in such cases, though there seems to be
very little use of it at present and it's not clear if the Deferred
patches are correctly listed as such; Deferred patches need occasional
review to see if the conditions of the deferral still apply or if the
patch can now be reviewed, rather than it being a dumping ground we don't
go back to). Nor should we presume a submitter to have dropped a patch
because we've been unduly slow at reviewing it.
--
Joseph S. Myers
joseph@codesourcery.com