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

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


Patch Set 1:

(1 comment)

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

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
The compilation errors were related to boost.

It seems I'm not good at explaining things clearly, my bad :)

Try the things below, and I'll try to update the message meanwhile.

cd /tmp
mkdir -p a
mkdir -p b

cat << EOF > x.cc
#include <iostream>

#include <inc.h>

using namespace std;

int main() {
  cout << sizeof(A) << endl;
  return 0;
}
EOF


cat << EOF > a/inc.h
struct A {
  int f0;
};
EOF

cat << EOF > b/inc.h
struct A {
  int f0;
  long f1;
  double f2;
};
EOF

c++ -isystem ./a -I ./b x.cc  && ./a.out

c++ -isystem ./a -isystem ./b x.cc && ./a.out

c++ -isystem ./b -I ./a x.cc && ./a.out



--
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: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 23 Aug 2018 23:22:10 +0000
Gerrit-HasComments: Yes

Reply via email to