Skip to content

Commit

Permalink
Use CMake built-in dir create function instead of hard-coded mkdir (#…
Browse files Browse the repository at this point in the history
…108)

... Because all systems may not play nice with `mkdir -p`.
  • Loading branch information
mortbopet committed Sep 3, 2024
1 parent 1dbf98b commit ef33fd8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/substrait/proto/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,9 @@ foreach(PROTO_FILE IN LISTS PROTOBUF_FILELIST)
${GENERATED_PROTO_MIDLEVEL_DIR}/${RELATIVE_PROTO_PATH})

cmake_path(GET GENERATED_PROTO_FILE PARENT_PATH GENERATED_PROTO_DIR)
file(MAKE_DIRECTORY ${GENERATED_PROTO_DIR})
add_custom_command(
OUTPUT ${GENERATED_PROTO_FILE}
COMMAND mkdir -p ${GENERATED_PROTO_DIR}
COMMAND ${PERL_EXECUTABLE} ${UPDATE_PROTO_PACKAGE_TOOL} ${PROTO_FILE} >
${GENERATED_PROTO_FILE}
DEPENDS ${PROTO_FILE} ${UPDATE_PROTO_PACKAGE_TOOL}
Expand Down

0 comments on commit ef33fd8

Please sign in to comment.