jorisvandenbossche commented on a change in pull request #11778:
URL: https://github.com/apache/arrow/pull/11778#discussion_r759961208



##########
File path: docs/source/developers/cpp/building.rst
##########
@@ -145,45 +238,34 @@ Minimal release build (1GB of RAM for building or more 
recommended):
 
 .. code-block:: shell
 
-   git clone https://github.com/apache/arrow.git
-   cd arrow/cpp
-   mkdir release
-   cd release
-   cmake ..
-   make
+   $ mkdir build-release
+   $ cd build-release
+   $ cmake ..
+   $ make -j8       # if you have 8 CPU cores, otherwise adjust
 
 Minimal debug build with unit tests (4GB of RAM for building or more 
recommended):
 
 .. code-block:: shell
 
-   git clone https://github.com/apache/arrow.git
-   cd arrow
-   git submodule update --init --recursive
-   export ARROW_TEST_DATA=$PWD/testing/data
-   cd cpp
-   mkdir debug
-   cd debug
-   cmake -DCMAKE_BUILD_TYPE=Debug -DARROW_BUILD_TESTS=ON ..
-   make unittest
+   $ git submodule update --init --recursive

Review comment:
       In https://github.com/pydata/pydata-sphinx-theme/issues/41 I listed some 
options around this. It seems sphinx now does this out of the box, but only for 
>>> python prompts. https://github.com/sbrunner/sphinx-prompt might be the 
closest thing.
   
   (but to be clear I don't mean this as a suggestion that needs to be done in 
this PR)




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