# [PATCH] Check that a tag exists using show-ref instead of looking for the ref file.

4 messages from 2006-10-01 to 2006-10-03. Participants: Christian Couder, Junio C Hamano.
Thread: https://gitlist.dev/t/5782

## Christian Couder, 2006-10-01 20:33

Subject: [PATCH] Check that a tag exists using show-ref instead of looking for the ref file.
Message-ID: <20061001223304.d37df36f.chriscool@tuxfamily.org>
URL: https://gitlist.dev/e/20061001223304.d37df36f.chriscool%40tuxfamily.org

```
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
---
 git-tag.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-tag.sh b/git-tag.sh
index 6463b31..a3f1819 100755
--- a/git-tag.sh
+++ b/git-tag.sh
@@ -66,7 +66,7 @@ done
 name="$1"
 [ "$name" ] || usage
 prev=0000000000000000000000000000000000000000
-if test -e "$GIT_DIR/refs/tags/$name"
+if git-show-ref --verify --quiet -- "refs/tags/$name"
 then
     test -n "$force" || die "tag '$name' already exists"
     prev=`git rev-parse "refs/tags/$name"`
-- 
1.4.2.1.gd567

```

## Junio C Hamano, 2006-10-03 09:09

Subject: Re: [PATCH] Check that a tag exists using show-ref instead of looking for the ref file.
Message-ID: <7vlknxrcja.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vlknxrcja.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <20061001223304.d37df36f.chriscool@tuxfamily.org>

```
Thanks.  We also need this to avoid refetching already packed
tags.

-- >8 --
[PATCH] git-fetch: do not look into $GIT_DIR/refs to see if a tag exists.

Signed-off-by: Junio C Hamano <junkio@cox.net>
---
 git-fetch.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/git-fetch.sh b/git-fetch.sh
index f1522bd..e8a7668 100755
--- a/git-fetch.sh
+++ b/git-fetch.sh
@@ -417,7 +417,7 @@ case "$no_tags$tags" in
 		sed -ne 's|^\([0-9a-f]*\)[ 	]\(refs/tags/.*\)^{}$|\1 \2|p' |
 		while read sha1 name
 		do
-			test -f "$GIT_DIR/$name" && continue
+			git-show-ref --verify --quiet -- $name && continue
 			git-check-ref-format "$name" || {
 				echo >&2 "warning: tag ${name} ignored"
 				continue
-- 
1.4.2.3.gd1e9e

```

## Christian Couder, 2006-10-03 20:11

Subject: Re: [PATCH] Check that a tag exists using show-ref instead of looking for the ref file.
Message-ID: <200610032211.05847.chriscool@tuxfamily.org>
URL: https://gitlist.dev/e/200610032211.05847.chriscool%40tuxfamily.org
In-Reply-To: <7vlknxrcja.fsf@assigned-by-dhcp.cox.net>

```
Junio C Hamano wrote:
> Thanks.  We also need this to avoid refetching already packed
> tags.

Yeah, I did a very quick check on the shell scripts and I obviously missed 
some places that need updates.

In fact, I couldn't resist to start implementing the deleted-refs/$frotz~ref 
stuff we talked about...

Christian.

```

## Junio C Hamano, 2006-10-03 20:26

Subject: Re: [PATCH] Check that a tag exists using show-ref instead of looking for the ref file.
Message-ID: <7virj1m9hl.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7virj1m9hl.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <200610032211.05847.chriscool@tuxfamily.org>

```
Christian Couder <chriscool@tuxfamily.org> writes:

> Junio C Hamano wrote:
>> Thanks.  We also need this to avoid refetching already packed
>> tags.
>
> Yeah, I did a very quick check on the shell scripts and I obviously missed 
> some places that need updates.
>
> In fact, I couldn't resist to start implementing the deleted-refs/$frotz~ref 
> stuff we talked about...

How would that help?  We need to audit the existing scripts
since they would not look under $GIT_DIR/deleted-refs currently
anyway.

```