jorisvandenbossche commented on a change in pull request #11778: URL: https://github.com/apache/arrow/pull/11778#discussion_r761008478
########## 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: Apparently, sphinx actually already supports having a `$` not copy-pastable out of the box (see eg https://pydata-sphinx-theme.readthedocs.io/en/latest/contributing.html#set-up-a-basic-development-environment), I _think_ it's a matter of specifying the correct highlighting language for the code blocks (`console` was used in the case of the linked example) -- 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]
