Comments and names changes

apache · Jul 1, 2022 · 997a9b9 · 997a9b9
1 parent bb0a96b
commit 997a9b9
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/cpp/cmake_modules/FindArrowPython.cmake b/cpp/cmake_modules/FindArrowPython.cmake
@@ -15,7 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-# - Find Arrow Python (python/pyarrrow/src_arrow/api.h, libarrow_python.a, libarrow_python.so)
+# - Find Arrow Python (arrow/python/api.h, libarrow_python.a, libarrow_python.so)
 #
 # This module requires Arrow from which it uses
 #  arrow_find_package()

diff --git a/cpp/src/arrow/public_api_test.cc b/cpp/src/arrow/public_api_test.cc
@@ -50,10 +50,6 @@
 #include "arrow/json/api.h"  // IWYU pragma: keep
 #endif
 
-// #ifdef ARROW_PYTHON
-// #include "arrow/python/api.h"  // IWYU pragma: keep
-// #endif
-
 #ifdef DCHECK
 #error "DCHECK should not be visible from Arrow public headers."
 #endif

diff --git a/python/pyarrow/src_arrow/CMakeLists.txt b/python/pyarrow/src_arrow/CMakeLists.txt
@@ -33,10 +33,10 @@ endif()
 # CMAKE_MODULE_PATH: location of cmake_modules in python
 #
 
-get_filename_component(PYARROW_SOURCE_DIR ${CMAKE_SOURCE_DIR} DIRECTORY)
-get_filename_component(PYTHON_SOURCE_DIR ${PYARROW_SOURCE_DIR} DIRECTORY)
-get_filename_component(ARROW_SOURCE ${PYTHON_SOURCE_DIR} DIRECTORY)
-set(ARROW_SOURCE_DIR "${ARROW_SOURCE}/cpp")
+get_filename_component(ARROW_PYTHON_SOURCE_DIR ${CMAKE_SOURCE_DIR} DIRECTORY)
+get_filename_component(PYARROW_SOURCE_DIR ${ARROW_PYTHON_SOURCE_DIR} DIRECTORY)
+get_filename_component(ARROW_SOURCE_DIR ${PYARROW_SOURCE_DIR} DIRECTORY)
+set(ARROW_CPP_SOURCE_DIR "${ARROW_SOURCE_DIR}/cpp")
 
 # normalize ARROW_HOME path
 cmake_path(CONVERT "$ENV{ARROW_HOME}" TO_CMAKE_PATH_LIST ARROW_HOME)