diff options
| author | Johannes Sixt <j6t@kdbg.org> | 2009-02-28 21:12:57 +0100 |
|---|---|---|
| committer | Johannes Sixt <j6t@kdbg.org> | 2009-03-19 21:47:14 +0100 |
| commit | 1f553918a8482ea792e8cd4d5d2c75fe60f68aae (patch) | |
| tree | 1227021499212fafc4d70e2030813598f68292c0 /t/t4014-format-patch.sh | |
| parent | 8586f98bd29bd4ad3d0e62a3be4a4d59ff8b27cd (diff) | |
| download | git-1f553918a8482ea792e8cd4d5d2c75fe60f68aae.tar.gz | |
test-lib: Introduce test_chmod and use it instead of update-index --chmod
This function replaces sequences of 'chmod +x' and 'git update-index
--chmod=+x' in the test suite, whose purpose is to help filesystems
that need core.filemode=false. Two places where only 'chmod +x' was used
we also use this new function.
The function calls 'git update-index --chmod' without checking
core.filemode (unlike some of the call sites did). We do this because the
call sites *expect* that the executable bit ends up in the index (ie. it
is not the purpose of the call sites to *test* whether git treats
'chmod +x' and 'update-index --chmod=+x' correctly). Therefore, on
filesystems with core.filemode=true the 'git update-index --chmod' is a
no-op.
The function uses --add with update-index to help one call site in
t6031-merge-recursive. It makes no difference for the other callers.
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Diffstat (limited to 't/t4014-format-patch.sh')
| -rwxr-xr-x | t/t4014-format-patch.sh | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/t/t4014-format-patch.sh b/t/t4014-format-patch.sh index ebfc4a6590..f187d15e32 100755 --- a/t/t4014-format-patch.sh +++ b/t/t4014-format-patch.sh @@ -16,9 +16,7 @@ test_expect_success setup ' git checkout -b side && for i in 1 2 5 6 A B C 7 8 9 10; do echo "$i"; done >file && - chmod +x elif && - git update-index file elif && - git update-index --chmod=+x elif && + test_chmod +x elif && git commit -m "Side changes #1" && for i in D E F; do echo "$i"; done >>file && |