Skip to content
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

compile error: google has not been declared #3

Closed
liudf0716 opened this issue Sep 15, 2017 · 8 comments
Closed

compile error: google has not been declared #3

liudf0716 opened this issue Sep 15, 2017 · 8 comments
Labels
enhancement improvements on existing features

Comments

@liudf0716
Copy link

following this article:
https://github.com/brpc/brpc/blob/master/docs/cn/getting_started.md
I encounter the following compiling error:

image

@chenzhangyi
Copy link
Member

chenzhangyi commented Sep 15, 2017

It seems that gflags has modified its namespace from google to gflags (see gflags/gflags#145).
We will resolve this in the future, right now you'd better use an old version of gflags(before 2.2) to compile brpc.
Sorry for the inconvenience.

@jamesge
Copy link
Contributor

jamesge commented Sep 19, 2017

This issue should be solved at HEAD, need to be confirmed by @liudf0716 config_brpc.sh must be re-run before re-compilation.

@liudf0716
Copy link
Author

in my centos7 platform

git pull
sh config_brpc.sh --headers=/usr/include --libs=/usr/lib64
Fail to find google/profiler.h from --headers

it's my recent report.

@jamesge
Copy link
Contributor

jamesge commented Sep 20, 2017

gperftools changes its dir and namespace from google to gperftools in newer versions, try to manually change google to gperftools in source code temporarily. We'll verify this problem.

@jamesge
Copy link
Contributor

jamesge commented Sep 20, 2017

@liudf0716 check again

@liudf0716
Copy link
Author

@jamesge It's ok now. :+1:

@liudf0716
Copy link
Author

I have a suggestion:
how about support cmake way to compile brpc?

@jamesge
Copy link
Contributor

jamesge commented Sep 20, 2017

will support

@jamesge jamesge closed this as completed Sep 20, 2017
@jamesge jamesge added the enhancement improvements on existing features label Sep 28, 2017
@xiedeacc xiedeacc mentioned this issue Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improvements on existing features
Projects
None yet
Development

No branches or pull requests

3 participants