DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 23D413858D28
From: Duncan Roe <duncan_roe@optusnet.com.au>
Date: Wed, 24 Nov 2021 09:36:49 +1100
To: cygwin@cygwin.com
Subject: Re: raise(-1) has stopped returning an error recently
Message-ID: <YZ1tAfzwlW8C84z4@slk1.local.net>
Mail-Followup-To: cygwin@cygwin.com
References: <YZsoj6UvpF6pcbtt@slk1.local.net>
 <YZtwMZ1LUbx+b5+s@calimero.vinschen.de>
 <YZuVy5+nbzPtiqdw@calimero.vinschen.de>
 <YZyl69ODRcBVnMed@slk1.local.net>
 <YZy5bRsZuulb6FUV@calimero.vinschen.de>
 <42c9bb90-dd78-edfa-99ff-f65f7e000956@SystematicSw.ab.ca>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <42c9bb90-dd78-edfa-99ff-f65f7e000956@SystematicSw.ab.ca>
Precedence: list
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cygwin-bounces+archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie.com@cygwin.com>

On Tue, Nov 23, 2021 at 11:18:25AM -0700, Brian Inglis wrote:
> On 2021-11-23 02:50, Corinna Vinschen via Cygwin wrote:
> > On Nov 23 19:27, Duncan Roe wrote:
> > > On Mon, Nov 22, 2021 at 02:06:19PM +0100, cygwin wrote:
> > > > On Nov 22 11:25, Corinna Vinschen via Cygwin wrote:
> > > > > On Nov 22 16:20, Duncan Roe wrote:
> > > > > > #include <stdio.h>
> > > > > > #include <signal.h>
> > > > > >
> > > > > > int
> > > > > > main (void)
> > > > > > {
> > > > > >    int retcod;
> > > > > >
> > > > > >    retcod = raise (-1);
> > > > > >    printf("raise(-1) returned %d\n", retcod);
> > > > > > }
> > > > >
> > > > > Thanks for the STC.  There are actually two long-standing problems,
> > > > > one in raise, calling pthread_kill even for non-threaded processes,
> > > > > one in pthread_kill, not checking the signal number.
> > > > >
> > > > > I'll patch them right away.
> > > >
> > > > I created a patchset:
> > > > https://sourceware.org/git?p=newlib-cygwin.git;a=commitdiff;h=d54d8f173d66
> > > > https://sourceware.org/git?p=newlib-cygwin.git;a=commitdiff;h=24bffff63eb7
> > > > https://sourceware.org/git?p=newlib-cygwin.git;a=commitdiff;h=afb7c557d2af
> > > >
> > > > Please test the latest developer snapshot from http://cygwin.com/snapshots/
> > >
> > > Thanks, that fixed it.
> >
> > Thanks for testing.
> >
> > > Btw to whoever maintains grep for cygwin: 'make check' should pass on
> > > next release (I patched out the surrogate-pair failre).
>
> I had no problems with test-raise last release.

I don't remember having a problem with it even a few weeks ago.

> I did with surrogate pairs but after spending too much time on all the test
> infrastructure around that, decided it was a low probability event, and wait
> until anyone complains to refer it upstream.

I wasted time on that too. That's why I patched surrogate-pair to not do its 3rd
test if 'uname -s' indicates Cygwin.

For the full story, see https://debbugs.gnu.org/cgi/bugreport.cgi?bug=27555#5

> Do Cygwin and/or Windows support surrogate pairs in UTF-8?

There are 3 tests in surrogate-pair and only the 3rd one failed. So I guess
surrogate pairs in UTF-8 "mostly work".

Cheers ... Duncan.

-- 
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple