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

Re: [PATCH v1] git-instaweb: Add Python builtin http.server support

From
brian m. carlson <sandals@crustytoothpaste.net>
Date
Jan 25, 2019, 02:04 UTC
Message-ID
<20190125020451.GU423984@genre.crustytoothpaste.net>
In-Reply-To
<20190124161331.25945-1-arti.zirk@gmail.com>
On Thu, Jan 24, 2019 at 06:13:31PM +0200, Arti Zirk wrote:
Show 13 quoted lines
> With this patch it is possible to launch git-instaweb by using
> Python 3 http.server CGI handler via `-d python` option.
> 
> git-instaweb generates a small wrapper around the http.server
> (in GIT_DIR/gitweb/) that address a limitation of the CGI handler
> where CGI scripts have to be in a cgi-bin subdirectory and
> directory index can't be easily changed. To keep the implementation
> small, gitweb is running on url `/cgi-bin/gitweb.cgi` and an automatic
> redirection is done when opening `/`.
> 
> Python 3 is by default installed on most modern Linux distributions
> which enables running `git instaweb -d python` without needing
> anything else.

I'm glad we're using Python 3 here, but I wonder if the name "python" will lead people to think it will work with Python 2 as well. There are people using up-to-date Git on systems like CentOS 6 and 7, where Python 3 is not generally available.

Could we call this "python3" instead, or provide some other way to communicate this to the user?

-- 
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 11 in “git-instaweb: Add Python builtin http.server support”
  1. git-instaweb: Add Python builtin http.server supportArti Zirk, Jan 24, 2019
  2. Junio C HamanoJan 24, 2019
  3. brian m. carlsonJan 25, 2019
  4. Junio C HamanoJan 25, 2019
  5. Arti ZirkJan 25, 2019
  6. brian m. carlsonJan 25, 2019
  7. Junio C HamanoJan 28, 2019
  8. git-instaweb: Add Python builtin http.server supportArti Zirk, Jan 28, 2019
  9. brian m. carlsonJan 28, 2019
  10. Arti ZirkJan 28, 2019
  11. Junio C HamanoJan 28, 2019

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.