Skip to content

Commit

Permalink
fix: Set TEXTPLAN_SOURCE_DIR based on current dir in planloader test. (
Browse files Browse the repository at this point in the history
…#101)

That variable was previously computed based on `CMAKE_SOURCE_DIR`, which
is the wrong value if `substrait-cpp` is used as a dependency from a
different top-level CMake project. The PR applies the pattern based on
`CMAKE_CURRENT_SOURCE_DIR` used in other tests.
  • Loading branch information
ingomueller-net authored Mar 15, 2024
1 parent fd555a6 commit 579d884
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion export/planloader/tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,8 @@ add_test_case(
gtest
gtest_main)

set(TEXTPLAN_SOURCE_DIR "${CMAKE_SOURCE_DIR}/src/substrait/textplan")
set(TEXTPLAN_SOURCE_DIR
"${CMAKE_CURRENT_SOURCE_DIR}/../../../src/substrait/textplan")

add_custom_command(
TARGET planloader_test
Expand Down

0 comments on commit 579d884

Please sign in to comment.