Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-18 Thread Beat Bolli
On 18.07.15 14:23, Paul Mackerras wrote: > On Fri, Jul 17, 2015 at 08:30:24AM -0700, Junio C Hamano wrote: >> Paul Mackerras writes: >> >>> We have an item in the preferences menu to control the SHA1 length >>> that is automatically selected when going to a new commit. It's >>> stored in the vari

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-18 Thread Paul Mackerras
On Fri, Jul 17, 2015 at 08:30:24AM -0700, Junio C Hamano wrote: > Paul Mackerras writes: > > > We have an item in the preferences menu to control the SHA1 length > > that is automatically selected when going to a new commit. It's > > stored in the variable $autosellen. That seems like it would

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-17 Thread Junio C Hamano
Paul Mackerras writes: > We have an item in the preferences menu to control the SHA1 length > that is automatically selected when going to a new commit. It's > stored in the variable $autosellen. That seems like it would be a > reasonable choice for the SHA1 length to use here. Reusing a confi

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-17 Thread Paul Mackerras
On Thu, Jul 16, 2015 at 05:29:25PM +0200, Beat Bolli wrote: > When referring to earlier commits in commit messages or other text, one > of the established formats is > > ("", ) > > Add a "Copy commit summary" command to the context menu that puts this > text for the currently selected commit

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-17 Thread Beat Bolli
On 2015-07-17 10:50, li...@haller-berlin.de wrote: Junio C Hamano wrote: Beat Bolli writes: > When referring to earlier commits in commit messages or other text, one > of the established formats is > > ("", ) > ... > +proc copysummary {} { > +global rowmenuid commitinfo > + > +

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-17 Thread Stefan Haller
Junio C Hamano wrote: > Beat Bolli writes: > > > When referring to earlier commits in commit messages or other text, one > > of the established formats is > > > > ("", ) > > ... > > +proc copysummary {} { > > +global rowmenuid commitinfo > > + > > +set id [string range $rowmenuid 0

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-16 Thread Johannes Sixt
Am 16.07.2015 um 17:29 schrieb Beat Bolli: When referring to earlier commits in commit messages or other text, one of the established formats is ("", ) Add a "Copy commit summary" command to the context menu that puts this text for the currently selected commit on the clipboard. This make

Re: [PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-16 Thread Junio C Hamano
Beat Bolli writes: > When referring to earlier commits in commit messages or other text, one > of the established formats is > > ("", ) > ... > +proc copysummary {} { > +global rowmenuid commitinfo > + > +set id [string range $rowmenuid 0 7] > +set info $commitinfo($rowmenuid) >

[PATCH v2] gitk: Add a "Copy commit summary" command

2015-07-16 Thread Beat Bolli
When referring to earlier commits in commit messages or other text, one of the established formats is ("", ) Add a "Copy commit summary" command to the context menu that puts this text for the currently selected commit on the clipboard. This makes it easy for our users to create well-formatt