Skip to content
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

KAA-1510: Fix instruction for Ubuntu 16 #1273

Merged
merged 1 commit into from
Nov 17, 2016
Merged

KAA-1510: Fix instruction for Ubuntu 16 #1273

merged 1 commit into from
Nov 17, 2016

Conversation

Dubland
Copy link
Contributor

@Dubland Dubland commented Nov 15, 2016

/cc @vadimol

Copy link
Contributor

@vadimol vadimol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vadimol
Copy link
Contributor

vadimol commented Nov 16, 2016

@Dubland please, open PR on master as well

@@ -54,7 +54,7 @@ wget https://sourceforge.net/projects/boost/files/boost/1.55.0/boost_1_55_0.tar.
tar -xf boost_1_55_0.tar.gz
cd boost_1_55_0
./bootstrap.sh
sudo ./bjam install
sudo ./bjam cxxflags=-std=c++11 install
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just wondering why this is needed?

Copy link
Contributor

@GamovCoder GamovCoder Nov 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The option is needed to enable 11th standard support for Ubuntu 16.04 default g++ compiler. It is not enabled by default and boost configuration system for some reasons does not pass the necessary flag to a compiler.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why it was not necessary in Ubuntu 14.04?

@rasendubi rasendubi merged commit c789944 into kaaproject:release-0.10 Nov 17, 2016
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.

4 participants