2006-05-17 13:55:40 +04:00
|
|
|
#!/bin/sh
|
|
|
|
#
|
|
|
|
# Copyright (c) 2006 Shawn Pearce
|
|
|
|
#
|
|
|
|
|
2007-07-03 09:52:14 +04:00
|
|
|
test_description='Test git update-ref and basic ref logging'
|
2006-05-17 13:55:40 +04:00
|
|
|
. ./test-lib.sh
|
|
|
|
|
2011-04-24 09:34:13 +04:00
|
|
|
Z=$_z40
|
2008-01-16 02:50:17 +03:00
|
|
|
|
|
|
|
test_expect_success setup '
|
|
|
|
|
|
|
|
for name in A B C D E F
|
|
|
|
do
|
|
|
|
test_tick &&
|
|
|
|
T=$(git write-tree) &&
|
|
|
|
sha1=$(echo $name | git commit-tree $T) &&
|
|
|
|
eval $name=$sha1
|
|
|
|
done
|
|
|
|
|
|
|
|
'
|
|
|
|
|
2006-05-17 13:55:40 +04:00
|
|
|
m=refs/heads/master
|
2006-07-29 07:44:51 +04:00
|
|
|
n_dir=refs/heads/gu
|
|
|
|
n=$n_dir/fixes
|
2006-05-17 13:55:40 +04:00
|
|
|
|
|
|
|
test_expect_success \
|
|
|
|
"create $m" \
|
2007-07-03 09:52:14 +04:00
|
|
|
"git update-ref $m $A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"create $m" \
|
2007-07-03 09:52:14 +04:00
|
|
|
"git update-ref $m $B $A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $B"' = $(cat .git/'"$m"')'
|
2008-05-25 20:14:29 +04:00
|
|
|
test_expect_success "fail to delete $m with stale ref" '
|
|
|
|
test_must_fail git update-ref -d $m $A &&
|
|
|
|
test $B = "$(cat .git/$m)"
|
|
|
|
'
|
|
|
|
test_expect_success "delete $m" '
|
|
|
|
git update-ref -d $m $B &&
|
|
|
|
! test -f .git/$m
|
|
|
|
'
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -f .git/$m
|
|
|
|
|
2008-06-03 03:34:53 +04:00
|
|
|
test_expect_success "delete $m without oldvalue verification" "
|
|
|
|
git update-ref $m $A &&
|
|
|
|
test $A = \$(cat .git/$m) &&
|
|
|
|
git update-ref -d $m &&
|
|
|
|
! test -f .git/$m
|
|
|
|
"
|
|
|
|
rm -f .git/$m
|
|
|
|
|
2006-07-29 07:44:51 +04:00
|
|
|
test_expect_success \
|
|
|
|
"fail to create $n" \
|
2010-10-31 10:36:19 +03:00
|
|
|
"touch .git/$n_dir &&
|
|
|
|
test_must_fail git update-ref $n $A >out 2>err"
|
2006-07-29 07:44:51 +04:00
|
|
|
rm -f .git/$n_dir out err
|
|
|
|
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"create $m (by HEAD)" \
|
2007-07-03 09:52:14 +04:00
|
|
|
"git update-ref HEAD $A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"create $m (by HEAD)" \
|
2007-07-03 09:52:14 +04:00
|
|
|
"git update-ref HEAD $B $A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $B"' = $(cat .git/'"$m"')'
|
2008-05-25 20:14:29 +04:00
|
|
|
test_expect_success "fail to delete $m (by HEAD) with stale ref" '
|
|
|
|
test_must_fail git update-ref -d HEAD $A &&
|
|
|
|
test $B = $(cat .git/$m)
|
|
|
|
'
|
|
|
|
test_expect_success "delete $m (by HEAD)" '
|
|
|
|
git update-ref -d HEAD $B &&
|
|
|
|
! test -f .git/$m
|
|
|
|
'
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -f .git/$m
|
|
|
|
|
2012-10-21 14:40:31 +04:00
|
|
|
test_expect_success \
|
|
|
|
"create $m (by HEAD)" \
|
|
|
|
"git update-ref HEAD $A &&
|
|
|
|
test $A"' = $(cat .git/'"$m"')'
|
|
|
|
test_expect_success \
|
|
|
|
"pack refs" \
|
|
|
|
"git pack-refs --all"
|
|
|
|
test_expect_success \
|
|
|
|
"move $m (by HEAD)" \
|
|
|
|
"git update-ref HEAD $B $A &&
|
|
|
|
test $B"' = $(cat .git/'"$m"')'
|
2012-10-21 14:40:32 +04:00
|
|
|
test_expect_success "delete $m (by HEAD) should remove both packed and loose $m" '
|
2012-10-21 14:40:31 +04:00
|
|
|
git update-ref -d HEAD $B &&
|
|
|
|
! grep "$m" .git/packed-refs &&
|
|
|
|
! test -f .git/$m
|
|
|
|
'
|
|
|
|
rm -f .git/$m
|
|
|
|
|
2008-10-26 05:33:58 +03:00
|
|
|
cp -f .git/HEAD .git/HEAD.orig
|
|
|
|
test_expect_success "delete symref without dereference" '
|
|
|
|
git update-ref --no-deref -d HEAD &&
|
|
|
|
! test -f .git/HEAD
|
|
|
|
'
|
|
|
|
cp -f .git/HEAD.orig .git/HEAD
|
|
|
|
|
2008-11-01 02:25:44 +03:00
|
|
|
test_expect_success "delete symref without dereference when the referred ref is packed" '
|
|
|
|
echo foo >foo.c &&
|
|
|
|
git add foo.c &&
|
|
|
|
git commit -m foo &&
|
|
|
|
git pack-refs --all &&
|
|
|
|
git update-ref --no-deref -d HEAD &&
|
|
|
|
! test -f .git/HEAD
|
|
|
|
'
|
|
|
|
cp -f .git/HEAD.orig .git/HEAD
|
|
|
|
git update-ref -d $m
|
|
|
|
|
2014-09-11 05:22:48 +04:00
|
|
|
test_expect_success 'update-ref -d is not confused by self-reference' '
|
|
|
|
git symbolic-ref refs/heads/self refs/heads/self &&
|
|
|
|
test_when_finished "rm -f .git/refs/heads/self" &&
|
|
|
|
test_path_is_file .git/refs/heads/self &&
|
|
|
|
test_must_fail git update-ref -d refs/heads/self &&
|
|
|
|
test_path_is_file .git/refs/heads/self
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'update-ref --no-deref -d can delete self-reference' '
|
|
|
|
git symbolic-ref refs/heads/self refs/heads/self &&
|
|
|
|
test_when_finished "rm -f .git/refs/heads/self" &&
|
|
|
|
test_path_is_file .git/refs/heads/self &&
|
|
|
|
git update-ref --no-deref -d refs/heads/self &&
|
|
|
|
test_path_is_missing .git/refs/heads/self
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'update-ref --no-deref -d can delete reference to bad ref' '
|
|
|
|
>.git/refs/heads/bad &&
|
|
|
|
test_when_finished "rm -f .git/refs/heads/bad" &&
|
|
|
|
git symbolic-ref refs/heads/ref-to-bad refs/heads/bad &&
|
|
|
|
test_when_finished "rm -f .git/refs/heads/ref-to-bad" &&
|
|
|
|
test_path_is_file .git/refs/heads/ref-to-bad &&
|
|
|
|
git update-ref --no-deref -d refs/heads/ref-to-bad &&
|
|
|
|
test_path_is_missing .git/refs/heads/ref-to-bad
|
|
|
|
'
|
|
|
|
|
2008-02-01 12:50:53 +03:00
|
|
|
test_expect_success '(not) create HEAD with old sha1' "
|
2008-07-12 19:47:52 +04:00
|
|
|
test_must_fail git update-ref HEAD $A $B
|
2008-02-01 12:50:53 +03:00
|
|
|
"
|
|
|
|
test_expect_success "(not) prior created .git/$m" "
|
|
|
|
! test -f .git/$m
|
|
|
|
"
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -f .git/$m
|
|
|
|
|
|
|
|
test_expect_success \
|
|
|
|
"create HEAD" \
|
2007-07-03 09:52:14 +04:00
|
|
|
"git update-ref HEAD $A"
|
2008-02-01 12:50:53 +03:00
|
|
|
test_expect_success '(not) change HEAD with wrong SHA1' "
|
2008-07-12 19:47:52 +04:00
|
|
|
test_must_fail git update-ref HEAD $B $Z
|
2008-02-01 12:50:53 +03:00
|
|
|
"
|
|
|
|
test_expect_success "(not) changed .git/$m" "
|
|
|
|
! test $B"' = $(cat .git/'"$m"')
|
|
|
|
'
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -f .git/$m
|
|
|
|
|
2006-12-15 01:41:17 +03:00
|
|
|
: a repository with working tree always has reflog these days...
|
|
|
|
: >.git/logs/refs/heads/master
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"create $m (logged by touch)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:30" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD '"$A"' -m "Initial Creation" &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"update $m (logged by touch)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:31" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD'" $B $A "'-m "Switch" &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$B"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"set $m (logged by touch)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:41" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD'" $A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
|
|
|
|
cat >expect <<EOF
|
|
|
|
$Z $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150200 +0000 Initial Creation
|
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150260 +0000 Switch
|
|
|
|
$B $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150860 +0000
|
|
|
|
EOF
|
|
|
|
test_expect_success \
|
|
|
|
"verifying $m's log" \
|
2009-03-16 23:18:42 +03:00
|
|
|
"test_cmp expect .git/logs/$m"
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -rf .git/$m .git/logs expect
|
|
|
|
|
|
|
|
test_expect_success \
|
|
|
|
'enable core.logAllRefUpdates' \
|
2007-07-03 09:52:14 +04:00
|
|
|
'git config core.logAllRefUpdates true &&
|
|
|
|
test true = $(git config --bool --get core.logAllRefUpdates)'
|
2006-05-17 13:55:40 +04:00
|
|
|
|
|
|
|
test_expect_success \
|
|
|
|
"create $m (logged by config)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:32" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD'" $A "'-m "Initial Creation" &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"update $m (logged by config)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:33" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD'" $B $A "'-m "Switch" &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$B"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
test_expect_success \
|
|
|
|
"set $m (logged by config)" \
|
|
|
|
'GIT_COMMITTER_DATE="2005-05-26 23:43" \
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref HEAD '"$A &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test $A"' = $(cat .git/'"$m"')'
|
2006-05-17 13:55:40 +04:00
|
|
|
|
|
|
|
cat >expect <<EOF
|
|
|
|
$Z $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150320 +0000 Initial Creation
|
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150380 +0000 Switch
|
|
|
|
$B $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150980 +0000
|
|
|
|
EOF
|
|
|
|
test_expect_success \
|
|
|
|
"verifying $m's log" \
|
2009-03-16 23:18:42 +03:00
|
|
|
'test_cmp expect .git/logs/$m'
|
2006-05-17 13:55:40 +04:00
|
|
|
rm -f .git/$m .git/logs/$m expect
|
|
|
|
|
2007-07-03 09:52:14 +04:00
|
|
|
git update-ref $m $D
|
2006-05-19 11:28:19 +04:00
|
|
|
cat >.git/logs/$m <<EOF
|
2008-07-08 08:38:54 +04:00
|
|
|
0000000000000000000000000000000000000000 $C $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150320 -0500
|
|
|
|
$C $A $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150350 -0500
|
2006-05-19 11:28:19 +04:00
|
|
|
$A $B $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150380 -0500
|
|
|
|
$F $Z $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150680 -0500
|
|
|
|
$Z $E $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150980 -0500
|
|
|
|
EOF
|
|
|
|
|
|
|
|
ed="Thu, 26 May 2005 18:32:00 -0500"
|
|
|
|
gd="Thu, 26 May 2005 18:33:00 -0500"
|
|
|
|
ld="Thu, 26 May 2005 18:43:00 -0500"
|
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{May 25 2005}" (before history)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{May 25 2005}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$C"' = $(cat o) &&
|
2007-01-19 12:19:05 +03:00
|
|
|
test "warning: Log for '\'master\'' only goes back to $ed." = "$(cat e)"'
|
2006-05-19 11:28:19 +04:00
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
"Query master@{2005-05-25} (before history)" \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify master@{2005-05-25} >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$C"' = $(cat o) &&
|
2007-01-19 12:19:05 +03:00
|
|
|
echo test "warning: Log for '\'master\'' only goes back to $ed." = "$(cat e)"'
|
2006-05-19 11:28:19 +04:00
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{May 26 2005 23:31:59}" (1 second before history)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{May 26 2005 23:31:59}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$C"' = $(cat o) &&
|
2007-01-19 12:19:05 +03:00
|
|
|
test "warning: Log for '\''master'\'' only goes back to $ed." = "$(cat e)"'
|
2006-05-19 11:28:19 +04:00
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{May 26 2005 23:32:00}" (exactly history start)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{May 26 2005 23:32:00}" >o 2>e &&
|
2008-07-08 08:38:54 +04:00
|
|
|
test '"$C"' = $(cat o) &&
|
|
|
|
test "" = "$(cat e)"'
|
|
|
|
test_expect_success \
|
|
|
|
'Query "master@{May 26 2005 23:32:30}" (first non-creation change)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2008-07-08 08:38:54 +04:00
|
|
|
git rev-parse --verify "master@{May 26 2005 23:32:30}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$A"' = $(cat o) &&
|
2006-05-19 11:28:19 +04:00
|
|
|
test "" = "$(cat e)"'
|
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{2005-05-26 23:33:01}" (middle of history with gap)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{2005-05-26 23:33:01}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$B"' = $(cat o) &&
|
2014-06-03 20:09:59 +04:00
|
|
|
test "warning: Log for ref '"$m has gap after $gd"'." = "$(cat e)"'
|
2006-05-19 11:28:19 +04:00
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{2005-05-26 23:38:00}" (middle of history)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{2005-05-26 23:38:00}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$Z"' = $(cat o) &&
|
2006-05-19 11:28:19 +04:00
|
|
|
test "" = "$(cat e)"'
|
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{2005-05-26 23:43:00}" (exact end of history)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{2005-05-26 23:43:00}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$E"' = $(cat o) &&
|
2006-05-19 11:28:19 +04:00
|
|
|
test "" = "$(cat e)"'
|
|
|
|
test_expect_success \
|
2006-05-19 11:29:43 +04:00
|
|
|
'Query "master@{2005-05-28}" (past end of history)' \
|
2010-10-31 10:30:58 +03:00
|
|
|
'rm -f o e &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git rev-parse --verify "master@{2005-05-28}" >o 2>e &&
|
2006-09-17 00:41:14 +04:00
|
|
|
test '"$D"' = $(cat o) &&
|
2014-06-03 20:09:59 +04:00
|
|
|
test "warning: Log for ref '"$m unexpectedly ended on $ld"'." = "$(cat e)"'
|
2006-05-19 11:28:19 +04:00
|
|
|
|
2006-05-19 11:29:43 +04:00
|
|
|
|
|
|
|
rm -f .git/$m .git/logs/$m expect
|
|
|
|
|
|
|
|
test_expect_success \
|
|
|
|
'creating initial files' \
|
2006-05-25 07:33:18 +04:00
|
|
|
'echo TEST >F &&
|
2007-07-03 09:52:14 +04:00
|
|
|
git add F &&
|
2006-05-25 07:33:18 +04:00
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:30" \
|
2008-09-03 12:59:27 +04:00
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:30" git commit -m add -a &&
|
2010-10-31 10:30:58 +03:00
|
|
|
h_TEST=$(git rev-parse --verify HEAD) &&
|
2006-05-25 07:33:18 +04:00
|
|
|
echo The other day this did not work. >M &&
|
|
|
|
echo And then Bob told me how to fix it. >>M &&
|
|
|
|
echo OTHER >F &&
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:41" \
|
2008-09-03 12:59:27 +04:00
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:41" git commit -F M -a &&
|
2007-07-03 09:52:14 +04:00
|
|
|
h_OTHER=$(git rev-parse --verify HEAD) &&
|
2006-07-11 06:48:47 +04:00
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:44" \
|
2008-09-03 12:59:27 +04:00
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:44" git commit --amend &&
|
2007-07-03 09:52:14 +04:00
|
|
|
h_FIXED=$(git rev-parse --verify HEAD) &&
|
2006-07-11 06:48:47 +04:00
|
|
|
echo Merged initial commit and a later commit. >M &&
|
|
|
|
echo $h_TEST >.git/MERGE_HEAD &&
|
|
|
|
GIT_AUTHOR_DATE="2005-05-26 23:45" \
|
2008-09-03 12:59:27 +04:00
|
|
|
GIT_COMMITTER_DATE="2005-05-26 23:45" git commit -F M &&
|
2007-11-09 02:41:39 +03:00
|
|
|
h_MERGED=$(git rev-parse --verify HEAD) &&
|
2006-05-25 07:33:18 +04:00
|
|
|
rm -f M'
|
|
|
|
|
|
|
|
cat >expect <<EOF
|
2006-07-11 06:48:47 +04:00
|
|
|
$Z $h_TEST $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150200 +0000 commit (initial): add
|
2006-05-25 07:33:18 +04:00
|
|
|
$h_TEST $h_OTHER $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150860 +0000 commit: The other day this did not work.
|
2006-07-11 06:48:47 +04:00
|
|
|
$h_OTHER $h_FIXED $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117151040 +0000 commit (amend): The other day this did not work.
|
|
|
|
$h_FIXED $h_MERGED $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117151100 +0000 commit (merge): Merged initial commit and a later commit.
|
2006-05-25 07:33:18 +04:00
|
|
|
EOF
|
|
|
|
test_expect_success \
|
2008-09-03 12:59:27 +04:00
|
|
|
'git commit logged updates' \
|
2009-03-16 23:18:42 +03:00
|
|
|
"test_cmp expect .git/logs/$m"
|
2006-07-11 06:48:47 +04:00
|
|
|
unset h_TEST h_OTHER h_FIXED h_MERGED
|
2006-05-19 11:29:43 +04:00
|
|
|
|
|
|
|
test_expect_success \
|
2007-07-03 09:52:14 +04:00
|
|
|
'git cat-file blob master:F (expect OTHER)' \
|
|
|
|
'test OTHER = $(git cat-file blob master:F)'
|
2006-05-19 11:29:43 +04:00
|
|
|
test_expect_success \
|
2007-07-03 09:52:14 +04:00
|
|
|
'git cat-file blob master@{2005-05-26 23:30}:F (expect TEST)' \
|
|
|
|
'test TEST = $(git cat-file blob "master@{2005-05-26 23:30}:F")'
|
2006-05-19 11:29:43 +04:00
|
|
|
test_expect_success \
|
2007-07-03 09:52:14 +04:00
|
|
|
'git cat-file blob master@{2005-05-26 23:42}:F (expect OTHER)' \
|
|
|
|
'test OTHER = $(git cat-file blob "master@{2005-05-26 23:42}:F")'
|
2006-05-19 11:29:43 +04:00
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
a=refs/heads/a
|
|
|
|
b=refs/heads/b
|
|
|
|
c=refs/heads/c
|
|
|
|
E='""'
|
|
|
|
F='%s\0'
|
|
|
|
pws='path with space'
|
|
|
|
|
|
|
|
test_expect_success 'stdin test setup' '
|
|
|
|
echo "$pws" >"$pws" &&
|
|
|
|
git add -- "$pws" &&
|
|
|
|
git commit -m "$pws"
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success '-z fails without --stdin' '
|
|
|
|
test_must_fail git update-ref -z $m $m $m 2>err &&
|
|
|
|
grep "usage: git update-ref" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin works with no input' '
|
|
|
|
>stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse --verify -q $m
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on empty line' '
|
|
|
|
echo "" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: empty command in input" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on only whitespace' '
|
|
|
|
echo " " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on leading whitespace' '
|
|
|
|
echo " create $a $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: whitespace before command: create $a $m" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails on unknown command' '
|
|
|
|
echo "unknown $a" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: unknown $a" err
|
|
|
|
'
|
|
|
|
|
2014-04-07 17:47:55 +04:00
|
|
|
test_expect_success 'stdin fails on unbalanced quotes' '
|
2013-09-11 16:46:18 +04:00
|
|
|
echo "create $a \"master" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: badly quoted argument: \\\"master" err
|
|
|
|
'
|
|
|
|
|
2014-04-07 17:47:55 +04:00
|
|
|
test_expect_success 'stdin fails on invalid escape' '
|
|
|
|
echo "create $a \"ma\zter\"" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: badly quoted argument: \\\"ma\\\\zter\\\"" err
|
|
|
|
'
|
|
|
|
|
2014-04-07 17:47:54 +04:00
|
|
|
test_expect_success 'stdin fails on junk after quoted argument' '
|
2013-09-11 16:46:18 +04:00
|
|
|
echo "create \"$a\"master" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:47:54 +04:00
|
|
|
grep "fatal: unexpected character after quoted argument: \\\"$a\\\"master" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with no ref' '
|
|
|
|
echo "create " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with no new value' '
|
|
|
|
echo "create $a" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create $a: missing <newvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails create with too many arguments' '
|
|
|
|
echo "create $a $m $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create $a: extra input: $m" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with no ref' '
|
|
|
|
echo "update " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: update: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with no new value' '
|
|
|
|
echo "update $a" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: update $a: missing <newvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails update with too many arguments' '
|
|
|
|
echo "update $a $m $m $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: update $a: extra input: $m" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails delete with no ref' '
|
|
|
|
echo "delete " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: delete: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails delete with too many arguments' '
|
|
|
|
echo "delete $a $m $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: delete $a: extra input: $m" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails verify with too many arguments' '
|
|
|
|
echo "verify $a $m $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: verify $a: extra input: $m" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails option with unknown name' '
|
|
|
|
echo "option unknown" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: option unknown: unknown" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin fails with duplicate refs' '
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
create $a $m
|
|
|
|
create $b $m
|
|
|
|
create $a $m
|
|
|
|
EOF
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: Multiple updates for ref '"'"'$a'"'"' not allowed." err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref works' '
|
|
|
|
echo "create $a $m" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2014-04-07 17:47:55 +04:00
|
|
|
test_expect_success 'stdin succeeds with quoted argument' '
|
|
|
|
git update-ref -d $a &&
|
|
|
|
echo "create $a \"$m\"" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin succeeds with escaped character' '
|
|
|
|
git update-ref -d $a &&
|
|
|
|
echo "create $a \"ma\\163ter\"" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
test_expect_success 'stdin update ref creates with zero old value' '
|
|
|
|
echo "update $b $m $Z" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git update-ref -d $b
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref creates with empty old value' '
|
|
|
|
echo "update $b $m $E" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref works with path with space to blob' '
|
|
|
|
echo "create refs/blobs/pws \"$m:$pws\"" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse "$m:$pws" >expect &&
|
|
|
|
git rev-parse refs/blobs/pws >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git update-ref -d refs/blobs/pws
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref fails with wrong old value' '
|
|
|
|
echo "update $c $m $m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref fails with bad old value' '
|
|
|
|
echo "update $c $m does-not-exist" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:02 +04:00
|
|
|
grep "fatal: update $c: invalid <oldvalue>: does-not-exist" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref fails with bad new value' '
|
|
|
|
echo "create $c does-not-exist" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:02 +04:00
|
|
|
grep "fatal: create $c: invalid <newvalue>: does-not-exist" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin create ref fails with zero new value' '
|
|
|
|
echo "create $c " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create $c: zero <newvalue>" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update ref works with right old value' '
|
|
|
|
echo "update $b $m~1 $m" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete ref fails with wrong old value' '
|
|
|
|
echo "delete $a $m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$a'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete ref fails with zero old value' '
|
|
|
|
echo "delete $a " >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: delete $a: zero <oldvalue>" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update symref works option no-deref' '
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
option no-deref
|
|
|
|
update TESTSYMREF $a $b
|
|
|
|
EOF
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse TESTSYMREF >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete symref works option no-deref' '
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
option no-deref
|
|
|
|
delete TESTSYMREF $b
|
|
|
|
EOF
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREF &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete ref works with right old value' '
|
|
|
|
echo "delete $b $m~1" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q $b
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update/create/verify combination works' '
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
update $a $m
|
|
|
|
create $b $m
|
|
|
|
verify $c
|
|
|
|
EOF
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
2014-12-11 02:47:51 +03:00
|
|
|
test_expect_success 'stdin verify succeeds for correct value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
echo "verify $m $m" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify succeeds for missing reference' '
|
|
|
|
echo "verify refs/heads/missing $Z" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify treats no value as missing' '
|
|
|
|
echo "verify refs/heads/missing" >stdin &&
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify fails for wrong value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
echo "verify $m $m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin verify fails for mistaken null value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
echo "verify $m $Z" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2014-12-11 02:47:52 +03:00
|
|
|
test_expect_success 'stdin verify fails for mistaken empty value' '
|
2014-12-11 02:47:51 +03:00
|
|
|
M=$(git rev-parse $m) &&
|
|
|
|
test_when_finished "git update-ref $m $M" &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
echo "verify $m" >stdin &&
|
|
|
|
test_must_fail git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
test_expect_success 'stdin update refs works with identity updates' '
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
update $a $m $m
|
|
|
|
update $b $m $m
|
|
|
|
update $c $Z $E
|
|
|
|
EOF
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin update refs fails with wrong old value' '
|
|
|
|
git update-ref $c $m &&
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
update $a $m $m
|
|
|
|
update $b $m $m
|
|
|
|
update $c ''
|
|
|
|
EOF
|
|
|
|
test_must_fail git update-ref --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $c >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin delete refs works with packed and loose refs' '
|
|
|
|
git pack-refs --all &&
|
|
|
|
git update-ref $c $m~1 &&
|
|
|
|
cat >stdin <<-EOF &&
|
|
|
|
delete $a $m
|
|
|
|
update $b $Z $m
|
|
|
|
update $c $E $m~1
|
|
|
|
EOF
|
|
|
|
git update-ref --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q $a &&
|
|
|
|
test_must_fail git rev-parse --verify -q $b &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z works on empty input' '
|
|
|
|
>stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse --verify -q $m
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on empty line' '
|
|
|
|
echo "" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on empty command' '
|
|
|
|
printf $F "" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: empty command in input" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on only whitespace' '
|
|
|
|
printf $F " " >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: whitespace before command: " err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on leading whitespace' '
|
|
|
|
printf $F " create $a" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: whitespace before command: create $a" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails on unknown command' '
|
|
|
|
printf $F "unknown $a" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: unknown $a" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with no ref' '
|
|
|
|
printf $F "create " >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with no new value' '
|
|
|
|
printf $F "create $a" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: create $a: unexpected end of input when reading <newvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails create with too many arguments' '
|
|
|
|
printf $F "create $a" "$m" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with no ref' '
|
|
|
|
printf $F "update " >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: update: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
2014-04-07 17:48:07 +04:00
|
|
|
test_expect_success 'stdin -z fails update with too few args' '
|
|
|
|
printf $F "update $a" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
2014-04-07 17:48:06 +04:00
|
|
|
test_expect_success 'stdin -z emits warning with empty new value' '
|
2014-04-07 17:48:04 +04:00
|
|
|
git update-ref $a $m &&
|
|
|
|
printf $F "update $a" "" "" >stdin &&
|
2014-04-07 17:48:06 +04:00
|
|
|
git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "warning: update $a: missing <newvalue>, treating as zero" err &&
|
2014-04-07 17:48:04 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $a
|
|
|
|
'
|
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
test_expect_success 'stdin -z fails update with no new value' '
|
|
|
|
printf $F "update $a" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <newvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with no old value' '
|
|
|
|
printf $F "update $a" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: update $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails update with too many arguments' '
|
|
|
|
printf $F "update $a" "$m" "$m" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with no ref' '
|
|
|
|
printf $F "delete " >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: delete: missing <ref>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with no old value' '
|
|
|
|
printf $F "delete $a" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: delete $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails delete with too many arguments' '
|
|
|
|
printf $F "delete $a" "$m" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails verify with too many arguments' '
|
|
|
|
printf $F "verify $a" "$m" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: unknown command: $m" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails verify with no old value' '
|
|
|
|
printf $F "verify $a" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:08 +04:00
|
|
|
grep "fatal: verify $a: unexpected end of input when reading <oldvalue>" err
|
2013-09-11 16:46:18 +04:00
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails option with unknown name' '
|
|
|
|
printf $F "option unknown" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: option unknown: unknown" err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z fails with duplicate refs' '
|
|
|
|
printf $F "create $a" "$m" "create $b" "$m" "create $a" "$m" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
|
|
|
grep "fatal: Multiple updates for ref '"'"'$a'"'"' not allowed." err
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z create ref works' '
|
|
|
|
printf $F "create $a" "$m" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref creates with zero old value' '
|
|
|
|
printf $F "update $b" "$m" "$Z" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git update-ref -d $b
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref creates with empty old value' '
|
|
|
|
printf $F "update $b" "$m" "" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z create ref works with path with space to blob' '
|
|
|
|
printf $F "create refs/blobs/pws" "$m:$pws" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse "$m:$pws" >expect &&
|
|
|
|
git rev-parse refs/blobs/pws >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git update-ref -d refs/blobs/pws
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref fails with wrong old value' '
|
|
|
|
printf $F "update $c" "$m" "$m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref fails with bad old value' '
|
|
|
|
printf $F "update $c" "$m" "does-not-exist" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:02 +04:00
|
|
|
grep "fatal: update $c: invalid <oldvalue>: does-not-exist" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
2014-04-02 12:09:54 +04:00
|
|
|
test_expect_success 'stdin -z create ref fails when ref exists' '
|
|
|
|
git update-ref $c $m &&
|
|
|
|
git rev-parse "$c" >expect &&
|
|
|
|
printf $F "create $c" "$m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$c'"'"'" err &&
|
2014-04-02 12:09:54 +04:00
|
|
|
git rev-parse "$c" >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
test_expect_success 'stdin -z create ref fails with bad new value' '
|
2014-04-02 12:09:54 +04:00
|
|
|
git update-ref -d "$c" &&
|
2013-09-11 16:46:18 +04:00
|
|
|
printf $F "create $c" "does-not-exist" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:02 +04:00
|
|
|
grep "fatal: create $c: invalid <newvalue>: does-not-exist" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
update-ref.c: extract a new function, parse_next_sha1()
Replace three functions, update_store_new_sha1(),
update_store_old_sha1(), and parse_next_arg(), with a single function,
parse_next_sha1(). The new function takes care of a whole argument,
including checking whether it is there, converting it to an SHA-1, and
emitting errors on EOF or for invalid values. The return value
indicates whether the argument was present or absent, which requires
a bit of intelligence because absent values are represented
differently depending on whether "-z" was used.
The new interface means that the calling functions, parse_cmd_*(),
don't have to interpret the result differently based on the
line_termination mode that is in effect. It also means that
parse_cmd_create() can distinguish unambiguously between an empty new
value and a zeros new value, which fixes a failure in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2014-04-07 17:48:05 +04:00
|
|
|
test_expect_success 'stdin -z create ref fails with empty new value' '
|
2013-09-11 16:46:18 +04:00
|
|
|
printf $F "create $c" "" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: create $c: missing <newvalue>" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update ref works with right old value' '
|
|
|
|
printf $F "update $b" "$m~1" "$m" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref fails with wrong old value' '
|
|
|
|
printf $F "delete $a" "$m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$a'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref fails with zero old value' '
|
|
|
|
printf $F "delete $a" "$Z" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2014-04-07 17:48:09 +04:00
|
|
|
grep "fatal: delete $a: zero <oldvalue>" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update symref works option no-deref' '
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
printf $F "option no-deref" "update TESTSYMREF" "$a" "$b" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse TESTSYMREF >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete symref works option no-deref' '
|
|
|
|
git symbolic-ref TESTSYMREF $b &&
|
|
|
|
printf $F "option no-deref" "delete TESTSYMREF" "$b" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q TESTSYMREF &&
|
|
|
|
git rev-parse $m~1 >expect &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete ref works with right old value' '
|
|
|
|
printf $F "delete $b" "$m~1" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q $b
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update/create/verify combination works' '
|
|
|
|
printf $F "update $a" "$m" "" "create $b" "$m" "verify $c" "" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
2014-12-11 02:47:51 +03:00
|
|
|
test_expect_success 'stdin -z verify succeeds for correct value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
printf $F "verify $m" "$m" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify succeeds for missing reference' '
|
|
|
|
printf $F "verify refs/heads/missing" "$Z" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify treats no value as missing' '
|
|
|
|
printf $F "verify refs/heads/missing" "" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/missing
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify fails for wrong value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
printf $F "verify $m" "$m~1" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z verify fails for mistaken null value' '
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
printf $F "verify $m" "$Z" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2014-12-11 02:47:52 +03:00
|
|
|
test_expect_success 'stdin -z verify fails for mistaken empty value' '
|
2014-12-11 02:47:51 +03:00
|
|
|
M=$(git rev-parse $m) &&
|
|
|
|
test_when_finished "git update-ref $m $M" &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
printf $F "verify $m" "" >stdin &&
|
|
|
|
test_must_fail git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
2013-09-11 16:46:18 +04:00
|
|
|
test_expect_success 'stdin -z update refs works with identity updates' '
|
|
|
|
printf $F "update $a" "$m" "$m" "update $b" "$m" "$m" "update $c" "$Z" "" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z update refs fails with wrong old value' '
|
|
|
|
git update-ref $c $m &&
|
2014-04-07 17:47:53 +04:00
|
|
|
printf $F "update $a" "$m" "$m" "update $b" "$m" "$m" "update $c" "$m" "$Z" >stdin &&
|
2013-09-11 16:46:18 +04:00
|
|
|
test_must_fail git update-ref -z --stdin <stdin 2>err &&
|
2015-05-11 18:25:19 +03:00
|
|
|
grep "fatal: Cannot lock ref '"'"'$c'"'"'" err &&
|
2013-09-11 16:46:18 +04:00
|
|
|
git rev-parse $m >expect &&
|
|
|
|
git rev-parse $a >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $b >actual &&
|
|
|
|
test_cmp expect actual &&
|
|
|
|
git rev-parse $c >actual &&
|
|
|
|
test_cmp expect actual
|
|
|
|
'
|
|
|
|
|
|
|
|
test_expect_success 'stdin -z delete refs works with packed and loose refs' '
|
|
|
|
git pack-refs --all &&
|
|
|
|
git update-ref $c $m~1 &&
|
|
|
|
printf $F "delete $a" "$m" "update $b" "$Z" "$m" "update $c" "" "$m~1" >stdin &&
|
|
|
|
git update-ref -z --stdin <stdin &&
|
|
|
|
test_must_fail git rev-parse --verify -q $a &&
|
|
|
|
test_must_fail git rev-parse --verify -q $b &&
|
|
|
|
test_must_fail git rev-parse --verify -q $c
|
|
|
|
'
|
|
|
|
|
2015-04-15 01:25:06 +03:00
|
|
|
run_with_limited_open_files () {
|
|
|
|
(ulimit -n 32 && "$@")
|
|
|
|
}
|
|
|
|
|
|
|
|
test_lazy_prereq ULIMIT_FILE_DESCRIPTORS 'run_with_limited_open_files true'
|
|
|
|
|
ref_transaction_commit(): fix atomicity and avoid fd exhaustion
The old code was roughly
for update in updates:
acquire locks and check old_sha
for update in updates:
if changing value:
write_ref_to_lockfile()
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This has two problems.
Non-atomic updates
==================
The atomicity of the reference transaction depends on all pre-checks
being done in the first loop, before any changes have started being
committed in the second loop. The problem is that
write_ref_to_lockfile() (previously part of write_ref_sha1()), which
is called from the second loop, contains two more checks:
* It verifies that new_sha1 is a valid object
* If the reference being updated is a branch, it verifies that
new_sha1 points at a commit object (as opposed to a tag, tree, or
blob).
If either of these checks fails, the "transaction" is aborted during
the second loop. But this might happen after some reference updates
have already been permanently committed. In other words, the
all-or-nothing promise of "git update-ref --stdin" could be violated.
So these checks have to be moved to the first loop.
File descriptor exhaustion
==========================
The old code locked all of the references in the first loop, leaving
all of the lockfiles open until later loops. Since we might be
updating a lot of references, this could result in file descriptor
exhaustion.
The solution
============
After this patch, the code looks like
for update in updates:
acquire locks and check old_sha
if changing value:
write_ref_to_lockfile()
else:
close_ref()
for update in updates:
if changing value:
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This fixes both problems:
1. The pre-checks in write_ref_to_lockfile() are now done in the first
loop, before any changes have been committed. If any of the checks
fails, the whole transaction can now be rolled back correctly.
2. All lockfiles are closed in the first loop immediately after they
are created (either by write_ref_to_lockfile() or by close_ref()).
This means that there is never more than one open lockfile at a
time, preventing file descriptor exhaustion.
To simplify the bookkeeping across loops, add a new REF_NEEDS_COMMIT
bit to update->flags, which keeps track of whether the corresponding
lockfile needs to be committed, as opposed to just unlocked. (Since
"struct ref_update" is internal to the refs module, this change is not
visible to external callers.)
This change fixes two tests in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-04-24 14:35:49 +03:00
|
|
|
test_expect_success ULIMIT_FILE_DESCRIPTORS 'large transaction creating branches does not burst open file limit' '
|
2015-04-15 01:25:06 +03:00
|
|
|
(
|
|
|
|
for i in $(test_seq 33)
|
|
|
|
do
|
|
|
|
echo "create refs/heads/$i HEAD"
|
|
|
|
done >large_input &&
|
|
|
|
run_with_limited_open_files git update-ref --stdin <large_input &&
|
|
|
|
git rev-parse --verify -q refs/heads/33
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
ref_transaction_commit(): fix atomicity and avoid fd exhaustion
The old code was roughly
for update in updates:
acquire locks and check old_sha
for update in updates:
if changing value:
write_ref_to_lockfile()
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This has two problems.
Non-atomic updates
==================
The atomicity of the reference transaction depends on all pre-checks
being done in the first loop, before any changes have started being
committed in the second loop. The problem is that
write_ref_to_lockfile() (previously part of write_ref_sha1()), which
is called from the second loop, contains two more checks:
* It verifies that new_sha1 is a valid object
* If the reference being updated is a branch, it verifies that
new_sha1 points at a commit object (as opposed to a tag, tree, or
blob).
If either of these checks fails, the "transaction" is aborted during
the second loop. But this might happen after some reference updates
have already been permanently committed. In other words, the
all-or-nothing promise of "git update-ref --stdin" could be violated.
So these checks have to be moved to the first loop.
File descriptor exhaustion
==========================
The old code locked all of the references in the first loop, leaving
all of the lockfiles open until later loops. Since we might be
updating a lot of references, this could result in file descriptor
exhaustion.
The solution
============
After this patch, the code looks like
for update in updates:
acquire locks and check old_sha
if changing value:
write_ref_to_lockfile()
else:
close_ref()
for update in updates:
if changing value:
commit_ref_update()
for update in updates:
if deleting value:
unlink()
rewrite packed-refs file
for update in updates:
if reference still locked:
unlock_ref()
This fixes both problems:
1. The pre-checks in write_ref_to_lockfile() are now done in the first
loop, before any changes have been committed. If any of the checks
fails, the whole transaction can now be rolled back correctly.
2. All lockfiles are closed in the first loop immediately after they
are created (either by write_ref_to_lockfile() or by close_ref()).
This means that there is never more than one open lockfile at a
time, preventing file descriptor exhaustion.
To simplify the bookkeeping across loops, add a new REF_NEEDS_COMMIT
bit to update->flags, which keeps track of whether the corresponding
lockfile needs to be committed, as opposed to just unlocked. (Since
"struct ref_update" is internal to the refs module, this change is not
visible to external callers.)
This change fixes two tests in t1400.
Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
2015-04-24 14:35:49 +03:00
|
|
|
test_expect_success ULIMIT_FILE_DESCRIPTORS 'large transaction deleting branches does not burst open file limit' '
|
2015-04-15 01:25:06 +03:00
|
|
|
(
|
|
|
|
for i in $(test_seq 33)
|
|
|
|
do
|
|
|
|
echo "delete refs/heads/$i HEAD"
|
|
|
|
done >large_input &&
|
|
|
|
run_with_limited_open_files git update-ref --stdin <large_input &&
|
|
|
|
test_must_fail git rev-parse --verify -q refs/heads/33
|
|
|
|
)
|
|
|
|
'
|
|
|
|
|
2006-05-17 13:55:40 +04:00
|
|
|
test_done
|