jorisvandenbossche commented on code in PR #14350: URL: https://github.com/apache/arrow/pull/14350#discussion_r998112416
########## docs/source/developers/python.rst: ########## @@ -518,13 +518,11 @@ We set a number of environment variables: - the path of the installation directory of the Arrow C++ libraries as ``ARROW_HOME`` -- add the path of installed DLL libraries to ``PATH`` - and the CMake generator to be used as ``PYARROW_CMAKE_GENERATOR`` .. code-block:: - $ set ARROW_HOME=%cd%\arrow-dist - $ set PATH=%ARROW_HOME%\bin;%PATH% + $ set ARROW_HOME=%CONDA_PREFIX% Review Comment: But given that our appveyor setup actually uses `%CONDA_PREFIX%\Library`, let's keep this consistent with that? Also in windows.rst for cpp, `%CONDA_PREFIX%\Library` is mentioned. According to https://docs.conda.io/projects/conda-build/en/stable/user-guide/environment-variables.html#environment-variables-set-during-the-build-process, conda uses a `Library` subdirectory in the conda env. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
