You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2016/11/14 18:47:20 UTC

[kudu-CR] macOS: fix shared ptr.h includes

Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/5037

to look at the new patch set (#2).

Change subject: macOS: fix shared_ptr.h includes
......................................................................

macOS: fix shared_ptr.h includes

_LIBCPP_VERSION was not being defined appropriately in
client_samples-test due to header include order.

Change-Id: Iff56316ea55244235f3a44df8e0c3f53da1b35c6
---
M src/kudu/client/shared_ptr.h
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/37/5037/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5037
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iff56316ea55244235f3a44df8e0c3f53da1b35c6
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins