# [PATCH] tag: --force is quiet about new tags 4 messages from 2013-03-12 to 2013-03-13. Participants: Phil Hord, Junio C Hamano, Michael J Gruber. Thread: https://gitlist.dev/t/33163 ## Phil Hord, 2013-03-12 23:13 Subject: [PATCH] tag: --force is quiet about new tags Message-ID: <1363130021-28624-1-git-send-email-hordp@cisco.com> URL: https://gitlist.dev/e/1363130021-28624-1-git-send-email-hordp%40cisco.com ``` git tag --force is used to replace an existing tag with a new reference. Git helpfully tells the user the old ref when this happens. But if the tag name is new and does not exist, git tells the user the old ref anyway (000000). Teach git to ignore --force if the tag is new. Add a test for this and also to ensure --force can replace tags at all. Signed-off-by: Phil Hord --- builtin/tag.c | 2 +- t/t7004-tag.sh | 12 ++++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/builtin/tag.c b/builtin/tag.c index f826688..af3af3f 100644 --- a/builtin/tag.c +++ b/builtin/tag.c @@ -582,7 +582,7 @@ int cmd_tag(int argc, const char **argv, const char *prefix) die(_("%s: cannot lock the ref"), ref.buf); if (write_ref_sha1(lock, object, NULL) < 0) die(_("%s: cannot update the ref"), ref.buf); - if (force && hashcmp(prev, object)) + if (force && !is_null_sha1(prev) && hashcmp(prev, object)) printf(_("Updated tag '%s' (was %s)\n"), tag, find_unique_abbrev(prev, DEFAULT_ABBREV)); strbuf_release(&buf); diff --git a/t/t7004-tag.sh b/t/t7004-tag.sh index f5a79b1..c8d6e9f 100755 --- a/t/t7004-tag.sh +++ b/t/t7004-tag.sh @@ -104,6 +104,18 @@ test_expect_success 'creating a tag using HEAD directly should succeed' ' tag_exists myhead ' +test_expect_success '--force can create a tag with the name of one existing' ' + tag_exists mytag && + git tag --force mytag && + tag_exists mytag' + +test_expect_success '--force is moot with a non-existing tag name' ' + git tag newtag >expect && + git tag --force forcetag >actual && + test_cmp expect actual +' +git tag -d newtag forcetag + # deleting tags: test_expect_success 'trying to delete an unknown tag should fail' ' -- 1.8.2.rc3.394.g2617418.dirty ``` ## Junio C Hamano, 2013-03-13 03:33 Subject: Re: [PATCH] tag: --force is quiet about new tags Message-ID: <7vy5ds3re0.fsf@alter.siamese.dyndns.org> URL: https://gitlist.dev/e/7vy5ds3re0.fsf%40alter.siamese.dyndns.org In-Reply-To: <1363130021-28624-1-git-send-email-hordp@cisco.com> ``` Phil Hord writes: > git tag --force is used to replace an existing tag with > a new reference. Git helpfully tells the user the old > ref when this happens. But if the tag name is new and does > not exist, git tells the user the old ref anyway (000000). > > Teach git to ignore --force if the tag is new. Add a test > for this and also to ensure --force can replace tags at all. > > Signed-off-by: Phil Hord > --- I think we would still want to allow the operation to go through, even when the --force option is given, to create a new tag. I agree that the message should not say "Updated". So teaching Git not to issue the "Updated" message makes perfect sense. It is somewhat misleading to say we are teaching Git to ignore the option, though. Thanks. ``` ## Phil Hord, 2013-03-13 04:21 Subject: Re: [PATCH] tag: --force is quiet about new tags Message-ID: URL: https://gitlist.dev/e/CABURp0oT8ZN5fKtKh67Gm1P5rf_a3diqcYbw8mUga-gv5MPu6Q%40mail.gmail.com In-Reply-To: <7vy5ds3re0.fsf@alter.siamese.dyndns.org> ``` On Tue, Mar 12, 2013 at 11:33 PM, Junio C Hamano wrote: > Phil Hord writes: > >> git tag --force is used to replace an existing tag with >> a new reference. Git helpfully tells the user the old >> ref when this happens. But if the tag name is new and does >> not exist, git tells the user the old ref anyway (000000). >> >> Teach git to ignore --force if the tag is new. Add a test >> for this and also to ensure --force can replace tags at all. >> >> Signed-off-by: Phil Hord >> --- > > I think we would still want to allow the operation to go through, > even when the --force option is given, to create a new tag. I agree > that the message should not say "Updated". So teaching Git not to > issue the "Updated" message makes perfect sense. It is somewhat > misleading to say we are teaching Git to ignore the option, though. > > Thanks. My phrasing was too ambiguous. What you described is exactly what the patch does. --force is superfluous when the tag does not already exist. It is only checked in two places, and one of those is to decide whether to print the "Updated" message. How's this? Teach 'git tag --force' to suppress the update message if the tag is new. Add a test for this and also to ensure --force can replace tags at all. Phil ``` ## Michael J Gruber, 2013-03-13 09:16 Subject: Re: [PATCH] tag: --force is quiet about new tags Message-ID: <51404404.4030005@drmicha.warpmail.net> URL: https://gitlist.dev/e/51404404.4030005%40drmicha.warpmail.net In-Reply-To: ``` Phil Hord venit, vidit, dixit 13.03.2013 05:21: > On Tue, Mar 12, 2013 at 11:33 PM, Junio C Hamano wrote: >> Phil Hord writes: >> >>> git tag --force is used to replace an existing tag with >>> a new reference. Git helpfully tells the user the old >>> ref when this happens. But if the tag name is new and does >>> not exist, git tells the user the old ref anyway (000000). >>> >>> Teach git to ignore --force if the tag is new. Add a test >>> for this and also to ensure --force can replace tags at all. >>> >>> Signed-off-by: Phil Hord >>> --- >> >> I think we would still want to allow the operation to go through, >> even when the --force option is given, to create a new tag. I agree >> that the message should not say "Updated". So teaching Git not to >> issue the "Updated" message makes perfect sense. It is somewhat >> misleading to say we are teaching Git to ignore the option, though. >> >> Thanks. > > My phrasing was too ambiguous. What you described is exactly what the > patch does. --force is superfluous when the tag does not already > exist. It is only checked in two places, and one of those is to > decide whether to print the "Updated" message. How's this? > > Teach 'git tag --force' to suppress the update message if > the tag is new. Add a test for this and also to ensure > --force can replace tags at all. > > Phil Looks good to me, both the patch and the (updated) commit message. Michael ```