a small but vexing inconsistency

------=_Part_6039_57173131.1433256373935
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Today I finally realized why my cvs2git conversion was littered with 
extra no-change commits and such: contrib/git-move-refs.py wasn't doing 
anything. it took not too long to figure out why:

ctx.symbol_commit_message = (
     "This commit was manufactured by cvs2git to create %(symbol_type)s "
     "'%(symbol_name)s'."
     )

versus

             if subject.startswith("This commit was manufactured by 
cvs2svn") \

Now this is mostly my fault for not validating that git-move-refs was 
doing what it's supposed to, but I have a trivial patch anyway.

regards,

   --jh

------------------------------------------------------
http://cvs2svn.tigris.org/ds/viewMessage.do?dsForumId=1667&dsMessageId=3120338

To unsubscribe from this discussion, e-mail: [[email protected]].
------=_Part_6039_57173131.1433256373935
Content-Type: text/x-patch; name=svn-move-refs.diff
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename=svn-move-refs.diff

Index: contrib/git-move-refs.py
===================================================================
--- contrib/git-move-refs.py	(revision 5462)
+++ contrib/git-move-refs.py	(working copy)
@@ -111,7 +111,7 @@
             elif objtype != "commit":
                 continue
 
-            if subject.startswith("This commit was manufactured by cvs2svn") \
+            if subject.startswith("This commit was manufactured by cvs2") \
                    or not subject:
                 # We shall try to move this ref, if possible
                 parent_list = []

------=_Part_6039_57173131.1433256373935--