yylex error when rebuilding lexer.c

[ghost]

When my system builds "src/lexer.c" and "src/lexer.h", it adds this line among
others:

#define yylex jq_yylex

This causes a big problem:

In file included from src/parser.y:124:0:
src/lexer.h:44:19: error: conflicting types for ‘jq_yylex’
src/parser.y:149:5: note: in expansion of macro ‘yylex’
 int yylex(YYSTYPE* yylval, YYLTYPE* yylloc, block* answer, int* errors,
     ^
In file included from src/parser.y:124:0:
src/lexer.h:405:12: note: previous declaration of ‘jq_yylex’ was here
src/parser.y: In function ‘jq_yylex’:
src/parser.y:152:13: error: too few arguments to function ‘jq_yylex’
   int tok = jq_yylex(yylval, yylloc, lexer);
             ^
In file included from src/parser.y:124:0:
src/lexer.h:44:19: note: declared here
src/parser.y:149:5: note: in expansion of macro ‘yylex’
 int yylex(YYSTYPE* yylval, YYLTYPE* yylloc, block* answer, int* errors,
     ^

As a workaround, I can:

touch src/lexer.c

Or:

./configure --disable-maintainer-mode

but all these do is prevent "src/lexer.c" and "src/lexer.h" from being remade.

[wtlangford]

What are your system's flex/bison versions? (Also, are they flex/bison or lex/yacc?)
Also, what's your system?

[ghost]

@wtlangford see below:

$ flex --version
flex 2.6.3

$ bison --version
bison (GNU Bison) 3.0.4

$ uname --all
CYGWIN_NT-6.1 Steven-PC 2.8.0(0.309/5/3) 2017-04-01 20:47 x86_64 Cygwin

[ghost]

@wtlangford it is the newer Flex that is causing the problem:

• Cygwin 2.6.3 bad
• Cygwin 2.6.1 good
• Msys2 2.6.3 bad
• Msys2 2.6.1 good

[wtlangford]

That's... Odd. I would never have found that. Any idea what specifically broke? Are there maybe some new #defines we need?

…

On Sat, Apr 8, 2017, 17:55 Steven Penny ***@***.***> wrote: @wtlangford <https://github.com/wtlangford> it is the newer Flex that is causing the problem: - cygwin 2.6.3 bad - cygwin 2.6.1 good - msys2 2.6.3 bad - msys2 2.6.1 good — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1386 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADQ4VyIwMRBT7apBAGvKzkNjGqi2H5IXks5ruAJQgaJpZM4M3y2Q> .

[ghost]

@wtlangford see westes/flex#162 - issue has been fixed
on master and will be on 2.6.4 when it is released.

I think I will close this. It is a Flex issue not a JQ issue. Plus I dont really
have a good reason to rebuild lexer.c anyway. I have added
--disable-maintainer-mode to my build script:

http://github.com/svnpenn/glade/blob/1cb67d6/jq/install.sh#L6-L7

and removed Flex and Bison from my makefile:

http://github.com/svnpenn/glade/blob/1cb67d6/jq/makefile#L4

[therealromster]

Just patch flex 2.6.3 or downgrade to 2.6.2 https://crux.nu/ports/crux-3.2/core/flex/flex-2.6.3-add_defines.patch