kou commented on code in PR #36296:
URL: https://github.com/apache/arrow/pull/36296#discussion_r1252449624


##########
docs/source/developers/java/building.rst:
##########
@@ -195,7 +195,7 @@ CMake
 
 - To build all JNI libraries (MacOS / Linux) except the JNI C Data Interface 
library:
 
-  .. code-block::
+  .. code-block:: text

Review Comment:
   ```suggestion
     .. code-block:: console
   ```



##########
docs/Makefile:
##########
@@ -150,6 +150,8 @@ cpp-live:
 
 .PHONY: python
 python:
+       # Remove the following line when sphinx issue 
(https://github.com/sphinx-doc/sphinx/issues/10943) is closed
+       python -m pip install ../python --quiet --no-dependencies

Review Comment:
   Can we move this to caller of `make python`?
   If we put this here, this command is always executed for `make python`.
   (I think that we only need to execute this once.)



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to