On Mon, May 17, 2021 at 04:35:28PM +0200, Yann Ylavic wrote: > Since APR-1.7 (r1854995), SIGUSR2 is removed from the list of > uncatchable synchronous signals so it could be used instead of > SIGWINCH for graceful stop in httpd. > > The problem with SIGWINCH is when httpd is run from a pseudo-terminal > (not detached), it raises for any window resizing (which is not ideal, > notably when debugging). > > The patch would look like the attached one. Not so bad idea?
I'm not sure about this, it seems a bit dubious to change behaviour across an APR version upgrade (& httpd rebuild). That signal can get hard-coded in other places, e.g. the systemd unit. Would be better to require APR 1.7 and have the signal fixed, IMO, I don't know if that's desirable in itself. (Definitely +1 on removing the hard-coded references to SIGWINCH through the comments, FWIW) Regards, Joe