You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by kw...@apache.org on 2022/10/09 13:40:55 UTC
[pulsar-client-cpp] branch main updated: Fix README.md build path (#29)
This is an automated email from the ASF dual-hosted git repository.
kwang pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pulsar-client-cpp.git
The following commit(s) were added to refs/heads/main by this push:
new 94500a8 Fix README.md build path (#29)
94500a8 is described below
commit 94500a844784ea400f4f5d3ba17a62ad13507cc1
Author: Kai Wang <kw...@apache.org>
AuthorDate: Sun Oct 9 21:40:51 2022 +0800
Fix README.md build path (#29)
### Motivation
For the new repo, we don't need cd to `${PULSAR_PATH}` when building, so we can remove the `${PULSAR_PATH}`.
### Modifications
Remove `${PULSAR_PATH}`.
---
README.md | 36 +++++++++++++++++-------------------
1 file changed, 17 insertions(+), 19 deletions(-)
diff --git a/README.md b/README.md
index 1d15b79..e6045ad 100644
--- a/README.md
+++ b/README.md
@@ -126,7 +126,7 @@ sudo cp ${GMOCK_LIB_PATH}/*.a /usr/lib
#### Compile Pulsar client library:
```shell
-cd pulsar/pulsar-client-cpp
+cd pulsar-client-cpp
cmake .
make
```
@@ -164,8 +164,7 @@ brew install googletest
#### Compile Pulsar client library:
```shell
-export PULSAR_PATH=<Path where you cloned pulsar repo>
-cd ${PULSAR_PATH}/pulsar-client-cpp/
+cd pulsar-client-cpp/
cmake .
make
```
@@ -173,15 +172,15 @@ make
#### Checks
##### Client library will be placed in
```
-${PULSAR_PATH}/pulsar-client-cpp/lib/libpulsar.dylib
-${PULSAR_PATH}/pulsar-client-cpp/lib/libpulsar.a
+lib/libpulsar.dylib
+lib/libpulsar.a
```
##### Tools will be placed in:
```
-${PULSAR_PATH}/pulsar-client-cpp/perf/perfProducer
-${PULSAR_PATH}/pulsar-client-cpp/perf/perfConsumer
+perf/perfProducer
+perf/perfConsumer
```
### Compile on Windows
@@ -228,10 +227,10 @@ If you installed the dependencies manually, you need to run
```shell
#If all dependencies are in your path, all that is necessary is
-${PULSAR_PATH}/pulsar-client-cpp/cmake .
+pulsar-client-cpp/cmake .
#if all dependencies are not in your path, then passing in a PROTOC_PATH and CMAKE_PREFIX_PATH is necessary
-${PULSAR_PATH}/pulsar-client-cpp/cmake -DPROTOC_PATH=C:/protobuf/bin/protoc -DCMAKE_PREFIX_PATH="C:/boost;C:/openssl;C:/zlib;C:/curl;C:/protobuf;C:/googletest;C:/dlfcn-win32" .
+pulsar-client-cpp/cmake -DPROTOC_PATH=C:/protobuf/bin/protoc -DCMAKE_PREFIX_PATH="C:/boost;C:/openssl;C:/zlib;C:/curl;C:/protobuf;C:/googletest;C:/dlfcn-win32" .
#This will generate pulsar-cpp.sln. Open this in Visual Studio and build the desired configurations.
```
@@ -240,37 +239,37 @@ ${PULSAR_PATH}/pulsar-client-cpp/cmake -DPROTOC_PATH=C:/protobuf/bin/protoc -DCM
##### Client libraries are available in the following places.
```
-${PULSAR_PATH}/pulsar-client-cpp/build/lib/Release/pulsar.lib
-${PULSAR_PATH}/pulsar-client-cpp/build/lib/Release/pulsar.dll
+pulsar-client-cpp/build/lib/Release/pulsar.lib
+pulsar-client-cpp/build/lib/Release/pulsar.dll
```
#### Examples
##### Add windows environment paths.
```
-${PULSAR_PATH}/pulsar-client-cpp/build/lib/Release
-${PULSAR_PATH}/pulsar-client-cpp/vcpkg_installed
+pulsar-client-cpp/build/lib/Release
+pulsar-client-cpp/vcpkg_installed
```
##### Examples are available in.
```
-${PULSAR_PATH}/pulsar-client-cpp/build/examples/Release
+pulsar-client-cpp/build/examples/Release
```
## Tests
```shell
# Source code
-${PULSAR_PATH}/pulsar-client-cpp/tests/
+pulsar-client-cpp/tests/
# Execution
# Start standalone broker
-${PULSAR_PATH}/pulsar-test-service-start.sh
+pulsar-test-service-start.sh
# Run the tests
-${PULSAR_PATH}/pulsar-client-cpp/tests/main
+pulsar-client-cpp/tests/main
# When no longer needed, stop standalone broker
-${PULSAR_PATH}/pulsar-test-service-stop.sh
+pulsar-test-service-stop.sh
```
## Requirements for Contributors
@@ -280,4 +279,3 @@ It's required to install [LLVM](https://llvm.org/builds/) for `clang-tidy` and `
Use `pulsar-client-cpp/docker-format.sh` to ensure the C++ sources are correctly formatted.
We welcome contributions from the open source community, kindly make sure your changes are backward compatible with GCC 4.8 and Boost 1.53.
-