On 20 July 2018 at 17:36, Richard Henderson <richard.hender...@linaro.org> wrote: > On 07/20/2018 09:25 AM, Peter Maydell wrote: >> On 20 July 2018 at 17:14, Richard Henderson >> <richard.hender...@linaro.org> wrote: >>> You already print the file, just include the line. Perhaps >>> >>> fprintf(stderr, >>> "%s:%d: kill_qemu tried to terminate QEMU " >>> "process but it dumped core with signal %s\n", >>> __FILE__, __LINE__, strsignal(WTERMSIG(wstatus))); >>> abort(); >> >> I wasn't convinced that strsignal() would be available >> on all the host OSes we build on (we don't currently use >> it outside linux-user/), and I definitely didn't think that >> it merited a configure test for its presence just for a >> test error message :-) > > Hmm. It has been in _GNU_SOURCE since the dawn of time > and in POSIX since 2008. > > For non-linux, I peeked at the OpenBSD man page, which says > > The strsignal() function first appeared in AT&T System V > Release 4 UNIX and was reimplemented for NetBSD 1.0. > > That suggests all of the extant BSDs should have it. > > MinGW has had the function since 2008. > > What other hosts do we support?
OSX, but that's I think OK as it inherits it from BSD. The configure script also has support for Solaris-variants and Haiku... thanks -- PMM
