-
Notifications
You must be signed in to change notification settings - Fork 58
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 fsl build - add require and bison/flex versions to cmakelists #199
fix fsl build - add require and bison/flex versions to cmakelists #199
Conversation
guards for macros in flex files add versions for bison/flex in cmakelists add werror wall to cmake
Checks have failed for MacOS. CMake Error at /Users/runner/miniconda3/envs/riscv_perf_model/share/cmake-3.25/Modules/FindPackageHandleStandardArgs.cmake:230 (message): Does MacOS have support for bison 3.8.2 or later? |
Yeah, it's supported: https://anaconda.org/conda-forge/bison, but not part of the For giggles, can you try adding this line to that file?
|
did not work. It's finding bison 2.3, which was released apparently 18 years ago. I don't know anything about MacOs but is the image out of date ? Ubuntu image seems to find the package. Supposedly Bison 3.8.3 was released nov/dec 2021. Not cutting edge. We've got people that work on their Macs and can compile our model with FSL.
|
Well bummer. For those that have Macs what is the path to bison? Is it in conda? |
Well, let me retract my claim. I'm asking, but I might be wrong about the time frame. We only really support 22.04 ubuntu but people have been able to run on their Macs, as I said I might be wrong about when the last time someone did that. I am double checking. There are a number of hits for problem w/ bison and macos and it putting it into an unexpected PATH. What other options might there be if this is can not be solved? |
I don't see Bison in the Sparta conda env but it is installed on my Mac at |
This is a particularly MacOs thing, seemingly broken, maybe something in the OS requires 2.3. I can't imagine. The default bison location has 2.3. Installing a more modern version with brew places the new binary elsewhere but does not update the path. See the links below. https://stackoverflow.com/questions/31805431/how-to-install-bison-on-mac-osx
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yahoo!
build fix for aaron's issue
#189
I have added checks to CMakeLists for Flex 2.6.4 and Bison 3.8.2.
This is a likely cause but I can not duplicate the problem on my system.
I have specified the versions I have tested with.
I have added guards for the macros in the Flex file (fsl.l), just in case
I have added werror and wall to fsl cmakefile