Skip to content

not making corruption worse

Jeff King git 2 versions New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest5 patches 0 repliesNew
  2. v15 patches 29 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5t5312: test object deletion code paths in a corrupted repositoryt/t5312-prune-corruption.sh+114 -00 replies
2/5refs: introduce a "ref paranoia" flagDocumentation/git.txt cache.h environment.c +1 more+25 -00 replies
3/5prune: turn on ref_paranoia flagbuiltin/prune.c t/t5312-prune-corruption.sh+3 -20 replies
4/5repack: turn on "ref paranoia" when doing a destructive repackbuiltin/repack.c t/t5312-prune-corruption.sh+7 -30 replies
5/5refs.c: drop curate_packed_refsrefs.c t/t5312-prune-corruption.sh+2 -670 replies

Reviews trailers per patch

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

Read the v2 thread

Cover letter [PATCH v2 0/5] not making corruption worse

This is a re-roll of the series to make "git prune" in a corrupted repository safer.

There are only minor tweaks from v1, but I think all of the raised issues were addressed (there was discussion on some other points, but I think they are OK as-is; more discussion is of course welcome).

The changes from v1 are:

- use "bogus..name" as a bad refname instead of "bogus:name", as the latter has problems on Windows

- fix broken &&-chains in tests

- better commenting of missing-commit setup in t5312

- typo-fixes in commit messages

Patches:

[1/5]: t5312: test object deletion code paths in a corrupted repository [2/5]: refs: introduce a "ref paranoia" flag
[3/5]: prune: turn on ref_paranoia flag
[4/5]: repack: turn on "ref paranoia" when doing a destructive repack [5/5]: refs.c: drop curate_packed_refs

-Peff