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
build: kill warnings #7397
build: kill warnings #7397
Conversation
@badone i am fixing the warning we discussed just now here. |
needs rebase |
Signed-off-by: Kefu Chai <kchai@redhat.com>
the CollectionHandle variant methods are hidden by the subclasses who do not override it. so the compiler complains. by adding "using" statement we expose them again. Signed-off-by: Kefu Chai <kchai@redhat.com>
fixes the `"HAVE_EXECINFO_H" redefined` warning Signed-off-by: Kefu Chai <kchai@redhat.com>
ef47a58
to
4804261
Compare
Signed-off-by: Kefu Chai <kchai@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
4804261
to
ef94d68
Compare
use ${MAKE} so the make command can be properly propagated to their children Signed-off-by: Kefu Chai <kchai@redhat.com>
ef94d68
to
e7bdf1c
Compare
with this change and ceph/civetweb#8, we will be closer to a green build in master. |
@liewegas rebased and added more fixes. |
* and mark the WorkQueue::_process(T *, TPHandle&) a pure virtual method. so we must override it * and mark all override methods with the "override" specifier. Signed-off-by: Kefu Chai <kchai@redhat.com>
Signed-off-by: Kefu Chai <kchai@redhat.com>
b9ec3c3
to
e9872ad
Compare
@cbodley could you help review the cmake bits in this pr? as @yehudasa gave his blessing to the civetweb change at ceph/civetweb#8, so i assume the submodule change is good. |
i also pushed the branch to gitbuilder, see http://gitbuilder.sepia.ceph.com/gitbuilder-ceph-tarball-trusty-amd64-cmake/#origin/wip-kill-warnings . it's green now. |
build: kill warnings Reviewed-by: Sage Weil <sage@redhat.com>
No description provided.