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

Re: [PATCH 0/8] tightening ref handling outside of refs/

From
Patrick Steinhardt <ps@pks.im>
Date
Apr 29, 2024, 09:28 UTC
Message-ID
<Zi9oRuknlhfG1-WX@tanuki>
In-Reply-To
<20240429084238.GA242085@coredump.intra.peff.net>
On Mon, Apr 29, 2024 at 04:42:38AM -0400, Jeff King wrote:
Show 10 quoted lines
> On Mon, Apr 29, 2024 at 04:15:13AM -0400, Jeff King wrote:
> 
> >   [1/8]: t0600: don't create ref outside of refs/
> >   [2/8]: t5619: use fully qualified refname for branch
> 
> You can probably guess that I found these test cleanups only after
> writing the rest of the series and seeing them fail. :)
> 
> It turns out there is one more spot that is run only with reftables (so
> CI caught it, but my local testing did not):

Yeah, that's an issue by itself in my opinion. It's ultimately the reason why I change this to always run the backend-specific tests in [1].

Show 20 quoted lines
> diff --git a/t/t0610-reftable-basics.sh b/t/t0610-reftable-basics.sh
> index 178791e086..c6dbd2b5c4 100755
> --- a/t/t0610-reftable-basics.sh
> +++ b/t/t0610-reftable-basics.sh
> @@ -343,11 +343,11 @@ test_expect_success 'ref transaction: env var disables compaction' '
>  	for i in $(test_seq $iterations)
>  	do
>  		GIT_TEST_REFTABLE_AUTOCOMPACTION=false \
> -		git -C repo update-ref branch-$i HEAD || return 1
> +		git -C repo update-ref refs/heads/branch-$i HEAD || return 1
>  	done &&
>  	test_line_count = $expected repo/.git/reftable/tables.list &&
>  
> -	git -C repo update-ref foo HEAD &&
> +	git -C repo update-ref refs/heads/foo HEAD &&
>  	test_line_count -lt $expected repo/.git/reftable/tables.list
>  '
>  
> I'll wait for comments before re-rolling, but I'll make sure that gets
> added in.
The fix looks reasonable.
Patrick
[1]: https://lore.kernel.org/git/acf0c285506fe7ba275b08cdaf6b2245ec66b565.1712896869.git.ps@pks.im/
Previous: Jeff KingNext: Jeff King
Message 5 of 7 in “tightening ref handling outside of refs/”
  1. 0/8 tightening ref handling outside of refs/Jeff King, Apr 29, 2024
  2. 1/8 t0600: don't create ref outside of refs/Jeff King, Apr 29, 2024
  3. Jeff KingApr 29, 2024
  4. Jeff KingApr 29, 2024
  5. Patrick SteinhardtApr 29, 2024
  6. Jeff KingApr 30, 2024
  7. Junio C HamanoApr 29, 2024

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.