diff options
| author | Denton Liu <liu.denton@gmail.com> | 2019-11-27 11:53:11 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2019-11-29 13:20:14 -0800 |
| commit | 9b5a9fa60a444058c07c6f8df08fd779aaebfaae (patch) | |
| tree | 815354765890f166321d3459f9c1b55daa84c497 /t/t0090-cache-tree.sh | |
| parent | 17aa9d9c1af316d60ce6573daf0ff7b28bf009ed (diff) | |
| download | git-9b5a9fa60a444058c07c6f8df08fd779aaebfaae.tar.gz | |
t0090: stop losing return codes of git commands
In generate_expected_cache_tree_rec(), there are currently two instances
of `git ls-files` in the upstream of a pipe. In the case where the
upstream git command fails, its return code will be lost. Extract the
`git ls-files` into its own call so that if it ever fails, its return
code is not lost.
Signed-off-by: Denton Liu <liu.denton@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t0090-cache-tree.sh')
| -rwxr-xr-x | t/t0090-cache-tree.sh | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/t/t0090-cache-tree.sh b/t/t0090-cache-tree.sh index ce9a4a5f32..5a633690bf 100755 --- a/t/t0090-cache-tree.sh +++ b/t/t0090-cache-tree.sh @@ -21,9 +21,10 @@ generate_expected_cache_tree_rec () { parent="$2" && # ls-files might have foo/bar, foo/bar/baz, and foo/bar/quux # We want to count only foo because it's the only direct child - subtrees=$(git ls-files|grep /|cut -d / -f 1|uniq) && + git ls-files >files && + subtrees=$(grep / files|cut -d / -f 1|uniq) && subtree_count=$(echo "$subtrees"|awk -v c=0 '$1 != "" {++c} END {print c}') && - entries=$(git ls-files|wc -l) && + entries=$(wc -l <files) && printf "SHA $dir (%d entries, %d subtrees)\n" "$entries" "$subtree_count" && for subtree in $subtrees do |