refactor(c)!: Updated include/install paths for adbc.h

.github/workflows/integration.yml

@@ -22,15 +22,13 @@ on:
     branches:
       - main
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "go/**"
       - "python/**"
       - ".github/workflows/integration.yml"
   push:
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "go/**"

.github/workflows/native-unix.yml

@@ -22,7 +22,6 @@ on:
     branches:
       - main
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "docs/**"
@@ -33,7 +32,6 @@ on:
       - ".github/workflows/native-unix.yml"
   push:
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "docs/**"

.github/workflows/native-windows.yml

@@ -22,7 +22,6 @@ on:
     branches:
       - main
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "glib/**"
@@ -32,7 +31,6 @@ on:
       - ".github/workflows/native-windows.yml"
   push:
     paths:
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "glib/**"

.github/workflows/packaging.yml

@@ -23,7 +23,6 @@ on:
       - main
     paths:
       - ".env"
-      - "adbc.h"
       - "c/**"
       - "ci/**"
       - "glib/**"

.github/workflows/r-basic.yml

@@ -23,7 +23,6 @@ on:
     branches:
       - main
     paths:
-      - "adbc.h"
       - "c/**"
       - "go/adbc/driver/**"
       - "go/adbc/pkg/**"

.github/workflows/r-standard.yml

@@ -31,7 +31,6 @@ on:
     branches:
       - main
     paths:
-      - "adbc.h"
       - "c/**"
       - "go/adbc/driver/**"
       - "go/adbc/pkg/**"

c/CMakeLists.txt

@@ -35,6 +35,9 @@ add_subdirectory(vendor/nanoarrow)
 add_subdirectory(driver/common)
 add_subdirectory(driver/framework)
 
+install(FILES "${REPOSITORY_ROOT}/c/include/arrow-adbc/adbc.h"
+        DESTINATION include/arrow-adbc)
+
 if(ADBC_BUILD_TESTS)
   add_subdirectory(validation)
 endif()
@@ -54,6 +57,8 @@ if(ADBC_DRIVER_FLIGHTSQL)
 endif()
 
 if(ADBC_DRIVER_MANAGER)
+  install(FILES "${REPOSITORY_ROOT}/c/include/arrow-adbc/adbc_driver_manager.h"
+          DESTINATION include/arrow-adbc)
   add_subdirectory(driver_manager)
 endif()
 

c/apidoc/Doxyfile

@@ -891,7 +891,7 @@ WARN_LOGFILE           =
 # spaces. See also FILE_PATTERNS and EXTENSION_MAPPING
 # Note: If this tag is empty the current directory is searched.
 
-INPUT                  = ../../adbc.h ../../README.md ../../c/driver_manager/adbc_driver_manager.h
+INPUT                  = ../../c/include/arrow-adbc/adbc.h ../../README.md ../../c/include/arrow-adbc/adbc_driver_manager.h
 
 # This tag can be used to specify the character encoding of the source files
 # that doxygen parses. Internally doxygen uses the UTF-8 encoding. Doxygen uses

c/driver/bigquery/CMakeLists.txt

@@ -61,8 +61,8 @@ if(ADBC_BUILD_TESTS)
                 ${TEST_LINK_LIBS})
   target_compile_features(adbc-driver-bigquery-test PRIVATE cxx_std_17)
   target_include_directories(adbc-driver-bigquery-test SYSTEM
-                             PRIVATE ${REPOSITORY_ROOT}
-                                     ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/
+                                     ${REPOSITORY_ROOT}/c/include/
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver
                                      ${REPOSITORY_ROOT}/c/driver/common)

c/driver/bigquery/bigquery_test.cc

@@ -15,7 +15,7 @@
 // specific language governing permissions and limitations
 // under the License.
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <gmock/gmock-matchers.h>
 #include <gtest/gtest-matchers.h>
 #include <gtest/gtest-param-test.h>

c/driver/common/CMakeLists.txt

@@ -18,7 +18,7 @@
 add_library(adbc_driver_common STATIC utils.c)
 adbc_configure_target(adbc_driver_common)
 set_target_properties(adbc_driver_common PROPERTIES POSITION_INDEPENDENT_CODE ON)
-target_include_directories(adbc_driver_common PRIVATE "${REPOSITORY_ROOT}"
+target_include_directories(adbc_driver_common PRIVATE "${REPOSITORY_ROOT}/c/include"
                                                       "${REPOSITORY_ROOT}/c/vendor")
 
 if(ADBC_BUILD_TESTS)
@@ -35,6 +35,7 @@ if(ADBC_BUILD_TESTS)
                 nanoarrow)
   target_compile_features(adbc-driver-common-test PRIVATE cxx_std_17)
   target_include_directories(adbc-driver-common-test
-                             PRIVATE "${REPOSITORY_ROOT}" "${REPOSITORY_ROOT}/c/vendor")
+                             PRIVATE "${REPOSITORY_ROOT}/c/include"
+                                     "${REPOSITORY_ROOT}/c/vendor")
   adbc_configure_target(adbc-driver-common-test)
 endif()

c/driver/common/driver_base.h

@@ -23,7 +23,7 @@
 #include <utility>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 
 // This file defines a developer-friendly way to create an ADBC driver, currently intended
 // for testing the R driver manager. It handles errors, option getting/setting, and

c/driver/common/driver_test.cc

@@ -18,7 +18,7 @@
 #include <gtest/gtest.h>
 #include <cstring>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include "driver_base.h"
 
 // Self-contained version of the Handle

c/driver/common/meson.build

@@ -18,7 +18,7 @@
 adbc_common_lib = library(
     'adbc_driver_common',
     sources: ['utils.c'],
-    include_directories: [root_dir],
+    include_directories: [include_dir],
     dependencies: [nanoarrow_dep],
     install: true,
 )
@@ -28,7 +28,7 @@ if get_option('tests')
         'adbc-driver-common-test',
         'utils_test.cc',
         'driver_test.cc',
-        include_directories: [root_dir],
+        include_directories: [include_dir],
         link_with: [adbc_common_lib],
         dependencies: [nanoarrow_dep, gtest_main_dep, gmock_dep],
     )

c/driver/common/utils.c

@@ -23,7 +23,7 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 
 static size_t kErrorBufferSize = 1024;
 

c/driver/common/utils.h

@@ -22,7 +22,7 @@
 #include <stdint.h>
 #include <string.h>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include "nanoarrow/nanoarrow.h"
 
 #ifdef __cplusplus

c/driver/flightsql/CMakeLists.txt

@@ -35,7 +35,8 @@ add_go_lib("${REPOSITORY_ROOT}/go/adbc/pkg/flightsql/"
 
 foreach(LIB_TARGET ${ADBC_LIBRARIES})
   target_include_directories(${LIB_TARGET} SYSTEM
-                             INTERFACE ${REPOSITORY_ROOT} ${REPOSITORY_ROOT}/c/
+                             INTERFACE ${REPOSITORY_ROOT}/c/
+                                       ${REPOSITORY_ROOT}/c/include/
                                        ${REPOSITORY_ROOT}/c/vendor
                                        ${REPOSITORY_ROOT}/c/driver)
 endforeach()
@@ -62,7 +63,7 @@ if(ADBC_BUILD_TESTS)
                 ${TEST_LINK_LIBS})
   target_compile_features(adbc-driver-flightsql-test PRIVATE cxx_std_17)
   target_include_directories(adbc-driver-flightsql-test SYSTEM
-                             PRIVATE ${REPOSITORY_ROOT} ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/ ${REPOSITORY_ROOT}/c/include/
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver)
   adbc_configure_target(adbc-driver-flightsql-test)

c/driver/flightsql/dremio_flightsql_test.cc

@@ -15,7 +15,7 @@
 // specific language governing permissions and limitations
 // under the License.
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <gmock/gmock-matchers.h>
 #include <gtest/gtest-matchers.h>
 #include <gtest/gtest-param-test.h>

c/driver/flightsql/meson.build

@@ -52,7 +52,7 @@ if get_option('tests')
         'adbc-driver-flightsql-test',
         'dremio_flightsql_test.cc',
         'sqlite_flightsql_test.cc',
-        include_directories: [root_dir, driver_dir],
+        include_directories: [include_dir, c_dir, driver_dir],
         link_with: [
             adbc_common_lib,
             adbc_driver_flightsql_lib

c/driver/flightsql/sqlite_flightsql_test.cc

@@ -20,7 +20,7 @@
 #include <random>
 #include <thread>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <gmock/gmock-matchers.h>
 #include <gtest/gtest-matchers.h>
 #include <gtest/gtest-param-test.h>

c/driver/framework/CMakeLists.txt

@@ -21,7 +21,7 @@ add_library(adbc_driver_framework STATIC base_driver.cc catalog.cc objects.cc)
 adbc_configure_target(adbc_driver_framework)
 set_target_properties(adbc_driver_framework PROPERTIES POSITION_INDEPENDENT_CODE ON)
 target_include_directories(adbc_driver_framework
-                           PRIVATE "${REPOSITORY_ROOT}" "${REPOSITORY_ROOT}/c/"
+                           PRIVATE "${REPOSITORY_ROOT}/c/" "${REPOSITORY_ROOT}/c/include"
                                    "${REPOSITORY_ROOT}/c/vendor")
 target_link_libraries(adbc_driver_framework PUBLIC adbc_driver_common fmt::fmt)
 

c/driver/framework/base_connection.h

@@ -24,7 +24,7 @@
 #include <utility>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <nanoarrow/nanoarrow.h>
 #include <nanoarrow/nanoarrow.hpp>
 

c/driver/framework/base_database.h

@@ -20,7 +20,7 @@
 #include <string_view>
 #include <utility>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 
 #include "driver/framework/base_driver.h"
 #include "driver/framework/status.h"

c/driver/framework/base_driver.h

@@ -28,7 +28,7 @@
 #include <variant>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <fmt/core.h>
 #include <fmt/format.h>
 

c/driver/framework/catalog.h

@@ -25,7 +25,7 @@
 #include <variant>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 
 #include "driver/framework/status.h"
 

c/driver/framework/meson.build

@@ -22,7 +22,7 @@ adbc_framework_lib = library(
         'catalog.cc',
         'objects.cc',
     ],
-    include_directories: [root_dir],
+    include_directories: [include_dir, c_dir],
     link_with: [adbc_common_lib],
     dependencies: [nanoarrow_dep, fmt_dep],
     install: true,

c/driver/framework/status.h

@@ -25,7 +25,7 @@
 #include <variant>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <fmt/core.h>
 
 /// \file status.h

c/driver/postgresql/CMakeLists.txt

@@ -57,8 +57,8 @@ add_arrow_lib(adbc_driver_postgresql
 foreach(LIB_TARGET ${ADBC_LIBRARIES})
   target_compile_definitions(${LIB_TARGET} PRIVATE ADBC_EXPORTING)
   target_include_directories(${LIB_TARGET} SYSTEM
-                             PRIVATE ${REPOSITORY_ROOT}
-                                     ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/
+                                     ${REPOSITORY_ROOT}/c/include/
                                      ${LIBPQ_INCLUDE_DIRS}
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver)
@@ -86,8 +86,8 @@ if(ADBC_BUILD_TESTS)
                 ${TEST_LINK_LIBS})
   target_compile_features(adbc-driver-postgresql-test PRIVATE cxx_std_17)
   target_include_directories(adbc-driver-postgresql-test SYSTEM
-                             PRIVATE ${REPOSITORY_ROOT}
-                                     ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/
+                                     ${REPOSITORY_ROOT}/c/include/
                                      ${LIBPQ_INCLUDE_DIRS}
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver)
@@ -108,8 +108,8 @@ if(ADBC_BUILD_TESTS)
                 ${TEST_LINK_LIBS})
   target_compile_features(adbc-driver-postgresql-copy-test PRIVATE cxx_std_17)
   target_include_directories(adbc-driver-postgresql-copy-test SYSTEM
-                             PRIVATE ${REPOSITORY_ROOT}
-                                     ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/
+                                     ${REPOSITORY_ROOT}/c/include/
                                      ${LIBPQ_INCLUDE_DIRS}
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver)
@@ -128,7 +128,7 @@ if(ADBC_BUILD_BENCHMARKS)
                 benchmark::benchmark)
   # add_benchmark replaces _ with - when creating target
   target_include_directories(postgresql-benchmark
-                             PRIVATE ${REPOSITORY_ROOT} ${REPOSITORY_ROOT}/c/
+                             PRIVATE ${REPOSITORY_ROOT}/c/ ${REPOSITORY_ROOT}/c/include/
                                      ${REPOSITORY_ROOT}/c/vendor
                                      ${REPOSITORY_ROOT}/c/driver)
 endif()

c/driver/postgresql/connection.cc

@@ -28,7 +28,7 @@
 #include <utility>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <libpq-fe.h>
 
 #include "database.h"

c/driver/postgresql/connection.h

@@ -20,7 +20,7 @@
 #include <cstdint>
 #include <memory>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <libpq-fe.h>
 
 #include "postgres_type.h"

c/driver/postgresql/database.cc

@@ -23,7 +23,7 @@
 #include <utility>
 #include <vector>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <libpq-fe.h>
 #include <nanoarrow/nanoarrow.h>
 

c/driver/postgresql/database.h

@@ -21,7 +21,7 @@
 #include <memory>
 #include <string>
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <libpq-fe.h>
 
 #include "postgres_type.h"

c/driver/postgresql/error.h

@@ -19,7 +19,7 @@
 
 #pragma once
 
-#include <adbc.h>
+#include <arrow-adbc/adbc.h>
 #include <libpq-fe.h>
 
 namespace adbcpq {