{"thread":{"id":"30724","subject":"Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","startedAt":"2012-06-06T10:28:51Z","lastAt":"2012-06-07T18:47:14Z","messageCount":9,"participants":["John Wiegley","Junio C Hamano","Thomas Adam","Michael Haggerty","Andreas Schwab"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"192971","messageId":"m24nqoohss.fsf@gmail.com","threadId":"30724","inReplyTo":null,"subject":"Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"John Wiegley","fromEmail":"jwiegley@gmail.com","sentAt":"2012-06-06T10:28:51Z","receivedAt":"2012-06-06T10:28:51Z","isPatch":false,"sender":{"key":"jwiegley@gmail.com","avatar":"https://gravatar.com/avatar/65ad91a7e7d1b6423bcb82a5bcfd91167f7064b752bd259141e9dc2d48fe76be?d=mp&s=160"},"body":"I've spoken to the author of this commit, Matthias Urlichs. Here is an\nexcerpt of our conversation:\n\n> On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:\n> > Just wanted to let you know that this bit me. I have a client whose CVS\n> > repository I'm converting to Git, and they have _many_ log messages that\n> > are larger than 32k in size.\n> \n> Feel free to submit a patch that reverts this. These days, there's probably\n> no user of cvs2git left, but at that time it was important to get the same\n> commit IDs back.\n\nThis just needs to be reverted:\n\n git revert e371046b6473907aa6d62b7862a3afe9d33561e1\n\nThanks,\n John\n"},{"id":"193000","messageId":"7vfwa8mj7l.fsf@alter.siamese.dyndns.org","threadId":"30724","inReplyTo":"m24nqoohss.fsf@gmail.com","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-06-06T17:41:18Z","receivedAt":"2012-06-06T17:41:18Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Wiegley writes:\n\n> I've spoken to the author of this commit, Matthias Urlichs. Here is an\n> excerpt of our conversation:\n>\n>> On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:\n>> > Just wanted to let you know that this bit me. I have a client whose CVS\n>> > repository I'm converting to Git, and they have _many_ log messages that\n>> > are larger than 32k in size.\n>> \n>> Feel free to submit a patch that reverts this. These days, there's probably\n>> no user of cvs2git left, but at that time it was important to get the same\n>> commit IDs back.\n>\n> This just needs to be reverted:\n>\n> git revert e371046b6473907aa6d62b7862a3afe9d33561e1\n\nThat ancient commit does two things, and one thing that it claims to\ndo does not have anything to do with 32k limit.\n\nPlease send in a patch that exactly addresses the issue (it is\nunclear if you want to keep or drop the removal of trailing\nwhitespaces that is done by that commit). The proposed log message\nneeds to justify why breaking other people's repositories that were\nconverted by an ancient version of cvs2git is lessor of two evils\n(the other one being logs longer than 32k are not kept for you).\n\nThanks.\n"},{"id":"193001","messageId":"CA+39Oz4f_Wn1cVzqNWO76HZWa4AswSBpbriaRc0OznapVLJfGg@mail.gmail.com","threadId":"30724","inReplyTo":"m24nqoohss.fsf@gmail.com","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Thomas Adam","fromEmail":"thomas@xteddy.org","sentAt":"2012-06-06T17:54:10Z","receivedAt":"2012-06-06T17:54:10Z","isPatch":false,"sender":{"key":"thomas@xteddy.org","avatar":"https://gravatar.com/avatar/e7256db4738e501e5d2e84f00bb0bd99503165729573848a03330301fc2adc4a?d=mp&s=160"},"body":"On 6 June 2012 11:28, John Wiegley wrote:\n> I've spoken to the author of this commit, Matthias Urlichs.  Here is an\n> excerpt of our conversation:\n>\n>> On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:\n>> > Just wanted to let you know that this bit me.  I have a client whose CVS\n>> > repository I'm converting to Git, and they have _many_ log messages that\n>> > are larger than 32k in size.\n>>\n>> Feel free to submit a patch that reverts this. These days, there's probably\n>> no user of cvs2git left, but at that time it was important to get the same\n\nThis assertion is not only wrong, it's just ludicrous. The intended\nfunctionality has a statement of intent with regards to its\nfunctionality -- and as a user of cvs2git, I'd not want to lose *any*\nof that functionality.\n\nDon't be stupid with this. Please.\n\n-- Thomas Adam\n"},{"id":"193004","messageId":"m262b4uwiz.fsf@gmail.com","threadId":"30724","inReplyTo":"CA+39Oz4f_Wn1cVzqNWO76HZWa4AswSBpbriaRc0OznapVLJfGg@mail.gmail.com","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"John Wiegley","fromEmail":"johnw@newartisans.com","sentAt":"2012-06-06T18:26:28Z","receivedAt":"2012-06-06T18:26:28Z","isPatch":false,"sender":{"key":"johnw@newartisans.com","avatar":null},"body":">>>>> Thomas Adam writes:\n\n> This assertion is not only wrong, it's just ludicrous. The intended\n> functionality has a statement of intent with regards to its functionality --\n> and as a user of cvs2git, I'd not want to lose *any* of that functionality.\n> \n> Don't be stupid with this. Please.\n\nMy needs would be satisfied with an command-line option that removes the\nlimit, while keeping the status quo unaffected if it's not used. How does\nthat sound?\n\nJohn\n"},{"id":"193035","messageId":"4FD05B45.2090006@alum.mit.edu","threadId":"30724","inReplyTo":"CA+39Oz4f_Wn1cVzqNWO76HZWa4AswSBpbriaRc0OznapVLJfGg@mail.gmail.com","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2012-06-07T07:41:57Z","receivedAt":"2012-06-07T07:41:57Z","isPatch":false,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"On 06/06/2012 07:54 PM, Thomas Adam wrote:\n> On 6 June 2012 11:28, John Wiegley wrote:\n>> I've spoken to the author of this commit, Matthias Urlichs. Here is an\n>> excerpt of our conversation:\n>>\n>>> On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:\n>>>> Just wanted to let you know that this bit me. I have a client whose CVS\n>>>> repository I'm converting to Git, and they have _many_ log messages that\n>>>> are larger than 32k in size.\n>>>\n>>> Feel free to submit a patch that reverts this. These days, there's probably\n>>> no user of cvs2git left, but at that time it was important to get the same\n>\n> This assertion is not only wrong, it's just ludicrous. The intended\n> functionality has a statement of intent with regards to its\n> functionality -- and as a user of cvs2git, I'd not want to lose *any*\n> of that functionality.\n\nI was confused about this conversation. The commit that John Wiegley \nproposes to revert is from 2005. The \"cvs2git\" functionality in cvs2svn \nwas not added until 2007. So it must be that commit e371046b64 was \nadded for compatibility with some other cvs2git script (i.e., not the \none that is part of the cvs2svn project). Nowadays the only script \ncalled \"cvs2git\" that I ever see mentioned (and I maintain a Google \nsearch on that string) is the one from the cvs2svn project. So I assume \nthat the old \"cvs2git\" script (the one mentioned in commit e371046b64's \nlog message) has died off.\n\nThe current cvs2svn-based cvs2git script doesn't have any limitation on \nthe size of log messages and doesn't clean up their whitespace. The \nonly things that it does, in the default configuration, is check that \nthe message is ASCII (if not there are options to reencode it as UTF-8) \nand convert all EOL sequences into LF.\n\nTherefore I don't believe that there is any reason to preserve the \nfunctionality of commit e371046b64 in the name of compatibility with \ncvs2git.\n\nI have no opinion about whether it makes sense to revert/preserve the \ncommit for other reasons.\n\nMichael\n(the cvs2svn/cvs2git maintainer)\n\n-- \nMichael Haggerty\nmhagger@alum.mit.edu\nhttp://softwareswirl.blogspot.com/\n"},{"id":"193078","messageId":"7vd35bjcd6.fsf@alter.siamese.dyndns.org","threadId":"30724","inReplyTo":"4FD05B45.2090006@alum.mit.edu","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-06-07T16:49:41Z","receivedAt":"2012-06-07T16:49:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Michael Haggerty writes:\n\n> On 06/06/2012 07:54 PM, Thomas Adam wrote:\n>> On 6 June 2012 11:28, John Wiegley wrote:\n>>> I've spoken to the author of this commit, Matthias Urlichs. Here is an\n>>> excerpt of our conversation:\n>>>\n>>>> On Sat, 2012-04-21 at 00:08 -0500, John Wiegley wrote:\n>>>>> Just wanted to let you know that this bit me. I have a client whose CVS\n>>>>> repository I'm converting to Git, and they have _many_ log messages that\n>>>>> are larger than 32k in size.\n>>>>\n>>>> Feel free to submit a patch that reverts this. These days, there's probably\n>>>> no user of cvs2git left, but at that time it was important to get the same\n>>\n>> This assertion is not only wrong, it's just ludicrous. The intended\n>> functionality has a statement of intent with regards to its\n>> functionality -- and as a user of cvs2git, I'd not want to lose *any*\n>> of that functionality.\n>\n> I was confused about this conversation. The commit that John Wiegley\n> proposes to revert is from 2005. The \"cvs2git\" functionality in\n> cvs2svn was not added until 2007. So it must be that commit\n> e371046b64 was added for compatibility with some other cvs2git script\n> (i.e., not the one that is part of the cvs2svn project). Nowadays the\n> only script called \"cvs2git\" that I ever see mentioned (and I maintain\n> a Google search on that string) is the one from the cvs2svn project.\n> So I assume that the old \"cvs2git\" script (the one mentioned in commit\n> e371046b64's log message) has died off.\n\nThe way I read the log message of e371046b64 is that the repository\nresulting from the conversion without this patch will be different\nfor repositories that were originally converted by the older\ncvs2git. So my impression is that it does not matter if the older\ntool died long time ago. As long as repositories converted by it\nstill lives in the field, reverting the patch will start producing\ndifferent (and arguably more correct) results for people who are\nusing such a repository that started its life long time ago.\n\nThe potential negative impact is not huge for projects that used\ncvsimport for a one-time converion, and further developments all\nhappen in git, of course...\n"},{"id":"193079","messageId":"m23967vynk.fsf@igel.home","threadId":"30724","inReplyTo":"7vd35bjcd6.fsf@alter.siamese.dyndns.org","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Andreas Schwab","fromEmail":"schwab@linux-m68k.org","sentAt":"2012-06-07T17:07:27Z","receivedAt":"2012-06-07T17:07:27Z","isPatch":false,"sender":{"key":"schwab@linux-m68k.org","avatar":"https://avatars.githubusercontent.com/u/2175493?v=4"},"body":"Junio C Hamano writes:\n\n> The way I read the log message of e371046b64 is that the repository\n> resulting from the conversion without this patch will be different\n> for repositories that were originally converted by the older\n> cvs2git. So my impression is that it does not matter if the older\n> tool died long time ago. As long as repositories converted by it\n> still lives in the field, reverting the patch will start producing\n> different (and arguably more correct) results for people who are\n> using such a repository that started its life long time ago.\n\nOnly if the conversion is restarted from scratch. Otherwise, any\nexisting converted commits are preserved due to the incremental nature\nof git cvsimport.\n\nAndreas.\n\n-- \nAndreas Schwab, schwab@linux-m68k.org\nGPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5\n\"And now for something completely different.\"\n"},{"id":"193088","messageId":"7v3967huss.fsf@alter.siamese.dyndns.org","threadId":"30724","inReplyTo":"m23967vynk.fsf@igel.home","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-06-07T17:54:27Z","receivedAt":"2012-06-07T17:54:27Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Andreas Schwab writes:\n\n> Only if the conversion is restarted from scratch.\n\nYes, that was the use case I was the most worried about.\n\nOften a re-import is one way to validate what you have (and worse\nyet, what you based your recent work on), so unmatching commit\nobject names are red flags.\n"},{"id":"193097","messageId":"m2sje7ufgt.fsf@igel.home","threadId":"30724","inReplyTo":"7v3967huss.fsf@alter.siamese.dyndns.org","subject":"Re: Please revert e371046b6473907aa6d62b7862a3afe9d33561e1","fromName":"Andreas Schwab","fromEmail":"schwab@linux-m68k.org","sentAt":"2012-06-07T18:47:14Z","receivedAt":"2012-06-07T18:47:14Z","isPatch":false,"sender":{"key":"schwab@linux-m68k.org","avatar":"https://avatars.githubusercontent.com/u/2175493?v=4"},"body":"Junio C Hamano writes:\n\n> Andreas Schwab writes:\n>\n>> Only if the conversion is restarted from scratch.\n>\n> Yes, that was the use case I was the most worried about.\n>\n> Often a re-import is one way to validate what you have (and worse\n> yet, what you based your recent work on), so unmatching commit\n> object names are red flags.\n\nGiven the notorious unreliability of cvsps that doesn't look like a very\nserious change in comparison.\n\nAndreas.\n\n-- \nAndreas Schwab, schwab@linux-m68k.org\nGPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5\n\"And now for something completely different.\"\n"}]}