git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v3 2/3] Allow help.htmlpath to be a URL prefix

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 28, 2012, 21:24 UTC
Message-ID
<7vvcib3zam.fsf@alter.siamese.dyndns.org>
In-Reply-To
<cebaecfaa217a20794e8278b4db3ce5b8d667834.1340866684.git.chris@arachsys.com>
Chris Webb <chris@arachsys.com> writes:
Show 34 quoted lines
> Setting this to a URL prefix instead of a path to a local directory allows
> git-help --web to work even when HTML docs aren't locally installed, by
> pointing the browser at a copy accessible on the web. For example,
>
>     [help]
>       format = html
>       htmlpath = http://git-scm.com/docs
>
> will use the publicly available documentation on the git homepage.
>
> Signed-off-by: Chris Webb <chris@arachsys.com>
> ---
>  builtin/help.c |    8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/builtin/help.c b/builtin/help.c
> index b467db2..92f2349 100644
> --- a/builtin/help.c
> +++ b/builtin/help.c
> @@ -399,9 +399,11 @@ static void get_html_page_path(struct strbuf *page_path, const char *page)
>  		html_path = system_path(GIT_HTML_PATH);
>  
>  	/* Check that we have a git documentation directory. */
> -	if (stat(mkpath("%s/git.html", html_path), &st)
> -	    || !S_ISREG(st.st_mode))
> -		die(_("'%s': not a documentation directory."), html_path);
> +	if (!strstr(html_path, "://")) {
> +		if (stat(mkpath("%s/git.html", html_path), &st)
> +				|| !S_ISREG(st.st_mode))
> +			die("'%s': not a documentation directory.", html_path);
> +	}
>  
>  	strbuf_init(page_path, 0);
>  	strbuf_addf(page_path, "%s/%s.html", html_path, page);
Sounds sensible and looks reasonable.  Thanks.
I do not think [PATCH 3/3] is a positive change at all, though.
Previous: Chris WebbNext: Chris Webb
Message 3 of 6 in “Add config variable to set HTML path for git-help --web”
  1. 1/3 Add config variable to set HTML path for git-help --webChris Webb, Jun 28, 2012
  2. 2/3 Allow help.htmlpath to be a URL prefixChris Webb, Jun 28, 2012
  3. Junio C HamanoJun 28, 2012
  4. Chris WebbJun 28, 2012
  5. 3/3 Add a help format 'usage' to provide brief command usageChris Webb, Jun 28, 2012
  6. Junio C HamanoJun 28, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.