Re: [AGL] Small patch for recent allegro changes.

[Milan M. <mil...@gm...>]

Thomas Fjellstrom wrote:
> Index: include/alleggl.h
> ===================================================================
> --- include/alleggl.h   (revision 1240)
> +++ include/alleggl.h   (working copy)
> @@ -68,7 +68,7 @@
>  #include "allegrogl/gl_ext.h"
> 
>  #ifdef ALLEGRO_WITH_XWINDOWS
> -#ifndef HAVE_LIBPTHREAD
> +#ifndef ALLEGRO_HAVE_LIBPTHREAD
>  #   error AllegroGL requires Allegro to have pthread support enabled!
>  #endif
>  #include <alleggl_config.h>
> 

> Yup. AGL source is now broken on allegro svn and any future releases.

Is with this patch AGL broken on allegro 4.2.1 and older? Should we keep 
both HAVE_LIBPTHREAD and ALLEGRO_HAVE_LIBPTHREAD?

like:
#ifdef ALLEGRO_WITH_XWINDOWS
    #if GET_ALLEGRO_VERSION() <= MAKE_VER(4, 2, 1)
       #ifndef HAVE_LIBPTHREAD
       #   error AllegroGL requires Allegro to have pthread support enabled!
       #endif
    #else
       #ifndef ALLEGRO_HAVE_LIBPTHREAD
       #   error AllegroGL requires Allegro to have pthread support enabled!
       #endif
    #endif
#endif


-- 
Milan Mimica
http://sparklet.sf.net