Re: [PATCH 0/7] refs: remove use of `the_repository`

On Fri, Jul 10, 2026 at 07:56:19AM +0200, Patrick Steinhardt wrote:

On Thu, Jul 09, 2026 at 01:39:03PM -0700, Junio C Hamano wrote:

quoted

Patrick Steinhardt [off-list ref] writes:

quoted

The series is built on top of f85a7e6620 (Start Git 2.56 cycle,
2026-07-06) with ps/refs-writing-subcommands at 002fe677ca
(builtin/refs: add "rename" subcommand, 2026-07-06) merged into it.
Despite that, there's a small set of conflicts with "seen" that can be
merged like this:

Thanks for a heads-up.

This seems to break so many tests when merged to either 'jch' or
'seen', even though all of them pass standalone.  I did not have
time to figure out what interactions with which other topic are
causing the breakages.

Oh, interesting. I'll investigate what other topic this has interactions
with. Thanks!

Hm, curious, I cannot reproduce any of these failures at all, everything
is passing locally when merging "seen" into my branch. Did you maybe
mismerge the changes in "setup.c" by accident? That seems like the most
likely reason as you mention that it breaks lots of tests, and "setup.c"
is of course involved with all of them.

For reference, this is what the final result of the conflicting part
looks like on my side:

	if (real_git_dir) {
		struct stat st;

		if (!exist_ok && !stat(git_dir, &st))
			die(_("%s already exists"), git_dir);

		if (!exist_ok && !stat(real_git_dir, &st))
			die(_("%s already exists"), real_git_dir);

		apply_and_export_relative_gitdir(repo, real_git_dir, 1);
		git_dir = repo_get_git_dir(repo);
		separate_git_dir(repo, git_dir, original_git_dir);
	} else {
		apply_and_export_relative_gitdir(repo, git_dir, 1);
		git_dir = repo_get_git_dir(repo);
	}

Thanks!

Patrick