Boost C++ Libraries: Ticket #12349: no type named ‘types’ in ‘boost::spirit::qi::detail::find_substitute

anonymous — Tue, 26 Jul 2016 15:37:13 GMT

I confirm

anonymous — Mon, 01 Aug 2016 16:04:58 GMT

Same problem here.

anonymous — Mon, 08 Aug 2016 17:10:53 GMT

Hello,

the same issue affects FreeOrion in Debian, compiled with Boost 1.61 and GCC-6.

Build log:

https://buildd.debian.org/status/fetch.php?pkg=freeorion&arch=amd64&ver=0.4.5%2Bgit20160707-1%2Bb1&stamp=1470433494

Debian bug report:

https://bugs.debian.org/833773

This is apparently a regression because FreeOrion builds fine with Boost 1.58 and GCC-6.

component changed; owner set

viboes — Fri, 19 Aug 2016 21:18:43 GMT

owner set to Joel de Guzman
component None → spirit

anonymous — Mon, 28 Nov 2016 19:41:53 GMT

Similar issue in compiling rasdaman with Boost 1.61; 1.60 works fine.

anonymous — Wed, 19 Apr 2017 08:11:58 GMT

Similar issue in compiling rasdaman with Boost 1.61; 1.60 works fine.

Still same in Boost 1.62.

attachment set

juanpablo.canepa@… — Wed, 09 Aug 2017 20:40:24 GMT

attachment → spirit_test.cpp

Reduced gramar that fails to build with boost 1.63 / gcc 6.2

juanpablo.canepa@… — Wed, 09 Aug 2017 20:44:34 GMT

I managed to reduce the problem to a very simple grammar, that I've attached to this ticket. The problem seems to be that boost.spirit, for some reason, assumes there is a nested field called "types" in boost::optional, that was removed after 1.61, that was defined:

typedef BOOST_DEDUCED_TYPENAME mpl::if_<is_reference_predicate,types_when_ref,types_when_not_ref>::type types ;

This is blocking a release of one of our tools, so if there was any workaround that we can patch locally, I'll take it for the time being.

severity changed

anonymous — Fri, 18 Aug 2017 18:15:37 GMT

severity Problem → Showstopper

Joel de Guzman — Wed, 30 Aug 2017 23:48:07 GMT

Confirmed on OSX Clang. Let's see what we can do...

Joel de Guzman — Wed, 30 Aug 2017 23:57:46 GMT

Alright, I have a fix. Will commit to devel as soon as I can.

anonymous — Thu, 31 Aug 2017 01:17:43 GMT

OK, seems this is is already fixed in the latest devel which should already make it to the latest Boost release. See line 141 alternative_function.hpp

fix for alternative.cpp test case, FHE 2016-07-28

juanpablo.canepa@… — Sat, 02 Sep 2017 02:35:43 GMT

Joel,

Thanks for the reply. I did try that patch that you mention while browsing the github repo, but my sample test failed to compile with another error (I'll provide the info ass soon as possible). Maybe it is because I just patched that single file (alternative_function.hpp) but there are more changes that it builds upon(*).

I'll re-try the patch on a clean build and report back.

Thanks for paying attention to the ticket!

Cheers, Juan Pablo

(*) There are corporate reasons shy I can't just move on to the latest version of boost, so I'm trying to figure out the minimum subset of changes needed to patch our current 1.63 build.

Tue, 05 Dec 2017 19:37:29 GMT

Fixed in https://github.com/boostorg/spirit/pull/271

status changed; resolution set

Joel de Guzman — Wed, 13 Dec 2017 23:04:47 GMT

status new → closed
resolution → fixed