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

Re: [PATCH v3 4/4] urlmatch: allow globbing for the URL host part

From
Junio C Hamano <gitster@pobox.com>
Date
Jan 27, 2017, 17:45 UTC
Message-ID
<xmqq1svo1jne.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<20170127062142.GA760@pks-pc>
Patrick Steinhardt <patrick.steinhardt@elego.de> writes:
Show 9 quoted lines
>> This is probably a useful improvement.
>> 
>> Having said that, when I mentioned "glob", I meant to also support
>> something like this:
>> 
>> 	https://www[1-4].ibm.com/
>
> The problem with additional extended syntax like proposed by you
> is that we would indeed need an escaping mechanism here.

True. I think a true shell globbing is overkill (so is regexp) and just a simple wildcarding with '*' would be a good first step that is easy to explain and later extend as needed.

Thanks.
Previous: Patrick SteinhardtNext: Patrick Steinhardt
Message 8 of 10 in “urlmatch: allow regexp-based matches”
  1. 0/4 urlmatch: allow regexp-based matchesPatrick Steinhardt, Jan 25, 2017
  2. 2/4 urlmatch: enable normalization of URLs with globsPatrick Steinhardt, Jan 25, 2017
  3. 4/4 urlmatch: allow globbing for the URL host partPatrick Steinhardt, Jan 25, 2017
  4. Junio C HamanoJan 26, 2017
  5. Junio C HamanoJan 26, 2017
  6. Junio C HamanoJan 26, 2017
  7. Patrick SteinhardtJan 27, 2017
  8. Junio C HamanoJan 27, 2017
  9. 1/4 mailmap: add Patrick Steinhardt's work addressPatrick Steinhardt, Jan 25, 2017
  10. 3/4 urlmatch: split host and port fields in `struct url_info`Patrick Steinhardt, Jan 25, 2017

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.