kou commented on code in PR #46501:
URL: https://github.com/apache/arrow/pull/46501#discussion_r2110355841
##########
ci/scripts/cpp_build.sh:
##########
@@ -287,6 +287,11 @@ fi
if [ "${ARROW_USE_MESON:-OFF}" = "ON" ]; then
time meson compile -j ${ARROW_BUILD_PARALLEL}
meson install
+ # Remove all added files in cpp/subprojects/ because they may have
+ # unreadable permissions on Docker host.
+ pushd "${source_dir}"
+ meson subprojects purge --confirm --include-cache
+ popd
Review Comment:
Yes. If we don't have this, `cpp/subprojects/` has unreadable files. It
causes an actions/cache error.
(I think that Meson should not change the source directory.)
##########
ci/scripts/cpp_test.sh:
##########
@@ -92,7 +92,9 @@ fi
if [ "${ARROW_USE_MESON:-OFF}" = "ON" ]; then
ARROW_BUILD_EXAMPLES=OFF # TODO: Remove this
meson test \
+ --no-rebuild \
Review Comment:
Yes. If we don't have this, `meson test` re-downloads `cpp/subprojects/`. It
produces unreadable files under source directory. It causes an `actions/cache`
error.
--
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]
