From: Eric S. Raymond <esr@thyrsus.com>
Date: Wed, 10 Nov 2010 21:56:12 +0000 (-0500)
Subject: Warning about lost metadata.
X-Git-Tag: 0.1~72
X-Git-Url: http://thyrsus.com/gitweb/?a=commitdiff_plain;h=c592cbaafb31da73971ecc18e5d0d8e83a445441;p=rcs-fast-import.git

Warning about lost metadata.
---

diff --git a/rcs-fast-import b/rcs-fast-import
index 01cd98d..69bfdc0 100755
--- a/rcs-fast-import
+++ b/rcs-fast-import
@@ -36,9 +36,9 @@ If D, R, or C operations are present, the conversion will not round-trip
 properly (that is, rcs-fast-export | rcs-fast-import won't be an exact
 copy) with differences near deleted, renamed, and copied files.
 
-Warning: All tag metadata other than the name will be lost.  The person
-and date imputed to commits is from the author metadata if that exists,
-otherwise from the committer metadata.
+Warning: All tag metadata other than the name will be lost.  The date
+imputed to commits is from the author metadata if that exists, otherwise
+from the committer metadata. Committer and author information is lost.
 """
 
 import sys, os, getopt, re, cStringIO, subprocess, shutil
@@ -595,13 +595,15 @@ class RCS:
                     except OSError:
                         raise FatalException("can't create %s." % rcsdir)
                 if verbose >= DEBUG_SHUFFLE:
-                    announce("copying %s to %s" \
-                             % (copyfile, os.path.dirname(path) or '.'))
+                    announce("copying %s to %s" % (copyfile, path))
                 if not dryrun:
-                    shutil.copy(copyfile, os.path.dirname(path))
+                    shutil.copy(copyfile, path)
             responsible = event.authors[0] or event.committer
-            command = "ci "
-            command += " -w%s" % responsible.email
+            command = "ci -l"
+            # If we were going to try to match committer or author
+            # email addresses to local names, this would be the place
+            # to do it.  Not a whole lot of point in trying, though.
+            #command += " -w%s" % responsible.email
             command += " -d'%s'" % responsible.date.rfc822()
             command += " " + " ".join(inargs)
             with popen_or_die(command, "checkin", "w") as fp: