{"thread":{"id":"40774","subject":"[PATCH] Documentation: make formatting more consistent","startedAt":"2015-11-11T15:47:25Z","lastAt":"2015-11-13T05:45:05Z","messageCount":3,"participants":["Андрей Рыбак","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"273197","messageId":"CACayv=gEGY-JNfu26fOzztXwhV_bkG8pv_iOXLC5L-Mig9f0kw@mail.gmail.com","threadId":"40774","inReplyTo":null,"subject":"[PATCH] Documentation: make formatting more consistent","fromName":"Андрей Рыбак","fromEmail":"rybak.a.v@gmail.com","sentAt":"2015-11-11T15:47:25Z","receivedAt":"2015-11-11T15:47:25Z","isPatch":true,"sender":{"key":"rybak.a.v@gmail.com","avatar":"https://avatars.githubusercontent.com/u/624072?v=4"},"body":"Documentation/git.txt: enclose all Environment Variables in single quotes.\n\nSigned-off-by: Andrey Rybak <rybak.a.v@gmail.com>\n---\nDocumentation/git.txt is not consistent in the way it stylizes mentions\nof Environment Variables. Most of them are enclosed in single quotes,\nsome are enclosed in backticks, some are not enclosed.\n\n Documentation/git.txt | 12 ++++++------\n 1 file changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git.txt b/Documentation/git.txt\nindex c2e2a94..900272b 100644\n--- a/Documentation/git.txt\n+++ b/Documentation/git.txt\n@@ -1056,7 +1056,7 @@ of clones and fetches.\n  cloning of shallow repositories.\n  See 'GIT_TRACE' for available trace output options.\n\n-GIT_LITERAL_PATHSPECS::\n+'GIT_LITERAL_PATHSPECS'::\n  Setting this variable to `1` will cause Git to treat all\n  pathspecs literally, rather than as glob patterns. For example,\n  running `GIT_LITERAL_PATHSPECS=1 git log -- '*.c'` will search\n@@ -1065,15 +1065,15 @@ GIT_LITERAL_PATHSPECS::\n  literal paths to Git (e.g., paths previously given to you by\n  `git ls-tree`, `--raw` diff output, etc).\n\n-GIT_GLOB_PATHSPECS::\n+'GIT_GLOB_PATHSPECS'::\n  Setting this variable to `1` will cause Git to treat all\n  pathspecs as glob patterns (aka \"glob\" magic).\n\n-GIT_NOGLOB_PATHSPECS::\n+'GIT_NOGLOB_PATHSPECS'::\n  Setting this variable to `1` will cause Git to treat all\n  pathspecs as literal (aka \"literal\" magic).\n\n-GIT_ICASE_PATHSPECS::\n+'GIT_ICASE_PATHSPECS'::\n  Setting this variable to `1` will cause Git to treat all\n  pathspecs as case-insensitive.\n\n@@ -1087,7 +1087,7 @@ GIT_ICASE_PATHSPECS::\n  variable when it is invoked as the top level command by the\n  end user, to be recorded in the body of the reflog.\n\n-`GIT_REF_PARANOIA`::\n+'GIT_REF_PARANOIA'::\n  If set to `1`, include broken or badly named refs when iterating\n  over lists of refs. In a normal, non-corrupted repository, this\n  does nothing. However, enabling it may help git to detect and\n@@ -1098,7 +1098,7 @@ GIT_ICASE_PATHSPECS::\n  an operation has touched every ref (e.g., because you are\n  cloning a repository to make a backup).\n\n-`GIT_ALLOW_PROTOCOL`::\n+'GIT_ALLOW_PROTOCOL'::\n  If set, provide a colon-separated list of protocols which are\n  allowed to be used with fetch/push/clone. This is useful to\n  restrict recursive submodule initialization from an untrusted\n-- \n2.5.3.windows.1\n"},{"id":"273231","messageId":"20151111214106.GA2208@sigill.intra.peff.net","threadId":"40774","inReplyTo":"CACayv=gEGY-JNfu26fOzztXwhV_bkG8pv_iOXLC5L-Mig9f0kw@mail.gmail.com","subject":"Re: [PATCH] Documentation: make formatting more consistent","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-11-11T21:41:06Z","receivedAt":"2015-11-11T21:41:06Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Nov 11, 2015 at 06:47:25PM +0300, Андрей Рыбак wrote:\n\n> Documentation/git.txt: enclose all Environment Variables in single quotes.\n\nThis tells \"what\", but we can already see that from the diff. The more\ninteresting part is \"why\". Later you say:\n\n> Signed-off-by: Andrey Rybak <rybak.a.v@gmail.com>\n> ---\n> Documentation/git.txt is not consistent in the way it stylizes mentions\n> of Environment Variables. Most of them are enclosed in single quotes,\n> some are enclosed in backticks, some are not enclosed.\n\nwhich I think is the more interesting bit, and should go into the commit\nmessage (but parts after the \"---\" generally do not). I've grabbed it\nmanually in this case.\n\nI know we've discussed this particular spot before, and I think there\nmay have been some disagreement about which style was the best. But\nsince clearly no patch came out of the last discussion, and since\nan inconsistent set of styles is probably worse than consistent use of\nany of the choices, this makes sense to me as an incremental step.\n\nIf we want to move to all-backticks (for example) later on, we can\neasily do so (or we can leave this as the final state).\n\n> diff --git a/Documentation/git.txt b/Documentation/git.txt\n> index c2e2a94..900272b 100644\n> --- a/Documentation/git.txt\n> +++ b/Documentation/git.txt\n> @@ -1056,7 +1056,7 @@ of clones and fetches.\n>   cloning of shallow repositories.\n>   See 'GIT_TRACE' for available trace output options.\n\nYour patch seems whitespace-damaged (the leading tabs were converted to\nspaces). It was pretty easy to fix up in this case, so I did so (i.e.,\nno need to re-send), but please check your mailer settings.\n\nThanks.\n\n-Peff\n"},{"id":"273266","messageId":"20151113054505.GA32233@sigill.intra.peff.net","threadId":"40774","inReplyTo":"20151111214106.GA2208@sigill.intra.peff.net","subject":"Re: [PATCH] Documentation: make formatting more consistent","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2015-11-13T05:45:05Z","receivedAt":"2015-11-13T05:45:05Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Nov 11, 2015 at 04:41:06PM -0500, Jeff King wrote:\n\n> I know we've discussed this particular spot before, and I think there\n> may have been some disagreement about which style was the best. But\n> since clearly no patch came out of the last discussion, and since\n> an inconsistent set of styles is probably worse than consistent use of\n> any of the choices, this makes sense to me as an incremental step.\n> \n> If we want to move to all-backticks (for example) later on, we can\n> easily do so (or we can leave this as the final state).\n\nAndrey helpfully dug up that thread off-list:\n\n  http://thread.gmane.org/gmane.comp.version-control.git/267990/\n\nIf we want to move to backticks, we probably want to also turn on\nMAN_BOLD_LITERAL by default, or it's a step backwards for some folks.\n\nAs I was the person who suggested backticks back in that thread, and I\ndo not want to spend the time myself on figuring out if MAN_BOLD_LITERAL\nis safe to use everywhere, I somewhat retract my suggestion.\n\nI've queued this patch as-is for now, but I'd be happy to hear other\nopinions, or if people want to dig into the MAN_BOLD_LITERAL thing.\n\n-Peff\n"}]}