Skip to content

Conversation

@BurningEnlightenment
Copy link
Contributor

DO NOT MERGE YET

So as implied by the headline, I'm currently working toward a green "build|passed" travis sticker.

I updated the travis configuration file, so that Ubuntu 14.04 LTS and GCC 4.8 will be used which in turn means that the C++0x (or better C++11) build no longer fails because of Boost.Thread.
Furthermore I fixed the selfpipe example (#52) and the fourth tutorial (#53).

Unfortunately the Travis GCC 4.8 C++11 build of the latter fails (see the build log). I located the problem in application/auto_handler.hpp where the base_selector of the auto_handler class seems to incorrectly pick handler_auto_set_u instead of handler_auto_set_c. Which of course can't find a standard constructor for myapp. Please note that MSVC 19 and GCC 4.8 with -std=c++03 didn't reproduce the misbehaviour... Any thoughts?

See https://svn.boost.org/trac/boost/wiki/TravisCoverals .
Switch to travis trusty distribution in order to get gcc 4.8 an passable
C++11 support.
Add missing <fstream> include.
@pmcenery
Copy link

Unfortunately the Travis GCC 4.8 C++11 build of the latter fails (see the build log). I located the problem in application/auto_handler.hpp where the base_selector of the auto_handler class seems to incorrectly pick handler_auto_set_u instead of handler_auto_set_c. Which of course can't find a standard constructor for myapp. Please note that MSVC 19 and GCC 4.8 with -std=c++03 didn't reproduce the misbehaviour... Any thoughts?

Is there any fix available that will allow auto_handler to work with GCC >= 5 with -std=c++11?

Many thanks,
Paul.

@BurningEnlightenment
Copy link
Contributor Author

BurningEnlightenment commented Oct 19, 2017

@pmcenery sadly I didn't continue the investigation back than and therefore I have no fix available. However, you could try replacing boost::conditional and boost::is_convertible with their respective standard library counterparts and see if that works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants