{"thread":{"id":"14593","subject":"[PATCH] In perforce, RCS keywords are case-sensitive","startedAt":"2008-07-22T16:48:57Z","lastAt":"2008-07-24T07:05:21Z","messageCount":2,"participants":["Daniel Barkalow","Simon Hausmann"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"84333","messageId":"alpine.LNX.1.00.0807221235000.19665@iabervon.org","threadId":"14593","inReplyTo":null,"subject":"[PATCH] In perforce, RCS keywords are case-sensitive","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2008-07-22T16:48:57Z","receivedAt":"2008-07-22T16:48:57Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"At least, this is true in 2007.2, according to the documentation.\n\nSigned-off-by: Daniel Barkalow <barkalow@iabervon.org>\n---\n\nA line of perl with the variable $file in a string followed by a different \nvariable not in the string, for example, doesn't get mangled by p4 and may \ntherefore appear in a p4 depot.\n\nI don't know if the \"old-style keyword expansion\" is case-sensitive, \nthough, so I'm leaving that alone.\n\n contrib/fast-import/git-p4 |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4\nindex d8de9f6..1ee612e 100755\n--- a/contrib/fast-import/git-p4\n+++ b/contrib/fast-import/git-p4\n@@ -902,7 +902,7 @@ class P4Sync(Command):\n             if stat['type'] in ('text+ko', 'unicode+ko', 'binary+ko'):\n                 text = re.sub(r'(?i)\\$(Id|Header):[^$]*\\$',r'$\\1$', text)\n             elif stat['type'] in ('text+k', 'ktext', 'kxtext', 'unicode+k', 'binary+k'):\n-                text = re.sub(r'(?i)\\$(Id|Header|Author|Date|DateTime|Change|File|Revision):[^$]*\\$',r'$\\1$', text)\n+                text = re.sub(r'\\$(Id|Header|Author|Date|DateTime|Change|File|Revision):[^$]*\\$',r'$\\1$', text)\n \n             contents[stat['depotFile']] = text\n \n-- \n1.5.6.rc2.26.g8c37.dirty\n"},{"id":"84680","messageId":"200807240912.18546.simon@lst.de","threadId":"14593","inReplyTo":"alpine.LNX.1.00.0807221235000.19665@iabervon.org","subject":"Re: [PATCH] In perforce, RCS keywords are case-sensitive","fromName":"Simon Hausmann","fromEmail":"simon@lst.de","sentAt":"2008-07-24T07:05:21Z","receivedAt":"2008-07-24T07:05:21Z","isPatch":true,"sender":{"key":"hausmann@kde.org","avatar":"https://gravatar.com/avatar/bc9aad4fb31dce17eb66e690e7b51fe980c62da3c225c785da35dd806b8da778?d=mp&s=160"},"body":"On Tuesday 22 July 2008 18:48:57 Daniel Barkalow wrote:\n> At least, this is true in 2007.2, according to the documentation.\n>\n> Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>\n\nAcked-By: Simon Hausmann <simon@lst.de>\n\nThanks,\nSimon\n\n> A line of perl with the variable $file in a string followed by a different\n> variable not in the string, for example, doesn't get mangled by p4 and may\n> therefore appear in a p4 depot.\n>\n> I don't know if the \"old-style keyword expansion\" is case-sensitive,\n> though, so I'm leaving that alone.\n>\n>  contrib/fast-import/git-p4 |    2 +-\n>  1 files changed, 1 insertions(+), 1 deletions(-)\n>\n> diff --git a/contrib/fast-import/git-p4 b/contrib/fast-import/git-p4\n> index d8de9f6..1ee612e 100755\n> --- a/contrib/fast-import/git-p4\n> +++ b/contrib/fast-import/git-p4\n> @@ -902,7 +902,7 @@ class P4Sync(Command):\n>              if stat['type'] in ('text+ko', 'unicode+ko', 'binary+ko'):\n>                  text = re.sub(r'(?i)\\$(Id|Header):[^$]*\\$',r'$\\1$', text)\n>              elif stat['type'] in ('text+k', 'ktext', 'kxtext',\n> 'unicode+k', 'binary+k'): -                text =\n> re.sub(r'(?i)\\$(Id|Header|Author|Date|DateTime|Change|File|Revision):[^$]*\\\n>$',r'$\\1$', text) +                text =\n> re.sub(r'\\$(Id|Header|Author|Date|DateTime|Change|File|Revision):[^$]*\\$',r\n>'$\\1$', text)\n>\n>              contents[stat['depotFile']] = text\n"}]}