git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* FEATURE REQUEST: Comment assignment on branches
@ 2010-03-15  8:33 Maxim Treskin
  2010-03-15 21:10 ` René Scharfe
  2010-03-15 21:32 ` Nicolas Sebrecht
  0 siblings, 2 replies; 13+ messages in thread
From: Maxim Treskin @ 2010-03-15  8:33 UTC (permalink / raw)
  To: git

Hello

Is it possible to add comments assignment to branches?
Something like:

$ git branch --comment="New branch with implementation of some features" br14

$ git branch
  br14
* master

$ git branch --comments
  br14           (New branch with implementation of some features)
* master

and when configuration variable branch.comments == true, this behavior
is default.

Thank you

-- 
Maxim Treskin

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15  8:33 FEATURE REQUEST: Comment assignment on branches Maxim Treskin
@ 2010-03-15 21:10 ` René Scharfe
  2010-03-15 21:32   ` Junio C Hamano
  2010-03-16 14:36   ` Nguyen Thai Ngoc Duy
  2010-03-15 21:32 ` Nicolas Sebrecht
  1 sibling, 2 replies; 13+ messages in thread
From: René Scharfe @ 2010-03-15 21:10 UTC (permalink / raw)
  To: Maxim Treskin; +Cc: git

Am 15.03.2010 09:33, schrieb Maxim Treskin:
> Hello
> 
> Is it possible to add comments assignment to branches?
> Something like:
> 
> $ git branch --comment="New branch with implementation of some features" br14
> 
> $ git branch
>   br14
> * master
> 
> $ git branch --comments
>   br14           (New branch with implementation of some features)
> * master
> 
> and when configuration variable branch.comments == true, this behavior
> is default.

Hmm.  You could name your branch
"br14/new-branch-with-implementation-of-some-features" instead of
"br14".  With command line completion you would only have to hit two
extra keys (slash tab) and could enjoy a meaningful branch name everywhere.

René

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15 21:10 ` René Scharfe
@ 2010-03-15 21:32   ` Junio C Hamano
  2010-03-16  0:35     ` Mark Lodato
  2010-03-16 14:36   ` Nguyen Thai Ngoc Duy
  1 sibling, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2010-03-15 21:32 UTC (permalink / raw)
  To: René Scharfe; +Cc: Maxim Treskin, git

René Scharfe <rene.scharfe@lsrfire.ath.cx> writes:

> Am 15.03.2010 09:33, schrieb Maxim Treskin:
>> Hello
>> 
>> Is it possible to add comments assignment to branches?
>> Something like:
>> 
>> $ git branch --comment="New branch with implementation of some features" br14
>> 
>> $ git branch
>>   br14
>> * master
>> 
>> $ git branch --comments
>>   br14           (New branch with implementation of some features)
>> * master
>> 
>> and when configuration variable branch.comments == true, this behavior
>> is default.
>
> Hmm.  You could name your branch
> "br14/new-branch-with-implementation-of-some-features" instead of
> "br14".  With command line completion you would only have to hit two
> extra keys (slash tab) and could enjoy a meaningful branch name everywhere.

Another thing to worry about is how "--comments" and "-v" would interact
with each other.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15  8:33 FEATURE REQUEST: Comment assignment on branches Maxim Treskin
  2010-03-15 21:10 ` René Scharfe
@ 2010-03-15 21:32 ` Nicolas Sebrecht
  2010-03-15 21:34   ` Junio C Hamano
  1 sibling, 1 reply; 13+ messages in thread
From: Nicolas Sebrecht @ 2010-03-15 21:32 UTC (permalink / raw)
  To: Maxim Treskin; +Cc: git, Nicolas Sebrecht

The 15/03/10, Maxim Treskin wrote:

> Is it possible to add comments assignment to branches?
> Something like:

Aren't you looking for 'git notes'?

-- 
Nicolas Sebrecht

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15 21:32 ` Nicolas Sebrecht
@ 2010-03-15 21:34   ` Junio C Hamano
  2010-03-16  4:45     ` Miles Bader
  0 siblings, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2010-03-15 21:34 UTC (permalink / raw)
  To: Nicolas Sebrecht; +Cc: Maxim Treskin, git

Nicolas Sebrecht <nicolas.s.dev@gmx.fr> writes:

> The 15/03/10, Maxim Treskin wrote:
>
>> Is it possible to add comments assignment to branches?
>> Something like:
>
> Aren't you looking for 'git notes'?

I don't think so.  Notes are fundamentally per-commit.

My understanding is that it is more like:

    [branch "frotz"]
    	comment = "This is to add frotz command to the system"

I do not have a fundamental objection to such a feature, but the
presentation needs to be well thought out.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15 21:32   ` Junio C Hamano
@ 2010-03-16  0:35     ` Mark Lodato
  2010-03-16  1:49       ` Junio C Hamano
  0 siblings, 1 reply; 13+ messages in thread
From: Mark Lodato @ 2010-03-16  0:35 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: René Scharfe, Maxim Treskin, git

On Mon, Mar 15, 2010 at 5:32 PM, Junio C Hamano <gitster@pobox.com> wrote:
>
> Another thing to worry about is how "--comments" and "-v" would interact
> with each other.

What about putting the comment on the line below, with the same level
of indentation?

  maint    8fcaca3 don't use default revision if a rev was specified
           (preparation for the next maintenance release)
* master   c24138b Merge branch 'sd/format-patch-to'
           (preparation for the next feature release)
  next     0ae494e Merge branch 'pb/log-first-parent-p-m' into next
           (semi-stable test branch for integration into master)

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-16  0:35     ` Mark Lodato
@ 2010-03-16  1:49       ` Junio C Hamano
  0 siblings, 0 replies; 13+ messages in thread
From: Junio C Hamano @ 2010-03-16  1:49 UTC (permalink / raw)
  To: Mark Lodato; +Cc: René Scharfe, Maxim Treskin, git

Mark Lodato <lodatom@gmail.com> writes:

> On Mon, Mar 15, 2010 at 5:32 PM, Junio C Hamano <gitster@pobox.com> wrote:
>>
>> Another thing to worry about is how "--comments" and "-v" would interact
>> with each other.
>
> What about putting the comment on the line below, with the same level
> of indentation?

That, or in an order that is the other way around.

I think eventually people realize that "branch -v" and "tag -l -n $n" are
similar and start making noises about "branch -v -n $n" to internally run
"log --oneline".  To prepare for such a feature creep, comment first then
commit would probably be a better idea, but I dunno.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15 21:34   ` Junio C Hamano
@ 2010-03-16  4:45     ` Miles Bader
  2010-03-16  6:03       ` Junio C Hamano
  0 siblings, 1 reply; 13+ messages in thread
From: Miles Bader @ 2010-03-16  4:45 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Nicolas Sebrecht, Maxim Treskin, git

Junio C Hamano <gitster@pobox.com> writes:
> My understanding is that it is more like:
>
>     [branch "frotz"]
>     	comment = "This is to add frotz command to the system"
>
> I do not have a fundamental objection to such a feature, but the
> presentation needs to be well thought out.

This would seem especially useful for publicly visible branches...

-Miles

-- 
Absurdity, n. A statement or belief manifestly inconsistent with one's own
opinion.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-16  4:45     ` Miles Bader
@ 2010-03-16  6:03       ` Junio C Hamano
  2010-03-16  6:20         ` Miles Bader
  0 siblings, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2010-03-16  6:03 UTC (permalink / raw)
  To: Miles Bader; +Cc: Nicolas Sebrecht, Maxim Treskin, git

Miles Bader <miles@gnu.org> writes:

> Junio C Hamano <gitster@pobox.com> writes:
>> My understanding is that it is more like:
>>
>>     [branch "frotz"]
>>     	comment = "This is to add frotz command to the system"
>>
>> I do not have a fundamental objection to such a feature, but the
>> presentation needs to be well thought out.
>
> This would seem especially useful for publicly visible branches...

If you mean by "publicly visible" branches in public repositories, I
suspect not.  At places like repo.or.cz, github, or installations managed
by gitosis, you typically do not have direct access to $GIT_DIR/config
files (they belong to site administrators) in your repositories, and that
is not likely to change for security reasons.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-16  6:03       ` Junio C Hamano
@ 2010-03-16  6:20         ` Miles Bader
  2010-03-16  6:26           ` Sverre Rabbelier
  0 siblings, 1 reply; 13+ messages in thread
From: Miles Bader @ 2010-03-16  6:20 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Nicolas Sebrecht, Maxim Treskin, git

On Tue, Mar 16, 2010 at 3:03 PM, Junio C Hamano <gitster@pobox.com> wrote:
>> This would seem especially useful for publicly visible branches...
>
> If you mean by "publicly visible" branches in public repositories, I
> suspect not.  At places like repo.or.cz, github, or installations managed
> by gitosis, you typically do not have direct access to $GIT_DIR/config
> files (they belong to site administrators) in your repositories, and that
> is not likely to change for security reasons.

Hm, ok you're right... which is a shame... :(

'cause a "pushable branch description" would be ace!

[I take it git has no "limited remote metadata change" function...?]

-Miles

-- 
Do not taunt Happy Fun Ball.

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-16  6:20         ` Miles Bader
@ 2010-03-16  6:26           ` Sverre Rabbelier
  2010-03-16  7:33             ` Matthieu Moy
  0 siblings, 1 reply; 13+ messages in thread
From: Sverre Rabbelier @ 2010-03-16  6:26 UTC (permalink / raw)
  To: Miles Bader, Junio C Hamano; +Cc: Nicolas Sebrecht, Maxim Treskin, git

Heya,

On Tue, Mar 16, 2010 at 07:20, Miles Bader <miles@gnu.org> wrote:
> On Tue, Mar 16, 2010 at 3:03 PM, Junio C Hamano <gitster@pobox.com> wrote:
>> If you mean by "publicly visible" branches in public repositories, I
>> suspect not.  At places like repo.or.cz, github, or installations managed
>> by gitosis, you typically do not have direct access to $GIT_DIR/config
>> files (they belong to site administrators) in your repositories, and that
>> is not likely to change for security reasons.
>
> Hm, ok you're right... which is a shame... :(

Which is also why I think it would be nice if we could teach notes to
annotate branches/tags/the whole shaboodle. I really think that a
generic way to annotate something under /refs/ would be useful.

-- 
Cheers,

Sverre Rabbelier

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-16  6:26           ` Sverre Rabbelier
@ 2010-03-16  7:33             ` Matthieu Moy
  0 siblings, 0 replies; 13+ messages in thread
From: Matthieu Moy @ 2010-03-16  7:33 UTC (permalink / raw)
  To: Sverre Rabbelier
  Cc: Miles Bader, Junio C Hamano, Nicolas Sebrecht, Maxim Treskin, git

Sverre Rabbelier <srabbelier@gmail.com> writes:

> Which is also why I think it would be nice if we could teach notes to
> annotate branches/tags/the whole shaboodle. I really think that a
> generic way to annotate something under /refs/ would be useful.

meetoo.

Today, we have .git/description which is used by gitweb (and others?)
to say what the repository contains. That would be nice to have the
same thing for branches. Take
http://git.kernel.org/?p=git/git.git;a=summary for example: commits
have a comment, tag have a comment, and branches just say:

10 hours ago 	man 	shortlog | log | tree
10 hours ago 	html 	shortlog | log | tree
22 hours ago 	pu 	shortlog | log | tree
22 hours ago 	next 	shortlog | log | tree
23 hours ago 	master 	shortlog | log | tree
2 days ago 	maint 	shortlog | log | tree
5 days ago 	todo 	shortlog | log | tree

A newbie will have a hard time understanding what "pu" means, while a
one-liner saying (proposed update, may be rewound at any time) would
give most of the required information. Actually, ideally, this could
be a message like commit message (i.e. one-liner, blank line, and
body).

(My 2 cents)

--
Matthieu Moy
http://www-verimag.imag.fr/~moy/

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

* Re: FEATURE REQUEST: Comment assignment on branches
  2010-03-15 21:10 ` René Scharfe
  2010-03-15 21:32   ` Junio C Hamano
@ 2010-03-16 14:36   ` Nguyen Thai Ngoc Duy
  1 sibling, 0 replies; 13+ messages in thread
From: Nguyen Thai Ngoc Duy @ 2010-03-16 14:36 UTC (permalink / raw)
  To: René Scharfe; +Cc: Maxim Treskin, git

On 3/16/10, René Scharfe <rene.scharfe@lsrfire.ath.cx> wrote:
> Am 15.03.2010 09:33, schrieb Maxim Treskin:
>
> > Hello
>  >
>  > Is it possible to add comments assignment to branches?
>  > Something like:
>  >
>  > $ git branch --comment="New branch with implementation of some features" br14
>  >
>  > $ git branch
>  >   br14
>  > * master
>  >
>  > $ git branch --comments
>  >   br14           (New branch with implementation of some features)
>  > * master
>  >
>  > and when configuration variable branch.comments == true, this behavior
>  > is default.
>
>
> Hmm.  You could name your branch
>  "br14/new-branch-with-implementation-of-some-features" instead of
>  "br14".  With command line completion you would only have to hit two
>  extra keys (slash tab) and could enjoy a meaningful branch name everywhere.

If only completion works across shells. Another idea: put notes in a
blob, tagged with "notes/branchname" or another convention.

Then if you want to see description of branch "br14", do "git show
notes/br14". Teaching "git branch" to show it is easy.
-- 
Duy

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

end of thread, other threads:[~2010-03-16 14:44 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-15  8:33 FEATURE REQUEST: Comment assignment on branches Maxim Treskin
2010-03-15 21:10 ` René Scharfe
2010-03-15 21:32   ` Junio C Hamano
2010-03-16  0:35     ` Mark Lodato
2010-03-16  1:49       ` Junio C Hamano
2010-03-16 14:36   ` Nguyen Thai Ngoc Duy
2010-03-15 21:32 ` Nicolas Sebrecht
2010-03-15 21:34   ` Junio C Hamano
2010-03-16  4:45     ` Miles Bader
2010-03-16  6:03       ` Junio C Hamano
2010-03-16  6:20         ` Miles Bader
2010-03-16  6:26           ` Sverre Rabbelier
2010-03-16  7:33             ` Matthieu Moy

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).