Use the same BNFC version

[deemp]

• Make the version of BNFC fixed for devs so that changes to grammar don't produce larger than necessary diffs (like this one).

BNFC 2.9.6 doesn't exist on Hackage. The latest version is 2.9.5.

It also doesn't exist in our current resolver.

https://github.com/objectionary/normalizer/blob/c1b2a139e981f589b69f7e98b32c58db9c731f72/stack.yaml#L1

https://www.stackage.org/nightly-2023-12-14

However, I see comments that some files were generated by bnfc 2.9.6.

https://github.com/objectionary/normalizer/blob/c1b2a139e981f589b69f7e98b32c58db9c731f72/eo-phi-normalizer/src/Language/EO/Phi/Syntax/Abs.hs#L1

https://github.com/objectionary/normalizer/blob/c1b2a139e981f589b69f7e98b32c58db9c731f72/eo-phi-normalizer/src/Language/EO/Phi/Syntax/Print.hs#L1

[aabounegm]

That was my bad. I had BNFC built from source (master branch, which has the unreleased v2.9.6) from when I was contributing a PR. I have now removed my local build and installed v2.9.5 using Stack.
I see that the generated files now use v2.9.5 since #22, so I think we can close this issue.

[deemp]

Can we make use of full positions version? Related to #19.

If yes, pls, add your version to extra-deps in stack.yaml and make a PR.

[aabounegm]

Do we really need it (at least right now)?
Even if so, I think #19 is enough and no need to keep this unrelated issue open.

[fizruk]

Can we close this issue?

[deemp]

I don't mind.