You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2016/02/17 00:35:51 UTC

[kudu-CR] Fix CSD and C++11 doc

Todd Lipcon has posted comments on this change.

Change subject: Fix CSD and C++11 doc
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2182/1/docs/installation.adoc
File docs/installation.adoc:

Line 245: the devtoolset will result in `Host compiler appears to require libatomic, but cannot find it.`
I think this is fine for this 0.7.0 release, but would be better, I think, to just insert a quick check in the top of the build script that verifies a sufficiently new compiler is being used and prints a nice error message.


-- 
To view, visit http://gerrit.cloudera.org:8080/2182
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia29cc1dc2f5180ca5c32843aa6c9fedb13ee8b56
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones <mi...@apache.org>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes