Re: gitk drawing bug

2015-04-05 Thread Paul Mackerras
On Fri, Apr 03, 2015 at 09:28:00PM -0400, Martin d'Anjou wrote:
 On 15-04-03 07:05 PM, Alex Henrie wrote:
 2015-02-18 12:27 GMT-07:00 Martin d'Anjou martin.danjo...@gmail.com:
 It appears I have uncovered inconsistent behaviour in gitk. Looks like
 a bug. I have a picture here:
 https://docs.google.com/document/d/19TTzGD94B9EEIrVU5mRMjfJFvF5Ar3MlPblRJfP5OdQ/edit?usp=sharing
 
 Essentially, when I hit shift-F5, it sometimes draw the history
 differently (still valid, but drawn differently). There is no change
 in the repository between the shift-F5 keystrokes.

That's not a bug, it's a consequence of the fact that gitk is designed
to be fast.  It only lays out as much of the graph is visible plus a
little more, not the whole graph, and it doesn't use any global
analysis.  The reason for that is speed.  Gitk is usable on a
repository with half a million commits, such as the linux kernel, and
to achieve that we can't afford to do wait until we have all the
commits read in and then do some computation over the whole topology;
it all has to be done incrementally.  Also, the underlying git log
sometimes gives gitk a parent commit before one of its children, and
when that happens the topology has to be modified and thus the graph
does too, if any topology that has already been drawn gets modified.

As long as the graph correctly shows the relationships between
commits, it has achieved its purpose.  If you (or anyone) can come up
with improvements that make it look nicer, that's great, and I'll
consider them as long as they don't slow down gitk on large
repositories to any noticeable extent.

Paul.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: gitk drawing bug

2015-04-03 Thread Martin d'Anjou

On 15-04-03 07:05 PM, Alex Henrie wrote:

2015-02-18 12:27 GMT-07:00 Martin d'Anjou martin.danjo...@gmail.com:

It appears I have uncovered inconsistent behaviour in gitk. Looks like
a bug. I have a picture here:
https://docs.google.com/document/d/19TTzGD94B9EEIrVU5mRMjfJFvF5Ar3MlPblRJfP5OdQ/edit?usp=sharing

Essentially, when I hit shift-F5, it sometimes draw the history
differently (still valid, but drawn differently). There is no change
in the repository between the shift-F5 keystrokes.

Did you ever contact the gitk maintainer, Paul Mackerras
pau...@samba.org, about this bug?

-Alex

I do not remember contacting Paul Mackerras, but thank you for doing so.

Martin

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: gitk drawing bug

2015-04-03 Thread Alex Henrie
2015-02-18 12:27 GMT-07:00 Martin d'Anjou martin.danjo...@gmail.com:
 It appears I have uncovered inconsistent behaviour in gitk. Looks like
 a bug. I have a picture here:
 https://docs.google.com/document/d/19TTzGD94B9EEIrVU5mRMjfJFvF5Ar3MlPblRJfP5OdQ/edit?usp=sharing

 Essentially, when I hit shift-F5, it sometimes draw the history
 differently (still valid, but drawn differently). There is no change
 in the repository between the shift-F5 keystrokes.

Did you ever contact the gitk maintainer, Paul Mackerras
pau...@samba.org, about this bug?

-Alex
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


gitk drawing bug

2015-02-18 Thread Martin d'Anjou
Hi,

It appears I have uncovered inconsistent behaviour in gitk. Looks like
a bug. I have a picture here:
https://docs.google.com/document/d/19TTzGD94B9EEIrVU5mRMjfJFvF5Ar3MlPblRJfP5OdQ/edit?usp=sharing

Essentially, when I hit shift-F5, it sometimes draw the history
differently (still valid, but drawn differently). There is no change
in the repository between the shift-F5 keystrokes.

Thanks,
Martin
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html