Re: [PATCH 0/4] Fixes from the per-repository object store series

2017-09-14 Thread Junio C Hamano
Jonathan Nieder  writes:

> This is a continuation of the series at [1].  That was part 1 ---
> you can think of this as part 0, since it contains the simplest and
> least controversial part of the series --- each patch in this series
> is a bugfix in its own right.
>
> Patch 1 should be familiar if you reviewed the series [1].  It is
> unchanged from the patch there, except to note Peff's ack.
>
> Patches 2-4 have not yet visited the list but are fixes (or, in the
> case of patch 4, cleanups) noticed as part of the same process of
> teaching object store code to handle multiple repositories.
>
> We hope that splitting these out should make them easier to review
> and for people to benefit from these fixes without having to wait
> for the rest of the series to settle.

One thing that is unclear is if you are tentatively withdrawing the
longer series with this update, but I'd assume that it is the case
because these two obviously will conflict with each other, and the
most of what these four patches do are subset of what the larger one
you sent earlier do.

Thanks.


[PATCH 0/4] Fixes from the per-repository object store series

2017-09-12 Thread Jonathan Nieder
Hi,

This is a continuation of the series at [1].  That was part 1 ---
you can think of this as part 0, since it contains the simplest and
least controversial part of the series --- each patch in this series
is a bugfix in its own right.

Patch 1 should be familiar if you reviewed the series [1].  It is
unchanged from the patch there, except to note Peff's ack.

Patches 2-4 have not yet visited the list but are fixes (or, in the
case of patch 4, cleanups) noticed as part of the same process of
teaching object store code to handle multiple repositories.

We hope that splitting these out should make them easier to review
and for people to benefit from these fixes without having to wait
for the rest of the series to settle.

Thoughts of all kinds welcome.

Jonathan Nieder (1):
  pack: make packed_git_mru global a value instead of a pointer

Stefan Beller (3):
  push, fetch: error out for submodule entries not pointing to commits
  replace-objects: evaluate replacement refs without using the object
store
  packed refs: pass .git dir instead of packed-refs path to init_fn

 builtin/pack-objects.c |  4 ++--
 cache.h|  4 ++--
 packfile.c | 12 +---
 refs.c |  2 +-
 refs/files-backend.c   |  4 ++--
 refs/packed-backend.c  |  4 ++--
 submodule.c| 33 +
 t/t5531-deep-submodule-push.sh | 10 ++
 8 files changed, 49 insertions(+), 24 deletions(-)

[1] 
https://public-inbox.org/git/20170830064634.ga153...@aiede.mtv.corp.google.com/