mesa3d-dev

[Mesa3d-dev] Current tinderbox regression (dri: fix dri_test.c for non-TLS build)

[Chris B. <cj...@la...>]

Hi,

   > dri: fix dri_test.c for non-TLS build
   > 
   > _glapi_Context and _glapi_Dispatch have different constness
   > between TLS and non-TLS builds.

Breaks the build here, on Fedora 11/ppc64:

http://tinderbox.x.org/builds/2010-03-23-0038/logs/libGL/#build

../../../../../src/mesa/drivers/dri/common/dri_test.c:22: error: conflicting types for '_glapi_Dispatch'
../../../../../src/mesa/glapi/glapi.h:97: note: previous declaration of '_glapi_Dispatch' was here
../../../../../src/mesa/drivers/dri/common/dri_test.c:23: error: conflicting types for '_glapi_Context'
../../../../../src/mesa/glapi/glapi.h:99: note: previous declaration of '_glapi_Context' was here

- Chris.
-- 
Chris Ball   <cj...@la...>
One Laptop Per Child

Re: [Mesa3d-dev] Current tinderbox regression (dri: fix dri_test.c for non-TLS build)

[Luca B. <luc...@gm...>]

According to the logs, that build was not based on that commit, which
instead actually fixes that issue.

http://tinderbox.x.org/builds/2010-03-23-0040/ was actually the first
tinderbox build using that, and it went past that issue to fail on
xeglthreads problem, which is unrelated.

Thanks anyway for reporting this.

Re: [Mesa3d-dev] Current tinderbox regression (dri: fix dri_test.c for non-TLS build)

[Chris B. <cj...@la...>]

Hi Luca,

   > According to the logs, that build was not based on that commit,
   > which instead actually fixes that issue.
   > 
   > http://tinderbox.x.org/builds/2010-03-23-0040/ was actually the
   > first tinderbox build using that, and it went past that issue to
   > fail on xeglthreads problem, which is unrelated.

Thanks!  You're right, this one's fixed on the latest run.

- Chris.
-- 
Chris Ball   <cj...@la...>
One Laptop Per Child