Skip to content

sb/packfiles-in-repository updates

Nguyễn Thái Ngọc Duy git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest12 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/12packfile: allow prepare_packed_git_mru to handle arbitrary repositoriespackfile.c+5 -50 replies
2/12packfile: allow rearrange_packed_git to handle arbitrary repositoriespackfile.c+4 -40 replies
3/12packfile: allow install_packed_git to handle arbitrary repositoriesfast-import.c http.c packfile.c +1 more+7 -70 replies
4/12packfile: add repository argument to prepare_packed_git_onepackfile.c+4 -30 replies
5/12packfile: add repository argument to prepare_packed_gitbuiltin/count-objects.c builtin/fsck.c builtin/gc.c +9 more+18 -170 replies
6/12packfile: add repository argument to reprepare_packed_gitbuiltin/gc.c builtin/receive-pack.c bulk-checkin.c +4 more+11 -70 replies
7/12packfile: allow prepare_packed_git_one to handle arbitrary repositoriespackfile.c+3 -40 replies
8/12packfile: allow prepare_packed_git to handle arbitrary repositoriespackfile.c packfile.h+10 -110 replies
9/12packfile: allow reprepare_packed_git to handle arbitrary repositoriespackfile.c packfile.h+5 -60 replies
10/12packfile: add repository argument to find_pack_entrypackfile.c packfile.h sha1_file.c+11 -100 replies
11/12packfile: allow find_pack_entry to handle arbitrary repositoriespackfile.c packfile.h+6 -80 replies
12/12packfile: keep prepare_packed_git() privatebuiltin/count-objects.c builtin/fsck.c builtin/gc.c +9 more+5 -160 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 00/12] sb/packfiles-in-repository updates

This is the rebased version on the updated sb/object-store I just sent out plus the fix for get_object_directory(). The interdiff (after rebased) looks small and nice

packfile.c +1 -1
index e02136bebb..63c89ee31a 100644
@@ -890,7 +890,7 @@ static void prepare_packed_git(struct repository *r)
 
 	if (r->objects->packed_git_initialized)
 		return;
-	prepare_packed_git_one(r, get_object_directory(), 1);
+	prepare_packed_git_one(r, r->objects->objectdir, 1);
 	prepare_alt_odb(r);
 	for (alt = r->objects->alt_odb_list; alt; alt = alt->next)
 		prepare_packed_git_one(r, alt->path, 0);

I notice there's still one get_object_directory() left in packfile.c but that should not cause problems with converted functions. That could be done in "phase 2".

Nguyễn Thái Ngọc Duy (1):
packfile: keep prepare_packed_git() private

Stefan Beller (11):
packfile: allow prepare_packed_git_mru to handle arbitrary
repositories
packfile: allow rearrange_packed_git to handle arbitrary repositories
packfile: allow install_packed_git to handle arbitrary repositories
packfile: add repository argument to prepare_packed_git_one
packfile: add repository argument to prepare_packed_git
packfile: add repository argument to reprepare_packed_git
packfile: allow prepare_packed_git_one to handle arbitrary
repositories
packfile: allow prepare_packed_git to handle arbitrary repositories
packfile: allow reprepare_packed_git to handle arbitrary repositories
packfile: add repository argument to find_pack_entry
packfile: allow find_pack_entry to handle arbitrary repositories

 builtin/count-objects.c  |  3 +-
 builtin/fsck.c           |  2 --
 builtin/gc.c             |  3 +-
 builtin/pack-objects.c   |  1 -
 builtin/pack-redundant.c |  2 --
 builtin/receive-pack.c   |  3 +-
 bulk-checkin.c           |  3 +-
 fast-import.c            |  3 +-
 fetch-pack.c             |  3 +-
 http-backend.c           |  1 -
 http.c                   |  2 +-
 pack-bitmap.c            |  1 -
 packfile.c               | 76 +++++++++++++++++++---------------------
 packfile.h               | 11 +++---
 server-info.c            |  1 -
 sha1_file.c              |  8 ++---
 sha1_name.c              |  2 --
 17 files changed, 58 insertions(+), 67 deletions(-)
Signature · 1 line
2.17.0.rc0.348.gd5a49e0b6f