git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonathan Nieder <jrnieder@gmail.com>
To: Stephen Bash <bash@genarts.com>
Cc: Git Mailing List <git@vger.kernel.org>,
	Sverre Rabbelier <srabbelier@gmail.com>,
	Ramkumar Ramachandra <artagnon@gmail.com>,
	Eric Wong <normalperson@yhbt.net>,
	David Barr <david.barr@cordelta.com>, Sam Vilain <sam@vilain.net>
Subject: Re: [PATCH] contrib/svn-fe: Fast script to remap svn history
Date: Mon, 22 Nov 2010 11:42:42 -0600	[thread overview]
Message-ID: <20101122174242.GD2084@burratino> (raw)
In-Reply-To: <15144262.32618.1290434477881.JavaMail.root@mail.hq.genarts.com>

Stephen Bash wrote:

> This has been discussed (and IMO it is essentially required to achieve high accuracy in the mapping):
> 
> http://thread.gmane.org/gmane.comp.version-control.git/158940/focus=159331

I think the suggestion of that thread was (tweaked a little)
something like this:

 - List of directories with copyfrom information.
   Prune them so no listed directory is an ancestor of another.
   The result would usually be a single directory name.

 - Record that directory's (or those directories') copyfrom
   information in the log message.

In general, I don't like limiting the information accessible to branch
mappers this way.  Maybe a branch mapper would like to look at the
copyfrom information for files instead of directories.  But this does
have the advantages of being simple and of not littering imported
trees with spurious files.

It also leaves open the question of how we would record unhandled node
properties (like svn:ignore and svn:eol) and empty directories, if at
all.

Probably in the end we will have to give up and provide multiple
options to choose between. :)

Jonathan

      reply	other threads:[~2010-11-22 17:43 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-07  6:06 [PATCH] contrib/svn-fe: Fast script to remap svn history David Barr
2010-10-07  6:29 ` Sverre Rabbelier
2010-10-07  7:17   ` David Michael Barr
2010-10-07  8:28   ` Jonathan Nieder
2010-11-21  5:17 ` Jonathan Nieder
2010-11-22 14:01   ` Stephen Bash
2010-11-22 17:42     ` Jonathan Nieder [this message]

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=20101122174242.GD2084@burratino \
    --to=jrnieder@gmail.com \
    --cc=artagnon@gmail.com \
    --cc=bash@genarts.com \
    --cc=david.barr@cordelta.com \
    --cc=git@vger.kernel.org \
    --cc=normalperson@yhbt.net \
    --cc=sam@vilain.net \
    --cc=srabbelier@gmail.com \
    /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 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 NNTP newsgroup(s).