On Thu, Mar 7, 2019 at 4:12 PM William A Rowe Jr <wr...@rowe-clan.net> wrote:
>
> did you want to backport this to 1.7.0 branch?

Done in r1854995, thanks for the reminder!

>
> Do we want to identify the delta and old pre-1.7 behavior in the doxygen
> of this function?

Something like:

Index: include/apr_thread_proc.h
===================================================================
--- include/apr_thread_proc.h    (revision 1854994)
+++ include/apr_thread_proc.h    (working copy)
@@ -816,6 +816,8 @@ APR_DECLARE(apr_status_t) apr_setup_signal_thread(
  * functions should return 1 if the signal has been handled, 0 otherwise.
  * @param signal_handler The function to call when a signal is received
  * apr_status_t apr_signal_thread((int)(*signal_handler)(int signum))
+ * @note From APR 1.7, SIGUSR2 is no more ignored by apr_signal_thread() and
+ * thus can by caught by the signal_handler.
  */
 APR_DECLARE(apr_status_t) apr_signal_thread(int(*signal_handler)(int signum));

?

Reply via email to