Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix build with bison 3.8 #73

Closed
wants to merge 1 commit into from
Closed

fix build with bison 3.8 #73

wants to merge 1 commit into from

Conversation

dylanaraps
Copy link

Changes in POSIX which were made in bison 3.8 cause builds of
OpenDoas to fail.

From the NEWS file:

To comply with the latest POSIX standard, in Yacc compatibility
mode (options -y/--yacc) Bison now generates prototypes for
yyerror and yylex. In some situations, this is breaking
compatibility: if the user has already declared these functions
but with some differences (e.g., to declare them as static, or
to use specific attributes), the generated parser will fail to
compile. To disable these prototypes, #define yyerror (to
yyerror), and likewise for yylex.

More information:

https://lists.gnu.org/archive/html/bug-bison/2021-09/msg00006.html

Changes in POSIX which were made in bison 3.8 cause builds of
OpenDoas to fail.

From the NEWS file:

  To comply with the latest POSIX standard, in Yacc compatibility
  mode (options `-y`/`--yacc`) Bison now generates prototypes for
  yyerror and yylex.  In some situations, this is breaking
  compatibility: if the user has already declared these functions
  but with some differences (e.g., to declare them as static, or
  to use specific attributes), the generated parser will fail to
  compile.  To disable these prototypes, #define yyerror (to
  `yyerror`), and likewise for yylex.

More information:

https://lists.gnu.org/archive/html/bug-bison/2021-09/msg00006.html
@dylanaraps
Copy link
Author

Not needed as of bison 3.8.1.

@dylanaraps dylanaraps closed this Sep 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant