Hello Adding another case that wasn't covered in the install testsuite!
Thanks Sylvestre
From 6882a006f9a9ec882b4cd46a1fb4adf6f6d18b4d Mon Sep 17 00:00:00 2001 From: Sylvestre Ledru <[email protected]> Date: Sun, 16 Nov 2025 23:19:14 +0100 Subject: [PATCH] tests: install supports comma-separated mode strings Identified here: <https://github.com/uutils/coreutils/pull/9298> * tests/install/basic-1.sh: Add the check. --- tests/install/basic-1.sh | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/tests/install/basic-1.sh b/tests/install/basic-1.sh index 741de7aef..64636e591 100755 --- a/tests/install/basic-1.sh +++ b/tests/install/basic-1.sh @@ -168,4 +168,17 @@ ginstall file1 file2 --mode=+w || fail=1 mode=$(ls -l file2|cut -b-10) test "$mode" = --w--w--w- || fail=1 +# Test comma-separated mode strings (like chmod) +touch file3 || framework_failure_ +ginstall file3 file4 --mode='ug+rw,o+r' || fail=1 +# Check that file4 has permissions -rw-rw-r-- +mode=$(ls -l file4|cut -b-10) +test "$mode" = -rw-rw-r-- || fail=1 + +# Test comma-separated mode with directory creation +ginstall -d testdir --mode='u+rwx,g+rx,o+r' || fail=1 +# Check that testdir has permissions drwxr-xr-- +mode=$(ls -ld testdir|cut -b-10) +test "$mode" = drwxr-xr-- || fail=1 + Exit $fail -- 2.47.2
