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

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

From
Jonathan Nieder <jrnieder@gmail.com>
Date
Dec 4, 2010, 00:42 UTC
Message-ID
<20101204004225.GA15906@burratino>
In-Reply-To
<AANLkTinWD53M2VjiWgeA0Qwx3OHzR2A09Y+AB2B9o1df@mail.gmail.com>
Giuseppe Bilotta wrote:
> I do believe that Debian encourages the use of sensible-browser (that
> does the BROWSER and *www-browser check itself) rather than manually
> going to look at those specifications.

My impression (by analogy with policy §11.4 "Editors and pagers") is that one is encouraged to make the default configurable at compile time and use

 - first $BROWSER
 - then something desktop-specific
 - then the configured default

and set that default to www-browser or x-www-browser, depending on whether your program uses X. That way, non-Debian systems benefit from the changes you introduce, too.

> An alternative approach would be to get rid of the *www-browser and
> BROWSER patches, and just use xdg-open if it's available. Which again
> raises the issue of how to enforce opening the page in a new tab.

Yes, in this case that is the ideal (assuming xdg-utils has wide enough adoption).

I think xdg-open has just as much a reason as we do to encourage opening the page in a new tab. Would it be hard to make that happen?

Previous: Giuseppe BilottaNext: Giuseppe Bilotta
Message 21 of 25 in “[PATCHv2 0/7] web--browse cleanup and extensions”
  1. Giuseppe BilottaDec 3, 2010
  2. 1/7 CodingGuidelines: mention whitespace preferences for shell scriptsGiuseppe Bilotta, Dec 3, 2010
  3. Junio C HamanoDec 3, 2010
  4. Giuseppe BilottaDec 3, 2010
  5. Junio C HamanoDec 3, 2010
  6. Giuseppe BilottaDec 3, 2010
  7. 2/7 web--browse: coding styleGiuseppe Bilotta, Dec 3, 2010
  8. Junio C HamanoDec 7, 2010
  9. 3/7 web--browse: split valid_tool listGiuseppe Bilotta, Dec 3, 2010
  10. 4/7 web--browse: support opera, seamonkey and elinksGiuseppe Bilotta, Dec 3, 2010
  11. Junio C HamanoDec 3, 2010
  12. Giuseppe BilottaDec 3, 2010
  13. 5/7 web--browse: better support for chromiumGiuseppe Bilotta, Dec 3, 2010
  14. Junio C HamanoDec 3, 2010
  15. Giuseppe BilottaDec 3, 2010
  16. 6/7 web--browse: use (x-)www-browser if availableGiuseppe Bilotta, Dec 3, 2010
  17. Christian CouderDec 3, 2010
  18. Giuseppe BilottaDec 3, 2010
  19. Junio C HamanoDec 3, 2010
  20. Giuseppe BilottaDec 3, 2010
  21. Jonathan NiederDec 4, 2010
  22. Giuseppe BilottaDec 4, 2010
  23. 7/7 web--browse: look at the BROWSER env varGiuseppe Bilotta, Dec 3, 2010
  24. Jonathan NiederDec 3, 2010
  25. Jonathan NiederDec 3, 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.