kou commented on PR #925: URL: https://github.com/apache/arrow-java/pull/925#issuecomment-3716519303
> Should I revert the repository in `rc.yml` ? Yes. We should use apache/arrow not kou/arrow because the change was merged into apache/arrow. > For the MacOS JNI, it's still failing (I removed protobuf). Let me investigate. Ah, sorry. Bundled Protobuf directory in apache/arrow was changed by https://github.com/apache/arrow/pull/48183 . Could you try this instead? ```diff diff --git a/ci/scripts/jni_macos_build.sh b/ci/scripts/jni_macos_build.sh index 13c0675d3..6c5cca409 100755 --- a/ci/scripts/jni_macos_build.sh +++ b/ci/scripts/jni_macos_build.sh @@ -77,7 +77,7 @@ cmake \ cmake --build "${build_dir}/cpp" --target install github_actions_group_end -export JAVA_JNI_CMAKE_ARGS="-DProtobuf_ROOT=${build_dir}/cpp/protobuf_ep-install" +export JAVA_JNI_CMAKE_ARGS="-DProtobuf_ROOT=${build_dir}/cpp/_deps/protobuf-build" "${source_dir}/ci/scripts/jni_build.sh" \ "${source_dir}" \ "${install_dir}" \ ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
