On Thu, 2011-09-01 at 21:15 -0500, Yaakov (Cygwin/X) wrote:

> On Thu, 2011-09-01 at 10:52 -0700, Gaetan Nadon wrote:
> > New commits:
> > commit daaa028c111276c3ba4d60b454f0fe39bf10666e
> > Author: Gaetan Nadon <[email protected]>
> > Date:   Tue Aug 30 22:06:36 2011 -0400
> > 
> >     Add missing AC_CONFIG_HEADERS
> >     
> >     This is a requirement for all modules.
> >     www.gnu.org/software/autoconf/manual/autoconf.html#Configuration-Headers
> >     
> >     Signed-off-by: Gaetan Nadon <[email protected]>
> 
> This commit causes a major problem; now none of the AC_DEFINEs (or the
> results of AC_CHECK_*) are seen by the source file because none of them
> #include "config.h".
> 
> Either you need to add:
> 
> #ifdef HAVE_CONFIG_H
> #include "config.h"
> #endif
> 
> to the beginning of each source file, or revert commit
> daaa028c111276c3ba4d60b454f0fe39bf10666e.  This is a release blocker.
> 
> 

I am terribly sorry. Migh as well finish the job and change the source
file. It's the first time I actually add one of these. I thought all the
defines were handled  by Automake by some other mechanism. Should be
done shortly.


> Yaakov
> Cygwin/X
> 
> 
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel


Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to