Skip to content

Commit fe9dc12

Browse files
pks-tgitster
authored andcommitted
setup: stop using the_repository in upgrade_repository_format()
Stop using `the_repository` in `upgrade_repository_format()` and instead accept the repository as a parameter. The injection of `the_repository` is thus bumped one level higher, where callers now pass it in explicitly. Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>
1 parent 75f1d6f commit fe9dc12

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

list-objects-filter-options.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,7 @@ void partial_clone_register(
378378
*/
379379
return;
380380
} else {
381-
if (upgrade_repository_format(1) < 0)
381+
if (upgrade_repository_format(the_repository, 1) < 0)
382382
die(_("unable to upgrade repository format to support partial clone"));
383383

384384
/* Add promisor config for the remote */

repository.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,6 +271,6 @@ void repo_update_index_if_able(struct repository *, struct lock_file *);
271271
* Return 1 if upgrade repository format to target_version succeeded,
272272
* 0 if no upgrade is necessary, and -1 when upgrade is not possible.
273273
*/
274-
int upgrade_repository_format(int target_version);
274+
int upgrade_repository_format(struct repository *repo, int target_version);
275275

276276
#endif /* REPOSITORY_H */

setup.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -808,15 +808,15 @@ static int check_repository_format_gently(struct repository *repo,
808808
return 0;
809809
}
810810

811-
int upgrade_repository_format(int target_version)
811+
int upgrade_repository_format(struct repository *repo, int target_version)
812812
{
813813
struct strbuf sb = STRBUF_INIT;
814814
struct strbuf err = STRBUF_INIT;
815815
struct strbuf repo_version = STRBUF_INIT;
816816
struct repository_format repo_fmt = REPOSITORY_FORMAT_INIT;
817817
int ret;
818818

819-
repo_common_path_append(the_repository, &sb, "config");
819+
repo_common_path_append(repo, &sb, "config");
820820
read_repository_format(&repo_fmt, sb.buf);
821821
strbuf_release(&sb);
822822

@@ -838,7 +838,7 @@ int upgrade_repository_format(int target_version)
838838
}
839839

840840
strbuf_addf(&repo_version, "%d", target_version);
841-
repo_config_set(the_repository, "core.repositoryformatversion", repo_version.buf);
841+
repo_config_set(repo, "core.repositoryformatversion", repo_version.buf);
842842

843843
ret = 1;
844844

worktree.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1104,7 +1104,7 @@ void write_worktree_linking_files(const char *dotgit, const char *gitdir,
11041104
strbuf_realpath(&repo, repo.buf, 1);
11051105

11061106
if (use_relative_paths && !the_repository->repository_format_relative_worktrees) {
1107-
if (upgrade_repository_format(1) < 0)
1107+
if (upgrade_repository_format(the_repository, 1) < 0)
11081108
die(_("unable to upgrade repository format to support relative worktrees"));
11091109
if (repo_config_set_gently(the_repository, "extensions.relativeWorktrees", "true"))
11101110
die(_("unable to set extensions.relativeWorktrees setting"));

0 commit comments

Comments
 (0)