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

Re: [PATCH 5/6] web--browse: use (x-)www-browser if available

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Dec 1, 2010, 15:49 UTC
Message-ID
<20101201154959.GA26120@burratino>
In-Reply-To
<AANLkTikfAr-KUVyqeTooxYKvKWYM4pRmw51if+349dpJ@mail.gmail.com>
Giuseppe Bilotta wrote:
> I will look into adding support for the BROWSER environment variable
> to select the browser if no browser is defined
Thanks, that sounds excellent.

BTW, the semantics of quoting characters/shell metacharacters in $BROWSER are not well defined (well, they're well defined in one way by the original spec, which is ignored by xdg-open and some other tools). xdg-open's semantics are probably the most reliable standard here but thought I should give a heads up.

Thinking out loud: longer term, it would be nice to just use xdg-utils and drop most of this logic from git. Presumably the ideal behavior for most apps opening web pages is already "new tab", so the cases of "new window" not explicitly configured that way by the user are bugs. Otherwise a request to portland@lists.freedesktop.org (subscription required) for xdg-open --new-tab/--new-window might be useful.

Regards, Jonathan

Previous: Giuseppe BilottaNext: Jonathan Nieder
Message 19 of 22 in “web--browse cleanups and extensions”
  1. 0/6 web--browse cleanups and extensionsGiuseppe Bilotta, Nov 29, 2010
  2. 1/6 CodingGuidelines: mention whitespace preferences for shell scriptsGiuseppe Bilotta, Nov 29, 2010
  3. Junio C HamanoNov 30, 2010
  4. Giuseppe BilottaNov 30, 2010
  5. 2/6 web--browse: coding styleGiuseppe Bilotta, Nov 29, 2010
  6. 3/6 web--browse: split valid_tool listGiuseppe Bilotta, Nov 29, 2010
  7. Jonathan NiederNov 29, 2010
  8. Giuseppe BilottaNov 29, 2010
  9. Junio C HamanoNov 29, 2010
  10. 4/6 web--browse: support opera, seamonkey and elinksGiuseppe Bilotta, Nov 29, 2010
  11. 5/6 web--browse: use (x-)www-browser if availableGiuseppe Bilotta, Nov 29, 2010
  12. Christian CouderNov 29, 2010
  13. Christian CouderNov 29, 2010
  14. Giuseppe BilottaNov 29, 2010
  15. Christian CouderNov 30, 2010
  16. Giuseppe BilottaNov 30, 2010
  17. Christian CouderDec 1, 2010
  18. Giuseppe BilottaDec 1, 2010
  19. Jonathan NiederDec 1, 2010
  20. Jonathan NiederNov 29, 2010
  21. Giuseppe BilottaNov 29, 2010
  22. 6/6 web--browse: special-case chromium pathGiuseppe Bilotta, Nov 29, 2010

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.