Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11311 )

Change subject: [cmake] add OpenSSL include path into -isystem
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/11311/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11311/1//COMMIT_MSG@13
PS1, Line 13: the compiler.  I.e., the path specific with '-I' option is chosen 
over
specified


http://gerrit.cloudera.org:8080/#/c/11311/1//COMMIT_MSG@15
PS1, Line 15: in the command line.  That could lead to mysterious compilation 
errors.
So the compilation errors you saw were regarding boost? Or openssl? If the 
former, how does changing openssl's inclusion help?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I345bf5f72fb9c899340339452ba1464becd6ada3
Gerrit-Change-Number: 11311
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 23 Aug 2018 23:05:49 +0000
Gerrit-HasComments: Yes

Reply via email to