Karthik Nayak <karthik.188@xxxxxxxxx> writes:
> Overall, this series is a bit more involved, and I would appreciate it
> if it receives a bit more scrutiny.
>
> The series is based on top of e66fd72e97 (The fourteenth batch,
> 2024-12-06) with `kn/reftable-writer-log-write-verify` merged in.
t1460.6 does not pass for me. I noticed it after I merged it to
'seen', but it fails standalone as well.
$ git log --first-parent --boundary master..kn/reflog-migration
65a841566a refs: add support for migrating reflogs
44ffb130f3 refs: allow multiple reflog entries for the same refname
6134624eaf refs: introduce the `ref_transaction_update_reflog` function
65a73fce30 refs: add `committer_info` to `ref_transaction_add_update()`
5ed33e32c7 refs: extract out refname verification in transactions
e9851924a2 refs/files: add count field to ref_lock
cdfa2c379a refs: add `index` field to `struct ref_udpate`
ee4d52c7f2 refs: include committer info in `ref_update` struct
df5d7a7ba5 Merge branch 'kn/reftable-writer-log-write-verify' into kn/reflog-migration
- 49c6b912e2 reftable/writer: ensure valid range for log's update_index
- e66fd72e97 The fourteenth batch
...
ok 5 - files: migration to same format fails
expecting success of 1460.6 'files -> reftable: migration with worktree fails':
test_when_finished "rm -rf repo" &&
git init --ref-format=$from_format repo &&
git -C repo worktree add wt &&
test_must_fail git -C repo refs migrate \
--ref-format=$to_format 2>err &&
cat >expect <<-EOF &&
error: migrating repositories with worktrees is not supported yet
EOF
test_cmp expect err
++ test_when_finished 'rm -rf repo'
++ test 0 = 0
++ test_cleanup='{ rm -rf repo
} && (exit "$eval_ret"); eval_ret=$?; :'
++ git init --ref-format=files repo
Initialized empty Git repository in /home/gitster/w/git.git/t/trash directory.t1460-refs-migrate/repo/.git/
++ git -C repo worktree add wt
No possible source branch, inferring '--orphan'
Preparing worktree (new branch 'wt')
++ test_must_fail git -C repo refs migrate --ref-format=reftable
++ case "$1" in
++ _test_ok=
++ test_must_fail_acceptable git -C repo refs migrate --ref-format=reftable
++ test git = env
++ test git = nongit
++ case "$1" in
++ return 0
++ git -C repo refs migrate --ref-format=reftable
/home/gitster/w/git.git/t/test-lib-functions.sh: line 1175: 3403665 Aborted "$@" 2>&7
++ exit_code=134
++ test 134 -eq 0
++ test_match_signal 13 134
++ test 134 = 141
++ test 134 = 269
++ return 1
++ test 134 -gt 129
++ test 134 -le 192
++ echo 'test_must_fail: died by signal 6: git -C repo refs migrate --ref-format=reftable'
test_must_fail: died by signal 6: git -C repo refs migrate --ref-format=reftable
++ return 1
error: last command exited with $?=1
not ok 6 - files -> reftable: migration with worktree fails
#
# test_when_finished "rm -rf repo" &&
# git init --ref-format=$from_format repo &&
# git -C repo worktree add wt &&
# test_must_fail git -C repo refs migrate \
# --ref-format=$to_format 2>err &&
# cat >expect <<-EOF &&
# error: migrating repositories with worktrees is not supported yet
# EOF
# test_cmp expect err
#
1..6
