Deskin Miller <deskinm@xxxxxxxxx> wrote: > When initializing a git-svn repository from a Subversion repoository, > it is common to be interested in a path which did not exist in the > initial commit to Subversion. In a large repository, the initial fetch > may take some looking for the earliest existence of the path time while > the user receives no additional feedback. Print the highest revision > number scanned thus far to let the user know something is still > happening. > > Signed-off-by: Deskin Miller <deskinm@xxxxxxxxx> Thanks Deskin, The carriage return actually looks quite good :) Acked-by: From: Eric Wong <normalperson@xxxxxxxx> > --- > I'd never looked closely enough at gs_fetch_loop_common to grok what was > going on, and my previous experience with svn's get_log led me to think > that git-svn would slurp up log info for all commits at once. Silly of > me. > > Eric, things are much cleaner doing as you suggest. I added a > carriage-return at the end of the message because I like it that way, > and the line will be overwritten once it actually starts fetching data > from svn. I don't feel strongly about it though, so if it's better to > make it a newline, please feel free to change it and apply. > > Deskin Miller > > git-svn.perl | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/git-svn.perl b/git-svn.perl > index 79888a0..601e2a3 100755 > --- a/git-svn.perl > +++ b/git-svn.perl > @@ -4348,6 +4348,9 @@ sub gs_fetch_loop_common { > } > $self->get_log([$longest_path], $min, $max, 0, 1, 1, > sub { $revs{$_[1]} = _cb(@_) }); > + if ($err) { > + print "Checked through r$max\r"; > + } > if ($err && $max >= $head) { > print STDERR "Path '$longest_path' ", > "was probably deleted:\n", > -- -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html