Re: OS X compile issue

["John S. Estabrook" <jestabro --at-- ncsa.uiuc.edu>]

Mat, this is a known typo, fixed for the next release:

replace "oldTID" with "thread->mTID"


On Fri, 30 Jun 2006, Mat Schaffer wrote:

> I had a little trouble getting 2.0.2 to compile on OS X.  Here's the make 
> output:
>
> make  all-recursive
> Making all in compat
> if gcc -DHAVE_CONFIG_H -I. -I. -I..  -I../include -I../include   -O2  -MT 
> Thread.o -MD -MP -MF ".deps/Thread.Tpo" -c -o Thread.o Thread.c; \
> then mv -f ".deps/Thread.Tpo" ".deps/Thread.Po"; else rm -f 
> ".deps/Thread.Tpo"; exit 1; fi
> Thread.c: In function 'thread_stop':
> Thread.c:205: error: 'oldTID' undeclared (first use in this function)
> Thread.c:205: error: (Each undeclared identifier is reported only once
> Thread.c:205: error: for each function it appears in.)
> make[2]: *** [Thread.o] Error 1
> make[1]: *** [all-recursive] Error 1
> make: *** [all] Error 2
>
> My solution was to add #undef HAVE_PTHREAD_CANCEL just before the #ifdef 
> HAVE_PTHREAD_CANCEL.  (line 204)
>
> I'm sure this isn't ideal.  My linux machine passes over line 205.  I don't 
> know enough about PTHREAD_CANCEL to know the difference, but I get the 
> feeling that the 'pthread_cancel( oldTID) ' might be old code because 
> 'oldTID' doesn't exist anywhere else in the project.
>
> I'm up and running, either way.  Just thought the developers (assuming they 
> read this) might want to look into it.
> -Mat

John S. Estabrook
jestabro --at-- ncsa.uiuc.edu
jestabro --at-- dast.nlanr.net