Skip to content

Commit

Permalink
ORC-1585: [C++] Add orc-format_ep as a dependency of orc
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?
Makes orc-format_ep a dependency of orc

### Why are the changes needed?
Building breaks if `make orc-format_ep` or `make` have not been run.

### How was this patch tested?
`mkdir build && cd build && cmake .. -DBUILD_JAVA=OFF && make orc-scan -j 100` success

### Was this patch authored or co-authored using generative AI tooling?
No

Closes #1745 from ffacs/ORC-1585.

Authored-by: ffacs <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit 00b77b2)
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
ffacs authored and dongjoon-hyun committed Jan 12, 2024
1 parent 80bab41 commit 9da39ea
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions c++/src/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -217,4 +217,6 @@ target_link_libraries (orc
${LIBHDFSPP_LIBRARIES}
)

add_dependencies(orc orc-format_ep)

install(TARGETS orc DESTINATION lib)

0 comments on commit 9da39ea

Please sign in to comment.