{"thread":{"id":"27217","subject":"[PATCH RESEND] git-svn: don't confuse editors with an apostrophe","startedAt":"2011-04-29T11:40:14Z","lastAt":"2011-05-01T02:12:06Z","messageCount":5,"participants":["Carlos Martín Nieto","Junio C Hamano","Eric Wong"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"166730","messageId":"1304077214-1707-1-git-send-email-cmn@elego.de","threadId":"27217","inReplyTo":null,"subject":"[PATCH RESEND] git-svn: don't confuse editors with an apostrophe","fromName":"Carlos Martín Nieto","fromEmail":"cmn@elego.de","sentAt":"2011-04-29T11:40:14Z","receivedAt":"2011-04-29T11:40:14Z","isPatch":true,"sender":{"key":"cmn@elego.de","avatar":"https://avatars.githubusercontent.com/u/335443?v=4"},"body":"The quotation mark serving as an apostrophe confuses editors and they\nthink a large part of the rest of the file is a string. Change\n\"you're\" to \"you are\" to avoid this.\n\nSigned-off-by: Carlos Martín Nieto \nAcked-by: Eric Wong \n---\n\nI sent this on 31-03-2011 but apart from Eric's ack, there was no\nreaction and I don't see it in the patch queue.\n\n cmn\n\n git-svn.perl | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/git-svn.perl b/git-svn.perl\nindex bf0451b..bc5981d 100755\n--- a/git-svn.perl\n+++ b/git-svn.perl\n@@ -348,7 +348,7 @@ Usage: git svn [options] [arguments]\\n\n \t}\n \tprint $fd <<\"\";\n \\nGIT_SVN_ID may be set in the environment or via the --id/-i switch to an\n-arbitrary identifier if you're tracking multiple SVN branches/repositories in\n+arbitrary identifier if you are tracking multiple SVN branches/repositories in\n one git repository and want to keep them separate. See git-svn(1) for more\n information.\n \n-- \n1.7.4.2.437.g4fc7e.dirty\n"},{"id":"166749","messageId":"7viptxxbr7.fsf@alter.siamese.dyndns.org","threadId":"27217","inReplyTo":"1304077214-1707-1-git-send-email-cmn@elego.de","subject":"Re: [PATCH RESEND] git-svn: don't confuse editors with an apostrophe","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-29T16:28:44Z","receivedAt":"2011-04-29T16:28:44Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Carlos Martín Nieto writes:\n\n> The quotation mark serving as an apostrophe confuses editors and they\n> think a large part of the rest of the file is a string. Change\n> \"you're\" to \"you are\" to avoid this.\n>\n> Signed-off-by: Carlos Martín Nieto \n> Acked-by: Eric Wong \n> ---\n>\n> I sent this on 31-03-2011 but apart from Eric's ack, there was no\n> reaction and I don't see it in the patch queue.\n\nIt was on purpose that I did not apply it, as the usual mode of operation\nhas been for me to pull git-svn changes via Eric, and more importantly, I\ndid not think it was important.\n\nIt did not look like a bugfix that needed to be handled quickly (even by\nbypassing Eric if needed), and more importantly, I did not think your\njustification of the patch was sound. So I (implicitly) made it up to\nEric either to discard it or to include it as part of his pull-request.\n\nThe explanation Eric gave (Avoiding contractions may make life easier for\nfolks less familiar with English overall) makes some sense to me, but\nafter re-reading your justification today, it still does not make sense.\n\nFor one thing, many editors would not even think \"you\" is not a string and\n\"re\" and everything follows is a string, when \"you're\" is given to\nthem. Everything is a string in a text editor, isn't it?\n\nI suspect that you are talking about syntax highlighting, and some syntax\nhighlighter may be stupid to be confused, but in that case, at least you\nwould need s/editors/some editors/.\n\nMost fundamentally, I do not see why output from \"sub usage\" that lists\navailable commands when \"git svn -h\" is asked for has anything to do with\nan editor getting confused. Why do you have that output in your editor in\nthe first place?\n\nEven if the reason you have that output in your editor were sane, do you\nmean to say that _all_ messages we output from our program, not just \"git\nsvn\", need to be cleansed of contractions? That is madness, and I do not\nthink that is what you meant. You need to clarify when your \"do not\ncontract\" rule applies fairly narrowly to avoid such a misunderstanding.\n"},{"id":"166774","messageId":"20110429205130.GA2727@bee.lab.cmartin.tk","threadId":"27217","inReplyTo":"7viptxxbr7.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH RESEND] git-svn: don't confuse editors with an apostrophe","fromName":"Carlos Martín Nieto","fromEmail":"cmn@elego.de","sentAt":"2011-04-29T20:51:33Z","receivedAt":"2011-04-29T20:51:33Z","isPatch":true,"sender":{"key":"cmn@elego.de","avatar":"https://avatars.githubusercontent.com/u/335443?v=4"},"body":"On Fri, Apr 29, 2011 at 09:28:44AM -0700, Junio C Hamano wrote:\n> Carlos Martín Nieto writes:\n> \n> > The quotation mark serving as an apostrophe confuses editors and they\n> > think a large part of the rest of the file is a string. Change\n> > \"you're\" to \"you are\" to avoid this.\n> >\n> > Signed-off-by: Carlos Martín Nieto \n> > Acked-by: Eric Wong \n> > ---\n> >\n> > I sent this on 31-03-2011 but apart from Eric's ack, there was no\n> > reaction and I don't see it in the patch queue.\n> \n> It was on purpose that I did not apply it, as the usual mode of operation\n> has been for me to pull git-svn changes via Eric, and more importantly, I\n> did not think it was important.\n\n From his reply (adding Ack) I inferred he expected you to apply it.\n\n> \n> It did not look like a bugfix that needed to be handled quickly (even by\n> bypassing Eric if needed), and more importantly, I did not think your\n> justification of the patch was sound. So I (implicitly) made it up to\n> Eric either to discard it or to include it as part of his pull-request.\n> \n> The explanation Eric gave (Avoiding contractions may make life easier for\n> folks less familiar with English overall) makes some sense to me, but\n> after re-reading your justification today, it still does not make sense.\n> \n> For one thing, many editors would not even think \"you\" is not a string and\n> \"re\" and everything follows is a string, when \"you're\" is given to\n> them. Everything is a string in a text editor, isn't it?\n> \n> I suspect that you are talking about syntax highlighting, and some syntax\n> highlighter may be stupid to be confused, but in that case, at least you\n> would need s/editors/some editors/.\n\nIt confuses emacs and vim and I'd be surprised if any weren't, but\nI'll change the message a bit.\n\n> \n> Most fundamentally, I do not see why output from \"sub usage\" that lists\n> available commands when \"git svn -h\" is asked for has anything to do with\n> an editor getting confused. Why do you have that output in your editor in\n> the first place?\n\nIt's not the output of the tool, it's the perl script. It uses the cat\n<< EOF trick/method so there is a stray quotation mark and the editor\nthinks that's the start of a string, so half the file is green or red\nor whatever color your strings are.\n\nI'll mention this more explicitly in the commit log, as it looks like\nit's not clear enough.\n\n cmn\n"},{"id":"166776","messageId":"7v4o5gwx0o.fsf@alter.siamese.dyndns.org","threadId":"27217","inReplyTo":"20110429205130.GA2727@bee.lab.cmartin.tk","subject":"Re: [PATCH RESEND] git-svn: don't confuse editors with an apostrophe","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2011-04-29T21:47:03Z","receivedAt":"2011-04-29T21:47:03Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Carlos Martín Nieto writes:\n\n> It's not the output of the tool, it's the perl script. It uses the cat\n> << EOF trick/method so ...\n\nAhhhhh (a lightbulb dimly lits, and then explodes).\n\nAs it is generally a bad practice to cater to those who edit this file by\nchanging the output that is served to the end user, _unless_ the updated\noutput clearly gives a better experience to the end user [*1*], it never\noccurred to me that that is what you are doing in this patch.\n\nYes, the message should be made clearer.\n\nThanks for an explanation.\n\n[Footnote]\n\n*1* In other words, updating the output to help users is fine. And if\nsuch a change unconfuses your editor, that would be a nice side effect.\nDoing things the other way around is simply the tail wagging the dog.\n"},{"id":"166805","messageId":"20110501021206.GA23269@dcvr.yhbt.net","threadId":"27217","inReplyTo":"20110429205130.GA2727@bee.lab.cmartin.tk","subject":"Re: [PATCH RESEND] git-svn: don't confuse editors with an apostrophe","fromName":"Eric Wong","fromEmail":"normalperson@yhbt.net","sentAt":"2011-05-01T02:12:06Z","receivedAt":"2011-05-01T02:12:06Z","isPatch":true,"sender":{"key":"e@80x24.org","avatar":null},"body":"Carlos Martín Nieto wrote:\n> On Fri, Apr 29, 2011 at 09:28:44AM -0700, Junio C Hamano wrote:\n> From his reply (adding Ack) I inferred he expected you to apply it.\n\nOops, I Acked but forgot to commit + push\n\n> > I suspect that you are talking about syntax highlighting, and some syntax\n> > highlighter may be stupid to be confused, but in that case, at least you\n> > would need s/editors/some editors/.\n> \n> It confuses emacs and vim and I'd be surprised if any weren't, but\n> I'll change the message a bit.\n\nI've pushed the patch with the following message:\n\nSubject: [PATCH] git-svn: avoid contraction in usage text\n\nContractions may be difficult for non-native English speakers.\nThe quotation mark may also confuse syntax highlighting in\nsome text editors.\n\n[ew: reworded commit message]\n\nSigned-off-by: Carlos Martín Nieto \nAcked-by: Eric Wong \n-- \nEric Wong\n"}]}