git
17 years agogit-gui: Fix 'Select All' action on Windows.
Shawn O. Pearce [Mon, 22 Jan 2007 23:31:12 +0000 (18:31 -0500)] 
git-gui: Fix 'Select All' action on Windows.

Sometimes the Select All action from our context menus doesn't work
unless the text field its supposed to act on has focus.  I'm not
really sure why adding the sel tag requires having focus.  It
technically should not be required to update the sel tag membership,
but perhaps there is a bug in Tcl/Tk 8.4.1 on Windows which is
causing this odd behavior.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't attempt to tag new file/deleted file headers in diffs.
Shawn O. Pearce [Mon, 22 Jan 2007 23:24:45 +0000 (18:24 -0500)] 
git-gui: Don't attempt to tag new file/deleted file headers in diffs.

We don't want to tag these new file/delete file lines, as they aren't
actually that interesting.  Its quite clear from the diff itself that
the file is a new file or is a deleted file (as the entire thing will
appear in the diff).

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Force an update-index --refresh on unchanged files.
Shawn O. Pearce [Mon, 22 Jan 2007 22:10:38 +0000 (17:10 -0500)] 
git-gui: Force an update-index --refresh on unchanged files.

Its possible for external programs to update file modification dates of
many files within a repository.  I've seen this on Windows with a popular
virus scanner, sadly enough.  If the user has Trust File Modification
Timestamp enabled and the virus scanner touches a large number of files
it can be annoying trying to clear them out of the 'Changed But Not
Updated' file list by clicking on them one at a time to load the diff.

So now we force a rescan as soon as one such file is found, and for
just that rescan we disable the Trust File Modification Timestamp option
thereby allowing Git to update the modification dates in the index.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't format the mode line of a diff.
Shawn O. Pearce [Mon, 22 Jan 2007 04:11:47 +0000 (23:11 -0500)] 
git-gui: Don't format the mode line of a diff.

We sometimes see a mode line show up in a diff if the file mode was
changed.  But its not something we format specially.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Create missing branch head on initial commit.
Shawn O. Pearce [Sun, 21 Jan 2007 23:00:03 +0000 (18:00 -0500)] 
git-gui: Create missing branch head on initial commit.

If we are making an initial commit our branch head did not exist when
we scanned for all heads during startup.  Consequently we won't have
it in our branch menu.  So force it to be put there after the ref was
created.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Slightly tweak new window geometry.
Shawn O. Pearce [Sun, 21 Jan 2007 22:50:42 +0000 (17:50 -0500)] 
git-gui: Slightly tweak new window geometry.

I didn't really like the way a new git-gui launched in a new repository
as the window geometry wasn't quite the best layou.  So this is a minor
tweak to try and get space distributed around the window better.

By decreasing the widths we're also able to shrink the gui smaller
without Tk clipping content at the edge of the window.  A nice feature.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Update todo list with finished and new items.
Shawn O. Pearce [Sun, 21 Jan 2007 22:26:22 +0000 (17:26 -0500)] 
git-gui: Update todo list with finished and new items.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correctly categorize tracking branches and heads.
Shawn O. Pearce [Sun, 21 Jan 2007 22:22:40 +0000 (17:22 -0500)] 
git-gui: Correctly categorize tracking branches and heads.

Up until now git-gui did not support the new wildcard syntax used to
fetch any remote branch into a tracking branch during 'git fetch'. Now
if we identify a tracking branch as ending with the string '/*' then
we use for-each-ref to print out the reference names which may have
been fetched by that pattern.  We also now correctly filter any
tracking branches out of refs/heads, if they user has placed any there.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Automatically toggle the relevant radio buttons.
Shawn O. Pearce [Sun, 21 Jan 2007 22:02:25 +0000 (17:02 -0500)] 
git-gui: Automatically toggle the relevant radio buttons.

When the user selects a starting revision from one of our offered popup
lists (local branches or tracking branches) or enters in an expression
in the expression input field we should automatically activate the
corresponding radio button for them.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Fully select a field when entering into it.
Shawn O. Pearce [Sun, 21 Jan 2007 21:43:14 +0000 (16:43 -0500)] 
git-gui: Fully select a field when entering into it.

If the user is tabbing through fields in the options dialog they are
likely to want to just enter a new value for the field, rather than
edit the value in-place.  This is easier if we select the entire value
upon focusing into the field.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Improve keyboard traversal in dialogs.
Shawn O. Pearce [Sun, 21 Jan 2007 21:37:05 +0000 (16:37 -0500)] 
git-gui: Improve keyboard traversal in dialogs.

When we are in a dialog such as the new branch dialog or our options
dialog we should permit the user to traverse around through the available
widgets with their Tab/Shift-Tab key combinations.  So in any single
line text field where we don't want tab characters to actually be
inserted into the value rebind Tab and Shift-Tab to honor what the
tk_focusPrev and tk_focusNext scripts recommend.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Allow user to specify a branch name pattern.
Shawn O. Pearce [Sun, 21 Jan 2007 21:28:59 +0000 (16:28 -0500)] 
git-gui: Allow user to specify a branch name pattern.

Typically I'm creating all new branches with the same prefix, e.g. 'sp/'.
So its handy to be able to setup a repository (or global) level config
option for git gui which contains this initial prefix.  Once set then
git-gui will load it into the new branch name field whenever a new
branch is being created.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Give a better error message on an empty branch name.
Shawn O. Pearce [Sun, 21 Jan 2007 20:40:55 +0000 (15:40 -0500)] 
git-gui: Give a better error message on an empty branch name.

New branches must have a name.  An empty one is not a valid ref, but the
generic message "We do not like '' as a branch name." is just too vague
or difficult to read.  So detect the missing name early and tell the
user it must be entered.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't offer tracking branches if none exist.
Shawn O. Pearce [Sun, 21 Jan 2007 20:38:09 +0000 (15:38 -0500)] 
git-gui: Don't offer tracking branches if none exist.

I refactored the common code related to tracking branch listing into
a new procedure all_tracking_branches.  This saves a few lines and
should make the create and delete dialogs easier to maintain.

We now don't offer a radio button to create from a tracking branch
or merge-check a tracking branch if there are no tracking branches
known to git-gui.  This prevents us from creating an empty option
list and letting the user try to shoot themselves in the foot by
asking us to work against an empty initial revision.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Never line wrap in file lists.
Shawn O. Pearce [Sun, 21 Jan 2007 19:58:01 +0000 (14:58 -0500)] 
git-gui: Never line wrap in file lists.

Some of my file paths in some of my repositories are very long, this
is rather typical in Java projects where the path name contains a deep
package structure and then the file name itself is rather long and
(hopefully) descriptive.  Seeing these paths line wrap in the file lists
looks absolutely horrible.  The entire rendering is almost unreadable.

Now we draw both horizontal and vertical scrollbars for both file lists,
and we never line wrap within the list text itself.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Make diff viewer colors match gitk's defaults.
Shawn O. Pearce [Sun, 21 Jan 2007 19:49:45 +0000 (14:49 -0500)] 
git-gui: Make diff viewer colors match gitk's defaults.

Because users who use git-gui are likely to also be using gitk, we
should at least match gitk's default colors and formatting within the
diff viewer.

Unfortunately this meant that I needed to change the background colors
of the hunks in a 'diff --cc' output, as the green used for 'added line'
was completely unreadable on the old color.  We now use ivory1 to show
hunks which came from HEAD/parent^1, which are the portions that the
current branch has contributed, and are probably the user's own changes.
We use a very light blue for the portions which came from FETCH_HEAD,
as this makes the changes made by the other branch stand out more in the
diff.

I've also modified the hunk header lines to be blue, as that is how gitk
is showing them.

Apparently I forgot to raise the sel tag above everything else in the
diff viewer, which meant that selections in the diff viewer were not
visible if they were made on a 'diff --cc' hunk which had a background.
Its now the higest priority tag, ensuring the selection is always visible
and readable.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correctly ignore '* Unmerged path' during diff.
Shawn O. Pearce [Sun, 21 Jan 2007 19:23:51 +0000 (14:23 -0500)] 
git-gui: Correctly ignore '* Unmerged path' during diff.

If a path is really unmerged, such as because it has been deleted and
also modifed, we cannot obtain a diff for it.  Instead Git is sending
back '* Unmerged path <blah>' for file <blah>.  We should display this
line as-is as our tag selecting switches don't recognize it.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Change rude error popup to info popup.
Shawn O. Pearce [Sun, 21 Jan 2007 19:16:40 +0000 (14:16 -0500)] 
git-gui: Change rude error popup to info popup.

If the user has not added any files yet they cannot commit.  But
telling them this isn't an error, its really just an informational
note meant to push the user in the correct direction.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Improve the merge check interface for branch deletion.
Shawn O. Pearce [Sun, 21 Jan 2007 19:14:54 +0000 (14:14 -0500)] 
git-gui: Improve the merge check interface for branch deletion.

Just like how we split out the local and remote branches into two
different pick lists for branch creation, we should do the same
thing for branch deletion.  This means that  there are really 3
modes of operation here:

  * delete only if merged into designated local branch;
  * delete only if merged into designated tracking (remote) branch;
  * delete no matter what

So we now use radio buttons to select between these operations.

We still default to checking for merge into the current branch,
as that is probably the most commonly used behavior. It also is
what core Git's command line tools do.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Use a grid layout for branch dialog.
Shawn O. Pearce [Sun, 21 Jan 2007 18:56:38 +0000 (13:56 -0500)] 
git-gui: Use a grid layout for branch dialog.

Using a stack of frames in the Starting Revision section of the new
branch dialog turned out to be a mess.  The varying lengths of each
label caused the optionMenu widgets to be spread around the screen
at unaligned locations, making the interface very kludgy looking.

Now we layout the major sections of the branch dialog using grid
rather than pack, allowing these widgets to line up vertically in
a nice neat column.  All extra space is given to column 1, which is
where we have located the text fields.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Pad new branch name input box.
Shawn O. Pearce [Sun, 21 Jan 2007 18:37:53 +0000 (13:37 -0500)] 
git-gui: Pad new branch name input box.

The new branch name input box was showing up too close to the labelframe
border, it was basically right on top of it on Windows.  This didn't
look right when compared to the Starting Revision's expression input
field, as that had a 5 pixel padding.

So I've put the new name input box into its own frame and padded that
frame by 5 pixels, making the UI more consistent.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correct unmerged file detection at commit time.
Shawn O. Pearce [Sun, 21 Jan 2007 18:34:00 +0000 (13:34 -0500)] 
git-gui: Correct unmerged file detection at commit time.

Its impossible to commit an index which has unmerged stages.

Unfortunately a bug in git-gui allowed the user to try to do exactly that,
as we broke out of our file scanning loop as soon as we found a valid AMD
index state.  That's wrong, as the files are coming back from our array
in pseudo-random order; an unmerged file may get returned only after all
merged files.

I also noticed the grammer around here in our dialog boxes still used
the term 'include', so this has been updated to reflect current usage.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Add Refresh to diff viewer context menu.
Shawn O. Pearce [Sun, 21 Jan 2007 18:27:43 +0000 (13:27 -0500)] 
git-gui: Add Refresh to diff viewer context menu.

Sometimes you want to just force the diff to redisplay itself without
rescanning every file in the filesystem (as that can be very costly
on large projects and slow operating systems).  Now you can force a
diff-only refresh from the context menu.  Previously you could also
do this by reclicking on the file name in the UI, but it may not be
obvious to all users, having a context menu option makes it more
clear.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correct disappearing unstaged files.
Shawn O. Pearce [Sun, 21 Jan 2007 18:25:06 +0000 (13:25 -0500)] 
git-gui: Correct disappearing unstaged files.

A prior commit tried to use the old index state for the old working
directory state during a UI refresh of a file.  This caused files
which were being unstaged (and thus becoming unmodified) to drop
out of the working directory side of the display, at least until
the user performed a rescan to force the UI to redisplay everything.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Clear diff from viewer if the side changed.
Shawn O. Pearce [Sun, 21 Jan 2007 18:22:26 +0000 (13:22 -0500)] 
git-gui: Clear diff from viewer if the side changed.

If the user switches the currently shown file from one side of the UI
to the other then how its diff is presented would be different.  And
leaving the old diff up is downright confusing.

Since the diff is probably not interesting to the user after the switch
we should just clear the diff viewer.  This saves the user time, as they
won't need to wait for us to reload the diff.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Fix bug in unmerged file display.
Shawn O. Pearce [Sun, 21 Jan 2007 18:18:11 +0000 (13:18 -0500)] 
git-gui: Fix bug in unmerged file display.

We were not correctly setting the old state of an index display to
_ if the index was previously unmerged.   This caused us to try and
update a U->M when resolving a merge conflict but we were unable to
do so as the icon did not exist in the index viewer.  Tk did not
like being asked to modify an icon which was undefined.

Now we always transform both the old and the new states for both
sides (index and working directory) prior to updating the UI.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Improve diff --cc viewing for unmerged files.
Shawn O. Pearce [Sun, 21 Jan 2007 18:12:02 +0000 (13:12 -0500)] 
git-gui: Improve diff --cc viewing for unmerged files.

Now that we are using 'git diff' to display unmerged working directory
files we are getting 'diff --cc' output rather than 'diff --combined'
output.  Further the markers in the first two columns actually make
sense here, we shouldn't attempt to rewrite them to something else.

I've added 'diff --cc *' to the skip list in our diff viewer, as that
particular line is not very interesting to display.

I've completely refactored how we perform detection of the state of a
line during diff parsing; we now report an error message if we don't
understand the particular state of any given line.  This way we know
if we aren't tagging something we maybe should have tagged in the UI.

I've also added special display of the standard conflict hunk markers
(<<<<<<<, =======, >>>>>>>).  These are formatted without a patch op
as the patch op is always '+' or '++' (meaning the line has been added
relative to the committed state) and are displayed in orange bold text,
sort of like the @@ or @@@ marker line is at the start of each hunk.

In a 3 way merge diff hunks which came from our HEAD are shown with a
azure2 background, and hunks which came from the incoming MERGE_HEAD
are displayed with a 'light goldenrod yellow' background.  This makes
the two different hunks clearly visible within the file.  Hunks which
are ++ or -- (added or deleted relative to both parents) are shown
without any background at all.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Improve the display of merge conflicts.
Shawn O. Pearce [Sun, 21 Jan 2007 17:30:51 +0000 (12:30 -0500)] 
git-gui: Improve the display of merge conflicts.

If a file has a merge conflict we want it to show up in the 'Changed
But Not Updated' file list rather than the 'Changes To Be Committed'
file list.  This way the user can mostly ignore the left side (the
HEAD<->index comparsion) while resolving a merge and instead focus
on the merge conflicts, which are just shown on the right hand side.

This requires detecting the U state in the index side and drawing
it as though it were _, then forcing the working directory side to
have a U state.  We have to delay this until presentation time as
we don't want to change our internal state data to be different
from what Git is telling us (I tried, the patch for that was ugly
and didn't work).

When showing a working directory diff and its a merge conflict we
don't want to use diff-files as this would wind up showing any
automatically merged hunks obtained from MERGE_HEAD in the diff.
These are not usually very interesting as they were completed by
the system.  Instead we just want to see the conflicts.  Fortunately
the diff porcelain-ish frontend (aka 'git diff') detects the case of
an unmerged file and generates a --cc diff against HEAD and MERGE_HEAD.
So we now force any working directory diff with an index state of 'U'
to go through that difference path.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Remove combined diff showing behavior.
Shawn O. Pearce [Sun, 21 Jan 2007 16:54:16 +0000 (11:54 -0500)] 
git-gui: Remove combined diff showing behavior.

The combined diff format can be very confusing, especially to new users
who may not even be familiar with a standard two way diff format.  So
for files which are already staged for commit and which are modifed in
the working directory we should show two different diffs, depending on
which side the user clicked on.

If the user clicks on the "Changes To Be Committed" side then we should
show them the PARENT<->index difference.  This is the set of changes they
will actually commit.

If the user clicks on the "Changed But Not Updated" side we should show
them the index<->working directory difference.  This is the set of changes
which will not be committed, as they have not been staged into the index.
This is especially useful when merging, as the "Changed But Not Updated"
files are the ones that need merge conflict resolution, and the diff here
is the conflict hunks and/or any evil merge created by the user.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Refactor current_diff -> current_diff_path.
Shawn O. Pearce [Sun, 21 Jan 2007 16:37:58 +0000 (11:37 -0500)] 
git-gui: Refactor current_diff -> current_diff_path.

We now need to keep track of which side the current diff is for,
HEAD<->index or index<->working directory.  Consequently we need
an additional "current diff" variable to tell us which side the
diff is for.  Since this is really only necessary in reshow_diff
I'm going to declare a new global, rather than try to shove both
the path and the side into current_diff.

To keep things clear later on, I'm renaming current_diff to
current_diff_path.  There is no functionality change in this
commit.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Attempt to checkout the new branch after creation.
Shawn O. Pearce [Sun, 21 Jan 2007 09:57:11 +0000 (04:57 -0500)] 
git-gui: Attempt to checkout the new branch after creation.

If the user asked us to checkout the branch after creating it then
we should try to do so.  This may fail, especially right now since
branch switching from within git-gui is not supported.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't delete the test target branch.
Shawn O. Pearce [Sun, 21 Jan 2007 09:54:01 +0000 (04:54 -0500)] 
git-gui: Don't delete the test target branch.

Its possible for the user to select a branch for the merge test
(while deleting branches) and also select that branch for deletion.
Doing so would have bypassed our merge check for that branch, as
a branch is always a strict subset of itself.  So we will simply
skip over a branch and not delete it if that is the branch which
the user selected for the merge check.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Improve the branch delete confirmation dialogs.
Shawn O. Pearce [Sun, 21 Jan 2007 09:51:45 +0000 (04:51 -0500)] 
git-gui: Improve the branch delete confirmation dialogs.

If the user is deleting a branch which is fully merged into the
selected test branch we should not confirm the delete with them,
the fact that the branch is fully merged means we can recover the
branch and no work will be lost.

If a branch is not fully merged, we should warn the user about which
branch(es) that is and continue deleting those which are fully merged.

We should only delete a branch if the user disables the merge check,
and in that case we should confirm with the user that a delete should
occur as this may cause them to lose changes.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Move commit_prehook into commit_tree.
Shawn O. Pearce [Sun, 21 Jan 2007 09:28:22 +0000 (04:28 -0500)] 
git-gui: Move commit_prehook into commit_tree.

The only reason the commit_prehook logic was broken out into its own
proc was so it could be invoked after the current set of files that
were already added to the commit could be refreshed if 'Allow Partially
Added Files' was set to false.  Now that we no longer even offer that
option to the user there is no reason to keep this code broken out
into its own procedure.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Remove 'Allow Partially Added Files' option.
Shawn O. Pearce [Sun, 21 Jan 2007 09:19:33 +0000 (04:19 -0500)] 
git-gui: Remove 'Allow Partially Added Files' option.

Now that we take the approach of core Git where we allow the user to
stage their changes directly into the index all of the time there is
absolutely no reason to have the Allow Partially Added Files option,
nor is there a reason or desire to default that option to false.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Use borders on text fields in branch dialog.
Shawn O. Pearce [Sun, 21 Jan 2007 08:13:13 +0000 (03:13 -0500)] 
git-gui: Use borders on text fields in branch dialog.

On Mac OS X wish does not draw borders around text fields, making the
field look like its not even there until the user focuses into it. I
don't know the Mac OS X UI standards very well, but that just seems
wrong.  Other applications (e.g. Terminal.app) show their input boxes
with a sunken relief, so we should do the same.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Allow creating branches from tracking heads.
Shawn O. Pearce [Sun, 21 Jan 2007 07:27:26 +0000 (02:27 -0500)] 
git-gui: Allow creating branches from tracking heads.

Sometimes you want to create a branch from a remote tracking branch.
Needing to enter it in the revision expression field is very annoying,
so instead let the user select it from a list of known tracking branches.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Allow users to delete branches merged upstream.
Shawn O. Pearce [Sun, 21 Jan 2007 07:21:45 +0000 (02:21 -0500)] 
git-gui: Allow users to delete branches merged upstream.

Most of the time when you are deleting branches you want to delete
those which have been merged into your upstream source.  Typically
that means it has been merged into the tip commit of some tracking
branch, and the current branch (or any other head) doesn't matter.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Implemented local branch deletion.
Shawn O. Pearce [Sun, 21 Jan 2007 07:14:00 +0000 (02:14 -0500)] 
git-gui: Implemented local branch deletion.

Users can now delete a local branch by selecting from a list of
available branches.  The list automatically does not include
the current branch, as deleting the current branch could be quite
dangerous and should not be supported.

The user may also chose to have us verify the branches are fully
merged into another branch before deleting them.  By default we
select the current branch, matching 'git branch -d' behavior,
but the user could also select any other local branch.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Bind M1-N to create branch.
Shawn O. Pearce [Sun, 21 Jan 2007 06:34:55 +0000 (01:34 -0500)] 
git-gui: Bind M1-N to create branch.

Creating branches is a common enough activity within a Git project
that we probably should give it a keyboard accelerator.  N is not
currently used and seems reasonable to stand for "New Branch". To
bad our menu calls it create.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Implemented create branch GUI.
Shawn O. Pearce [Sun, 21 Jan 2007 06:31:14 +0000 (01:31 -0500)] 
git-gui: Implemented create branch GUI.

Users may now create new branches by activating the Branch->Create menu
item.  This opens a dialog which lets the user enter the new branch
name and select the starting revision for the new branch.

For the starting revision we allow the user to either select from a
list of known heads (aka local branches) or to enter an arbitrary
SHA1 expression.  For either creation technique we run the starting
revision through rev-parse to verify it is valid before trying to
create the ref with update-ref.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Pad the cancel/save buttons in the options window.
Shawn O. Pearce [Sun, 21 Jan 2007 04:52:19 +0000 (23:52 -0500)] 
git-gui: Pad the cancel/save buttons in the options window.

It looks horrible to have the cancel and save buttons wedged up against
each other in our options dialog.  Therefore toss a 5 pixel pad between
them.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Only permit selection in one list at a time.
Shawn O. Pearce [Sun, 21 Jan 2007 04:46:53 +0000 (23:46 -0500)] 
git-gui: Only permit selection in one list at a time.

Now that our lists represent more defined states it no longer makes any
sense to permit a user to make selections from both lists at once, as
the each available operation acts only on files whose status corresponds
to only one of the lists.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Simplify printing of index info to update-index.
Shawn O. Pearce [Sun, 21 Jan 2007 04:35:39 +0000 (23:35 -0500)] 
git-gui: Simplify printing of index info to update-index.

During unstaging we can simplify the way we perform the output by
combining our four puts into a single call.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Refactor the add to commit state filters.
Shawn O. Pearce [Sun, 21 Jan 2007 04:33:34 +0000 (23:33 -0500)] 
git-gui: Refactor the add to commit state filters.

The list of states which are valid for update-index were a little
too verbose and fed a few too many cases to the program.  We can
do better with less lines of code by using more pattern matching,
and since we already were globbing here there's little change in
runtime cost.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Refactor the revert (aka checkout-index) implementation.
Shawn O. Pearce [Sun, 21 Jan 2007 04:20:17 +0000 (23:20 -0500)] 
git-gui: Refactor the revert (aka checkout-index) implementation.

We can revert any file which has a valid stage 0 (is not unmerged)
and which is has a working directory status of M or D.  This vastly
simplifies our pattern matching on file status when building up the
list of files to perform a checkout-index against.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Add or unstage based on the specific icon used.
Shawn O. Pearce [Sun, 21 Jan 2007 04:10:30 +0000 (23:10 -0500)] 
git-gui: Add or unstage based on the specific icon used.

Rather than relying on the file state and just inverting it, we should
look at which file icon the user clicked on.  If they clicked on the
one in the "Changes To Be Committed" list then they want to unstage
the file.  If they clicked on the icon in the "Changed But Not Updated"
list then they want to add the file to the commit.  This should be much
more reliable about capturing the user's intent then looking at the file
state.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Refactor add/remove proc names to align with reality.
Shawn O. Pearce [Sun, 21 Jan 2007 04:07:04 +0000 (23:07 -0500)] 
git-gui: Refactor add/remove proc names to align with reality.

Now that core Git refers to resetting paths in the index as "unstaging"
the paths we should do the same in git-gui, both internally in our code
and also within the menu action name.  The same follows for our staging
logic, as core Git refers to this as 'add'.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Cleanup state descriptions.
Shawn O. Pearce [Sun, 21 Jan 2007 04:00:28 +0000 (23:00 -0500)] 
git-gui: Cleanup state descriptions.

Updated the state descriptions for individual file states to try and
make them more closely align with what git-runstatus might display.
This way a user who is reading Git documentation will be less confused
by our descriptions.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Remove invalid DM state.
Shawn O. Pearce [Sun, 21 Jan 2007 03:58:52 +0000 (22:58 -0500)] 
git-gui: Remove invalid DM state.

The DM state cannot really happen.  Its implying that the file has
been deleted in the index, but the file in the working directory has
been modified relative to the file in the index.  This is complete
nonsense, the file doesn't exist in the index for it to be different
against!

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correct DD file state to be only D_.
Shawn O. Pearce [Sun, 21 Jan 2007 03:57:19 +0000 (22:57 -0500)] 
git-gui: Correct DD file state to be only D_.

Apparently my earlier suspicion that the file state DD was a bug was
correct.  A file which has been deleted from the working directory and
from the index will always get the state of D_ during a rescan.  Thus
the only valid state for this to have is D_.  We should always use only
D_ internally during our state changes.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Convert UI to use 'staged for commit' interface.
Shawn O. Pearce [Sun, 21 Jan 2007 03:45:19 +0000 (22:45 -0500)] 
git-gui: Convert UI to use 'staged for commit' interface.

This is a rather drastic change to the git-gui user interface, but it
doesn't really look any different yet.  I've taken the two lists and
converted them to being "changes to be committed" and "changed but
not updated".  These lists correspond to the same lists output by
git-runstatus based on how files differ in the HEAD<->index and the
index<->working directory comparsions it performs.

This change is meant to correlate with the change in Git 1.5.0 where
we have brought the index more into the foreground and are trying to
teach users to make use of it as part of their daily operations.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Start file status display refactoring.
Shawn O. Pearce [Sun, 21 Jan 2007 03:06:51 +0000 (22:06 -0500)] 
git-gui: Start file status display refactoring.

I'm going to refactor the way file status information gets displayed
so it more closely aligns with the way 'git-runstatus' displays the
differences between HEAD<->index and index<->working directory.  To
that end the other file list is going to be changed to be the working
directory difference.  So this change renames it.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Display the directory we are entering during startup.
Shawn O. Pearce [Sun, 21 Jan 2007 03:00:28 +0000 (22:00 -0500)] 
git-gui: Display the directory we are entering during startup.

If the user has many git-gui icons it may be confusing when they
start one which git-gui is still coming up.  So on the windows
systems we now include an echo statement which displays the full
pathname of the working directory we are trying to enter into.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Make the gitk starting message match our usual format.
Shawn O. Pearce [Sun, 21 Jan 2007 02:56:25 +0000 (21:56 -0500)] 
git-gui: Make the gitk starting message match our usual format.

Because we usually say "Operation... please wait..." we should do
the same thing when starting gitk.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Allow [gitdir ...] to act as [file join [gitdir] ...].
Shawn O. Pearce [Sun, 21 Jan 2007 02:55:05 +0000 (21:55 -0500)] 
git-gui: Allow [gitdir ...] to act as [file join [gitdir] ...].

Because it is such a common idiom to use [gitdir] along with [file join]
to locate the path of an item within the .git directory of the current
repository we might as well allow gitdir to act as a wrapper for the
file join operation.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Cleanup usage of gitdir global variable.
Shawn O. Pearce [Sun, 21 Jan 2007 02:48:56 +0000 (21:48 -0500)] 
git-gui: Cleanup usage of gitdir global variable.

The gitdir global variable is essentially read-only, and is used rather
frequently.  So are appname and reponame.  Needing to constantly declare
'global appname' just so we can access the value as $appname is downright
annoying and redundant.  So instead I'm declaring these as procedures and
changing all uses to invoke the procedure rather than access the global
directly.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Refactor reponame computation.
Shawn O. Pearce [Sun, 21 Jan 2007 02:36:21 +0000 (21:36 -0500)] 
git-gui: Refactor reponame computation.

We use reponame in a number of locations, and every time its always the
same value.  Instead of computing this multiple times with code that was
copied and pasted around we can compute it once immediately after the
global gitdir has been computed and set.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Suggest when running 'git gc' may be worthwhile.
Shawn O. Pearce [Sun, 21 Jan 2007 02:23:21 +0000 (21:23 -0500)] 
git-gui: Suggest when running 'git gc' may be worthwhile.

Users often forget to repack their object database, then start to
complain about how slow it is to perform common operations after
they have collected thousands of loose objects in their objects
directory.  A simple repack usually restores performance.

During startup git-gui now asks git-count-objects how many loose
objects exist, and if this number exceeds a hardcoded threshold
we suggest that the user compress the database (aka run 'git gc')
at this time.  I've hardcoded this to 2000 objects on non-Windows
systems as there the filesystems tend to handle the ~8 objects
per directory just fine.  On Windows NTFS and FAT are just so slow
that we really start to lag when more than 200 loose objects exist,
so the hardcoded threshold is much lower there.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't offer my miga hack if its configuration file isn't present.
Shawn O. Pearce [Sun, 21 Jan 2007 01:53:37 +0000 (20:53 -0500)] 
git-gui: Don't offer my miga hack if its configuration file isn't present.

I really hate that I have this specialized hack within git-gui, but
its here.  The hack shouldn't be offered unless miga's required .pvcsrc
file is in the top level of the repository's working directory.  If
this file is missing miga will fail to startup properly, and the user
cannot wouldn't be able to use it within this directory.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Allow the user to copy the version data to the clipboard.
Shawn O. Pearce [Sun, 21 Jan 2007 01:47:31 +0000 (20:47 -0500)] 
git-gui: Allow the user to copy the version data to the clipboard.

If a user wants to report an issue they will likely want to include
the version number with their issue report.  This may be difficult
to enter if the version number includes an abbreviated commit SHA1
on the end of it.  So we now give the user a context menu option
on the version box which allows them to copy all of the relevant
version data to the clipboard, ready for pasting into a report.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Ensure version number is always current.
Shawn O. Pearce [Sun, 21 Jan 2007 01:31:09 +0000 (20:31 -0500)] 
git-gui: Ensure version number is always current.

I'm stealing the exact logic used by core Git within its own Makefile to
setup the version number within scripts and executables.  This way we
can be sure that the version number is always updated after a commit,
and that the version number also reflects when it is coming from a dirty
working directory (and is thus pretty worthless).

I've cleaned up some of the version display code in the about dialog too.
There were simply too many blank lines in the bottom section where we
showed the version data.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Display the full GPL copyright notice in about dialog.
Shawn O. Pearce [Sun, 21 Jan 2007 01:08:20 +0000 (20:08 -0500)] 
git-gui: Display the full GPL copyright notice in about dialog.

We're a true GPL program, and we're interactive.  We should show the
entire GPL notice and disclaimer of warranty in our about dialog upon
request by the user, as well as include it in the header of our source.
Perhaps overkill, but is recommended by our license.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Display the git-gui version in the Help->About dialog.
Shawn O. Pearce [Sun, 21 Jan 2007 01:04:02 +0000 (20:04 -0500)] 
git-gui: Display the git-gui version in the Help->About dialog.

Now that we know what version git-gui is, the about dialog should
display it to the end-user.  This way users can find out what version
they have before they report a problem or request a feature.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Modified makefile to embed version into git-gui script.
Shawn O. Pearce [Sun, 21 Jan 2007 01:00:07 +0000 (20:00 -0500)] 
git-gui: Modified makefile to embed version into git-gui script.

We want to embed the version of git-gui directly into the script file,
so that we can display it properly in the about dialog.  Consequently
I've refactored the Makefile process to act like the one in core git.git
with regards to shell scripts, allowing git-gui to be constructed by a
sed replacement performed on git-gui.sh.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Hide the ugly bash command line from the windows desktop icon.
Shawn O. Pearce [Sun, 21 Jan 2007 00:45:26 +0000 (19:45 -0500)] 
git-gui: Hide the ugly bash command line from the windows desktop icon.

The user really doesn't need to see the technical details of how we
launch git-gui from within their "desktop icon".  Instead we should hide
the command line from being displayed when the icon launches by putting
@ at the start of the line.  If they really need to see the command we
are running they can edit the batch file.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Change more 'include' language to 'add'.
Shawn O. Pearce [Sun, 21 Jan 2007 00:07:46 +0000 (19:07 -0500)] 
git-gui: Change more 'include' language to 'add'.

I just found a whole slew of places where we still were using the term
'include' rather than 'add' to refer to the act of updating the index
with modifications from the working directory.  To be consistent with
all Git documentation and command line tools, these should be 'add'.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Work around odd cygpath bug on Windows.
Shawn O. Pearce [Sun, 21 Jan 2007 00:03:26 +0000 (19:03 -0500)] 
git-gui: Work around odd cygpath bug on Windows.

There appears to be a bug on one of my test systems where cygpath with
the --long-name option is generating a corrupt string that does not
actually refer to sh.exe.  This breaks any desktop icon created by
git-gui as the executable we are trying to invoke does not exist.
Since Cygwin is typically installed as C:\cygwin long path names is
probably not actually necessary to link to the shell.

I also added a small echo to the start of the icon script, as it can
take one of my test systems several seconds to startup git-gui.  This
way the user knows we're starting git-gui, and was politely asked to
wait for the action to complete.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Correct wording of the revert confirmation dialog.
Shawn O. Pearce [Sat, 20 Jan 2007 23:54:56 +0000 (18:54 -0500)] 
git-gui: Correct wording of the revert confirmation dialog.

We no longer describe updating the index as including changes, as we
now use the add notation used by core Git's command line tools.  So
its confusing to be talking about unincluded changes within the revert
dialog.  Instead we should used language like 'unadded changes'.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Corrected behavior of deleted (but existing in HEAD) files.
Shawn O. Pearce [Sat, 20 Jan 2007 23:50:14 +0000 (18:50 -0500)] 
git-gui: Corrected behavior of deleted (but existing in HEAD) files.

Apparently I did not account for the D_ file state.  This can occur when
a file has been marked for deletion by deleting it from the index, and
the file also does not exist in the working directory.  Typically this
happens when the user deletes the file, hits Rescan, then includes the
missing file in the commit, then hits Rescan again.  We don't find the
file in the working directory but its been removed in the index, so the
state becomes D_.

This state should be identical with DD.  I'm not entirely sure why DD
occurs sometimes and D_ others, it would seem like D_ is the state that
should be happening instead of DD, leading me to believe there is a quirk
in git-gui's state manipulation code.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Run git-gc rather than git-repack.
Shawn O. Pearce [Sat, 20 Jan 2007 23:38:12 +0000 (18:38 -0500)] 
git-gui: Run git-gc rather than git-repack.

Now that git 1.5.0-rc1 and later has a 'git gc' command which performs
all important repository management activites (including reflog pruning,
repacking local objects, unnecessary loose object pruning and rerere cache
expiration) we should run 'gc' when the user wants us to cleanup their
object database for them.

I think the name 'gc' is horrible for a GUI application like git-gui,
so I'm labeling the menu action 'Compress Database' instead.  Hopefully
this will provide some clue to the user about what the action does.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Show all fetched branches for remote pulls.
Shawn O. Pearce [Wed, 13 Dec 2006 03:44:38 +0000 (22:44 -0500)] 
git-gui: Show all fetched branches for remote pulls.

Loop through every remote.<name>.fetch entry and add it as a valid
option in the Pull menu.  This way users can pull any remote branch
that they track, without needing to leave the gui.  Its a rather crude
work around for not having a full merge interface.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Created very crude Tools menu, to support miga.
Shawn O. Pearce [Fri, 8 Dec 2006 03:07:38 +0000 (22:07 -0500)] 
git-gui: Created very crude Tools menu, to support miga.

In one particular case I have a tool called 'miga' which users may need
to invoke on their repository.  This is a homegrown tool which is not
(and should be) part of git-gui, but I still want to be able to run it
from within the gui.

Right now I'm taking a shortcut and adding it to the Tools menu if
we are not on Mac OS X and the support script used to launch the tool
exists in the local filesystem.  This is nothing but a complete and
utter hack.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Reworded 'Include' to 'Add' to match core Git.
Shawn O. Pearce [Fri, 8 Dec 2006 00:59:46 +0000 (19:59 -0500)] 
git-gui: Reworded 'Include' to 'Add' to match core Git.

Now that git-add is a first class citizen in core Git (Nico's 366bfcb6)
users may start to expect the term 'add' to refer to the act of including
a file's changes into a commit.  So I'm replacing all uses of the term
'Include' in the UI with 'Add'.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Auto-update any A? or M? files during rescan.
Shawn O. Pearce [Mon, 27 Nov 2006 00:46:45 +0000 (19:46 -0500)] 
git-gui: Auto-update any A? or M? files during rescan.

If the user has partial includes disabled then it doesn't matter what
state the working directory is in; if the file has been included in
the next commit its index state is A or M and we should immediately
run update-index on the working directory file to bring the index in
sync with the working directory.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Enable resolution of merge conflicts.
Shawn O. Pearce [Mon, 27 Nov 2006 00:45:39 +0000 (19:45 -0500)] 
git-gui: Enable resolution of merge conflicts.

If a file has a merge conflict (index state = U) the user will need to
run update-index on that file to resolve all stages down to stage 0,
by including the file in the working directory.

Like core Git we'll just trust the user that their resolution is
correct, and that they didn't just include the file into the commit
while merge conflicts still exist within the file.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Updated todo list.
Shawn O. Pearce [Sun, 26 Nov 2006 04:16:16 +0000 (23:16 -0500)] 
git-gui: Updated todo list.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Set a proper title on our revert confirm dialog box.
Shawn O. Pearce [Sat, 25 Nov 2006 17:40:29 +0000 (12:40 -0500)] 
git-gui: Set a proper title on our revert confirm dialog box.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Started implementation of switch_branch.
Shawn O. Pearce [Sat, 25 Nov 2006 09:04:24 +0000 (04:04 -0500)] 
git-gui: Started implementation of switch_branch.

This implementation of switch_branch is not yet finished, and thus
it throws a "NOT FINISHED" error rather than completing the switch.
But its a rough sketch of the procedure required.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Misc. comment and formatting cleanups.
Shawn O. Pearce [Sat, 25 Nov 2006 08:38:39 +0000 (03:38 -0500)] 
git-gui: Misc. comment and formatting cleanups.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Rename all_branches -> all_heads.
Shawn O. Pearce [Sat, 25 Nov 2006 08:35:33 +0000 (03:35 -0500)] 
git-gui: Rename all_branches -> all_heads.

Since this list is really the set of refs which match "refs/heads/*" it
really is the set of heads and not necessarily the set of all branches,
as the remote tracking branches are not listed in this set, even if it
appears in the "refs/heads/*" namespace (e.g. an old style repository).

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Automatically skip tracking branches in branch menu.
Shawn O. Pearce [Sat, 25 Nov 2006 08:33:03 +0000 (03:33 -0500)] 
git-gui: Automatically skip tracking branches in branch menu.

Since the user should not work on a tracking branch we automatically
hide any branch which is used as a tracking branch by either a
remote.<name>.fetch config entry or by a Pull: line in a remotes file.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Abort on not implemented branch switching.
Shawn O. Pearce [Sat, 25 Nov 2006 07:47:18 +0000 (02:47 -0500)] 
git-gui: Abort on not implemented branch switching.

I'm not currently ready to implement branch switching, so I'm just
going to punt on it for now.  :-)

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Parse off refs/remotes when showing current branch.
Shawn O. Pearce [Sat, 25 Nov 2006 07:45:19 +0000 (02:45 -0500)] 
git-gui: Parse off refs/remotes when showing current branch.

Even though the user shouldn't have a remote branch checked out, if
they do we should still show as short of the branch name as possible.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Created Branch menu.
Shawn O. Pearce [Fri, 24 Nov 2006 22:30:12 +0000 (17:30 -0500)] 
git-gui: Created Branch menu.

This is an early start at branch management from within git-gui.  The
branch menu has create/delete command entries to create and delete
branches as well as a list of radiobutton entries for each branch
found in the repository through for-each-ref.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Support file state MD (modified/deleted).
Shawn O. Pearce [Fri, 24 Nov 2006 20:59:34 +0000 (15:59 -0500)] 
git-gui: Support file state MD (modified/deleted).

Apparently I missed the file state MD, which is a file modified and
updated in the index but then removed from the working directory.  This
should be treated just like AD, an added file which has been deleted from
the working directory.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Display the current branch.
Shawn O. Pearce [Fri, 24 Nov 2006 20:38:18 +0000 (15:38 -0500)] 
git-gui: Display the current branch.

Users want to know what branch they are sitting on before making a commit,
as they may need to switch to a different branch first.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Added revert changes command.
Shawn O. Pearce [Fri, 24 Nov 2006 02:40:45 +0000 (21:40 -0500)] 
git-gui: Added revert changes command.

Users sometimes need to be able to throw away locally modified files
in order to go back to the last committed version of that file.  To
perform a revert the user must first uninclude each file from the new
commit as the working file must at least partially match the index,
and we use git-checkout-index to update the working directory.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Improve pull error dialogs.
Shawn O. Pearce [Thu, 23 Nov 2006 00:24:41 +0000 (19:24 -0500)] 
git-gui: Improve pull error dialogs.

Just like prior to a commit its only an informational message that
we refuse to perform a pull on a dirty working directory.  Therefore
we should not use an error icon.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Don't start 'gitk --all' on Mac OS X.
Shawn O. Pearce [Wed, 22 Nov 2006 03:58:28 +0000 (22:58 -0500)] 
git-gui: Don't start 'gitk --all' on Mac OS X.

Since gitk is currently broken on Mac OS X and is unable to start itself
when given command line parameters just don't offer the "Visual All
Branches" menu option on Mac OS X.

Once this feature of gitk is fixed we should change this section of code
to make sure a working version of gitk will be executed before we offer
the option up to the user.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Added menu command to visualize all branches.
Shawn O. Pearce [Wed, 22 Nov 2006 01:33:09 +0000 (20:33 -0500)] 
git-gui: Added menu command to visualize all branches.

Sometimes its useful to start gitk with the --all option, to view all of
the known branches and tags within this repository.  Rather than making
the user startup gitk and then edit the view we can pass the option along
for them.

This also makes it slightly more explicit, that when gitk starts up by
default its showing the current branch and not everything.  Yes gitk
isn't showing that to the user, but the fact that the user had to make
a decision between seeing this current branch or all branches will
hopefully make them study gitk's display before jumping to a conclusion.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Refactor M1 binding selection.
Shawn O. Pearce [Wed, 22 Nov 2006 01:21:11 +0000 (20:21 -0500)] 
git-gui: Refactor M1 binding selection.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Added configuration editor TODO list.
Shawn O. Pearce [Tue, 21 Nov 2006 20:38:49 +0000 (15:38 -0500)] 
git-gui: Added configuration editor TODO list.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Warn Cygwin users about possible environment issues.
Shawn O. Pearce [Tue, 21 Nov 2006 20:28:14 +0000 (15:28 -0500)] 
git-gui: Warn Cygwin users about possible environment issues.

Because the Tcl binary distributed with Cygwin tends to not pass along
its own environment (the env array) to its children, its unlikely that
any Git commands spawned by git-gui will receive the same environment
variables that git-gui itself received from the shell which started it.

If the user is counting on environment variables to pass down, like say
GIT_INDEX_FILE, they may not, so we warn them during git-gui startup
that things may not work out as the user intended.  Perhaps one day
when git-gui and git are running on native Windows (rather than through
the Cygwin emulation layers) things will work better.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Correct is_MacOSX platform test.
Shawn O. Pearce [Tue, 21 Nov 2006 17:00:50 +0000 (12:00 -0500)] 
git-gui: Correct is_MacOSX platform test.

Darwn based UNIX systems are not necessarily Mac OS X.  However the only
windowing system used by Tk that is Mac OS X is 'aqua', and only 'aqua'
exists on Mac OS X.  Therefore this is a more reliable test for the
Macintosh platform.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Abstract out windows platform test to is_Windows proc.
Shawn O. Pearce [Tue, 21 Nov 2006 16:57:41 +0000 (11:57 -0500)] 
git-gui: Abstract out windows platform test to is_Windows proc.

Like the is_MacOSX proc we shouldn't keep repeating the platform test
for Windows.  Instead abstract the code out into a procedure and use
the procedure whenever we need to do something special.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Include the Tcl/Tk version in the about dialog.
Shawn O. Pearce [Tue, 21 Nov 2006 07:46:51 +0000 (02:46 -0500)] 
git-gui: Include the Tcl/Tk version in the about dialog.

Users may need to know what version of Tcl they are running git-gui
under, in case there is an interesting interface quirk or other
compatability problem we don't know about right now that we may
need to explore (and maybe fix).  Since its simple enough to show
a line with this version data we should do so.

We also try to reduce the amount of text shown as often the Tcl and Tk
version numbers will be identical; when this happens we should only show
the one version number.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Make the copyright notice serve double duty.
Shawn O. Pearce [Tue, 21 Nov 2006 07:36:55 +0000 (02:36 -0500)] 
git-gui: Make the copyright notice serve double duty.

The copyright notice we display in the about dialog should be the same
as the one at the top of our source code.  By putting the copyright
notice that appears at the top of our source code into a global variable
rather than a comment we can trivially make them the same at all times.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Be more Macintosh like.
Shawn O. Pearce [Tue, 21 Nov 2006 07:33:56 +0000 (02:33 -0500)] 
git-gui: Be more Macintosh like.

It is tradition for applications to store their about and preferences
menu options within the application menu.  This is the first menu in
the menu bar, just after the apple menu.  Apparently the way to access
this menu from Tk on Mac OS X systems is to create a special menu whose
name ends in ".apple" and place it into the menu bar.

So now if we are on Mac OS X we move our about menu and our options menu
into the application menu, like other Mac OS X applications.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
18 years agogit-gui: Added about dialog box.
Shawn O. Pearce [Tue, 21 Nov 2006 04:55:51 +0000 (23:55 -0500)] 
git-gui: Added about dialog box.

Created a help menu with an about dialog box.  This about dialog
shows the copyright notice for the application, the fact that it
is covered by the GPL v2.0 or later, the authors, and the current
version of Git it is invoking when users perform actions within it.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>