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/09/22 07:45:22 UTC

[GitHub] [pulsar] tisonkun commented on a diff in pull request #17796: docs: Updating Python installation section

tisonkun commented on code in PR #17796:
URL: https://github.com/apache/pulsar/pull/17796#discussion_r977310599


##########
site2/docs/client-libraries-python.md:
##########
@@ -44,23 +36,10 @@ pip install 'pulsar-client[all]==@pulsar:version_number@'
 
 Installation via PyPi is available for the following Python versions:
 
-Platform | Supported Python versions
-:--------|:-------------------------
-MacOS >= 11.0 | 3.7, 3.8, 3.9 and 3.10
-Linux (including Alpine Linux) | 3.7, 3.8, 3.9 and 3.10
-
-
-### Install from source
-
-To install the `pulsar-client` library by building from source, follow [instructions](client-libraries-cpp.md#compilation) and compile the Pulsar C++ client library. That builds the Python binding for the library.
-
-To install the built Python bindings:
-
-```shell
-git clone https://github.com/apache/pulsar
-cd pulsar/pulsar-client-cpp/python
-sudo python setup.py install
-```

Review Comment:
   This is simply incorrect. We should update the doc in README.md under `pulsar-client-cpp` and add back this section to guide users there.



-- 
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