{"thread":{"id":"11766","subject":"[PATCH 1/2] Documentation: config: add \"browser.<tool>.path\".","startedAt":"2008-01-28T07:02:23Z","lastAt":"2008-01-29T06:07:56Z","messageCount":3,"participants":["Christian Couder","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"66710","messageId":"20080128080223.de8a67d2.chriscool@tuxfamily.org","threadId":"11766","inReplyTo":null,"subject":"[PATCH 1/2] Documentation: config: add \"browser.<tool>.path\".","fromName":"Christian Couder","fromEmail":"chriscool@tuxfamily.org","sentAt":"2008-01-28T07:02:23Z","receivedAt":"2008-01-28T07:02:23Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"Signed-off-by: Christian Couder <chriscool@tuxfamily.org>\n---\n Documentation/config.txt |    4 ++++\n 1 files changed, 4 insertions(+), 0 deletions(-)\n\ndiff --git a/Documentation/config.txt b/Documentation/config.txt\nindex 877eda9..c2fedd1 100644\n--- a/Documentation/config.txt\n+++ b/Documentation/config.txt\n@@ -367,6 +367,10 @@ branch.<name>.rebase::\n \tit unless you understand the implications (see linkgit:git-rebase[1]\n \tfor details).\n \n+browser.<tool>.path::\n+\tOverride the path for the given tool that may be used to\n+\tbrowse help. See linkgit:git-help[1].\n+\n clean.requireForce::\n \tA boolean to make git-clean do nothing unless given -f\n \tor -n.   Defaults to true.\n-- \n1.5.4.rc4.16.g1009\n"},{"id":"66715","messageId":"7v1w82wgev.fsf@gitster.siamese.dyndns.org","threadId":"11766","inReplyTo":"20080128080223.de8a67d2.chriscool@tuxfamily.org","subject":"Re: [PATCH 1/2] Documentation: config: add \"browser.<tool>.path\".","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-01-28T07:36:08Z","receivedAt":"2008-01-28T07:36:08Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Christian Couder <chriscool@tuxfamily.org> writes:\n\n> Signed-off-by: Christian Couder <chriscool@tuxfamily.org>\n> ---\n>  Documentation/config.txt |    4 ++++\n>  1 files changed, 4 insertions(+), 0 deletions(-)\n>\n> diff --git a/Documentation/config.txt b/Documentation/config.txt\n> index 877eda9..c2fedd1 100644\n> --- a/Documentation/config.txt\n> +++ b/Documentation/config.txt\n> @@ -367,6 +367,10 @@ branch.<name>.rebase::\n>  \tit unless you understand the implications (see linkgit:git-rebase[1]\n>  \tfor details).\n>  \n> +browser.<tool>.path::\n> +\tOverride the path for the given tool that may be used to\n> +\tbrowse help. See linkgit:git-help[1].\n> +\n\nThanks.\n\nI wonder if we want to stress the fact that this is about\nrendering HTML version of the manual.  IOW, the <tool>.path\nis about the path the chosen HTML renderer program is at, and\nnot consulted at all unless \"git help -w help\" is run.\n\nAlso, do we want to list the repertoire of <tool>?  Unlike\n<name> in \"remote.<name>.url\" and \"branch.<name>.remote\", this\nis not something the end user can arbitrarily name.\n\nIncidentally, I notice that git-help manual page does not list\nwhat browsers are supported (except that it hints that firefox\nis usable by having an example \"browser.firefox.path\"), even\nthough it tells how the browser is picked via \"*.browser\".\n"},{"id":"66811","messageId":"200801290707.57040.chriscool@tuxfamily.org","threadId":"11766","inReplyTo":"7v1w82wgev.fsf@gitster.siamese.dyndns.org","subject":"Re: [PATCH 1/2] Documentation: config: add \"browser.<tool>.path\".","fromName":"Christian Couder","fromEmail":"chriscool@tuxfamily.org","sentAt":"2008-01-29T06:07:56Z","receivedAt":"2008-01-29T06:07:56Z","isPatch":true,"sender":{"key":"christian.couder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/208954?v=4"},"body":"Le lundi 28 janvier 2008, Junio C Hamano a écrit :\n> Christian Couder <chriscool@tuxfamily.org> writes:\n> >\n> > +browser.<tool>.path::\n> > +\tOverride the path for the given tool that may be used to\n> > +\tbrowse help. See linkgit:git-help[1].\n> > +\n>\n> Thanks.\n>\n> I wonder if we want to stress the fact that this is about\n> rendering HTML version of the manual.  IOW, the <tool>.path\n> is about the path the chosen HTML renderer program is at, and\n> not consulted at all unless \"git help -w help\" is run.\n>\n> Also, do we want to list the repertoire of <tool>?  Unlike\n> <name> in \"remote.<name>.url\" and \"branch.<name>.remote\", this\n> is not something the end user can arbitrarily name.\n>\n> Incidentally, I notice that git-help manual page does not list\n> what browsers are supported (except that it hints that firefox\n> is usable by having an example \"browser.firefox.path\"), even\n> though it tells how the browser is picked via \"*.browser\".\n\nIt should be fixed in the following patch series.\n\nThanks,\nChristian.\n"}]}