git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Alexander Gavrilov <angavrilov@gmail.com>
To: git@vger.kernel.org
Cc: Paul Mackerras <paulus@samba.org>
Subject: [PATCH (GITK) 0/6] Runaway process and commit selection fixes
Date: Sun, 27 Jul 2008 10:17:22 +0400	[thread overview]
Message-ID: <200807271017.23272.angavrilov@gmail.com> (raw)

This series includes three patches that I sent 8 days ago,
because I haven't received any reply so far.

These patches address the following problems:

1) Runaway processes (resend)

	As in the 'git gui blame' case, gitk back-end processes can sometimes
	run for a while without producing any output, e.g. diff-files on a slow
	filesystem.

	These patches make gitk explicitly kill its back-end processes.

2) Gitk stopping showing any diffs under random conditions.

3) Broken commit selection on view reload: gitk tried to preserve
  the selected commit, but usually failed because of code rot.

	I added selection preservation on Reload and Edit View.
	Update still should reset the selection to HEAD if anything changed.

	Also, if the previously selected commit was not found in the new view,
	gitk should fall back to selecting HEAD.


Alexander Gavrilov (6):
      gitk: Kill back-end processes on window close.
      gitk: Register diff-files & diff-index in commfd, to ensure kill.
      gitk: On Windows use a Cygwin-specific flag for kill.

      gitk: Fixed broken exception handling in diff.
      gitk: Fixed automatic row selection during load.
      gitk: Fallback to selecting the head commit upon load.

 gitk |  148 +++++++++++++++++++++++++++++++++++++++++++-----------------------
 1 files changed, 97 insertions(+), 51 deletions(-)

             reply	other threads:[~2008-07-27  6:18 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-27  6:17 Alexander Gavrilov [this message]
2008-07-27  6:18 ` [PATCH (GITK) 1/6] gitk: Kill back-end processes on window close Alexander Gavrilov
2008-07-27  6:19   ` [PATCH (GITK) 2/6] gitk: Register diff-files & diff-index in commfd, to ensure kill Alexander Gavrilov
2008-07-27  6:20     ` [PATCH (GITK) 3/6] gitk: On Windows use a Cygwin-specific flag for kill Alexander Gavrilov
2008-07-27  6:20       ` [PATCH (GITK) 4/6] gitk: Fixed broken exception handling in diff Alexander Gavrilov
2008-07-27  6:21         ` [PATCH (GITK) 5/6] gitk: Fixed automatic row selection during load Alexander Gavrilov
2008-07-27  6:22           ` [PATCH (GITK) 6/6] gitk: Fallback to selecting the head commit upon load Alexander Gavrilov
2008-07-31 11:25           ` [PATCH (GITK) 5/6] gitk: Fixed automatic row selection during load Paul Mackerras
2008-07-31 12:41             ` Alexander Gavrilov
2008-08-03  8:49               ` [RFC PATCH (GITK)] gitk: Allow overriding the default commit Alexander Gavrilov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200807271017.23272.angavrilov@gmail.com \
    --to=angavrilov@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=paulus@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).