mhulden / foma

Automatically exported from code.google.com/p/foma

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Modify bison files to require minimum version

dowobeha opened this issue · comments

On macOS, bison is present but is an ancient version. On macOS 10.15.7, the version of bison is 2.3.

As a result, some of the newer fixes to bison files use syntax that bison 2.3 doesn't recognize, resulting in bison syntax errors.

Bison supports marking a minimum required version.

There are two options. One is rewriting the bison files to support bison 2.3 syntax. While this would be nice for those of us on macOS, a simpler solution would be to use the bison require syntax to require a specific version of bison.

Hi,
Package for macOS is built without errors on macOS 10.15.7
Here's log: https://github.com/mhulden/foma/runs/2773115543
and ZIP archive: https://github.com/mhulden/foma/suites/2939992105/artifacts/66053844

@dowobeha is right. Foma builds on macOS in your CI pipeline only because the most current version of bison is installed via Homebrew.

@AmbientLighter Can't you just extend your CI pipeline to put the compiled binaries as attachments on a tagged release? It would be really helpful if you created tagged releases when you finish a new version. This way, users would not have to compile from source themselves.

@pemistahl I'm not a maintainer of foma, so I've asked @mhulden to tag releases or grant me a permissions.
See also: #93
Unfortunately, he didn't respond to any comments or requests.
Surely, I can tag releases in my own fork, but I'm not really sure whether it would do more harm than good.

Fair enough, @AmbientLighter. Thank you for the compiled binaries.