jorisvandenbossche commented on a change in pull request #11505:
URL: https://github.com/apache/arrow/pull/11505#discussion_r745505038
##########
File path: docs/source/developers/python.rst
##########
@@ -335,6 +335,11 @@ virtualenv) enables cmake to choose the python executable
which you are using.
For any other C++ build challenges, see :ref:`cpp-development`.
+In case you may need to rebuild the C++ part due to errors in the process it is
+advisable to delete the content of the build folder with command ``rm -rf *``
+from the build folder itself. If the build has passed successfully and you
need to rebuild
Review comment:
> I would generally delete the build folder itself, rather than delete
its contents.
Yes, that was also the reason I mentioned to mention `rm -rf /cpp/build`
instead of `rm -rf *` from inside that directory.
I agree that we can recommend the `git clean` way instead of `python
setup.py clean` (your other inline comment below). But for explicitly deleting
the C++ build directory (which is the topic here), wouldn't `rm -rf /cpp/build`
be more explicit (also for understanding what is actually done / needed in this
case)?
--
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]
