Skip to content

Commit

Permalink
instrumentation/nginx: update documentation to drop deprecated options (
Browse files Browse the repository at this point in the history
#315)

opentelemetry-cpp has deprecated WITH_OTLP cmake option.
It suggests to use WITH_OTLP_GRPC/WITH_OTLP_HTTP instead.

Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
  • Loading branch information
mfrw authored Apr 13, 2024
1 parent 7230417 commit 9d64d9d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions instrumentation/nginx/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ Additional platforms and/or versions coming soon.

## Dependencies (for building)

1. [gRPC](https://github.com/grpc/grpc) - currently the only supported exporter is OTLP. This requirement will be lifted
1. [gRPC](https://github.com/grpc/grpc) - currently the only supported exporter is OTLP_GRPC. This requirement will be lifted
once more exporters become available.
2. [opentelemetry-cpp](https://github.com/open-telemetry/opentelemetry-cpp) - opentelemetry-cpp needs to be built with
position independent code and OTLP support, e.g.:
position independent code and OTLP_GRPC support, e.g.:
```
cmake -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DWITH_OTLP=ON ..
cmake -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DWITH_OTLP_GRPC=ON ..
```

## Building
Expand Down

0 comments on commit 9d64d9d

Please sign in to comment.