git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Sam Vilain <sam@vilain.net>
To: "João Abecasis" <joao@abecasis.name>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] git-svn: follow revisions of svm-mirrored repositories
Date: Mon, 16 Jun 2008 09:30:29 +1200	[thread overview]
Message-ID: <485589F5.6020409@vilain.net> (raw)
In-Reply-To: <7bf6f1d20806132102x71422617s26260fdc348a7c04@mail.gmail.com>

João Abecasis wrote:
> Hi,
> 
> When using an svk mirror repository as the source for git-svn,
> find-rev and rebase don't work.
> 
> find-rev takes a while, while it traverses and processes commit logs
> for the branch, and ultimately fails with the error message: "Unable
> to determine upstream SVN information from git-svn history". This
> happens because find-rev doesn't relate information in the commit
> messages to the internal svm-source revision maps.
> 
> Similarly, rebase is faster but still exits with the message "Unable
> to determine upstream SVN information from working tree history".
> 
> The attached patch fixes a couple of underlying issues to get at least
> these two commands working. AFAICT it still works well with plain svn
> repositories.
> 
> Can this be merged upstream? Any comments are welcome.

Can you give an approximate series of commands that led to this not
working?  Just to clarify what happened.  Ideally, it would be a test
case; see if you can add it to the existing SVM test case.  In fact this
might be a regression compared to the original support, due to lack of
tests - in which case it would be good to fix this "for good".

Also, please try to send your patches inline if possible, or at least
try to get them to be Content-Disposition: inline, it makes review
easier for casual list subscribers.

Thanks,
Sam.

  reply	other threads:[~2008-06-15 21:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-14  4:02 [PATCH] git-svn: follow revisions of svm-mirrored repositories João Abecasis
2008-06-15 21:30 ` Sam Vilain [this message]
2008-06-18  2:57   ` João Abecasis
2008-06-18  2:58     ` João Abecasis
  -- strict thread matches above, loose matches on Subject: below --
2008-06-26 16:24 João Abecasis

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

  List information: http://vger.kernel.org/majordomo-info.html

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

  git send-email \
    --in-reply-to=485589F5.6020409@vilain.net \
    --to=sam@vilain.net \
    --cc=git@vger.kernel.org \
    --cc=joao@abecasis.name \
    /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.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).