Skip to content
This repository was archived by the owner on Mar 22, 2023. It is now read-only.

[1.6] cmake: require c++11 for cow check#280

Merged
igchor merged 1 commit intopmem:stable-1.6from
kkajrewicz:cmake-fix
Apr 17, 2019
Merged

[1.6] cmake: require c++11 for cow check#280
igchor merged 1 commit intopmem:stable-1.6from
kkajrewicz:cmake-fix

Conversation

@kkajrewicz
Copy link
Copy Markdown
Contributor

@kkajrewicz kkajrewicz commented Apr 9, 2019

C++ version was incorrectly computed by cmake to c++98 which resulted in compilation error on ubuntu 16.04


This change is Reviewable

@pmem-bot
Copy link
Copy Markdown
Contributor

pmem-bot commented Apr 9, 2019

@igchor and @lplewa please review this pull request

@marcinslusarz
Copy link
Copy Markdown
Contributor

stable-1.6?

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2019

Codecov Report

Merging #280 into stable-1.6 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           stable-1.6    #280   +/-   ##
==========================================
  Coverage        96.5%   96.5%           
==========================================
  Files              30      30           
  Lines            2091    2091           
==========================================
  Hits             2018    2018           
  Misses             73      73
Flag Coverage Δ
#tests_clang_debug_cpp17 96.93% <ø> (ø) ⬆️
#tests_gcc_debug 97.88% <ø> (ø) ⬆️
#tests_gcc_release_cpp17_no_valgrind 96.63% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b73544f...d965fe2. Read the comment docs.

Copy link
Copy Markdown

@szyrom szyrom left a comment

Choose a reason for hiding this comment

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

  • stable 1.6

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@kkajrewicz kkajrewicz changed the base branch from master to stable-1.6 April 15, 2019 06:21
C++ version was incorrectly computed by cmake to c++98 which resulted in compilation error on ubuntu 16.04
@kkajrewicz kkajrewicz changed the title cmake: require c++11 for cow check [1.6] cmake: require c++11 for cow check Apr 15, 2019
Copy link
Copy Markdown
Contributor

@igchor igchor left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@igchor igchor merged commit 18bea23 into pmem:stable-1.6 Apr 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants