You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by li...@apache.org on 2022/08/01 13:29:54 UTC

[arrow-adbc] branch main updated: [Docs] Update CONTRIBUTING.md (#49)

This is an automated email from the ASF dual-hosted git repository.

lidavidm pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-adbc.git


The following commit(s) were added to refs/heads/main by this push:
     new cfe1752  [Docs] Update CONTRIBUTING.md (#49)
cfe1752 is described below

commit cfe175222f767d3b4ec3c3223a4e93562d234fe2
Author: David Li <li...@gmail.com>
AuthorDate: Mon Aug 1 09:29:49 2022 -0400

    [Docs] Update CONTRIBUTING.md (#49)
---
 CONTRIBUTING.md | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index f2479d0..0574e26 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -30,6 +30,11 @@ https://github.com/apache/arrow-adbc/issues
 
 The libraries here are all **individual** CMake projects.
 
+_Note:_ unlike the Arrow C++ build system, the CMake projects will
+**not** automatically download and build dependencies—you should
+configure CMake appropriately to find dependencies in system or
+package manager locations.
+
 For example, the driver manager is built as follows:
 
 ```shell
@@ -39,7 +44,11 @@ $ cmake ../../c/driver_manager
 $ make -j
 ```
 
-The SQLite3 and Apache Arrow Flight SQL drivers can be built similarly.
+The SQLite3 and Apache Arrow Flight SQL drivers can be built
+similarly.  Both drivers require an installation of the Arrow C++
+libraries (in the case of the Flight SQL driver, with Flight SQL
+enabled), and of course, the SQLite3 driver requires an installation
+of SQLite.
 
 To find dependencies, use CMake options such as `CMAKE_PREFIX_PATH`.
 A list of dependencies for Conda (conda-forge) is included, and can be
@@ -63,6 +72,7 @@ For example, to build and run tests for the SQLite3 driver:
 ```shell
 $ mkdir -p build/sqlite
 $ cd build/sqlite
+# You may need to set -DCMAKE_PREFIX_PATH such that googletest can be found
 $ cmake ../../c/drivers/sqlite -DADBC_BUILD_TESTS=ON
 $ make -j
 $ ctest
@@ -70,6 +80,11 @@ $ ctest
 
 ### Python
 
+The Python libraries require Cython during build time; they have no
+other build or runtime dependencies.  (The C++ sources for the driver
+manager are inlined into the Cython compilation process—so there is no
+need to separately build the driver manager.)
+
 ```shell
 $ cd python/adbc_driver_manager
 $ pip install -r requirements-dev.txt