[Date Prev][Date Next] [Chronological] [Thread] [Top]

Minor include/ac/signal.h bug



There was a recent patch submitted that solved the problem of differing
<signal.h> on various Linux platforms.  The logic of the patch seems fine,
but on Linux/Alpha you get compiler warnings about LDAP_SIGUSR1 being
redefined on line 39 of include/ac/signal.h.

I looked at it and it looks like a typo;  I changed two cases of
LDAP_SIGUSR1 to LDAP_SIGUSR2.  It's a two-character fix, but I attached a
`diff -u` for funsies.

(This is from a CVS pull of mere minutes ago.)

Dan Morrill
Computer Scientist, Physicist
--- include/ac/signal.h	Wed Nov 25 11:59:14 1998
+++ include/ac/signal2.h	Wed Nov 25 11:54:51 1998
@@ -36,9 +36,9 @@
 #		if defined( SIGUNUSED )
 #			define LDAP_SIGUSR2	SIGUNUSED
 #		elif defined ( SIGINFO )
-#			define LDAP_SIGUSR1	SIGINFO
+#			define LDAP_SIGUSR2	SIGINFO
 #		elif defined ( SIGEMT )
-#			define LDAP_SIGUSR1	SIGEMT
+#			define LDAP_SIGUSR2	SIGEMT
 #		endif
 #	endif
 #endif