[asterisk-dev] [asterisk-commits] file: trunk r89394 - in /trunk: ./ include/asterisk/

[Luigi Rizzo]

On Sun, Nov 18, 2007 at 08:18:27AM -0400, Joshua Colp wrote:
> > in my opinion these autoconf checks are completely overkill: you
> > are just replacing a macro name with another one, and hiding to the
> > programmer the (trivial) logic behind that particular choice.
> > 
> > autoconf makes sense for things you cannot do from the preprocessor,
> > such as passing information back to the Makefile, checking the
> > location of a file or the availability of a C symbol or the number
> > of arguments in a function.
> 
> In this case on Linux PTHREAD_MUTEX_RECURSIVE_NP was actually an enum, not a #define which caused your check in lock.h to fail and mutex locks to NOT get initialized as recursive causing Asterisk to deadlock upon startup on Linux platforms. If you want to try to change it back into preprocessor logic feel free to.

ok then, can't do in the preprocessor, makes sense to use autoconf.
sorry for the noise and thanks for the explaination

cheers
luigi