threads / discuss / 57751

What's cooking in git.git (Apr 2022, #05; Mon, 18)

Subject: What's cooking in git.git (Apr 2022, #05; Mon, 18)

## tl;dr

5 messages between Apr 18, 2022 and Apr 22, 2022.

replies: 4people: 3as markdown or json

Junio C Hamano· Apr 18, 2022, 16:27 UTC · lore

Here are the topics that have been cooking in my tree. Commits prefixed with '+' are in 'next' (being in 'next' is a sign that a topic is stable enough to be used and are candidate to be in a future release). Commits prefixed with '-' are only in 'seen', and aren't considered "accepted" at all.

Git 2.36 has been tagged. As usual, we'll keep the tree closed for a few days for anything other than brown-paper-bag bugfixes (if needed) and then by the end of the week hopefully we can start sifting what is in 'next' to those to graduate to 'master' and those to be kicked back to 'seen'. We'll start taking new topics probably next week at the earliest.

cCopies of the source code to Git live in many repositories, and the following is a list of the ones I push into or their mirrors. Some repositories have only a subset of branches.

With maint, master, next, seen, todo:
	git://git.kernel.org/pub/scm/git/git.git/
	git://repo.or.cz/alt-git.git/
	https://kernel.googlesource.com/pub/scm/git/git/
	https://github.com/git/git/
	https://gitlab.com/git-vcs/git/
With all the integration branches and topics broken out:
	https://github.com/gitster/git/

Even though the preformatted documentation in HTML and man format are not sources, they are published in these repositories for convenience (replace "htmldocs" with "manpages" for the manual pages):

	git://git.kernel.org/pub/scm/git/git-htmldocs.git/
	https://github.com/gitster/git-htmldocs.git/
Release tarballs are available at:
	https://www.kernel.org/pub/software/scm/git/

-------------------------------------------------- [Graduated to 'master']

* cb/buggy-gcc-12-workaround (2022-04-15) 2 commits
  (merged to 'next' on 2022-04-15 at 48e68dd0fd)
 + config.mak.dev: alternative workaround to gcc 12 warning in http.c
 + config.mak.dev: workaround gcc 12 bug affecting "pedantic" CI job
 A couple of work around for CI breaking warnings from gcc 12.
 source: <20220415231342.35980-1-carenas@gmail.com>

-------------------------------------------------- [New Topics]

* cm/reftable-0-length-memset (2022-04-15) 1 commit
 - reftable: avoid undefined behaviour breaking t0032
 Code clean-up.
 Will merge to 'next'?
 source: <20220415083058.29495-1-carenas@gmail.com>

-------------------------------------------------- [Stalled]

* ab/commit-plug-leaks (2022-02-16) 2 commits
 - commit: use strbuf_release() instead of UNLEAK()
 - commit: fix "author_ident" leak
 Leakfixes in the top-level called-once function.
 Expecting a reroll.
 I think UNLEAK->strbuf_release() is a regression.
 source: <cover-0.2-00000000000-20220216T081844Z-avarab@gmail.com>
* dl/prompt-pick-fix (2022-03-25) 1 commit
 . git-prompt: fix sequencer/todo detection
 Fix shell prompt script (in contrib/) for those who set
 rebase.abbreviateCommands; we failed to recognize that we were in a
 multi-step cherry-pick session.
 Is this even needed?  How?
 cf. <xmqqwngdzque.fsf@gitster.g>
 source: <20220325145301.3370-1-danny0838@gmail.com>
* es/superproject-aware-submodules (2022-03-09) 3 commits
 . rev-parse: short-circuit superproject worktree when config unset
 . introduce submodule.hasSuperproject record
 . t7400-submodule-basic: modernize inspect() helper
 A configuration variable in a repository tells if it is (or is not)
 a submodule of a superproject.
 Expecting a reroll.
 cf. <kl6l4k45s7cb.fsf@chooglen-macbookpro.roam.corp.google.com>
 source: <20220310004423.2627181-1-emilyshaffer@google.com>
* cw/remote-object-info (2022-03-30) 5 commits
 . fixup! transfer.advertiseObjectInfo: add object-info config
 . fixup! object-info: add option for retrieving object info
 . object-info: add option for retrieving object info
 . transfer.advertiseObjectInfo: add object-info config
 . fetch-pack: refactor packet writing and fetch options
 Attempt to add a client component to talk with object-info
 endpoint.
 Expecting a reroll.
 source: <20220328191112.3092139-1-calvinwan@google.com>

-------------------------------------------------- [Cooking]

* sa/t1011-use-helpers (2022-04-12) 1 commit
 - t1011: replace test -f with test_path_is_file
 A GSoC practice.
 Will merge to 'next'.
 source: <20220412203722.10484-1-siddharthasthana31@gmail.com>
* ep/coverage-report-wants-test-to-have-run (2022-04-13) 1 commit
 - Makefile: add a prerequisite to the coverage-report target
 source: <20220414022513.31465-1-gitter.spiros@gmail.com>
* pw/test-malloc-with-sanitize-address (2022-04-11) 1 commit
 - tests: make SANITIZE=address imply TEST_NO_MALLOC_CHECK
 Avoid problems from interaction between malloc_check and address
 sanitizer.
 Will merge to 'next'.
 source: <pull.1210.git.1649507317350.gitgitgadget@gmail.com>
* rs/t7812-pcre2-ws-bug-test (2022-04-11) 1 commit
 - t7812: test PCRE2 whitespace bug
 A test to ensure workaround for an earlier pcre2 bug does work.
 Will merge to 'next'.
 source: <3a49649d-8ff9-e5a7-e3fd-33fee5068ae8@web.de>
* ah/convert-warning-message (2022-04-08) 1 commit
 - convert: clarify line ending conversion warning
 Update a few end-user facing messages around eol conversion.
 Will merge to 'next'.
 source: <20220408044154.9947-1-alexhenrie24@gmail.com>
* cg/vscode-with-gdb (2022-04-08) 1 commit
 - contrib/vscode/: debugging with VS Code and gdb
 VS code configuration updates.
 Will merge to 'next'.
 source: <20220407204001.112287-2-cogoni.guillaume@gmail.com>
* gf/unused-includes (2022-04-06) 2 commits
 - apply.c: remove unnecessary include
 - serve.c: remove unnecessary include
 Remove unused includes.
 Will merge to 'next'.
 source: <20220331194436.58005-1-garrit@slashdev.space>
* km/t3501-use-test-helpers (2022-04-06) 1 commit
 - t3501: remove test -f and stop ignoring git <cmd> exit code
 Test script updates.
 Will merge to 'next'.
 source: <20220405134742.17526-2-khalid.masum.92@gmail.com>
* pb/submodule-recurse-mode-enum (2022-04-06) 1 commit
 - submodule.h: use a named enum for RECURSE_SUBMODULES_*
 Small code clean-up.
 Will merge to 'next'.
 source: <pull.1111.v2.git.1649092211419.gitgitgadget@gmail.com>
* rs/commit-summary-wo-break-rewrite (2022-04-06) 1 commit
 - commit, sequencer: turn off break_opt for commit summary
 The commit summary shown after making a commit is matched to what
 is given in "git status" not to use the break-rewrite heuristics.
 Will merge to 'next'.
 source: <c35bd0aa-2e46-e710-2b39-89f18bad0097@web.de>
* tk/p4-utf8-bom (2022-04-06) 1 commit
 - git-p4: preserve utf8 BOM when importing from p4 to git
 "git p4" update.
 Will merge to 'next'.
 source: <pull.1203.git.1649051436934.gitgitgadget@gmail.com>
* tk/p4-with-explicity-sync (2022-04-06) 1 commit
 - git-p4: support explicit sync of arbitrary existing git-p4 refs
 "git p4" update.
 Will merge to 'next'.
 source: <pull.1202.git.1649049054600.gitgitgadget@gmail.com>
* ab/env-array (2022-04-06) 3 commits
 - run-command API users: use "env" not "env_array" in comments & names
 - run-command API: rename "env_array" to "env"
 - cocci: add a rename of "struct child_process"'s "env_array" to "env"
 Rename .env_array member to .env in the child_process structure.
 On hold.
 source: <cover-0.3-00000000000-20220406T104134Z-avarab@gmail.com>
* ab/misc-cleanup (2022-04-01) 6 commits
  (merged to 'next' on 2022-04-04 at c5fb674865)
 + alloc.[ch]: remove alloc_report() function
 + object-store.h: remove unused has_sha1_file*()
 + pack-bitmap-write: remove unused bitmap_reset() function
 + xdiff/xmacros.h: remove unused XDL_PTRFREE
 + configure.ac: remove USE_PIC comment
 + run-command.h: remove always unused "clean_on_exit_handler_cbdata"
 Code clean-up.
 Will cook in 'next'.
 source: <cover-v4-0.6-00000000000-20220331T014349Z-avarab@gmail.com>
* gf/shorthand-version-and-help (2022-03-31) 1 commit
 - cli: add -v and -h shorthands
 "git -v" and "git -h" are now understood as "git --version" and
 "git --help".
 Will merge to 'next'.
 source: <20220331212709.36036-1-garrit@slashdev.space>
* ea/progress-partial-blame (2022-04-06) 1 commit
  (merged to 'next' on 2022-04-07 at 7df8392d71)
 + blame: report correct number of lines in progress when using ranges
 The progress meter of "git blame" was showing incorrect numbers
 when processing only parts of the file.
 Will cook in 'next'.
 source: <20220406181320.16911-1-eantoranz@gmail.com>
* ab/plug-leak-in-revisions (2022-04-13) 28 commits
 - revisions API: add a TODO for diff_free(&revs->diffopt)
 - revisions API: have release_revisions() release "topo_walk_info"
 - revisions API: have release_revisions() release "date_mode"
 - revisions API: call diff_free(&revs->pruning) in revisions_release()
 - revisions API: release "reflog_info" in release revisions()
 - revisions API: clear "boundary_commits" in release_revisions()
 - revisions API: have release_revisions() release "prune_data"
 - revisions API: have release_revisions() release "grep_filter"
 - revisions API: have release_revisions() release "filter"
 - revisions API: have release_revisions() release "cmdline"
 - revisions API: have release_revisions() release "mailmap"
 - revisions API: have release_revisions() release "commits"
 - revisions API users: use release_revisions() for "prune_data" users
 - revisions API users: use release_revisions() with UNLEAK()
 - revisions API users: use release_revisions() in builtin/log.c
 - revisions API users: use release_revisions() in http-push.c
 - revisions API users: add "goto cleanup" for release_revisions()
 - stash: always have the owner of "stash_info" free it
 - revisions API users: use release_revisions() needing REV_INFO_INIT
 - revision.[ch]: document and move code declared around "init"
 - revisions API users: add straightforward release_revisions()
 - revision.[ch]: provide and start using a release_revisions()
 - cocci: add and apply free_commit_list() rules
 - format-patch: don't leak "extra_headers" or "ref_message_ids"
 - string_list API users: use string_list_init_{no,}dup
 - blame: use "goto cleanup" for cleanup_scoreboard()
 - t/helper/test-fast-rebase.c: don't leak "struct strbuf"
 - Merge branch 'ds/partial-bundle-more' into ab/plug-leak-in-revisions
 Plug the memory leaks from the trickiest API of all, the revision
 walker.
 Will merge to 'next'.
 source: <cover-v6-00.27-00000000000-20220413T195935Z-avarab@gmail.com>
* ab/ci-github-workflow-markup (2022-04-14) 7 commits
 - ci: call `finalize_test_case_output` a little later
 - ci: use `--github-workflow-markup` in the GitHub workflow
 - ci: optionally mark up output in the GitHub workflow
 - test(junit): avoid line feeds in XML attributes
 - tests: refactor --write-junit-xml code
 - ci: make it easier to find failed tests' logs in the GitHub workflow
 - Merge branch 'ab/ci-setup-simplify' into ab/ci-github-workflow-markup
 (this branch uses ab/ci-setup-simplify.)
 Build a moral equivalent of js/ci-github-workflow-markup on top of
 ab/ci-setup-simplify.
 How well does this compare feature-wise with js/ci-github-workflow-markup?
 source: <RFC-cover-v4-0.6-00000000000-20220413T195514Z-avarab@gmail.com>
* ab/ci-setup-simplify (2022-04-14) 29 commits
 - CI: make it easy to use ci/*.sh outside of CI
 - CI: don't use "set -x" in "ci/lib.sh" output
 - CI: set PYTHON_PATH setting for osx-{clang,gcc} into "$jobname" case
 - CI: set SANITIZE=leak in MAKEFLAGS directly
 - CI: set CC in MAKEFLAGS directly, don't add it to the environment
 - CI: add more variables to MAKEFLAGS, except under vs-build
 - CI: narrow down variable definitions in --build and --test
 - CI: only invoke ci/lib.sh as "steps" in main.yml
 - CI: pre-select test slice in Windows & VS tests
 - ci/run-test-slice.sh: replace shelling out with "echo"
 - CI: move "env" definitions into ci/lib.sh
 - CI: combine ci/install{,-docker}-dependencies.sh
 - CI: split up and reduce "ci/test-documentation.sh"
 - CI: invoke "make artifacts-tar" directly in windows-build
 - CI: check ignored unignored build artifacts in "win[+VS] build" too
 - ci/lib.sh: use "test" instead of "["
 - CI: remove "run-build-and-tests.sh", run "make [test]" directly
 - CI: export variables via a wrapper
 - CI: consistently use "export" in ci/lib.sh
 - CI: move p4 and git-lfs variables to ci/install-dependencies.sh
 - CI: have "static-analysis" run "check-builtins", not "documentation"
 - CI: have "static-analysis" run a "make ci-static-analysis" target
 - CI: don't have "git grep" invoke a pager in tree content check
 - CI/lib.sh: stop adding leading whitespace to $MAKEFLAGS
 - CI: remove unused Azure ci/* code
 - CI: remove dead "tree skipping" code
 - CI: remove more dead Travis CI support
 - CI: make "$jobname" explicit, remove fallback
 - CI: run "set -ex" early in ci/lib.sh
 (this branch is used by ab/ci-github-workflow-markup.)
 Drive more actions done in CI via the Makefile instead of shell
 commands sprinkled in .github/workflows/main.yml
 Will merge to 'next'?
 source: <cover-v3-00.29-00000000000-20220413T194847Z-avarab@gmail.com>
* kf/p4-multiple-remotes (2022-03-21) 1 commit
 - git-p4: fix issue with multiple perforce remotes
 "git p4" update.
 Will merge to 'next'?
 source: <pull.1180.git.1647866603032.gitgitgadget@gmail.com>
* fr/vimdiff-layout (2022-04-03) 4 commits
  (merged to 'next' on 2022-04-04 at 5d1c8197d0)
 + mergetools: add description to all diff/merge tools
 + vimdiff: add tool documentation
 + vimdiff: integrate layout tests in the unit tests framework ('t' folder)
 + vimdiff: new implementation with layout support
 Reimplement "vimdiff[123]" mergetool drivers with a more generic
 layout mechanism.
 Will cook in 'next'.
 source: <20220330191909.294610-1-greenfoo@u92.eu>
* bc/stash-export (2022-04-08) 4 commits
 - builtin/stash: provide a way to import stashes from a ref
 - builtin/stash: provide a way to export stashes to a ref
 - builtin/stash: factor out revision parsing into a function
 - object-name: make get_oid quietly return an error
 A mechanism to export and import stash entries to and from a normal
 commit to transfer it across repositories has been introduced.
 Will merge to 'next'?
 source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>
* ns/batch-fsync (2022-04-06) 13 commits
 - core.fsyncmethod: performance tests for batch mode
 - t/perf: add iteration setup mechanism to perf-lib
 - core.fsyncmethod: tests for batch mode
 - test-lib-functions: add parsing helpers for ls-files and ls-tree
 - core.fsync: use batch mode and sync loose objects by default on Windows
 - unpack-objects: use the bulk-checkin infrastructure
 - update-index: use the bulk-checkin infrastructure
 - builtin/add: add ODB transaction around add_files_to_cache
 - cache-tree: use ODB transaction around writing a tree
 - core.fsyncmethod: batched disk flushes for loose-objects
 - bulk-checkin: rebrand plug/unplug APIs as 'odb transactions'
 - bulk-checkin: rename 'state' variable and separate 'plugged' boolean
 - Merge branch 'ns/core-fsyncmethod' into ns/batch-fsync
 Introduce a filesystem-dependent mechanism to optimize the way the
 bits for many loose object files are ensured to hit the disk
 platter.
 Will merge to 'next'?
 source: <pull.1134.v5.git.1648616734.gitgitgadget@gmail.com>
* en/sparse-cone-becomes-default (2022-03-13) 9 commits
 - Documentation: some sparsity wording clarifications
 - git-sparse-checkout.txt: mark non-cone mode as deprecated
 - git-sparse-checkout.txt: flesh out non-cone mode pattern discussion a bit
 - git-sparse-checkout.txt: add a new EXAMPLES section
 - git-sparse-checkout.txt: shuffle some sections and mark as internal
 - git-sparse-checkout.txt: update docs for deprecation of 'init'
 - git-sparse-checkout.txt: wording updates for the cone mode default
 - sparse-checkout: make --cone the default
 - tests: stop assuming --no-cone is the default mode for sparse-checkout
 Deprecate non-cone mode of the sparse-checkout feature.
 Will merge to 'next'?
 source: <pull.1148.v2.git.1647054681.gitgitgadget@gmail.com>
* tb/cruft-packs (2022-03-02) 17 commits
 - sha1-file.c: don't freshen cruft packs
 - builtin/gc.c: conditionally avoid pruning objects via loose
 - builtin/repack.c: add cruft packs to MIDX during geometric repack
 - builtin/repack.c: use named flags for existing_packs
 - builtin/repack.c: allow configuring cruft pack generation
 - builtin/repack.c: support generating a cruft pack
 - builtin/pack-objects.c: --cruft with expiration
 - reachable: report precise timestamps from objects in cruft packs
 - reachable: add options to add_unseen_recent_objects_to_traversal
 - builtin/pack-objects.c: --cruft without expiration
 - builtin/pack-objects.c: return from create_object_entry()
 - t/helper: add 'pack-mtimes' test-tool
 - pack-mtimes: support writing pack .mtimes files
 - chunk-format.h: extract oid_version()
 - pack-write: pass 'struct packing_data' to 'stage_tmp_packfiles'
 - pack-mtimes: support reading .mtimes files
 - Documentation/technical: add cruft-packs.txt
 A mechanism to pack unreachable objects into a "cruft pack",
 instead of ejecting them into loose form to be reclaimed later, has
 been introduced.
 Waiting for discussion to settle.
 cf. <YiZI99yeijQe5Jaq@google.com>
 source: <cover.1646266835.git.me@ttaylorr.com>
* js/ci-github-workflow-markup (2022-03-01) 9 commits
 . ci: call `finalize_test_case_output` a little later
 . ci: use `--github-workflow-markup` in the GitHub workflow
 . ci: optionally mark up output in the GitHub workflow
 . test(junit): avoid line feeds in XML attributes
 . tests: refactor --write-junit-xml code
 . ci/run-build-and-tests: add some structure to the GitHub workflow output
 . ci: make it easier to find failed tests' logs in the GitHub workflow
 . ci/run-build-and-tests: take a more high-level view
 . ci: fix code style
 Update the GitHub workflow support to make it quicker to get to the
 failing test.
 Waiting for discussion to settle.
 cf. <220309.86tuc6lwpj.gmgdl@evledraar.gmail.com>
 cf. <220302.86mti87cj2.gmgdl@evledraar.gmail.com>
 cf. <30dbc8fb-a1db-05bc-3dcb-070e11cf4715@gmail.com>
 source: <pull.1117.v2.git.1646130289.gitgitgadget@gmail.com>
* et/xdiff-indirection (2022-02-17) 1 commit
 - xdiff: provide indirection to git functions
 Insert a layer of preprocessor macros for common functions in xdiff
 codebase.
 Expecting a reroll.
 cf. <xmqqbkyudb8n.fsf@gitster.g>
 source: <20220217225408.GB7@edef91d97c94>
* tk/simple-autosetupmerge (2022-02-25) 2 commits
 - t3200: tests for new branch.autosetupmerge option "simple"
 - merge: new autosetupmerge option 'simple' for matching branches
 "git -c branch.autosetupmerge=simple branch $A $B" will set the $B
 as $A's upstream only when $A and $B shares the same name, and "git
 -c push.default=simple" on branch $A would push to update the
 branch $A at the remote $B came from.
 Needs review.
 source: <pull.1161.v2.git.1645815142.gitgitgadget@gmail.com>
* tk/untracked-cache-with-uall (2022-04-01) 2 commits
  (merged to 'next' on 2022-04-04 at 2e11f1ac0c)
 + untracked-cache: support '--untracked-files=all' if configured
 + untracked-cache: test untracked-cache-bypassing behavior with -uall
 The performance of the "untracked cache" feature has been improved
 when "--untracked-files=<mode>" and "status.showUntrackedFiles"
 are combined.
 Will cook in 'next'.
 source: <pull.985.v6.git.1648742535.gitgitgadget@gmail.com>
* jh/builtin-fsmonitor-part3 (2022-03-25) 28 commits
 - t7527: test Unicode NFC/NFD handling on MacOS
 - t/lib-unicode-nfc-nfd: helper prereqs for testing unicode nfc/nfd
 - fsmonitor: on macOS also emit NFC spelling for NFD pathname
 - t7527: test FSMonitor on case insensitive+preserving file system
 - fsmonitor: never set CE_FSMONITOR_VALID on submodules
 - t/perf/p7527: add perf test for builtin FSMonitor
 - t7527: FSMonitor tests for directory moves
 - fsmonitor: optimize processing of directory events
 - fsm-listen-darwin: shutdown daemon if worktree root is moved/renamed
 - fsm-health-win32: force shutdown daemon if worktree root moves
 - fsm-health-win32: add polling framework to monitor daemon health
 - fsmonitor--daemon: stub in health thread
 - fsmonitor--daemon: rename listener thread related variables
 - fsmonitor--daemon: prepare for adding health thread
 - fsmonitor--daemon: cd out of worktree root
 - fsm-listen-darwin: ignore FSEvents caused by xattr changes on macOS
 - unpack-trees: initialize fsmonitor_has_run_once in o->result
 - fsmonitor-settings: NTFS and FAT32 on MacOS are incompatible
 - fsmonitor-settings: remote repos on Windows are incompatible
 - fsmonitor-settings: remote repos on macOS are incompatible
 - fsmonitor-settings: stub in macOS-specific incompatibility checking
 - fsmonitor-settings: VFS for Git virtual repos are incompatible
 - fsmonitor-settings: stub in Win32-specific incompatibility checking
 - fsmonitor-settings: bare repos are incompatible with FSMonitor
 - t/helper/fsmonitor-client: create stress test
 - t7527: test FSMonitor on repos with Unicode root paths
 - fsm-listen-win32: handle shortnames
 - Merge branch 'jh/builtin-fsmonitor-part2' into jh/builtin-fsmonitor-part3
 More fsmonitor--daemon.
 source: <pull.1143.v4.git.1648140680.gitgitgadget@gmail.com>
* js/bisect-in-c (2022-02-23) 14 commits
 - bisect: no longer try to clean up left-over `.git/head-name` files
 - bisect: remove Cogito-related code
 - bisect: turn `git bisect` into a full built-in
 - bisect: move even the option parsing to `bisect--helper`
 - bisect--helper: return only correct exit codes in `cmd_*()`
 - bisect--helper: move the `BISECT_STATE` case to the end
 - bisect--helper: make `--bisect-state` optional
 - bisect--helper: align the sub-command order with git-bisect.sh
 - bisect--helper: using `--bisect-state` without an argument is a bug
 - bisect--helper: really retire `--bisect-autostart`
 - bisect--helper: really retire --bisect-next-check
 - bisect--helper: retire the --no-log option
 - bisect: avoid double-quoting when printing the failed command
 - bisect run: fix the error message
 Final bits of "git bisect.sh" have been rewritten in C.
 Expecting a reroll.
 cf. <220225.86ilt27uln.gmgdl@evledraar.gmail.com>
 source: <pull.1132.v2.git.1645547423.gitgitgadget@gmail.com>
* js/scalar-diagnose (2022-02-06) 6 commits
 - scalar: teach `diagnose` to gather loose objects information
 - scalar: teach `diagnose` to gather packfile info
 - scalar diagnose: include disk space information
 - scalar: add `diagnose`
 - scalar: validate the optional enlistment argument
 - archive: optionally add "virtual" files
 Implementation of "scalar diagnose" subcommand.
 On hold.
 cf. <nycvar.QRO.7.76.6.2203012353090.11118@tvgsbejvaqbjf.bet>
 source: <pull.1128.v2.git.1644187146.gitgitgadget@gmail.com>
* en/merge-tree (2022-02-23) 13 commits
 - git-merge-tree.txt: add a section on potentional usage mistakes
 - merge-tree: add a --allow-unrelated-histories flag
 - merge-tree: allow `ls-files -u` style info to be NUL terminated
 - merge-tree: provide easy access to `ls-files -u` style info
 - merge-tree: provide a list of which files have conflicts
 - merge-ort: provide a merge_get_conflicted_files() helper function
 - merge-tree: support including merge messages in output
 - merge-ort: split out a separate display_update_messages() function
 - merge-tree: implement real merges
 - merge-tree: add option parsing and initial shell for real merge function
 - merge-tree: move logic for existing merge into new function
 - merge-tree: rename merge_trees() to trivial_merge_trees()
 - Merge branch 'en/remerge-diff' into en/merge-trees
 A new command is introduced that takes two commits and computes a
 tree that would be contained in the resulting merge commit, if the
 histories leading to these two commits were to be merged, and is
 added as a new mode of "git merge-tree" subcommand.
 On hold.
 cf. <CABPp-BGZ7OAYRR5YKRsxJSo-C=ho+qcNAkqwkim8CkhCfCeHsA@mail.gmail.com>
 source: <pull.1122.v6.git.1645602413.gitgitgadget@gmail.com>
* jh/p4-various-fixups (2022-04-01) 22 commits
  (merged to 'next' on 2022-04-04 at 251b14976f)
 + git-p4: sort imports
 + git-p4: seperate multiple statements onto seperate lines
 + git-p4: move inline comments to line above
 + git-p4: only seperate code blocks by a single empty line
 + git-p4: compare to singletons with "is" and "is not"
 + git-p4: normalize indentation of lines in conditionals
 + git-p4: ensure there is a single space around all operators
 + git-p4: ensure every comment has a single #
 + git-p4: remove spaces between dictionary keys and colons
 + git-p4: remove redundant backslash-continuations inside brackets
 + git-p4: remove extraneous spaces before function arguments
 + git-p4: place a single space after every comma
 + git-p4: removed brackets when assigning multiple return values
 + git-p4: remove spaces around default arguments
 + git-p4: remove padding from lists, tuples and function arguments
 + git-p4: sort and de-duplcate pylint disable list
 + git-p4: remove commented code
 + git-p4: convert descriptive class and function comments into docstrings
 + git-p4: improve consistency of docstring formatting
 + git-p4: indent with 4-spaces
 + git-p4: remove unneeded semicolons from statements
 + git-p4: add blank lines between functions and class definitions
 Various cleanups to "git p4".
 Will cook in 'next'.
 source: <20220401142504.58995-1-jholdsworth@nvidia.com>
* js/use-builtin-add-i (2021-12-01) 2 commits
 - add -i: default to the built-in implementation
 - t2016: require the PERL prereq only when necessary
 "git add -i" was rewritten in C some time ago and has been in
 testing; the reimplementation is now exposed to general public by
 default.
 On hold.
 What's the status of the "known breakage"?
 Are we ready to switch if we wanted to?
 There are known breakages on macOS.
 cf. <nycvar.QRO.7.76.6.2112021832060.63@tvgsbejvaqbjf.bet>
 source: <pull.1087.git.1638281655.gitgitgadget@gmail.com>
Phillip Wood· Apr 19, 2022, 12:38 UTC · re: Junio C Hamano · lore

ab/ci-setup-simplify (was Re: What's cooking in git.git (Apr 2022, #05; Mon, 18))

Show 37 quoted lines
> * ab/ci-setup-simplify (2022-04-14) 29 commits
>   - CI: make it easy to use ci/*.sh outside of CI
>   - CI: don't use "set -x" in "ci/lib.sh" output
>   - CI: set PYTHON_PATH setting for osx-{clang,gcc} into "$jobname" case
>   - CI: set SANITIZE=leak in MAKEFLAGS directly
>   - CI: set CC in MAKEFLAGS directly, don't add it to the environment
>   - CI: add more variables to MAKEFLAGS, except under vs-build
>   - CI: narrow down variable definitions in --build and --test
>   - CI: only invoke ci/lib.sh as "steps" in main.yml
>   - CI: pre-select test slice in Windows & VS tests
>   - ci/run-test-slice.sh: replace shelling out with "echo"
>   - CI: move "env" definitions into ci/lib.sh
>   - CI: combine ci/install{,-docker}-dependencies.sh
>   - CI: split up and reduce "ci/test-documentation.sh"
>   - CI: invoke "make artifacts-tar" directly in windows-build
>   - CI: check ignored unignored build artifacts in "win[+VS] build" too
>   - ci/lib.sh: use "test" instead of "["
>   - CI: remove "run-build-and-tests.sh", run "make [test]" directly
>   - CI: export variables via a wrapper
>   - CI: consistently use "export" in ci/lib.sh
>   - CI: move p4 and git-lfs variables to ci/install-dependencies.sh
>   - CI: have "static-analysis" run "check-builtins", not "documentation"
>   - CI: have "static-analysis" run a "make ci-static-analysis" target
>   - CI: don't have "git grep" invoke a pager in tree content check
>   - CI/lib.sh: stop adding leading whitespace to $MAKEFLAGS
>   - CI: remove unused Azure ci/* code
>   - CI: remove dead "tree skipping" code
>   - CI: remove more dead Travis CI support
>   - CI: make "$jobname" explicit, remove fallback
>   - CI: run "set -ex" early in ci/lib.sh
>   (this branch is used by ab/ci-github-workflow-markup.)
> 
>   Drive more actions done in CI via the Makefile instead of shell
>   commands sprinkled in .github/workflows/main.yml
> 
>   Will merge to 'next'?
>   source: <cover-v3-00.29-00000000000-20220413T194847Z-avarab@gmail.com>

I haven't had time to read all 31 patches from v4 in detail but I have looked at the results in seen.

Looking at seen:ci/install-dependencies.sh the shebang has been changed to "#!/bin/sh" but it contains "BREW_PACKAGE=${CC_PACKAGE/-/@}" which is a bashism.

Looking at seen:.github/workflows/main.yaml to skip running the tests one has to set "skip-tests: no" which is utterly confusing.

 From what I saw scanning the patches there seemed to be a lot of churn, 
both of existing code and code that gets added and then moved/refactored 
within the series.

Looking at the output of a recent ci run of seen the steps to prepare the environment before building and testing print all the environment variables rather than just the ones being set for that step which seems to go against the aim of "CI: narrow down variable definitions in --build and --test". (Also the "SKIP" prefix in the output lacks a ":")

Dscho raised concerns that this removes any support for azure pipelines which he uses when preparing security patches.

I think splitting out the build and test steps is a good idea but I'm less convinced by some of the other changes.

Best Wishes
Phillip
Ævar Arnfjörð Bjarmason· Apr 21, 2022, 18:36 UTC · re: Phillip Wood · lore

Re: ab/ci-setup-simplify (was Re: What's cooking in git.git (Apr 2022, #05; Mon, 18))

On Tue, Apr 19 2022, Phillip Wood wrote:
Show 25 quoted lines
>> * ab/ci-setup-simplify (2022-04-14) 29 commits
>> [...]
>>   Will merge to 'next'?
>>   source: <cover-v3-00.29-00000000000-20220413T194847Z-avarab@gmail.com>
>
> I haven't had time to read all 31 patches from v4 in detail but I have
> looked at the results in seen.
>
> Looking at seen:ci/install-dependencies.sh the shebang has been
> changed to "#!/bin/sh" but it contains
> "BREW_PACKAGE=${CC_PACKAGE/-/@}" which is a bashism.
>
> Looking at seen:.github/workflows/main.yaml to skip running the tests
> one has to set "skip-tests: no" which is utterly confusing.
>
> From what I saw scanning the patches there seemed to be a lot of
> churn, both of existing code and code that gets added and then
> moved/refactored within the series.
>
> Looking at the output of a recent ci run of seen the steps to prepare
> the environment before building and testing print all the environment 
> variables rather than just the ones being set for that step which
> seems to go against the aim of "CI: narrow down variable definitions
> in --build and --test". (Also the "SKIP" prefix in the output lacks a
> ":")

Thanks. Those were all helpful. I replied to these in a re-roll CL at: https://lore.kernel.org/git/cover-v5-00.29-00000000000-20220421T181526Z-avarab@gmail.com/

> Dscho raised concerns that this removes any support for azure
> pipelines which he uses when preparing security patches.

In the v1 discussion of my series ~2 months ago I asked him how he'd prefer to proceed with that: https://lore.kernel.org/git/220222.86y2236ndp.gmgdl@evledraar.gmail.com/

There wasn't any reply to that for about a month so I submitted the v2, noting that we might want to do something different there: https://lore.kernel.org/git/cover-v2-00.25-00000000000-20220325T182534Z-avarab@gmail.com/

And then as a follow-up to that v2 there was a follow-up discussion ending (from my side) here: https://lore.kernel.org/git/220406.86bkxeecoi.gmgdl@evledraar.gmail.com/

So, in brief summary I'm still happy to accommodate any such use-case. But I don't think it's OK to say that code we don't even use in-tree in area X must be kept as-is, to the point where it blocks forward progress for things that are used in-tree.

To be clear: I'm not saying that's Johannes's position, it may or may not be: The point is that he hasn't replied to https://lore.kernel.org/git/220222.86y2236ndp.gmgdl@evledraar.gmail.com/ or any relevant follow-up discussions. So I don't know what he's expecting there, or if it can be reasonably accommodated in making changes to ci/* which by their nature must decide to support/keep/fork/remove some of that code.

> I think splitting out the build and test steps is a good idea but I'm
> less convinced by some of the other changes.
What other changes are you referring to here?
Phillip Wood· Apr 22, 2022, 09:30 UTC · re: Ævar Arnfjörð Bjarmason · lore

Re: ab/ci-setup-simplify (was Re: What's cooking in git.git (Apr 2022, #05; Mon, 18))

Hi Ævar
On 21/04/2022 19:36, Ævar Arnfjörð Bjarmason wrote:
Show 36 quoted lines
> 
> On Tue, Apr 19 2022, Phillip Wood wrote:
> 
>>> * ab/ci-setup-simplify (2022-04-14) 29 commits
>>> [...]
>>>    Will merge to 'next'?
>>>    source: <cover-v3-00.29-00000000000-20220413T194847Z-avarab@gmail.com>
>>
>> I haven't had time to read all 31 patches from v4 in detail but I have
>> looked at the results in seen.
>>
>> Looking at seen:ci/install-dependencies.sh the shebang has been
>> changed to "#!/bin/sh" but it contains
>> "BREW_PACKAGE=${CC_PACKAGE/-/@}" which is a bashism.
>>
>> Looking at seen:.github/workflows/main.yaml to skip running the tests
>> one has to set "skip-tests: no" which is utterly confusing.
>>
>>  From what I saw scanning the patches there seemed to be a lot of
>> churn, both of existing code and code that gets added and then
>> moved/refactored within the series.
>>
>> Looking at the output of a recent ci run of seen the steps to prepare
>> the environment before building and testing print all the environment
>> variables rather than just the ones being set for that step which
>> seems to go against the aim of "CI: narrow down variable definitions
>> in --build and --test". (Also the "SKIP" prefix in the output lacks a
>> ":")
> 
> Thanks. Those were all helpful. I replied to these in a re-roll CL at:
> https://lore.kernel.org/git/cover-v5-00.29-00000000000-20220421T181526Z-avarab@gmail.com/
> [...] 
>> I think splitting out the build and test steps is a good idea but I'm
>> less convinced by some of the other changes.
> 
> What other changes are you referring to here?

Here's a list from memory - apologies if anything here has changed in v5

Separating the environment setup from running the build/tests is quite github specific. If instead we just had scripts that setup the environment and ran the appropriate make command they could be used by any future CI setup. Doing that would avoid lib.sh being a top level script rather than a library as well.

I expected tput.sh to be some kind of wrapper around tput but it just sets $TERM if it is not already set. We already do that in lib.sh I think and I don't see what the point of changing that. I think we'd want to do that whenever TERM is not set, not just for github actions.

Moving things from environment variables into MAKEFLAGS adds unnecessary complexity as we still have to export those variables on windows.

The script to run docker builds and tests under a unprivileged user is just deleted rather than fixing our docker builds to not run as root. At the moment some of the httpd tests are skipped because they refuse to run as root.

One thing I do like is moving the environment setup out of the github actions yaml and into the scripts as it should make it easier to support other CI setups.

Best Wishes
Phillip
Ævar Arnfjörð Bjarmason· Apr 22, 2022, 11:28 UTC · re: Phillip Wood · lore

Re: ab/ci-setup-simplify (was Re: What's cooking in git.git (Apr 2022, #05; Mon, 18))

 
On Fri, Apr 22 2022, Phillip Wood wrote:
Show 40 quoted lines
> Hi Ævar
>
> On 21/04/2022 19:36, Ævar Arnfjörð Bjarmason wrote:
>> On Tue, Apr 19 2022, Phillip Wood wrote:
>> 
>>>> * ab/ci-setup-simplify (2022-04-14) 29 commits
>>>> [...]
>>>>    Will merge to 'next'?
>>>>    source: <cover-v3-00.29-00000000000-20220413T194847Z-avarab@gmail.com>
>>>
>>> I haven't had time to read all 31 patches from v4 in detail but I have
>>> looked at the results in seen.
>>>
>>> Looking at seen:ci/install-dependencies.sh the shebang has been
>>> changed to "#!/bin/sh" but it contains
>>> "BREW_PACKAGE=${CC_PACKAGE/-/@}" which is a bashism.
>>>
>>> Looking at seen:.github/workflows/main.yaml to skip running the tests
>>> one has to set "skip-tests: no" which is utterly confusing.
>>>
>>>  From what I saw scanning the patches there seemed to be a lot of
>>> churn, both of existing code and code that gets added and then
>>> moved/refactored within the series.
>>>
>>> Looking at the output of a recent ci run of seen the steps to prepare
>>> the environment before building and testing print all the environment
>>> variables rather than just the ones being set for that step which
>>> seems to go against the aim of "CI: narrow down variable definitions
>>> in --build and --test". (Also the "SKIP" prefix in the output lacks a
>>> ":")
>> Thanks. Those were all helpful. I replied to these in a re-roll CL
>> at:
>> https://lore.kernel.org/git/cover-v5-00.29-00000000000-20220421T181526Z-avarab@gmail.com/
>> [...] 
>>> I think splitting out the build and test steps is a good idea but I'm
>>> less convinced by some of the other changes.
>> What other changes are you referring to here?
>
> Here's a list from memory - apologies if anything here has changed in
> v5
Thanks for following up, I think "probably not" re "anything changed"...
Show 5 quoted lines
> Separating the environment setup from running the build/tests is quite
> github specific. If instead we just had scripts that setup the
> environment and ran the appropriate make command they could be used by
> any future CI setup. Doing that would avoid lib.sh being a top level
> script rather than a library as well.
The part where we do :
 - ci/lib.sh --build
 - make

Is very GitHub-specific, i.e. it's relying on the "setenv" wrapper setting things in $GITHUB_ENV, which is then carried across "steps".

Now, it could just use "export" and have every "step" be the equivalent of:

 - ci/lib.sh --build && make

I.e. to have it "export", and not use $GITHUB_ENV, but I think the resulting UX is much better. I.e. the second step shows what variables are in use by the "step" now.

And for future-proofing I think we're better off as far as CI portability is concerned, and in fact I mostly want this so I can trivially run things locally "like in CI".

Adding another CI is a trivial matter of either eval-ing the ci/lib.sh output (which the tip commit emits instructions about how to do), or echo-ing to whatever that CI's equivalent of $GITHUB_ENV is.

> I expected tput.sh to be some kind of wrapper around tput but it just
> sets $TERM if it is not already set. We already do that in lib.sh I
> think and I don't see what the point of changing that. I think we'd
> want to do that whenever TERM is not set, not just for github actions.

You're right that we could do that earlier, i.e. to stick it in $GITHUB_ENV, but for these variables that were used only for some implementation detail of the scripts themselves I wanted to "source" them so they wouldn't appear in the variable list at the top.

I.e. that variable (unlike MAKEFLAGS etc.) isn't important to see how a run failed, and how it was configured, it's just working around platform-specific nits in the GitHub CI itself.

On the other hand it's just used in two places, and it's two lines, I could just copy/paste it. Do you think that's better?

> Moving things from environment variables into MAKEFLAGS adds
> unnecessary complexity as we still have to export those variables on
> windows.

I may have gotten something wrong here, but I was careful to move only the ones that are *for the Makefile* into MAKEFLAGS.

In most cases it's equivalent, but re $GITHUB_ENV dump readability it makes it more obvious what's being set for what things. I.e. Makefile itself v.s. the actual tests it then runs.

> The script to run docker builds and tests under a unprivileged user
> is just deleted rather than fixing our docker builds to not run as
> root. At the moment some of the httpd tests are skipped because they
> refuse to run as root.

What tests are being skipped? I may have gotten this horribly wrong, but I "git rm'd" those as part of deleting more leftover dead Travis code. I don't see where our current CI uses these at all.

*looks a bit*

I think you mean this, but this is in next (and master, but for some reason that run wasn't there): https://github.com/git/git/runs/6064756450?check_suite_focus=true#step:5:1279

So we should probably fix that, but I think that's unrelated to my series. It looks like we've been skipping those tests on GitHub CI ever since we had it at all.

> One thing I do like is moving the environment setup out of the github
> actions yaml and into the scripts as it should make it easier to
> support other CI setups.
*nod*, I very much have that as a goal.

← back to recent threads