On 06/06/2012 07:54 PM, Thomas Adam wrote:
On 6 June 2012 11:28, John Wiegley<jwiegley@xxxxxxxxx> wrote:
I've spoken to the author of this commit, Matthias Urlichs. Here is an
excerpt of our conversation:
On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:
Just wanted to let you know that this bit me. I have a client whose CVS
repository I'm converting to Git, and they have _many_ log messages that
are larger than 32k in size.
Feel free to submit a patch that reverts this. These days, there's probably
no user of cvs2git left, but at that time it was important to get the same
This assertion is not only wrong, it's just ludicrous. The intended
functionality has a statement of intent with regards to its
functionality -- and as a user of cvs2git, I'd not want to lose *any*
of that functionality.
I was confused about this conversation. The commit that John Wiegley
proposes to revert is from 2005. The "cvs2git" functionality in cvs2svn
was not added until 2007. So it must be that commit e371046b64 was
added for compatibility with some other cvs2git script (i.e., not the
one that is part of the cvs2svn project). Nowadays the only script
called "cvs2git" that I ever see mentioned (and I maintain a Google
search on that string) is the one from the cvs2svn project. So I assume
that the old "cvs2git" script (the one mentioned in commit e371046b64's
log message) has died off.
The current cvs2svn-based cvs2git script doesn't have any limitation on
the size of log messages and doesn't clean up their whitespace. The
only things that it does, in the default configuration, is check that
the message is ASCII (if not there are options to reencode it as UTF-8)
and convert all EOL sequences into LF.
Therefore I don't believe that there is any reason to preserve the
functionality of commit e371046b64 in the name of compatibility with
cvs2git.
I have no opinion about whether it makes sense to revert/preserve the
commit for other reasons.
Michael
(the cvs2svn/cvs2git maintainer)
--
Michael Haggerty
mhagger@xxxxxxxxxxxx
http://softwareswirl.blogspot.com/
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html