You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/06/08 12:30:16 UTC

[GitHub] [pulsar] lgxbslgx opened a new pull request, #15980: [fix][doc] Adjust the install command of the gtest and gmock in document

lgxbslgx opened a new pull request, #15980:
URL: https://github.com/apache/pulsar/pull/15980

   
   Fixes #15935 
   
   ### Motivation
   
   Currently, the command `sudo cp *.a /usr/lib` in the file `pulsar-client-cpp/README.md` sometimes failed. Because when building gtest and gmock, the libs `*.a` may not always be placed at the source code root path `/usr/src/gtest` and `/usr/src/gmock`. The path depends on the version of the gtest and gmock and depends on the configuration when building. So when the developers use the command `sudo cp *.a /usr/lib` directly, it may fail.
   
   ### Modifications
   
   Adjust the command and explain it in the document `pulsar-client-cpp/README.md`.
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   ### Documentation
   
   - [x] `doc` 
   (Your PR contains doc changes)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Anonymitaet merged pull request #15980: [fix][doc] Adjust the install command of the gtest and gmock in document

Posted by GitBox <gi...@apache.org>.
Anonymitaet merged PR #15980:
URL: https://github.com/apache/pulsar/pull/15980


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] lgxbslgx commented on pull request #15980: [fix][doc] Adjust the install command of the gtest and gmock in document

Posted by GitBox <gi...@apache.org>.
lgxbslgx commented on PR #15980:
URL: https://github.com/apache/pulsar/pull/15980#issuecomment-1151857545

   @merlimat Thanks for the review.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org