Anton Korobeynikov <asl@xxxxxxxxxxxx> wrote: > Hello, Everyone. > > I noticed this bug several times. Consider the following conditions are > met: > - We're syncing from svn using git-svn :) > - We have authors file provided > - We have a changeset with author unlisted in the authors file. > > git-svn dies due to the following code: > sub check_author { > my ($author) = @_; > if (!defined $author || length $author == 0) { > $author = '(no author)'; > } > if (defined $::_authors && ! defined $::users{$author}) { > die "Author: $author not defined in $::_authors file\n"; > } > $author; > } > > Unfortunately it leaves repository in some middle state: git-svn itself > thinks, that it synced with everything, but git itself doesn't "see" any > changesets anymore. I found no way to repair tree after such situation, > so I had to repull from scratch. > > I found myself, that this should be warning (and fix in this case is > trivial), not error (maybe some commandline switch to control behaviour, > etc). It can be even error, but breaking tree is definitely bug in this > case. You should be able to change the numbers in *-maxRev back to an old revision in .git/svn/.metadata. Does that fix things for you so you can resume synching again? I'll have to investigate the die()-ing of check_authors since that should cause git-svn to quit before the maxRev numbers get incremented. -- Eric Wong - 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