28msec / zorba

JSONiq & XQuery Query Processor
http://zorba.28.io/
Apache License 2.0
114 stars 22 forks source link

Bison and flex should be installed on the travis PQ #80

Closed fcavalieri closed 10 years ago

fcavalieri commented 10 years ago

-- Looking for bison -- Looking for flex -- Bison: 0.0 < 2.6 -- Bison: 0.0 < 3.0 -- GNU Bison is not available -- the parser will not be regenerated -- Using repository files /home/travis/build/28msec/zorba/src/xquery_parser.cpp/home/travis/build/28msec/zorba/src/jsoniq_parser.cpp -- Flex: 0.0.0 < 2.5.35 -- GNU Flex is not available -- the lexer will not be regenerated

wcandillon commented 10 years ago

@fcavalieri We had a very tricky issue with that (cmake version range was different from the one that zorba really supports). This is maybe fixed with the latest changes in the parser which have been made?

wcandillon commented 10 years ago

It's in fact explicitly disabled at https://github.com/28msec/zorba/blob/master/.travis.yml#L5

fcavalieri commented 10 years ago

Yes, I fixed the range in the CMake. Bison 2.6.5 (last of series 2.6.x) and Flex 2.5.39 are the one I use.

On Sat, Aug 9, 2014 at 12:40 AM, William Candillon <notifications@github.com

wrote:

@fcavalieri https://github.com/fcavalieri We had a very tricky issue with that (cmake version range was different from the one that zorba really supports). This is maybe fixed with the latest changes in the parser which have been made?

— Reply to this email directly or view it on GitHub https://github.com/28msec/zorba/issues/80#issuecomment-51665614.

fcavalieri commented 10 years ago

Actually, if the issues we had with bison/flex before were the one I remember (failing tests) it was simply that the pregenerated parser C++ files were outdated. If bison was installed they were regenerated and they would behave differently.

wcandillon commented 10 years ago

It is possible that there is a typo in the cmake output: -- Bison: 2.5 < 2.6 -- Bison: 2.5 < 3.0 -- GNU Bison's version 2.5 is less than required (2.4) -- the parser will not be regenerated

I looked to install 2.6 or higher on ubuntu 12.04 but didn't find a repository. But maybe I'm missing trivial?

fcavalieri commented 10 years ago

You need to do it manually from the tar.gz they distribute

On Sat, Aug 9, 2014 at 9:23 PM, William Candillon notifications@github.com wrote:

It is possible that there is a typo in the cmake output: -- Bison: 2.5 < 2.6 -- Bison: 2.5 < 3.0 -- GNU Bison's version 2.5 is less than required (2.4) -- the parser will not be regenerated

I looked to install 2.6 or higher on ubuntu 12.04 but didn't find a repository. But maybe I'm missing trivial?

— Reply to this email directly or view it on GitHub https://github.com/28msec/zorba/issues/80#issuecomment-51696007.

fcavalieri commented 10 years ago

@wcandillon We need to add: wget http://launchpadlibrarian.net/160037457/bison_3.0.2.dfsg-2_amd64.deb wget http://launchpadlibrarian.net/160037458/libbison-dev_3.0.2.dfsg-2_amd64.deb sudo dpkg -i libbison-dev_3.0.2.dfsg-2_amd64.deb bison_3.0.2.dfsg-2_amd64.deb sudo apt-get install flex to the wercker-box.yml file

fcavalieri commented 10 years ago

@wcandillon I don't think I have access to it, though. Can you do it or give me write access?