This is an automated email from the ASF dual-hosted git repository.
mgreber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/master by this push:
new e24fb3a85 build: Add write permission to test data directories
e24fb3a85 is described below
commit e24fb3a85e02f0e1574c14025187a02cb31116dc
Author: Gabriella Lotz <[email protected]>
AuthorDate: Wed Oct 15 15:14:50 2025 +0200
build: Add write permission to test data directories
The ADD_KUDU_TEST function copies test data files with read and
execute permissions. However, directories were missing write
permissions, which could prevent cleanup operations during builds
from removing these directories.
This change adds OWNER_WRITE to DIRECTORY_PERMISSIONS to allow
proper cleanup of test data directories during the build process.
Change-Id: I5af277dce4a1f72b8d7f0e5dfed9d5018e3da0ca
Reviewed-on: http://gerrit.cloudera.org:8080/23549
Reviewed-by: Marton Greber <[email protected]>
Tested-by: Marton Greber <[email protected]>
---
CMakeLists.txt | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 9b93e7351..82aa2b16a 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -891,7 +891,8 @@ function(ADD_KUDU_TEST REL_TEST_NAME)
file(COPY ${CMAKE_CURRENT_SOURCE_DIR}/${DATA_FILE}
# Copy with read and execute permissions, since tests should not modify
# the data files in place, but data files may be scripts used by tests.
- DIRECTORY_PERMISSIONS OWNER_READ OWNER_EXECUTE GROUP_READ GROUP_EXECUTE
+ # Directories need write permission to allow cleanup during builds.
+ DIRECTORY_PERMISSIONS OWNER_READ OWNER_WRITE OWNER_EXECUTE GROUP_READ
GROUP_EXECUTE
FILE_PERMISSIONS OWNER_READ OWNER_EXECUTE GROUP_READ GROUP_EXECUTE
DESTINATION ${DST_DIR})
endforeach()
