{"thread":{"id":"9496","subject":"[PATCH] Documentation: List environmental variables which were missing","startedAt":"2007-08-11T22:38:37Z","lastAt":"2007-08-12T01:38:52Z","messageCount":2,"participants":["Jakub Narebski","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"50516","messageId":"1186871917846-git-send-email-jnareb@gmail.com","threadId":"9496","inReplyTo":null,"subject":"[PATCH] Documentation: List environmental variables which were missing","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2007-08-11T22:38:37Z","receivedAt":"2007-08-11T22:38:37Z","isPatch":true,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Add to list environmental variables in \"Environment variables\" section\nof git(7) which are documented on other manpages, but were missing in\nthe list, in the form of link to appropriate manpage (similarly to how\nGIT_AUTHOR_* variables are documented in git(7)).\n\nSigned-off-by: Jakub Narebski <jnareb@gmail.com>\n---\n Documentation/git.txt |   18 ++++++++++++++++++\n 1 files changed, 18 insertions(+), 0 deletions(-)\n\ndiff --git a/Documentation/git.txt b/Documentation/git.txt\nindex 18f8b6a..8a0399e 100644\n--- a/Documentation/git.txt\n+++ b/Documentation/git.txt\n@@ -411,6 +411,24 @@ parameter, <path>.\n \n other\n ~~~~~\n+'GIT_CONFIG'::\n+'GIT_CONFIG_LOCAL'::\n+\tSpecifies config file.\n+\tSee gitlink:git-config[1]\n+\n+'GIT_PROXY_COMMAND'::\n+'GIT_SSL_NO_VERIFY'::\n+'GIT_SSL_CERT'::\n+'GIT_SSL_KEY'::\n+'GIT_SSL_CAINFO'::\n+'GIT_SSL_CAPATH'::\n+'GIT_HTTP_MAX_REQUESTS'::\n+'GIT_HTTP_LOW_SPEED_LIMIT'::\n+'GIT_HTTP_LOW_SPEED_TIME'::\n+'GIT_CURL_FTP_NO_EPSV'::\n+\tOverrides appropriate configuration variable.\n+\tSee gitlink:git-config[1]\n+\n 'GIT_MERGE_VERBOSITY'::\n \tA number controlling the amount of output shown by\n \tthe recursive merge strategy.  Overrides merge.verbosity.\n-- \n1.5.2.4\n"},{"id":"50527","messageId":"7v643llefn.fsf@assigned-by-dhcp.cox.net","threadId":"9496","inReplyTo":"1186871917846-git-send-email-jnareb@gmail.com","subject":"Re: [PATCH] Documentation: List environmental variables which were missing","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2007-08-12T01:38:52Z","receivedAt":"2007-08-12T01:38:52Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jakub Narebski <jnareb@gmail.com> writes:\n\n> Add to list environmental variables in \"Environment variables\" section\n> of git(7)...\n\nThese are \"environment variables\", not \"environmental\".  You\nhave the same typo in gitweb/INSTALL, I think.\n\n>  other\n>  ~~~~~\n> +'GIT_CONFIG'::\n> +'GIT_CONFIG_LOCAL'::\n> +\tSpecifies config file.\n> +\tSee gitlink:git-config[1]\n\nI think this should probably be listed next to GIT_DIR and\nfriends.\n\nIsn't \"Specifies config file\" way too vague to be any useful?\n\nIf you do not describe them enough on this page, saying \"there\nare other environment variables that affect the operation of\ngit; see git-config[1] for details\" without listing them is just\nas useful (or, just as useless).  At least doing so would need\nmuch less maintenance.\n\n> +'GIT_PROXY_COMMAND'::\n> +'GIT_SSL_NO_VERIFY'::\n> +'GIT_SSL_CERT'::\n> +'GIT_SSL_KEY'::\n> +'GIT_SSL_CAINFO'::\n> +'GIT_SSL_CAPATH'::\n> +'GIT_HTTP_MAX_REQUESTS'::\n> +'GIT_HTTP_LOW_SPEED_LIMIT'::\n> +'GIT_HTTP_LOW_SPEED_TIME'::\n> +'GIT_CURL_FTP_NO_EPSV'::\n> +\tOverrides appropriate configuration variable.\n> +\tSee gitlink:git-config[1]\n\nI wonder if it is worth it to add them in this form (the same\n\"usefulness vs maintenance hassle\" comment applies to the\nvagueness of \"appropriate variable\").\n\nEverything other than GIT_PROXY_COMMAND is for http fetch, isn't\nit?  At least you could say something like:\n\n\tUsed to override http.* configuration variables (see\n\tgitlink:git-config[1])\n"}]}