Skip to content

Commit

Permalink
GH-37406: [C++][FlightSQL] Add missing ArrowFlight::arrow_flight_{sha…
Browse files Browse the repository at this point in the history
…red,static} dependencies (#37407)

### Rationale for this change

`ArrowFlightSql::arrow_flight_sql_shared` must depend on `ArrowFlight::arrow_flight_shared` because `arrow/flight/sql/*.h` uses ` arrow/flight/*.h` and all users must use Flight RPC API with Flight SQL API.

### What changes are included in this PR?

Add missing `{SHARED,STATIC}_INSTALL_INTERFACE_LIBS`.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

Yes.
* Closes: #37406

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
  • Loading branch information
kou committed Aug 28, 2023
1 parent f27dc02 commit 78b0c10
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions cpp/src/arrow/flight/sql/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,12 @@ add_arrow_lib(arrow_flight_sql
${ARROW_VERSION_SCRIPT_FLAGS} # Defined in cpp/arrow/CMakeLists.txt
SHARED_LINK_LIBS
arrow_flight_shared
SHARED_INSTALL_INTERFACE_LIBS
ArrowFlight::arrow_flight_shared
STATIC_LINK_LIBS
arrow_flight_static
STATIC_INSTALL_INTERFACE_LIBS
ArrowFlight::arrow_flight_static
PRIVATE_INCLUDES
"${Protobuf_INCLUDE_DIRS}")

Expand Down

0 comments on commit 78b0c10

Please sign in to comment.