On Thu, Apr 29 2021, Junio C Hamano wrote:
Updates on my topics & other things I have comments on:
> * hn/reftable-tables-doc-update (2021-04-12) 1 commit
> (merged to 'next' on 2021-04-13 at cdadb2c621)
> + reftable: document an alternate cleanup method on Windows
>
> Doc updte.
In case it makes it into release notes: s/updte/update/;
> * dl/complete-stash-updates (2021-04-27) 4 commits
> - git-completion.bash: consolidate cases in _git_stash()
> - git-completion.bash: use $__git_cmd_idx in more places
> - git-completion.bash: rename to $__git_cmd_idx
> - git-completion.bash: separate some commands onto their own line
> (this branch uses dl/complete-stash.)
>
> Further update the command line completion (in contrib/) for "git
> stash".
>
> Will merge to 'next'?
I think so, I reviewed this and it looks good to me.
> * ab/pretty-date-format-tests (2021-04-27) 2 commits
> - pretty tests: give --date/format tests a better description
> - pretty tests: simplify %aI/%cI date format test
> (this branch is used by zh/pretty-date-human.)
>
> Tweak a few tests for "log --format=..." that show timestamps in
> various formats.
>
> Will merge to 'next'.
Thanks. Good to know not all my "I read this series and here's some
small patches ..." are a distraction :)
> * ls/subtree (2021-04-28) 30 commits
> - subtree: be stricter about validating flags
> - subtree: push: allow specifying a local rev other than HEAD
> - subtree: allow 'split' flags to be passed to 'push'
> - subtree: allow --squash to be used with --rejoin
> - subtree: give the docs a once-over
> - subtree: have $indent actually affect indentation
> - subtree: don't let debug and progress output clash
> - subtree: add comments and sanity checks
> - subtree: remove duplicate check
> - subtree: parse revs in individual cmd_ functions
> - subtree: use "^{commit}" instead of "^0"
> - subtree: don't fuss with PATH
> - subtree: use "$*" instead of "$@" as appropriate
> - subtree: use more explicit variable names for cmdline args
> - subtree: use git-sh-setup's `say`
> - subtree: use `git merge-base --is-ancestor`
> - subtree: drop support for git < 1.7
> - subtree: more consistent error propagation
> - subtree: don't have loose code outside of a function
> - subtree: t7900: add porcelain tests for 'pull' and 'push'
> - subtree: t7900: add a test for the -h flag
> - subtree: t7900: rename last_commit_message to last_commit_subject
> - subtree: t7900: fix 'verify one file change per commit'
> - subtree: t7900: delete some dead code
> - subtree: t7900: use 'test' for string equality
> - subtree: t7900: comment subtree_test_create_repo
> - subtree: t7900: use consistent formatting
> - subtree: t7900: use test-lib.sh's test_count
> - subtree: t7900: update for having the default branch name be 'main'
> - .gitignore: ignore 'git-subtree' as a build artifact
>
> "git subtree" updates.
>
> Will merge to 'next'?
> Unless somebody familiar with subtree speaks up soon and causes a
> reroll.
I had some nit-y comments on it, I more skimmed it than did a full revie
on it, but overall it looks good / reasonable to me.
> * zh/pretty-date-human (2021-04-27) 1 commit
> - pretty: provide human date format
> (this branch uses ab/pretty-date-format-tests.)
>
> "git log --format=..." placeholders learned %ah/%ch placeholders to
> request the --date=human output.
>
> Will merge to 'next'?
Looks good to me.
> * ab/describe-tests-fix (2021-04-29) 5 commits
> - describe tests: support -C in "check_describe"
> - describe tests: fix nested "test_expect_success" call
> - describe tests: don't rely on err.actual from "check_describe"
> - describe tests: refactor away from glob matching
> - describe tests: improve test for --work-tree & --dirty
> (this branch uses ab/test-lib-updates.)
>
> Various updates to tests around "git describe"
>
>
> * ab/pickaxe-pcre2 (2021-04-29) 22 commits
> - xdiff-interface: replace discard_hunk_line() with a flag
> - xdiff users: use designated initializers for out_line
> - pickaxe -G: don't special-case create/delete
> - pickaxe -G: terminate early on matching lines
> - xdiff-interface: allow early return from xdiff_emit_line_fn
> - xdiff-interface: prepare for allowing early return
> - pickaxe -S: slightly optimize contains()
> - pickaxe: rename variables in has_changes() for brevity
> - pickaxe -S: support content with NULs under --pickaxe-regex
> - pickaxe: assert that we must have a needle under -G or -S
> - pickaxe: refactor function selection in diffcore-pickaxe()
> - perf: add performance test for pickaxe
> - pickaxe/style: consolidate declarations and assignments
> - diff.h: move pickaxe fields together again
> - pickaxe: die when --find-object and --pickaxe-all are combined
> - pickaxe: die when -G and --pickaxe-regex are combined
> - pickaxe tests: add missing test for --no-pickaxe-regex being an error
> - pickaxe tests: test for -G, -S and --find-object incompatibility
> - pickaxe tests: add test for "log -S" not being a regex
> - pickaxe tests: add test for diffgrep_consume() internals
> - pickaxe tests: refactor to use test_commit --append --printf
> - grep/pcre2 tests: reword comments referring to kwset
> (this branch uses ab/test-lib-updates.)
>
> Rewrite the backend for "diff -G/-S" to use pcre2 engine when
> available.
Hopefully the ab/test-lib-updates moving forward will un-stall these
(comment on that below...).
> * ps/config-env-option-with-separate-value (2021-04-19) 2 commits
> - git: support separate arg for `--config-env`'s value
> - git.txt: fix synopsis of `--config-env` missing the equals sign
>
> "git --config-env var=val cmd" weren't accepted (only
> --config-env=var=val was).
>
> Will merge to 'next'.
> possibly with minimum fix-up?
> cf. <YIKcZxEDsG7qsE4G@xxxxxxxxxxxxxxxxxxxxxxx>
I also looked this over, LGTM.
> * ab/fsck-unexpected-type (2021-04-13) 6 commits
> - fsck: report invalid object type-path combinations
> - fsck: report invalid types recorded in objects
> - object-store.h: move read_loose_object() below 'struct object_info'
> - fsck: don't hard die on invalid object types
> - fsck tests: refactor one test to use a sub-repo
> - cache.h: move object functions to object-store.h
>
> "git fsck" has been taught to report mismatch between expected and
> actual types of an object better.
>
> Expecting a reroll.
Have a local version of this partially re-done for a re-roll...
> * ab/doc-lint (2021-04-10) 7 commits
> - docs: fix linting issues due to incorrect relative section order
> - doc lint: lint relative section order
> - doc lint: lint and fix missing "GIT" end sections
> - doc lint: fix bugs in, simplify and improve lint script
> - doc lint: Perl "strict" and "warnings" in lint-gitlink.perl
> - Documentation/Makefile: make doc.dep dependencies a variable again
> - Documentation/Makefile: make $(wildcard howto/*.txt) a var
>
> Dev support.
>
> Will merge to 'next'.
Thanks, I have some other doc improvements waiting on this.
> * ab/rebase-no-reschedule-failed-exec (2021-04-10) 2 commits
> - rebase: don't override --no-reschedule-failed-exec with config
> - rebase tests: camel-case rebase.rescheduleFailedExec consistently
>
> "git rebase --[no-]reschedule-failed-exec" did not work well with
> its configuration variable, which has been corrected.
>
> Will merge to 'next'.
Thanks.
> * bc/hash-transition-interop-part-1 (2021-04-27) 13 commits
> - hex: print objects using the hash algorithm member
> - hex: default to the_hash_algo on zero algorithm value
> - builtin/pack-objects: avoid using struct object_id for pack hash
> - commit-graph: don't store file hashes as struct object_id
> - builtin/show-index: set the algorithm for object IDs
> - hash: provide per-algorithm null OIDs
> - hash: set, copy, and use algo field in struct object_id
> - builtin/pack-redundant: avoid casting buffers to struct object_id
> - Use the final_oid_fn to finalize hashing of object IDs
> - hash: add a function to finalize object IDs
> - http-push: set algorithm when reading object ID
> - Always use oidread to read into struct object_id
> - hash: add an algo member to struct object_id
>
> SHA-256 transition.
>
> Will merge to 'next'?
This looks good to me. An earier version had some hash-object.c changes
that conflicted with a series of mine, that's now no longer there. I
still need to get back to seeing what (if anything) I need to do to
follow-up on that alternate way of specifying the hash version via an
argument to "git", or if brian was going to tackle it...
> * ab/svn-tests-set-e-fix (2021-04-12) 2 commits
> - svn tests: refactor away a "set -e" in test body
> - svn tests: remove legacy re-setup from init-clone test
>
> Test clean-up.
>
> Will merge to 'next'.
Thanks.
> * ab/test-lib-updates (2021-04-29) 11 commits
> - test-lib: split up and deprecate test_create_repo()
> - test-lib: do not show advice about init.defaultBranch under --verbose
> - test-lib: reformat argument list in test_create_repo()
> - submodule tests: use symbolic-ref --short to discover branch name
> - test-lib functions: add --printf option to test_commit
> - describe tests: convert setup to use test_commit
> - test-lib functions: add an --annotated option to "test_commit"
> - test-lib-functions: document test_commit --no-tag
> - test-lib-functions: reword "test_commit --append" docs
> - test-lib tests: remove dead GIT_TEST_FRAMEWORK_SELFTEST variable
> - test-lib: bring $remove_trash out of retirement
> (this branch is used by ab/describe-tests-fix and ab/pickaxe-pcre2.)
>
> Test clean-up.
>
> Waiting for an Ack before merging them to 'next'.
Is the ack a reply to
https://lore.kernel.org/git/xmqqo8dx7dv4.fsf@gitster.g/ (or here, I
suppose)?. Sorry about the in-flight hassle.
But yes, I think from the reviews etc. this is ready for "next" at this
point.
