This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Wed, Aug 09, 2017 at 10:19:30AM -0400, Carlos O'Donell wrote: > On 08/09/2017 09:10 AM, Zack Weinberg wrote: > > On Wed, Aug 9, 2017 at 6:39 AM, Florian Weimer wrote: > >> On 08/08/2017 09:07 PM, Carlos O'Donell wrote: > >>> On 08/08/2017 12:30 PM, Dmitry V. Levin wrote: > >>>> On Tue, Aug 08, 2017 at 09:20:17AM -0400, Carlos O'Donell wrote: > >>>>> On 08/07/2017 11:33 AM, Dmitry V. Levin wrote: [...] > I'm happy for Dmitry to commit them now that both you (Zack) and > Florian have looked at the patches and consider the changes OK. > Three developers is a good belt-and-suspenders peer review for > multiple-machine header changes. Wow, that was fast, thanks! Committed. FIY, here is the timeline of this patch: Tue, 18 Jul 2017: patch submitted Wed, 02 Aug 2017: 2.26 released Wed, 02 Aug 2017: patch ping Mon, 07 Aug 2017: commit heads-up Tue, 08 Aug 2017: first objection raised Wed, 09 Aug 2017: first and second reviews Wed, 09 Aug 2017: first objection removed Wed, 09 Aug 2017: patch committed Note that the ping after release attracted no attention for 5 days, unlike the commit heads-up that caused all this discussion; as result, the patch has been reviewed and committed. Please do not consider this case as an example how one can expedite a patch review! :) -- ldv
Attachment:
signature.asc
Description: PGP signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |