Re: bug in AC_COMPUTE_INT

[Lars J. Aas]

On Thu, Nov 16, 2000 at 12:41:53PM +0100, Akim Demaille wrote:
: >>>>> "Lars" == Lars J Aas <address@hidden> writes:
: 
: Lars> I'm using AC_CHECK_SIZEOF on Cygwin and have found out that the
: Lars> `conftest' file contains a \r\n after the size.  The \r
: Lars> character follows the variable into config.status (the
: Lars> undefs.sed scripts) and ruins the sed scripts that produces
: Lars> config-headers (and the config-headers become empty,
: Lars> consequently).  I'm trying to find a good fix, but aren't all
: Lars> that familiar with the AC_COMPUTE_INT macro set yet, so if
: Lars> anyone else knows what has to be done, feel free to preempt my
: Lars> patch...
: 
: Just concentrate on 
: 
: AC_INIT
: _AC_COMPUTE_INT_RUN([sizeof (int)], result)
: 
: it should be enough.
: 
: The problem is bizarre: I don't think anything changed in this area.

I think I found it - checking now:

Index: aclang.m4
===================================================================
RCS file: /cvs/autoconf/aclang.m4,v
retrieving revision 1.89
diff -u -r1.89 aclang.m4
--- aclang.m4   2000/11/14 11:06:37     1.89
+++ aclang.m4   2000/11/16 11:56:22
@@ -397,7 +397,7 @@
 # inclusion of `stdio.h'.
 m4_define([AC_LANG_INT_SAVE(C)],
 [AC_LANG_PROGRAM([m4_default([$1], address@hidden:@include "stdio.h"])],
-[FILE *f = fopen ("conftestval", "w");
+[FILE *f = fopen ("conftestval", "wb");
 if (!f)
   exit (1);
 fprintf (f, "%d\n", ($2));])])