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

Re: [PATCH] checkout, clone: die if tree cannot be parsed

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Mar 9, 2022, 22:20 UTC
Message-ID
<nycvar.QRO.7.76.6.2203092319060.357@tvgsbejvaqbjf.bet>
In-Reply-To
<kl6lilswdu68.fsf@chooglen-macbookpro.roam.corp.google.com>
Hi Glen,
On Wed, 2 Mar 2022, Glen Choo wrote:
Show 13 quoted lines
> Junio C Hamano <gitster@pobox.com> writes:
>
> >          but one random-looking hexadecimal string is almost
> > indistinguishable from another, and neither would be a very useful
> > message from the end user's point of view.  I am wondering if we can
> > use old_branch_info to formulate something easier to understand for
> > the user.  update_refs_for_switch() seems to compute old_desc as a
> > human readable name by using old_branch_info->name if available
> > before falling back to old_branch_info->commit object name, which
> > might be a source of inspiration.
>
> I think it's actually more helpful to have the oid instead of a
> human-readable description like old_branch_info->name.

The most helpful would be to have both. That way, it would at least be potentially possible to figure out from a ref how to fetch a non-corrupt version from elsewhere.

Ciao, Dscho

Previous: Glen Choo
Message 4 of 4 in “checkout, clone: die if tree cannot be parsed”
  1. checkout, clone: die if tree cannot be parsedGlen Choo, Mar 2, 2022
  2. Junio C HamanoMar 2, 2022
  3. Glen ChooMar 2, 2022
  4. Johannes SchindelinMar 9, 2022

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.