Re: [PATCH 2/3] SubmittingPatches: mention subsystems with dedicated repositories

2013-01-01 Thread Jason Holden
On Tue, Jan 01, 2013 at 03:24:54PM -0800, Junio C Hamano wrote: > > > +Subsystems with dedicated maintainers > + > +Some parts of the system have dedicated maintainers with their own > +repositories. > + > + - git-gui/ comes from git-gui project,

Bug in latest gitk - can't click lines connecting commits

2013-01-01 Thread Jason Holden
I was testing some patches against the latest gitk, and noticed that when I click the mouse on the lines that connect the commits in the history graph, I get an error popup with: Error: can't read "cflist_top": no such variable Looks like this was introduced in gitk commit b967135d89e8d8461d059

Re: [RFC/PATCH] gitk: Visualize a merge commit with a right-click in gitk

2012-12-31 Thread Jason Holden
On Mon, Dec 31, 2012 at 03:27:36PM +1100, Paul Mackerras wrote: > > Thanks for the patch. I have a couple of comments about it. First, > the exec command waits for the process to complete, which means that > the initial gitk GUI will be unresponsive until the user quits the > gitk window showing

Re: [PATCH 0/2] Add MAINTAINERS file and clarify gui workflows

2012-12-31 Thread Jason Holden
On Mon, Dec 31, 2012 at 09:40:19AM +, Thomas Ackermann wrote: > Junio C Hamano pobox.com> writes: > > > > > Thanks; I just realized that nothing in Documentation/ hierarchy > > mentions these; they are only mentioned in "A Note from the > > Maintainer" I send out every once in a while (kept

[PATCH 0/2] Add MAINTAINERS file and clarify gui workflows

2012-12-30 Thread Jason Holden
uld be observed. Jason Holden (2): Add top-level maintainers file with email/canonical repository information Provide better guidance for submitting patches against git-gui, gitk Documentation/SubmittingPatches | 11 +++ MAINTAINERS | 17 + 2 fi

[RFC/PATCH] gitk: Visualize a merge commit with a right-click in gitk

2012-12-29 Thread Jason Holden
lick on a merge commit and bring up a new gitk window displaying only those commits involved in the merge. When right-clicking on a non-merge commit, this option is grayed out. This patch also supports correct visualization of octopus merges Signed-off-by: Jason Holden