All of lore.kernel.org
 help / color / mirror / Atom feed
From: Seth Falcon <sethfalcon@gmail.com>
To: Eric Wong <normalperson@yhbt.net>
Cc: Junio C Hamano <junkio@cox.net>, git@vger.kernel.org
Subject: Re: [PATCH] git-svn: avoid respewing similar error messages for missing paths
Date: Mon, 02 Apr 2007 03:30:37 -0700	[thread overview]
Message-ID: <m2lkhb6pbm.fsf@gmail.com> (raw)
In-Reply-To: <20070331005448.GA6948@muzzle> (Eric Wong's message of "Fri, 30 Mar 2007 17:54:48 -0700")

Hi Eric,

Eric Wong <normalperson@yhbt.net> writes:

> We ignore errors if the path we're tracking did not exist for
> a particular revision range, but we still print out warnings
> telling the user about that.
>
> As pointed out by Seth Falcon, this amounts to a lot of warnings
> that could confuse and worry users.  I'm not entirely comfortable
> completely silencing the warnings, but showing one warning per
> path that we track should be reasonable.

Thanks, this is much quieter.

I have the impression that the initial fetch phase is taking longer
than it used to.  This could easily be due to network and server load
related issues, but I wonder if something in the parent following /
path searching is different.  The first git svn fetch seems to do
nothing for a very long time (except emit a few of the warnings) and
then once it starts actually adding commits it goes quite fast.  This
isn't really a problem as the initial fetch is a fairly rare
operation, but I wondered if this made sense based on recent git-svn
changes...


+ seth

  reply	other threads:[~2007-04-02 10:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-26 17:02 git svn fetch spewing warnings Seth Falcon
2007-03-27 19:00 ` Eric Wong
2007-03-27 19:17   ` Seth Falcon
2007-03-31  0:54     ` [PATCH] git-svn: avoid respewing similar error messages for missing paths Eric Wong
2007-04-02 10:30       ` Seth Falcon [this message]
2007-04-02 18:46         ` Eric Wong

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=m2lkhb6pbm.fsf@gmail.com \
    --to=sethfalcon@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.net \
    --cc=normalperson@yhbt.net \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.