search

analogdevicesinc / gr-iio

IIO blocks for GNU Radio
GNU General Public License v3.0
95 stars 61 forks source link

Add versioning requirements for Bison #6

Closed tfcollins closed 7 years ago

tfcollins commented 7 years ago

On some distro's like centos bison is version 2.7 in their mirrors, which causes build failures. 3.0.4 works but not sure if earlier releases are valid.

rgetz commented 7 years ago

You are right -- Centos7.3-1611 seems to point to 2.7 (which is ancient - released Dec 2012) https://centos.pkgs.org/7/centos-x86_64/bison-2.7-4.el7.x86_64.rpm.html

Bison 3.0.4 was released Jan 2015. Time for them to upgrade. http://www.rpmfind.net/linux/rpm2html/search.php?query=bison&submit=Search+...&system=&arch=

lclausen-adi commented 7 years ago

I believe anything >= 3.0 is good. We might be able to re-write the grammar to be compatible with older versions, but I don't think we want to do that.

And it is only needed for the iio-math module (which isn't iio related at all), we could add an option to disable it, or even move it to its own gr-math package.

tfcollins commented 7 years ago

Yes you are correct Lars on the version. I do agree that just adding the version requirement is a better solution.

Maybe in the future we can look at partitioning those math blocks off (all 3 of them).