Skip to content

Commit

Permalink
MINOR: [C++] Fixed bundled dependency comment in ThirdpartyToolchain.…
Browse files Browse the repository at this point in the history 
…cmake (apache#34990)

Updated comment to the correct filename - I was confused when I saw this comment in the code which had incorrectly named the output bundled dependency lib file.

Authored-by: Diana Sulmone <diana@sulmone.com>
Signed-off-by: Will Jones <willjones127@gmail.com>
  • Loading branch information
@sulmone
sulmone committed Apr 24, 2023
1 parent cd14e20 commit 5de5692
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions cpp/cmake_modules/ThirdpartyToolchain.cmake
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ add_custom_target(toolchain-benchmarks)
add_custom_target(toolchain-tests)

# Accumulate all bundled targets and we will splice them together later as
# libarrow_dependencies.a so that third party libraries have something usable
# to create statically-linked builds with some BUNDLED dependencies, including
# allocators like jemalloc and mimalloc
# libarrow_bundled_dependencies.a so that third party libraries have something
# usable to create statically-linked builds with some BUNDLED dependencies,
# including allocators like jemalloc and mimalloc
set(ARROW_BUNDLED_STATIC_LIBS)

# Accumulate all system dependencies to provide suitable static link
Expand Down

0 comments on commit 5de5692

Please sign in to comment.