Looks like things are fixed. I committed the following:

-Melvin

Revision Changes Path
1.58 +8 -0 parrot/imcc/imc.h
Index: imc.h
===================================================================
RCS file: /cvs/public/parrot/imcc/imc.h,v
retrieving revision 1.57
retrieving revision 1.58
diff -u -w -r1.57 -r1.58
--- imc.h 17 Nov 2003 00:44:33 -0000 1.57
+++ imc.h 18 Nov 2003 20:58:27 -0000 1.58
@@ -17,6 +17,14 @@
#include "parrot/parrot.h"
+/* For people without unistd.h to compile Flex lexer
+ * unistd.h probably isn't required on most if any
+ * platforms anyway.
+ */
+#ifndef PARROT_HAS_HEADER_UNISTD
+# define YY_NO_UNISTD_H 1
+#endif
+
#define IMCC_MAX_REGS PARROT_MAX_ARGS
#if IMCC_MAX_REGS > 16
#error: flags wont fit





At 06:37 PM 11/18/2003 +0000, Jonathan Worthington wrote:
----- Original Message -----
From: "Melvin Smith" <[EMAIL PROTECTED]>
> Can you confirm that this is fixed? Upgrading my flex from 2.5.4 -> 2.5.6
> fixed the unist.d include issue. I checked in a new lexer just now.
'fraid not.  The #include for it is now enclosed in this block:-

/* Special case for "unistd.h", since it is non-ANSI. We include it way
* down here because we want the user's section 1 to have been scanned first.
* The user has a chance to override it with an option.
*/
#ifndef YY_NO_UNISTD_H
/* %- */
#include <unistd.h>
/* %+ */
/* %* */
#endif /* !YY_NO_UNISTD_H */

YY_NO_UNISTD_H is not defined at any point before this.  So, we still have
the problem.

Thanks,

Jonathan




Reply via email to