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

[Thomas F. <tfj...@st...>]

On June 22, 2007, Milan Mimica wrote:
> 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?

Its 4.2.2 (4.2.1-svn) and older. The change was made earlier this week.

> 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

Probably is necessary.

-- 
Thomas Fjellstrom
tfj...@st...