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

Optionally install the header files for API, and generated targets and header files for SDK at custom/default install location. #1414

Closed
zx8086 opened this issue May 21, 2022 · 5 comments

Comments

@zx8086
Copy link

zx8086 commented May 21, 2022

custom/default install location?

For what ? the build, the system ?

example ?

@owent
Copy link
Member

owent commented May 22, 2022

If you ares using cmake, you can set CMAKE_INSTALL_PREFIX to specify where to install all headers and library files.
If you are using bazel, there is no need to install or using cmake --install <build dir> --prefix <where to install>, because all headers are auto copied by deps

@zx8086
Copy link
Author

zx8086 commented May 22, 2022

I'm trying to instrument nginx via https://github.com/open-telemetry/opentelemetry-cpp-contrib/tree/main/instrumentation/nginx ...

Not sure what the locations should be for this given the errors are relating to missing headers.

@owent
Copy link
Member

owent commented May 22, 2022

cmake --install . --prefix=<installation path> is better way. I have replied to open-telemetry/opentelemetry-cpp-contrib#146

@lalitb
Copy link
Member

lalitb commented Jun 1, 2022

@zx8086 - Should be good to close this issue ?

@zx8086
Copy link
Author

zx8086 commented Jun 10, 2022

Good to close thanks @lalitb

@zx8086 zx8086 closed this as completed Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants