Commit graph

69 commits

Author SHA1 Message Date
Paul Mackerras a69875318b Change wish8.4 back to wish
Checking in the change from wish to wish8.4 was a mistake; I had
changed it for a test but forgot to change it back before checking
in a patch.
2005-09-26 10:22:43 +10:00
Robert Suetterlin 6e2dda35c2 [PATCH] Add new keybindings
This adds several new keybindings to allow history and selectline
navigation.  I basically added Opera-like history traversal, as well
as left-right-cursor history traversal and vi-like motion commands.

Signed-off-by: Robert Suetterlin <robert@mpe.mpg.de>
Signed-off-by: Paul Mackerras <paulus@samba.org>
2005-09-22 10:07:36 +10:00
Paul Mackerras 9843c3074d Draw selected graph line thicker and make arrowheads active. 2005-08-30 10:57:11 +10:00
Paul Mackerras 106288cb02 Display the contents of a tag when the user clicks on it.
This just displays the result of git-cat-file on the tag in the
details pane.  If the tag is a "direct" tag (the tag file contains
the SHA1 ID of a commit rather than a tag), we show the tag name
and SHA1 ID.
2005-08-19 23:11:39 +10:00
Paul Mackerras f1d83ba34c Added re-read refs command, and display all refs.
These are features requested by Junio.  Any plain file under .git/refs
whose contents start with 40 hex characters is taken as a reference
and displayed like a head but with a light blue background (unless it
is in .git/refs/tags or .git/refs/heads, in which case it is displayed
as before).  There is now a "Reread references" menu item in the File
menu which re-reads all the plain files under .git/refs and redisplays
any references that have changed.
2005-08-19 22:14:28 +10:00
Paul Mackerras 04c13d3877 Save the maxwidth setting in the ~/.gitk file. 2005-08-19 10:22:24 +10:00
Paul Mackerras 022bc2ac74 Fix a bug where commits with no children weren't marked as on-screen.
This problem was revealed by running gitk --all on Wolfgang Denk's
u-boot repository.
2005-08-19 10:22:04 +10:00
Paul Mackerras e5ea701b8e Use the --parents flag to git-rev-list.
With --parents, git-rev-list gives us the list of parents on the
first line of each commit.  We use that rather than looking for
the parent: lines in the commit body, since this way we get to
know about the grafts for free.
2005-08-18 20:40:39 +10:00
Paul Mackerras f6075ebadb Allow graph lines to jump through hyperspace.
When the graph gets too wide (as defined by the maxwidth variable,
which can be set in ~/.gitk), we can now terminate graph lines with
an arrow pointing downwards, and reintroduce them later with an
arrow pointing upwards when we need them.  This makes the graph much
less cluttered on large repositories such as the linux kernel.

Unfortunately this has made it slower; it takes about 10 seconds
user time on the linux-2.6 repository on my machine now, compared
to 6 seconds before.  I'll have to work on optimizing that.  Also
on the todo list are making the arrow heads active (so if you click
on them you jump to the other end) and improving the placement of
the null entry.
2005-08-18 09:30:10 +10:00
Paul Mackerras 244edd1241 Add graft support.
We read .git/info/grafts and use the information in there to
override the list of parents we get from git-rev-list or
git-cat-file.
2005-08-17 21:27:55 +10:00
Paul Mackerras b664550c06 Refine the update heuristic to improve responsiveness a bit.
The previous commit improved performance a lot but also meant that
we waited longer to see something drawn.  This refines the heuristics
for when to call update so that (1) when we have finished processing
a bufferfull of information from git-rev-list, we call update if
enough time has elapsed, regardless of how many commits we've drawn,
and (2) the number of commits drawn between updates scales with the
total number of commits drawn: 1 for 1-99 commits, 10 for 100-9999
commits, or 100 for >= 10000 commits.
2005-08-11 09:56:23 +10:00
Paul Mackerras 466e4fdd66 Only do an update every 100 commits when drawing the graph.
On a large repository with > 60,000 commits, each call to the Tk
update primitive (which gives Tk a chance to respond to events and
redraw the screen) was taking up to 0.2 seconds.  Because the logic
was to call update after drawing a commit if 0.1 seconds had passed
since the last update call, we were calling it for every commit,
which was slowing us down enormously.  Now we also require that we
have drawn 100 commits since the last update (as well as it being
at least 0.1 seconds since the last update).  Drawing 100 commits
takes around 0.1 - 0.2 seconds (even in this large repo) on my G5.
2005-08-10 22:50:28 +10:00
Linus Torvalds b1ba39e7e8 [PATCH] "Child" information in commit window - and cleanups
This adds "Child: " lines to the commit window, which tells what children
a commit has.

It also cleans things up: it marks the text widget as no-wrap, which means
that it doesn't need to truncate the commit description arbitrarily by
hand. Also, the description itself is now done by a common helper routine
that handles both the parent and the children.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
2005-08-09 21:09:27 +10:00
Paul Mackerras fa4da7b32e Better graph line details display and expand history coverage.
Now the history remembers when we have clicked on a graph line
and when we have asked for a diff between two commits, as well
as when we have displayed a commit.

The display when you click on a graph line now uses clickable
SHA1 IDs instead of the embedded "Go" buttons.  Also made the
IDs clickable in the header for a diff between two commits.
2005-08-08 09:47:22 +10:00
Linus Torvalds 8b19280930 [PATCH] gitk "parent information" in commit window
This adds a useful "Parent:" line to the git commit information window.

It looks something like this (from the infamous octopus merge):

	Author: Junio C Hamano <junkio@cox.net>  2005-05-05 16:16:54
	Committer: Junio C Hamano <junkio@cox.net>  2005-05-05 16:16:54
	Parent: fc54a9c30c  (Update git-apply-patch-script ...)
	Parent: 9e30dd7c0e  (Make git-prune-script executa ...)
	Parent: c4b83e618f  (Do not write out new index if ...)
	Parent: 660265909f  (diff-cache shows differences  ...)
	Parent: b28858bf65  (Update diff engine for symlin ...)

	    Octopus merge of the following five patches.

	      Update git-apply-patch-script for symbolic links.
	      Make git-prune-script executable again.
	      Do not write out new index if nothing has changed.
	      diff-cache shows differences for unmerged paths without --cache.
	      Update diff engine for symlinks stored in the cache.

	    Signed-off-by: Junio C Hamano <junkio@cox.net>

where all the parent commit ID's are clickable, because the new lines are
added as part of the "comment" string, and thus the regular clickability
thing will match them automatically.

I think this is good. And my random-tcl-monkey-skills are clearly getting
better (although it's perfectly possible that somebody who actually knows
what he is doing would have done things differently).

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
2005-08-08 08:40:24 +10:00
Paul Mackerras d6e8149bc5 Use lf translation rather than binary when reading commit data.
The effect of this is that it allows Tcl to do the locale-specific
conversion of the input data to its internal unicode representation.
That means that commit messages in Russian or other languages should
be displayed correctly now (according to the locale that is in effect.)
2005-08-07 20:01:24 +10:00
Paul Mackerras 94a2eede68 Change cursor to a hand cursor when over a SHA1 ID link.
This is based on suggestions by Jeff Epler and Linus Torvalds, but
extended so that we do the switching between the watch cursor and
the normal cursor correctly as well.

Also fixed a bug pointed out by Junio Hamano - I wasn't incrementing
the link number (duh!).
2005-08-07 15:27:57 +10:00
Paul Mackerras d698206c12 Add forward and back buttons and make SHA1 IDs clickable links.
When we display the commit message in the details pane, any string
of 40 [0-9a-f] characters that corresponds to a SHA1 ID that we
know about gets turned into a clickable link, and displayed in
blue and underlined.

We now keep a history of commits that we have looked at, and we
have forward and back buttons for moving within the history list.
2005-08-06 22:06:06 +10:00
Paul Mackerras 8d858d1a3a Compress the graph horizontally if it gets too wide.
If the graph gets to use more than a certain percentage (default 50%)
of the width of the top-left pane, we now reduce the amount of space
allowed for each graph line.  This means it doesn't look quite as
nice but you can still see the headline for the commit.  (Currently
the only way to customize the percentage is to edit your ~/.gitk
file manually.)
2005-08-05 09:52:16 +10:00
Paul Mackerras 1115fb38df Fix some bugs causing application error popups.
Specifically this should fix the following errors:

wrong # args: should be "startdiff ids" (fix from Junio Hamano)
can't read "filelines(....)": no such element in array
can't unset "treepending": no such variable
2005-07-31 21:35:21 +10:00
Paul Mackerras c8a4acbf4a Improve the merge display when the result differs from all parents.
Now we see if the result is quite similar to one of the parents, and
if it is, display the result as a diff from that parent.  If the result
is similar to more than one parent, pick the one that it's most
similar to.
2005-07-29 09:23:03 -05:00
Junio C Hamano 73b6a6cbda [PATCH] gitk: Use GIT_DIR where appropriate.
Some places assumed .git is the GIT_DIR, resulting heads and
tags not showing when it was run like "GIT_DIR=. gitk --all".
This is not a contrived example --- I rely on it to verify
my private copy of git.git repository before pushing it out.

Define a single procedure "gitdir" and use it.

Signed-off-by: Junio C Hamano <junkio@cox.net>
Signed-off-by: Paul Mackerras <paulus@samba.org>
2005-07-28 07:29:59 -05:00
Paul Mackerras 4ef1753763 Import the --topo-order change and fix the writing of ~/.gitk. 2005-07-27 22:16:51 -05:00
Paul Mackerras 9d2a52eca1 Display the diffs for a merge in a unified fashion.
Stuff that ended up in the result is shown in bold with a "+" at the
beginning of the line; stuff that didn't is in the normal font with
a "-" at the beginning of the line.  The color shows which parent
the stuff was in; red for the first parent, blue for the second, then
green, purple, brown, and the rest are grey.  If the result is different
from all of the parents it is shown in black (and bold).
2005-07-27 22:15:47 -05:00
Paul Mackerras 7eab29339b Handle the rename cases reported by git-diff-tree -C correctly. 2005-07-20 12:25:54 -04:00
Paul Mackerras 3c461ffe88 Calculate the list of interesting files for a merge.
If there is a GCA for the parents of the merge, then a file is
interesting if some parent has a version that is different from both
the child and the GCA.  If there is no GCA (e.g. for a merge that
pulls in an external project) then a file is interesting if the child's
version is different from all of the parents.

Next step is to actually show the differences for the interesting
files...
2005-07-20 09:13:46 -04:00
Paul Mackerras f06548617d Make the diff display less gaudy.
The old style is still available as an option (we still need a
preferences window so we can set/clear it though).
2005-07-18 14:29:03 -04:00
Paul Mackerras f3b8b3ce3e Allow short SHA1 IDs in the SHA1 entry field. 2005-07-18 12:16:35 -04:00
Paul Mackerras 4f2c2642ca Fix display of mode changes in details pane.
Also simplified the parsing of the git-diff-tree -p output and got
rid of the unused 'seenfile' variable.
2005-07-17 11:11:44 -04:00
Paul Mackerras e2ed4324b0 First cut at displaying the diffs for a merge.
We display the files where the child differs from the first parent
first in black in the file list window, followed by the second parent
in blue, and so on using different colors for each parent (provided
you don't have more than 6 parents; if you do we cycle around).
2005-07-17 03:39:44 -04:00
Paul Mackerras 14c9dbd69b Make searching in files changed faster, and fix some bugs.
We now kick off a single git-diff-tree -r --stdin and feed it all
the commit pairs we want to know about, instead of doing a separate
git-diff-tree invocation for each.
2005-07-16 21:53:55 -04:00
Paul Mackerras b74fd57966 Add "Files" and "Pickaxe" to the find menu.
"Files" matches the find string against each of the files modified
by each commit, and can do exact, case-ignoring or regexp matching.

"Pickaxe" uses git-diff-tree -S'string' and can only do exact
matching.  I called it "pickaxe" rather than "find within patch"
since it only finds commits where the string is present in the child
but not the parents or vice versa, and "pickaxe" is what the author
of that feature calls it.
2005-07-16 07:46:13 -04:00
Paul Mackerras 4a2139f52f Add a menu item for writing out a commit to a file.
This just invokes git-diff-tree.  Also changed the readonly entry
widgets in the "write patch" and "generate tag" windows to have
flat relief, so it doesn't look like you should be able to modify
what's in them.
2005-06-29 09:47:48 +10:00
Paul Mackerras bdbfbe3dc9 Add a menu item for creating tags. 2005-06-27 22:56:40 +10:00
Paul Mackerras 7e952e797c Fix a bug where we would corrupt the stuff read from git-rev-list.
If we have a very long commit message, and we end up getting a
bufferfull of data from git-rev-list that all belongs to one commit,
we ended up throwing away the data from a previous read that should
have been included.  The result was a error message about not being
able to parse the output of git-rev-list.

Also, if the git-rev-list output that we can't parse is long, only put
the first 80 chars in the error message.  Otherwise we end up with an
enormous error window.
2005-06-27 20:04:26 +10:00
Paul Mackerras 74daedb62c Add a menu entry for generating a patch between any two commits. 2005-06-27 19:27:32 +10:00
Paul Mackerras 8a0a74ad77 Fix behaviour in the case where we have no commits to display.
I had code in there to put "No commits selected" on the canvas
but it needed some globals.
2005-06-27 13:38:29 +10:00
Paul Mackerras e2ede2b9f5 Check for the existence of the git directory on startup.
Check that $GIT_DIR (or .git, if GIT_DIR is not set) is a directory.
This means we can give a more informative error message if the user
runs gitk somewhere that isn't a git repository.
2005-06-27 10:37:11 +10:00
Paul Mackerras ee3dc72e30 Clear the SHA1 entry field when we go to paste something into it
If the user pastes in the selection (with the middle mouse button) and
it already has 40 characters in it, clear it before pasting.
2005-06-25 16:37:13 +10:00
Paul Mackerras c8dfbcf995 Add commit row context menu and handle left-click on graph lines
Right-click on a context row now brings up a menu allowing the user to
generate a diff between that row and the selected row.  Left-click on
a graph line shows the parent and children connected by the line in
the details pane.  Left-click on a circle in the graph selects that
commit.  Left-click elsewhere in the graph does nothing.

When displaying a diff, the bottom-right file list box behaves
slightly differently now; instead of eliding all other files' diffs,
it now just scrolls the details pane so that the selected file's diff
starts at the top of the pane.

Since the diffs can be rather large, arrange for an update to be done
every 100ms while reading diffs.

Also removed the CVS revision keywords and bumped the version number
to 1.2.
2005-06-25 15:39:21 +10:00
Paul Mackerras 6c20ff3423 Try to assign colors so crossing lines have different colors
In particular try hard to give different colors to lines that cross
at a corner in one of the lines.
2005-06-22 19:53:32 +10:00
Paul Mackerras 806ce09718 Account for indentation of the checkin comments by git-rev-list
This involves adding indentation when we read a commit with
git-cat-file and trimming the whitespace from the headline.
2005-06-22 18:15:15 +10:00
Paul Mackerras b490a99176 Use git-rev-list --header.
With --header, git-rev-list gives us the contents of the commit
in-line, so we don't need to exec a git-cat-file to get it, and we
don't need the readobj command either.
Also fixed a residual problem with handling the commit that
has a parent listed twice.
2005-06-22 10:25:38 +10:00
Paul Mackerras 2efef4b9b5 Pass arguments through git-rev-parse.
This allows the user to specify ranges more flexibly; for instance
the user can now do "gitk v2.6.12.." and see all the changes since
2.6.12.
2005-06-21 10:20:04 +10:00
Paul Mackerras a823a91131 Handle the case of a parent being listed twice in a merge.
This happens in the linux-2.6 tree.  We draw the graph line
double-thick to show that this happened.

Also fix a bug where we got a bogus "No commit information available"
line at the end on simple repositories like this one.
2005-06-21 10:01:38 +10:00
Paul Mackerras 84ba734580 Draw graph lines as one continuous line where possible
Added context menu on lines
Added headline display when the mouse hovers over a line
Removed some debug messages
2005-06-17 00:12:26 +00:00
Paul Mackerras ea13cba175 Fix operation without libreadobj.so.0.0
Display a watch cursor when reading the commits initially
2005-06-16 10:54:04 +00:00
Paul Mackerras 9ccbdfbfbc Restructure to do incremental drawing
Some speedups from not doing update so often
2005-06-16 00:27:23 +00:00
Paul Mackerras c2f6a02251 Show heads as well as tags 2005-06-10 07:54:49 +00:00
Paul Mackerras d4e95cb6cf cope with changed git-diff-tree output format 2005-06-01 00:02:13 +00:00