On 10/10/2019 17:42, Fangrui Song wrote:
src/arith.y was deleted by commit f6e3b2 [ARITH] Add assignment and intmax_t support. We can delete AC_PROG_YACC to get rid of the build-time dependency.
Just a minor point of clarification: there is no build-time dependency on bison/yacc. The configure script checks whether bison/yacc exists, but if not, happily continues. This change does not get rid of a build-time dependency, it cleans up the configure script by removing an unused check.
Which is obviously still an improvement so your patch should still go in. Cheers, Harald van Dijk
