Skip to content

Commit e5ce5b0

Browse files
committed
t5604: do not expect that HEAD can be a valid tagname
09116a1 (refs: loosen over-strict "format" check, 2011-11-16) introduced a test piece (originally in t5700) that expects to be able to create a tag named "HEAD" and then a local clone using the repository as its own reference works correctly. Later, another test piece started using this tag starting at acede2e (t5700: document a failure of alternates to affect fetch, 2012-02-11). But the breakage 09116a1 fixed was not specific to the tagname HEAD. It would have failed exactly the same way if the tag used were foo instead of HEAD. Before forbidding "git tag" from creating "refs/tags/HEAD", update these tests to use 'foo', not 'HEAD', as the name of the test tag. Note that the test piece that uses the tag learned the value of the tag in unnecessarily inefficient and convoluted way with for-each-ref. Just use "rev-parse" instead. Signed-off-by: Junio C Hamano <[email protected]>
1 parent 93e5e04 commit e5ce5b0

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

t/t5604-clone-reference.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ test_expect_success 'cloning with multiple references drops duplicates' '
131131

132132
test_expect_success 'clone with reference from a tagged repository' '
133133
(
134-
cd A && git tag -a -m tagged HEAD
134+
cd A && git tag -a -m tagged foo
135135
) &&
136136
git clone --reference=A A I
137137
'
@@ -156,10 +156,10 @@ test_expect_success 'fetch with incomplete alternates' '
156156
git remote add J "file://$base_dir/J" &&
157157
GIT_TRACE_PACKET=$U.K git fetch J
158158
) &&
159-
main_object=$(cd A && git for-each-ref --format="%(objectname)" refs/heads/main) &&
159+
main_object=$(git -C A rev-parse --verify refs/heads/main) &&
160160
test -s "$U.K" &&
161161
! grep " want $main_object" "$U.K" &&
162-
tag_object=$(cd A && git for-each-ref --format="%(objectname)" refs/tags/HEAD) &&
162+
tag_object=$(git -C A rev-parse --verify refs/tags/foo) &&
163163
! grep " want $tag_object" "$U.K"
164164
'
165165

0 commit comments

Comments
 (0)