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

Require C++11 or later #77

Merged
merged 1 commit into from Apr 9, 2018
Merged

Conversation

mloskot
Copy link
Member

@mloskot mloskot commented Apr 9, 2018

Add necessary flags to Boost.Build and CMake configurations as well as CI builds.

Description

Following the agreement about C++11 requirement in develop branch, Boost.GIL now officially requires compiler with support for C++11 or later.

Tasklist

  • Review
  • Adjust for comments
  • All CI builds and checks have passed

Add necessary flags to Boost.Build and CMake configurations
as well as CI builds.
@mloskot mloskot added the cat/enhancement Improvements, but not fixes addressing identified bugs label Apr 9, 2018
@mloskot mloskot added this to the The Grand Merge milestone Apr 9, 2018
@@ -9,13 +9,14 @@
project boost-gil
:
requirements
<toolset>intel:<debug-symbols>off
Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to remove this instruction ? Seems unrelated...

Copy link
Member Author

Choose a reason for hiding this comment

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

No, not at all.

I shuffled the lines moving <toolset>msvc: to the top, so the # MSVC comments is in the beginning

Copy link
Member

Choose a reason for hiding this comment

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

Oh, I see. I had missed that before. Thanks.

Copy link
Member Author

Choose a reason for hiding this comment

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

No problem. Thx for extra pair of eyes

@mloskot mloskot merged commit 4bc495e into boostorg:develop Apr 9, 2018
@mloskot mloskot deleted the ml/require-cpp11 branch April 9, 2018 20:17
mloskot added a commit that referenced this pull request Apr 10, 2018
Remove #if _MSC_VER > 1310 as always true since switch to C++11

Follows #77
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/enhancement Improvements, but not fixes addressing identified bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants