Skip to content

Commit

Permalink
apacheGH-35179: [C++] Fix IMPORTED_LOCATION property for Arrow::bundl…
Browse files Browse the repository at this point in the history
…ed_dependencies (apache#35196)

### What changes are included in this PR?

The previous change used the `LOCATION_<CONFIGURATION>` property which is not set by default.

### Are these changes tested?

Crossbow
* Closes: apache#35179

Authored-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
assignUser authored and rtpsw committed May 16, 2023
1 parent 5fd7d85 commit d215be6
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions cpp/src/arrow/ArrowConfig.cmake.in
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,9 @@ if(TARGET Arrow::arrow_static AND NOT TARGET Arrow::arrow_bundled_dependencies)
get_target_property(arrow_static_location Arrow::arrow_static
LOCATION_${CONFIGURATION})
get_filename_component(arrow_lib_dir "${arrow_static_location}" DIRECTORY)
set_property(TARGET Arrow::arrow_bundled_dependencies
APPEND
PROPERTY IMPORTED_CONFIGURATIONS ${CONFIGURATION})
set_target_properties(Arrow::arrow_bundled_dependencies
PROPERTIES IMPORTED_LOCATION_${CONFIGURATION}
"${arrow_lib_dir}/${CMAKE_STATIC_LIBRARY_PREFIX}arrow_bundled_dependencies${CMAKE_STATIC_LIBRARY_SUFFIX}"
Expand Down

0 comments on commit d215be6

Please sign in to comment.