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

Re: [PATCH 1/2] bundle-uri: ignore bundles without uri

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 16, 2025, 16:59 UTC
Message-ID
<xmqq4it2nxjw.fsf@gitster.g>
In-Reply-To
<87bjnao1vd.fsf@iotcl.com>
Toon Claes <toon@iotcl.com> writes:
Show 8 quoted lines
> Junio C Hamano <gitster@pobox.com> writes:
>
>> I tend to agree.  Instead of papering over a misconfiguration, it
>> would be better to let the users know, so they have a chance to
>> report and/or correct such a misconfiguration.
>
> So are you okay if I do `return error("some message")` instead of
> `return -1`, or do you expect more of a change?

I am not sure what you are asking. The comment was more about my preference of honestly returning failure with explanation over pretending success without actually doing anything, wasn't it?

Previous: Toon ClaesNext: Toon Claes
Message 6 of 10 in “Make git-clone(1) more resilient when using bundle-URI”
  1. 0/2 Make git-clone(1) more resilient when using bundle-URIToon Claes, Sep 12, 2025
  2. 1/2 bundle-uri: ignore bundles without uriToon Claes, Sep 12, 2025
  3. Justin ToblerSep 12, 2025
  4. Junio C HamanoSep 12, 2025
  5. Toon ClaesSep 16, 2025
  6. Junio C HamanoSep 16, 2025
  7. 2/2 bundle-uri: do not abort on invalid packet lineToon Claes, Sep 12, 2025
  8. Justin ToblerSep 12, 2025
  9. Toon ClaesSep 17, 2025
  10. Justin ToblerSep 17, 2025

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.