The t1301-shared-repo.sh testscript uses /usr/bin/stat to get the file mode, which isn't portable. Implement the test in shell using 'ls' as shown by Junio. Signed-off-by: Arjen Laarhoven <arjen@xxxxxxxx> --- t/t1301-shared-repo.sh | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/t/t1301-shared-repo.sh b/t/t1301-shared-repo.sh index bb5f302..6bfe19a 100755 --- a/t/t1301-shared-repo.sh +++ b/t/t1301-shared-repo.sh @@ -21,7 +21,16 @@ test_expect_success 'update-server-info honors core.sharedRepository' ' git commit -m a1 && umask 0277 && git update-server-info && - test 444 = $(stat -c %a .git/info/refs) + actual="$(ls -l .git/info/refs)" && + case "$actual" in + -r--r--r--*) + : happy + ;; + *) + echo Oops, .git/info/refs is not 0444 + false + ;; + esac ' test_done -- 1.5.3.rc4.67.gf9286 > > - test 444 = $(stat -c %a .git/info/refs) > > + $(perl -e '\''exit !(((stat ".git/info/refs")[2] & 0777) == 0444)'\'') > > ' > > Why is this inside a $()? Bah. > I am just wondering if this is more portable and readable... > > ... && > current="$(ls -l .git/info/refs)" && > case "$current" in > -r--r--r--*) > : happy > ;; > *) > echo Oops, .git/info/refs is not 0444 > false > ;; > esac - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
