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

Re: [PATCH v8 4/4] config: allow http.<url>.* any user matching

From
Kyle J. McKay <mackyle@gmail.com>
Date
Jul 22, 2013, 22:18 UTC
Message-ID
<14CC7E43-3588-4B36-A8E9-79C22D548D8D@gmail.com>
In-Reply-To
<7v1u6qclfx.fsf@alter.siamese.dyndns.org>
On Jul 22, 2013, at 14:51, Junio C Hamano wrote:
Show 12 quoted lines
> "Kyle J. McKay" <mackyle@gmail.com> writes:
>
>>> A solid wall of text is somewhat hard to read, so I'd queue the
>>> equivalent of the following "git diff -w" output on top.
>>
>> Can I send out the change as a 'fixup!' patch?  Or do I need to  
>> send a
>> new v9 patch series with the documentation update?
>
> If you are OK with splitting it into two paragraphs with the
> "longest" clarification tweak (the "patch" I showed you), just
> saying so and I can squash ;-) so there is no need to resend.
The wording of:

"+ password part, if present in the url, is always ignored). A <url> + with longer path matches take precedence over shorter matches no matter + what order they occur in the configuration file.

needs to be fixed first. Replace "take" with "takes" and you can go ahead and squash it in. :)

Show 29 quoted lines
>>> diff --git a/Documentation/config.txt b/Documentation/config.txt
>>> index c418adf..635ed5d 100644
>>> --- a/Documentation/config.txt
>>> +++ b/Documentation/config.txt
>>> @@ -1521,9 +1521,11 @@ http.<url>.*::
>>> 	path portion is an exact match or a prefix that matches at a "/"
>>> 	boundary.  If <url> does not include a user name, it will match a  
>>> url
>>> 	with any username otherwise the user name must match as well (the
>>> -	password part, if present in the url, is always ignored).
>>> Longer <url>
>>> -	path matches take precedence over shorter matches no matter
>>> what order
>>> -	they occur in.  For example, if both
>>> "https://user@example.com/
>>> path" and
>>> +	password part, if present in the url, is always ignored).  A <url>
>>> +	with longer path matches take precedence over shorter matches
>>> no matter
>>> +	what order they occur in the configuration file.
>>> ++
>>> +For example, if both "https://user@example.com/path" and
>>> "https://example.com/path/name" are used as a config <url> value and
>>> then "https://user@example.com/path/name/here" is passed to a git
>>> command, the settings in the "https://example.com/path/name" section
>>
>> OK.
>
> ... which essentially is your "OK" ;-)

Yes, I meant that as "OK I will send out an update that includes something like this."

After replacing "take" with "takes" in the change I'm good with just squashing that diff in.

Previous: Junio C HamanoNext: Junio C Hamano
Message 15 of 19 in “config: add support for http.<url>.* settings”
  1. 0/4 config: add support for http.<url>.* settingsKyle J. McKay, Jul 22, 2013
  2. 1/4 config: add support for http.<url>.* settingsKyle J. McKay, Jul 22, 2013
  3. Jeff KingJul 24, 2013
  4. 2/4 config: improve support for http.<url>.* settingsKyle J. McKay, Jul 22, 2013
  5. 3/4 tests: add new test for the url_normalize functionKyle J. McKay, Jul 22, 2013
  6. Jeff KingJul 24, 2013
  7. Junio C HamanoJul 24, 2013
  8. Kyle J. McKayJul 24, 2013
  9. Kyle J. McKayJul 24, 2013
  10. Jeff KingJul 24, 2013
  11. 4/4 config: allow http.<url>.* any user matchingKyle J. McKay, Jul 22, 2013
  12. Junio C HamanoJul 22, 2013
  13. Kyle J. McKayJul 22, 2013
  14. Junio C HamanoJul 22, 2013
  15. Kyle J. McKayJul 22, 2013
  16. Junio C HamanoJul 22, 2013
  17. Jeff KingJul 24, 2013
  18. Junio C HamanoJul 24, 2013
  19. Jeff KingJul 24, 2013

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.