{"thread":{"id":"15453","subject":"Re: git-p4 and keyword expansion","startedAt":"2008-09-09T17:38:48Z","lastAt":"2008-09-09T17:38:48Z","messageCount":1,"participants":["Jing Xue"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"90242","messageId":"20080909133848.487451nasi1gnyww@intranet.digizenstudio.com","threadId":"15453","inReplyTo":null,"subject":"Re: git-p4 and keyword expansion","fromName":"Jing Xue","fromEmail":"jingxue@digizenstudio.com","sentAt":"2008-09-09T17:38:48Z","receivedAt":"2008-09-09T17:38:48Z","isPatch":false,"sender":{"key":"jingxue@digizenstudio.com","avatar":null},"body":"On Tue, Sep 09, 2008 at 04:44:11PM +0530, dhruva wrote:\n> Hello,\n\nHi,\n\n> I feel the configuration must be set the first time only, when you\n> clone using 'git-p4 clone'. Altering it in between will be very\n> confusing!\n> Ideally, the setting must be transferred when the git repo\n> (cloned from git-p4) is cloned using standard git. Is it something\n> possible (well, I am new to git and am exploring. Any extra\n> information would help).\n>\n> My proposal is as follows:\n> 1. Add an extra command line argument to 'git-p4 clone' to either  \n> enable/disable keyword expansion\n> 2. Store that information under the .git folder in a file that is  \n> copied when someone clones that repo\n> 3. Use the stored information in future 'git-p4 sync/rebase'\n\nAny way to make it optional would be welcome by me.\n\n> I agree that keyword expansion is bad but there is no alternative\n> when you work with a system that has keyword expansion and you\n> need interoperability (p4<->git)\n\nA patch would only fail if it has a hunk containing a line with\nkeyword expansion in it. So I for one am willing to _occasionally_\nhave to submit to perforce manually, for the benefits of no keyword\nexpansion - among other things, the ability to diff between\nbranches without the interferences is very important for me.\n\nBut that's just my 2 cents, and I'm just another git(-p4) user.\nIf you do come up with a \"formal\" patch, you might want to\nexplicitly add Simon Hausmann to the To list, for he's the git-p4\nauthor.\n\nCheers.\n-- \nJing\n\n----------------------------------------------------------------\nThis message was sent using IMP, the Internet Messaging Program.\n"}]}