From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Wong Subject: Re: t9107-git-svn-migrate.sh fails Date: Sun, 27 May 2007 10:23:36 -0700 Message-ID: <20070527172336.GA27725@muzzle> References: <20070526093534.GB12639@muzzle> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: git@vger.kernel.org To: Johannes Schindelin X-From: git-owner@vger.kernel.org Sun May 27 19:23:59 2007 Return-path: Envelope-to: gcvg-git@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1HsMTD-0000og-PM for gcvg-git@gmane.org; Sun, 27 May 2007 19:23:56 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751501AbXE0RXq (ORCPT ); Sun, 27 May 2007 13:23:46 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751694AbXE0RXq (ORCPT ); Sun, 27 May 2007 13:23:46 -0400 Received: from hand.yhbt.net ([66.150.188.102]:42361 "EHLO hand.yhbt.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751501AbXE0RXp (ORCPT ); Sun, 27 May 2007 13:23:45 -0400 Received: from hand.yhbt.net (localhost [127.0.0.1]) by hand.yhbt.net (Postfix) with SMTP id 55E482DC032; Sun, 27 May 2007 10:23:37 -0700 (PDT) Received: by hand.yhbt.net (sSMTP sendmail emulation); Sun, 27 May 2007 10:23:36 -0700 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.13 (2006-08-11) Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: Johannes Schindelin wrote: > Hi, > > On Sat, 26 May 2007, Eric Wong wrote: > > > Johannes Schindelin wrote: > > > > > > I recently wanted to use git-svn, and installed the svn perl bindings. > > > Since then, t9107 is failing: > > > > > > -- snip -- > > > [...] > > > Rebuilding .git/svn/git-svn/.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > ... > > > r1 = b9b82a419abdbb54f51a41bc8a3118b28c791ac1 > > > Done rebuilding > > > .git/svn/git-svn/.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > diff --git > > > a/home/gene099/my/git/t/trash/.git/svn/trunk/.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > b/home/gene099/my/git/t/trash/.git/svn/trunk/.rev_db > > > index d3f1b6e..01d8afd 100644 > > > --- > > > a/home/gene099/my/git/t/trash/.git/svn/trunk/.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > +++ b/home/gene099/my/git/t/trash/.git/svn/trunk/.rev_db > > > @@ -1,2 +1 @@ > > > -0000000000000000000000000000000000000000 > > > -6aa651a66730888e854a8de54199d62ffa402739 > > > +.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > \ No newline at end of file > > > * FAIL 7: .rev_db auto-converted to .rev_db.UUID > > > > > > git-svn fetch -i trunk && > > > [...] > > > -- snap -- > > > > > > Usually I try to fix things like this myself, but I really have to get > > > some dinner now. Besides, other people than me seem to be way more clever > > > with perl code. > > > > .git/svn/trunk/.rev_db should be > > a symlink to > > .git/svn/trunk/.rev_db.f944dda1-3e31-0410-9475-8f268450faf6 > > > > I keep .rev_db around as a symlink for backwards compatibility in case > > the user wants to downgrade. > > > > I'm running cmp(1) to compare the file and symlink. Are you running > > diff2[1] replacements in your tree and it's not understanding symlinks? > > D'oh! > > Yes, I did a wholesale replacement of all cmp and diff invocations to > git-diff. > > Two thoughts: > > - why don't you check it with readlink? I'm not sure about the portability of readlink(1). cmp(1) is used in several places already, whereas readlink(1) is not. > - git diff should really output something when a file is compared > to a symlink... > > I guess the second is something for my TODO list. > > > > Anybody knows how to fix this? > > > > Works for me(TM). > > Changing the check back to cmp make it work here, too. > > > [1] - I think that's what you called it. I have had trouble keeping > > up-to-date with git things lately. > > Yep, I called it diff2. But we found out that we can DWIM git-diff to do > it; if in doubt, just say "git diff --no-index". Cool :) -- Eric Wong