{"thread":{"id":"13319","subject":"cvsexportcommit and cvsimport's -k option","startedAt":"2008-04-29T17:15:06Z","lastAt":"2008-05-02T20:11:02Z","messageCount":3,"participants":["Lars Alexander Noschinski","Robin Rosenberg","Lars Noschinski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"75549","messageId":"f6c6e2a5cc39.481773ba@rwth-aachen.de","threadId":"13319","inReplyTo":null,"subject":"cvsexportcommit and cvsimport's -k option","fromName":"Lars Alexander Noschinski","fromEmail":"lars.noschinski@rwth-aachen.de","sentAt":"2008-04-29T17:15:06Z","receivedAt":"2008-04-29T17:15:06Z","isPatch":false,"sender":{"key":"lars@public.noschinski.de","avatar":"https://gravatar.com/avatar/ca62bd8b265f2e26c89d39a4bfe7e390bfa6b16d6400e186e222d1c2382c66f2?d=mp&s=160"},"body":"[Please CC me on answers, as I'm not subscribed.]\n\nHello!\n\nIm using git to work on an CVS project. My work cycle is as follows:\n\n... hack away ...\ngit-cvsimport -i -r cvs -kk -v -d $CVSREPOS\ngit-rebase cvs # If work was done in the meantime\ngit-cvsexportcommit -cpuv $COMMIT # Multiple times, if necessary\ngit-cvsimport -i -r cvs -kk -v -d $CVSREPOS\ngit-checkout -b master cvs # commit ids are changed now\n\nAs far as I am concerned, this works fairly well, accept when for any reason\na line containing a CVS keyword (e.g. $Id$) is contained in a changeset. In\nthis case, patching fails, and I have to investigate why it fails, eventually\njust copying the old version from the cvs repository.\n\nWould it be possible to add an option to cvsexportcommit to detect such\nkilled keywords and ignore them?\n\n\nGreetings,\n   Lars.\n"},{"id":"75564","messageId":"200804292146.42262.robin.rosenberg.lists@dewire.com","threadId":"13319","inReplyTo":"f6c6e2a5cc39.481773ba@rwth-aachen.de","subject":"Re: cvsexportcommit and cvsimport's -k option","fromName":"Robin Rosenberg","fromEmail":"robin.rosenberg.lists@dewire.com","sentAt":"2008-04-29T19:46:42Z","receivedAt":"2008-04-29T19:46:42Z","isPatch":false,"sender":{"key":"robin.rosenberg@dewire.com","avatar":"https://avatars.githubusercontent.com/u/46357?v=4"},"body":"tisdagen den 29 april 2008 19.15.06 skrev Lars Alexander Noschinski:\n> ... hack away ...\n> git-cvsimport -i -r cvs -kk -v -d $CVSREPOS\n> git-rebase cvs # If work was done in the meantime\n> git-cvsexportcommit -cpuv $COMMIT # Multiple times, if necessary\n> git-cvsimport -i -r cvs -kk -v -d $CVSREPOS\n> git-checkout -b master cvs # commit ids are changed now\n>\n> As far as I am concerned, this works fairly well, accept when for any\n> reason a line containing a CVS keyword (e.g. $Id$) is contained in a\n> changeset. In this case, patching fails, and I have to investigate why it\n> fails, eventually just copying the old version from the cvs repository.\n>\n> Would it be possible to add an option to cvsexportcommit to detect such\n> killed keywords and ignore them?\n\nSure, but is it necessary? If the cvs checkout you use is checked out\nwith -kk there shouldn't be any problem.\n\nIf not, start by thinking up a few testcase for inclusion in \nt9200-git-cvsexportcommit.sh that illustrates the problem. Then we can\nchange cvsexporcommit until the new tests pass.\n\n-- robin\n"},{"id":"75875","messageId":"20080502201102.GA24914@lars.home.noschinski.de","threadId":"13319","inReplyTo":"200804292146.42262.robin.rosenberg.lists@dewire.com","subject":"Re: cvsexportcommit and cvsimport's -k option","fromName":"Lars Noschinski","fromEmail":"lars.noschinski@rwth-aachen.de","sentAt":"2008-05-02T20:11:02Z","receivedAt":"2008-05-02T20:11:02Z","isPatch":false,"sender":{"key":"lars@public.noschinski.de","avatar":"https://gravatar.com/avatar/ca62bd8b265f2e26c89d39a4bfe7e390bfa6b16d6400e186e222d1c2382c66f2?d=mp&s=160"},"body":"Hello!\n\n* Robin Rosenberg <robin.rosenberg.lists@dewire.com> [08-04-29 22:56]:\n>tisdagen den 29 april 2008 19.15.06 skrev Lars Alexander Noschinski:\n>> ... hack away ...\n>> git-cvsimport -i -r cvs -kk -v -d $CVSREPOS\n>> git-rebase cvs # If work was done in the meantime\n>> git-cvsexportcommit -cpuv $COMMIT # Multiple times, if necessary\n>> git-cvsimport -i -r cvs -kk -v -d $CVSREPOS\n>> git-checkout -b master cvs # commit ids are changed now\n>>\n>> As far as I am concerned, this works fairly well, accept when for any\n>> reason a line containing a CVS keyword (e.g. $Id$) is contained in a\n>> changeset. In this case, patching fails, and I have to investigate why it\n>> fails, eventually just copying the old version from the cvs repository.\n>>\n>> Would it be possible to add an option to cvsexportcommit to detect such\n>> killed keywords and ignore them?\n>\n>Sure, but is it necessary? If the cvs checkout you use is checked out\n>with -kk there shouldn't be any problem.\n\nErrm, now as you mention it ... Even it is clearly mentioned in\ngit-cvsimport(1), it did not realize, that -k just invokes cvs with -kk.\nI think this is a perfectly fine solution.\n\n>If not, start by thinking up a few testcase for inclusion in\n>t9200-git-cvsexportcommit.sh that illustrates the problem. Then we can\n>change cvsexporcommit until the new tests pass.\n\nAs cvs can do this for us, I think it is not necessary to introduce\nanother source of potential errors.\n\n\nThanks for your quick answer and sorry for me being dumb,\n     Lars\n"}]}