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

dmclock: include missing <functional> header. #14923

Merged
merged 1 commit into from May 4, 2017

Conversation

Projects
None yet
2 participants
@joscollin
Member

joscollin commented May 3, 2017

The following error appears during make:

ceph/src/dmclock/test/test_test_client.cc:14:0:
ceph/src/dmclock/test/../sim/src/sim_recs.h:40:12: error: ‘std::function’ has not been declared
       std::function<void()> code) {

Signed-off-by: Jos Collin jcollin@redhat.com

dmclock: ‘std::function’ has not been declared
The following error appears during make:

ceph/src/dmclock/test/test_test_client.cc:14:0:
ceph/src/dmclock/test/../sim/src/sim_recs.h:40:12: error: ‘std::function’ has not been declared
       std::function<void()> code) {

Signed-off-by: Jos Collin <jcollin@redhat.com>
@joscollin

This comment has been minimized.

Member

joscollin commented May 4, 2017

Retest this please.

@tchaikov tchaikov changed the title from dmclock: ‘std::function’ has not been declared to dmclock: include missing <functional> header. May 4, 2017

@tchaikov tchaikov merged commit 3a659b8 into ceph:master May 4, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
@joscollin

This comment has been minimized.

Member

joscollin commented May 6, 2017

@tchaikov This is done as per the Header Dependency Changes in gcc 7. Please see: https://gcc.gnu.org/gcc-7/porting_to.html

@joscollin joscollin deleted the joscollin:wip-dmclocktest-function-error branch May 6, 2017

@joscollin joscollin added the cleanup label May 9, 2017

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