AlenkaF commented on a change in pull request #11778: URL: https://github.com/apache/arrow/pull/11778#discussion_r760930052
########## 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: I think using something like `sphinx-prompt` + having an icon to copy the content of the code box would be great! Something like [sphinx-copybutton](https://sphinx-copybutton.readthedocs.io/en/latest/). Could be a beginner ticket for the docs to add/change this in developers section =) How many ppl would be confused? Well, not much I would say. But if we could make life easier even for one that is not yet used to the terminal and needs to build Arrow C++ that could be worth it? -- 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]
