git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Eric Raible <raible@gmail.com>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Eric Raible <raible+git@gmail.com>, git@vger.kernel.org
Subject: Re: RFH: spawning pager takes long time when when unconnected from  network
Date: Tue, 10 Feb 2009 02:53:31 -0800	[thread overview]
Message-ID: <279b37b20902100253v3cfd8e45kefa6da7de2ea4a4b@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.1.00.0902101124160.10279@pacific.mpi-cbg.de>

On Tue, Feb 10, 2009 at 2:24 AM, Johannes Schindelin
<Johannes.Schindelin@gmx.de> wrote:
> Hi,
>
> On Tue, 10 Feb 2009, Eric Raible wrote:
>
>> Johannes Sixt <j.sixt <at> viscovery.net> writes:
>>
>> > Do you have a directory in your PATH that points to a disconnected network
>> > drive? Do you use the default pager settings, or did you customize the pager?
>>
>> All components in my path are local.
>
> Could you run it with GIT_TRACE=<file>?
>
> Ciao,
> Dscho
>

The first thing I tried was to set GIT_TRACE=1, and it doesn't give much:
trace: built-in: git 'diff' 'ss' '--name-only'
trace: run_command: 'sh' '-c' 'less'

The long pause in question begins after the above output is produced
but before the list of files is shown.  The output is identical when setting
GIT_TRACE to a file.

When I pipe the git diff to less or set core.pager="",
then obviously it doesn't call run_command.

Since I can't really reproduce this in the debugger
(because the "bug" is an intermittent delay of 3-10 seconds), I tried
littering the code with trace_printf's.  Didn't really help, partially because
after the dup2 (which is part of spawning the pager process)
they don't on stdout any longer.  Now that I know about setting
GIT_TRACE to a file perhaps this is worth trying again.

But once it started to look like the delay was in the xdiff machinery,
I figured I was chasing a side-effect, and that the real problem was
some interaction between the two processes.  That's when I sent
my first RFH  to the list.

- Eric

  reply	other threads:[~2009-02-10 10:55 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-10  2:07 RFH: spawning pager takes long time when when unconnected from network Eric Raible
2009-02-10  2:27 ` Eric Raible
2009-02-10  7:57 ` Johannes Sixt
2009-02-10  9:20   ` Eric Raible
2009-02-10 10:24     ` Johannes Schindelin
2009-02-10 10:53       ` Eric Raible [this message]
2009-02-10 11:03         ` Johannes Schindelin
2009-02-10 11:39           ` Eric Raible
2009-02-10 11:07         ` Jeff King
2009-02-10 11:55           ` Eric Raible
2009-02-10 16:34             ` Jay Soffian
2009-02-10 11:09         ` Andreas Ericsson

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

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

  git send-email \
    --in-reply-to=279b37b20902100253v3cfd8e45kefa6da7de2ea4a4b@mail.gmail.com \
    --to=raible@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=raible+git@gmail.com \
    /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.
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 NNTP newsgroup(s).