Re: [PATCH] abort: Do not flush stdio streams [BZ #15436]

[Florian Weimer <fweimer at redhat dot com>]

On 09/20/2017 11:30 PM, Carlos O'Donell wrote:
> On 09/18/2017 08:33 AM, Florian Weimer wrote:
> > On 08/30/2017 06:22 PM, Carlos O'Donell wrote:
> > > On 08/30/2017 10:52 AM, Florian Weimer wrote:
> > > > Here's an alternative patch which removes flushing completely.  This is
> > > > what Andreas suggested.
> > > >
> > > > I've added a short NEWS entry.
> > >
> > > Could you please also add a wiki note for
> > > https://sourceware.org/glibc/wiki/Release/2.27#Packaging_Changes
> > > which explains what happened and why?
> >
> > Do we have consensus for this patch?  I haven't pushed it yet.
>
> I thought we did :-)
>
> However, if you, the author of the patch, aren't sure, then it is certainly
> important to repost your patch and gather another round of review.
>
> The key point is that we don't have to do this flushing if we don't have to
> and it's hard to see anyone claiming they needed these exact semantics given
> that abort() is an abnormal termination scenario.

There were no objections to the patch:

  <https://sourceware.org/ml/libc-alpha/2017-08/msg01304.html>

I'm going to commit it shortly and update the wiki as well.

Thanks,
Florian