cfg-grammar.y: add location reporting to bison debug

[bazsi]

This will add location information to the grammar debug output.

[alltilla]

This did not work for me at all.

To make it work I had to set the YY_LOCATION_PRINT macro instead of YYLOCATION_PRINT, and had to change the signature as well from main_location_print (FILE *yyo, YYLTYPE const * const yylocp) to main_location_print (FILE *yyo, YYLTYPE yylocp).

I can see that YYLOCATION_PRINT is documented, but my generated c code used YY_LOCATION_PRINT everywhere.

@bazsi Have you read anything about this?

FTR:

$ bison --version
bison (GNU Bison) 3.7.6
Written by Robert Corbett and Richard Stallman.

Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

[bazsi]

This might be a bison version issue. I found the stuff while reading the generated source. I'll check what I have as bison, but I think that's relatively recent as dbld uses Debian testing.

…

On Tue, Jun 4, 2024, 15:55 Attila Szakacs ***@***.***> wrote: This did not work for me at all. To make it work I had to set the YY_LOCATION_PRINT macro instead of YYLOCATION_PRINT, and had to change the signature as well from main_location_print (FILE *yyo, YYLTYPE const * const yylocp) to main_location_print (FILE *yyo, YYLTYPE yylocp). I can see that YYLOCATION_PRINT is documented, but my generated c code used YY_LOCATION_PRINT everywhere. @bazsi <https://github.com/bazsi> Have you read anything about this? FTR: $ bison --version bison (GNU Bison) 3.7.6 Written by Robert Corbett and Richard Stallman. Copyright (C) 2021 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. — Reply to this email directly, view it on GitHub <#138 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFOK5TNO2HVT2GHNBG6YELZFXBORAVCNFSM6AAAAABIVZKCZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXGYYDGOJRGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[bazsi]

@alltilla added bison dependency check as requested IRL.

[alltilla]

I have tested the old bison version, it behaves as expected.

[alltilla]

@bazsi Feel free to merge this if it works for your bison version.