git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Match lightweight tags too in bash prompt
@ 2010-11-19 15:43 knittl
  2010-11-30 18:04 ` Shawn Pearce
  0 siblings, 1 reply; 4+ messages in thread
From: knittl @ 2010-11-19 15:43 UTC (permalink / raw)
  To: git

>From 1fea30e0167681917b4f40a3cb2cde9a90706011 Mon Sep 17 00:00:00 2001
From: Daniel Knittl-Frank <knittl89+git@googlemail.com>
Date: Fri, 19 Nov 2010 16:40:02 +0100
Subject: [PATCH] Match lightweight tags too in bash prompt

provide `--tags` to git describe
---

bash prompt would display a commit's sha1, when checking out a lightweight tag.
provide --tags to git describe in the completion script, so it will
display the name
of the tag (as it already does for annotated tags)

 contrib/completion/git-completion.bash |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/completion/git-completion.bash
b/contrib/completion/git-completion.bash
index f710469..604fa79 100755
--- a/contrib/completion/git-completion.bash
+++ b/contrib/completion/git-completion.bash
@@ -261,7 +261,7 @@ __git_ps1 ()
 				(describe)
 					git describe HEAD ;;
 				(* | default)
-					git describe --exact-match HEAD ;;
+					git describe --tags --exact-match HEAD ;;
 				esac 2>/dev/null)" ||

 				b="$(cut -c1-7 "$g/HEAD" 2>/dev/null)..." ||
-- 
1.7.3.2.334.gd1031


-- 
typed with http://neo-layout.org
myFtPhp -- visit http://myftphp.sf.net -- v. 0.4.7 released!

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] Match lightweight tags too in bash prompt
  2010-11-19 15:43 [PATCH] Match lightweight tags too in bash prompt knittl
@ 2010-11-30 18:04 ` Shawn Pearce
  2010-11-30 23:16   ` knittl
  0 siblings, 1 reply; 4+ messages in thread
From: Shawn Pearce @ 2010-11-30 18:04 UTC (permalink / raw)
  To: knittl; +Cc: git

On Fri, Nov 19, 2010 at 7:43 AM, knittl <knittl89@googlemail.com> wrote:
> From 1fea30e0167681917b4f40a3cb2cde9a90706011 Mon Sep 17 00:00:00 2001
> From: Daniel Knittl-Frank <knittl89+git@googlemail.com>
> Date: Fri, 19 Nov 2010 16:40:02 +0100
> Subject: [PATCH] Match lightweight tags too in bash prompt
>
> provide `--tags` to git describe

Missing signed-off-by-line.

> ---
>
> bash prompt would display a commit's sha1, when checking out a lightweight tag.
> provide --tags to git describe in the completion script, so it will
> display the name
> of the tag (as it already does for annotated tags)

This information is useful justification for the change.  It should
therefore appear in the commit message, above the --- line.

-- 
Shawn.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Match lightweight tags too in bash prompt
  2010-11-30 18:04 ` Shawn Pearce
@ 2010-11-30 23:16   ` knittl
  2010-12-01 13:08     ` Thiago Farina
  0 siblings, 1 reply; 4+ messages in thread
From: knittl @ 2010-11-30 23:16 UTC (permalink / raw)
  To: Shawn Pearce; +Cc: git

On Tue, Nov 30, 2010 at 7:04 PM, Shawn Pearce <spearce@spearce.org> wrote:
> On Fri, Nov 19, 2010 at 7:43 AM, knittl <knittl89@googlemail.com> wrote:
>> From 1fea30e0167681917b4f40a3cb2cde9a90706011 Mon Sep 17 00:00:00 2001
>> From: Daniel Knittl-Frank <knittl89+git@googlemail.com>
>> Date: Fri, 19 Nov 2010 16:40:02 +0100
>> Subject: [PATCH] Match lightweight tags too in bash prompt
>>
>> provide `--tags` to git describe
>
> Missing signed-off-by-line.

i mentioned the s-o-b line already in my forward-mail ;)

>> ---
>>
>> bash prompt would display a commit's sha1, when checking out a lightweight tag.
>> provide --tags to git describe in the completion script, so it will
>> display the name
>> of the tag (as it already does for annotated tags)
>
> This information is useful justification for the change.  It should
> therefore appear in the commit message, above the --- line.

ok, i'll resend v2 tomorrow morning to you and the mailing list


cheers, daniel

-- 
typed with http://neo-layout.org
myFtPhp -- visit http://myftphp.sf.net -- v. 0.4.7 released!

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Match lightweight tags too in bash prompt
  2010-11-30 23:16   ` knittl
@ 2010-12-01 13:08     ` Thiago Farina
  0 siblings, 0 replies; 4+ messages in thread
From: Thiago Farina @ 2010-12-01 13:08 UTC (permalink / raw)
  To: knittl; +Cc: Shawn Pearce, git

On Tue, Nov 30, 2010 at 9:16 PM, knittl <knittl89@googlemail.com> wrote:
> On Tue, Nov 30, 2010 at 7:04 PM, Shawn Pearce <spearce@spearce.org> wrote:
>> On Fri, Nov 19, 2010 at 7:43 AM, knittl <knittl89@googlemail.com> wrote:
>>> From 1fea30e0167681917b4f40a3cb2cde9a90706011 Mon Sep 17 00:00:00 2001
>>> From: Daniel Knittl-Frank <knittl89+git@googlemail.com>
>>> Date: Fri, 19 Nov 2010 16:40:02 +0100
>>> Subject: [PATCH] Match lightweight tags too in bash prompt
>>>
>>> provide `--tags` to git describe
>>
>> Missing signed-off-by-line.
>
> i mentioned the s-o-b line already in my forward-mail ;)
>

I don't see it. You mean the From: line? If yes, that is not the
convention used here.

You have to add a line:
Signed-off-by: Your Name <you@example.com>

Take a look at Documentation/SubmittingPatches

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-12-01 13:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-11-19 15:43 [PATCH] Match lightweight tags too in bash prompt knittl
2010-11-30 18:04 ` Shawn Pearce
2010-11-30 23:16   ` knittl
2010-12-01 13:08     ` Thiago Farina

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).