* What's cooking in git.git (Dec 2022, #07; Mon, 26) @ 2022-12-26 3:38 Junio C Hamano 2022-12-26 10:52 ` cmake topics & js/ci-disable-cmake-by-default (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason 2022-12-28 8:36 ` built-in-submodule & in-flight dependencies (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason 0 siblings, 2 replies; 6+ messages in thread From: Junio C Hamano @ 2022-12-26 3:38 UTC (permalink / raw) To: git 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. A topic without enough support may be discarded after a long period of no activity. Copies 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/ -------------------------------------------------- [New Topics] * ab/avoid-losing-exit-codes-in-tests (2022-12-20) 6 commits - tests: don't lose misc "git" exit codes - tests: don't lose "git" exit codes in "! ( git ... | grep )" - tests: don't lose exit status with "test <op> $(git ...)" - tests: don't lose exit status with "(cd ...; test <op> $(git ...))" - t/lib-patch-mode.sh: fix ignored exit codes - auto-crlf tests: don't lose exit code in loops and outside tests Test clean-up. Will merge to 'next'? source: <cover-v4-0.6-00000000000-20221219T101240Z-avarab@gmail.com> * cw/ci-whitespace (2022-12-20) 3 commits - ci (check-whitespace): move to actions/checkout@v3 - ci (check-whitespace): add links to job output - ci (check-whitespace): suggest fixes for errors CI updates. Will merge to 'next'? source: <pull.1444.v2.git.1671496548.gitgitgadget@gmail.com> * dh/mingw-ownership-check-typofix (2022-12-20) 1 commit - mingw: fix typo in an error message from ownership check Error message typofix. Will merge to 'next'? source: <pull.1393.git.git.1671416810927.gitgitgadget@gmail.com> * js/ci-disable-cmake-by-default (2022-12-20) 1 commit - ci: only run win+VS build & tests in Git for Windows' fork Stop running win+VS build by default. Will merge to 'next'? source: <pull.1445.git.1671461414191.gitgitgadget@gmail.com> * sg/test-oid-wo-incomplete-line (2022-12-19) 1 commit - tests: make 'test_oid' print trailing newline Test helper updates. Will merge to 'next'? source: <20221218162905.3508164-1-szeder.dev@gmail.com> * ab/bundle-wo-args (2022-12-25) 1 commit (merged to 'next' on 2022-12-26 at b086677fd0) + bundle: don't segfault on "git bundle <subcmd>" Fix to a small regression in 2.38 days. Will merge to 'master'. source: <patch-1.1-2319eb2ddbd-20221220T133941Z-avarab@gmail.com> * ad/test-record-count-when-harness-is-in-use (2022-12-25) 1 commit - test-lib: allow storing counts with test harnesses Allow summary results from tests to be written to t/test-results directory even when a test harness like 'prove' is in use. Comments? source: <20221224225200.1027806-1-adam@dinwoodie.org> * ps/fsync-refs-fix (2022-12-25) 1 commit (merged to 'next' on 2022-12-26 at 6358875b39) + refs: fix corruption by not correctly syncing packed-refs to disk Fix the sequence to fsync $GIT_DIR/packed-refs file that forgot to flush its output to the disk.. Will merge to 'master'. source: <8c8ecf8e3718cbca049ee7a283edd7b7887e464e.1671547905.git.ps@pks.im> * sk/win32-pthread-exit-fix (2022-12-25) 1 commit (merged to 'next' on 2022-12-26 at ebcb1fef4c) + win32: use _endthreadex to terminate threads, not ExitThread An API emulation fix. Will merge to 'master'. source: <pull.1414.v2.git.git.1671932510529.gitgitgadget@gmail.com> * sk/win32-close-handle-upon-pthread-join (2022-12-26) 2 commits - win32: close handles of threads that have been joined - win32-pthread: prepare pthread.c for change by formatting source: <pull.1406.v12.git.git.1671932405.gitgitgadget@gmail.com> -------------------------------------------------- [Graduated to 'master'] * ab/darwin-default-to-sha1dc (2022-12-16) 1 commit (merged to 'next' on 2022-12-16 at 31fb33b832) + Makefile: use sha1collisiondetection by default on OSX and Darwin Use the SHA1DC implementation on macOS, just like other platforms, by default. source: <patch-v2-1.1-3de7cdbd260-20221215T084129Z-avarab@gmail.com> * ab/t4023-avoid-losing-exit-status-of-diff (2022-12-05) 1 commit (merged to 'next' on 2022-12-16 at 75c7a06930) + t4023: fix ignored exit codes of git Test fix. source: <patch-v2-3.8-c5feef1c808-20221202T000227Z-avarab@gmail.com> * ab/t5314-avoid-losing-exit-status (2022-12-02) 1 commit (merged to 'next' on 2022-12-16 at 40f3acd249) + t5314: check exit code of "git" Test fix. source: <patch-v2-1.1-ca77a7249e6-20221128T141818Z-avarab@gmail.com> * ab/t7600-avoid-losing-exit-status-of-git (2022-12-05) 1 commit (merged to 'next' on 2022-12-16 at ef89e4f2d1) + t7600: don't ignore "rev-parse" exit code in helper Test fix. source: <patch-v3-1.8-64dfec31fb3-20221202T114733Z-avarab@gmail.com> * aw/complete-case-insensitive (2022-11-30) 2 commits (merged to 'next' on 2022-12-16 at 341974c2bb) + completion: add case-insensitive match of pseudorefs + completion: add optional ignore-case when matching refs Introduce a case insensitive mode to the Bash completion helpers. source: <pull.1374.v2.git.git.1668990419.gitgitgadget@gmail.com> * jh/fsmonitor-darwin-modernize (2022-12-15) 1 commit (merged to 'next' on 2022-12-16 at 197e1f8c43) + fsmonitor: eliminate call to deprecated FSEventStream function Stop using deprecated macOS API in fsmonitor. source: <pull.1436.v2.git.1671045153981.gitgitgadget@gmail.com> * jh/t7527-unflake-by-forcing-cookie (2022-12-02) 1 commit (merged to 'next' on 2022-12-16 at d64198c6c3) + fsmonitor: fix race seen in t7527 Make fsmonitor more robust to avoid the flakiness seen in t7527. source: <pull.1437.git.1669937534944.gitgitgadget@gmail.com> * jk/avoid-redef-system-functions (2022-12-05) 3 commits (merged to 'next' on 2022-12-16 at 55f7102dba) + Merge branch 'jk/avoid-redef-system-functions-2.30' into jk/avoid-redef-system-functions + Merge branch 'jk/avoid-redef-system-functions-2.30' + Merge branch 'jk/avoid-redef-system-functions-2.30' into maint (this branch uses jk/avoid-redef-system-functions-2.30.) The jk/avoid-redef-system-functions-2.30 topic pre-merged for more recent codebase. * jk/avoid-redef-system-functions-2.30 (2022-12-05) 2 commits (merged to 'next' on 2022-12-16 at aa101dfc0c) + git-compat-util: undefine system names before redeclaring them + git-compat-util: avoid redefining system function names (this branch is used by jk/avoid-redef-system-functions.) Redefining system functions for a few functions did not follow our usual "implement git_foo() and #define foo(args) git_foo(args)" pattern, which has broken build for some folks. source: <Y4fH4rhcSztHwKvK@coredump.intra.peff.net> * jk/ref-filter-error-reporting-fix (2022-12-15) 5 commits (merged to 'next' on 2022-12-16 at 8135d51462) + ref-filter: convert email atom parser to use err_bad_arg() + ref-filter: truncate atom names in error messages + ref-filter: factor out "unrecognized %(foo) arg" errors + ref-filter: factor out "%(foo) does not take arguments" errors + ref-filter: reject arguments to %(HEAD) Clean-ups in error messages produced by "git for-each-ref" and friends. source: <Y5n3S7cnD7s/AIRL@coredump.intra.peff.net> * jk/server-supports-v2-cleanup (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at 9b995dc263) + server_supports_v2(): use a separate function for die_on_error Code clean-up. source: <Y5hZirNRGLQUZfOZ@coredump.intra.peff.net> * jk/unused-post-2.39 (2022-12-13) 9 commits (merged to 'next' on 2022-12-16 at 4ce4a868e5) + userdiff: mark unused parameter in internal callback + list-objects-filter: mark unused parameters in virtual functions + diff: mark unused parameters in callbacks + xdiff: mark unused parameter in xdl_call_hunk_func() + xdiff: drop unused parameter in def_ff() + ws: drop unused parameter from ws_blank_line() + list-objects: drop process_gitlink() function + blob: drop unused parts of parse_blob_buffer() + ls-refs: use repository parameter to iterate refs Code clean-up around unused function parameters. source: <Y5hdvpbLpXySHFRz@coredump.intra.peff.net> * js/t3920-shell-and-or-fix (2022-12-05) 1 commit (merged to 'next' on 2022-12-16 at 59536ed758) + t3920: don't ignore errors of more than one command with `|| true` (this branch is used by rs/t3920-crlf-eating-grep-fix.) Test fix. source: <febcfb0a-c410-fb71-cff9-92acfcb269e2@kdbg.org> * jt/http-fetch-trace2-report-name (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at b26e38cdb2) + http-fetch: invoke trace2_cmd_name() "git http-fetch" (which is rarely used) forgot to identify itself in the trace2 output. source: <20221212224630.2553391-1-jonathantanmy@google.com> * pg/diff-stat-unmerged-regression-fix (2022-12-15) 1 commit (merged to 'next' on 2022-12-16 at 2a61376847) + diff: fix regression with --stat and unmerged file The output from "git diff --stat" on an unmerged path lost the terminating LF in Git 2.39, which has been corrected. source: <20221214174150.404821-1-pete@jpgrayson.net> * rj/branch-copy-and-rename (2022-11-17) 1 commit (merged to 'next' on 2022-12-16 at b88fa41a8a) + branch: force-copy a branch to itself via @{-1} is a no-op Fix a pair of bugs in 'git branch'. source: <f0b2d46c-2e9c-2630-2870-8ed550dd1606@gmail.com> * rr/status-untracked-advice (2022-12-01) 1 commit (merged to 'next' on 2022-12-16 at bed7d84d07) + status: modernize git-status "slow untracked files" advice The advice message given by "git status" when it takes long time to enumerate untracked paths has been updated. source: <pull.1384.v9.git.1669769536707.gitgitgadget@gmail.com> * rs/am-parse-options-cleanup (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at 9013ff2a19) + am: don't pass strvec to apply_parse_options() Code clean-up. source: <baf93e4a-7f05-857c-e551-09675496c03c@web.de> * rs/clarify-error-in-write-loose-object (2022-12-14) 1 commit (merged to 'next' on 2022-12-16 at 8ef4588405) + object-file: inline write_buffer() Code clean-up. source: <94dff140-489c-40b2-7212-201ed3b7d008@web.de> * rs/clear-commit-marks-cleanup (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at a9e52a7cef) + commit: skip already cleared parents in clear_commit_marks_1() Code clean-up. source: <7cf2ea1f-8dbf-5639-3874-86de391ae20a@web.de> * rs/diff-parseopts (2022-12-02) 3 commits (merged to 'next' on 2022-12-16 at 6fdfd67cb9) + diff: remove parseopts member from struct diff_options + diff: use add_diff_options() in diff_opt_parse() + diff: factor out add_diff_options() The way the diff machinery prepares the options array for the parse_options API has been refactored to avoid resource leaks. source: <19325420-5630-87bb-0c2c-362151214b36@web.de> * rs/plug-pattern-list-leak-in-lof (2022-12-02) 1 commit (merged to 'next' on 2022-12-16 at 5cc6d4165a) + list-objects-filter: plug pattern_list leak Leak fix. source: <b4361c3e-852b-e30c-f240-86c34bc9c474@web.de> * rs/reflog-expiry-cleanup (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at 8165a625dd) + reflog: clear leftovers in reflog_expiry_cleanup() Code clean-up. source: <a53d4df3-4b2c-585a-e8ad-2a9b9017fb8c@web.de> * rs/t3920-crlf-eating-grep-fix (2022-12-07) 1 commit (merged to 'next' on 2022-12-16 at 56ff27beaa) + t3920: support CR-eating grep (this branch uses js/t3920-shell-and-or-fix.) Test fix. source: <cbe88abc-c1fb-cb50-6057-47ff27f7a12d@web.de> * rs/t4205-do-not-exit-in-test-script (2022-12-02) 1 commit (merged to 'next' on 2022-12-16 at 71b4de7729) + t4205: don't exit test script on failure Test fix. source: <c5b4d091-23c1-5a75-a255-99ec83973d8d@web.de> * sg/help-autocorrect-config-fix (2022-12-13) 1 commit (merged to 'next' on 2022-12-16 at edcf99f34c) + help.c: fix autocorrect in work tree for bare repository The code to auto-correct a misspelt subcommand unnecessarily called into git_default_config() from the early config codepath, which was a no-no. This has bee corrected. source: <pull.1373.v2.git.git.1667073374852.gitgitgadget@gmail.com> * sk/remove-duplicate-includes (2022-12-15) 1 commit (merged to 'next' on 2022-12-16 at 846bc84e59) + git: remove duplicate includes Code clean-up. source: <pull.1394.git.git.1671036743272.gitgitgadget@gmail.com> * sx/pthread-error-check-fix (2022-12-05) 1 commit (merged to 'next' on 2022-12-16 at 60848153fe) + maintenance: compare output of pthread functions for inequality with 0 Correct pthread API usage. source: <pull.1389.git.git.1670000578395.gitgitgadget@gmail.com> -------------------------------------------------- [Cooking] * ab/no-more-git-global-super-prefix (2022-12-26) 9 commits - read-tree: add "--super-prefix" option, eliminate global - submodule--helper: convert "{update,clone}" to their own "--super-prefix" - submodule--helper: convert "status" to its own "--super-prefix" - submodule--helper: convert "sync" to its own "--super-prefix" - submodule--helper: convert "foreach" to its own "--super-prefix" - submodule--helper: don't use global --super-prefix in "absorbgitdirs" - submodule.c & submodule--helper: pass along "super_prefix" param - read-tree + fetch tests: test failing "--super-prefix" interaction - submodule absorbgitdirs tests: add missing "Migrating git..." tests Stop using "git --super-prefix" and narrow the scope of its use to the submodule--helper. Will merge to 'next'. source: <cover-v5-0.9-00000000000-20221220T104519Z-avarab@gmail.com> * cb/grep-fallback-failing-jit (2022-12-17) 1 commit - grep: fall back to interpreter mode if JIT fails In an environment where dynamically generated code is prohibited to run (e.g. SELinux), failure to JIT pcre patterns is expected. Fall back to interpreted execution in such a case. Expecting a reroll. cf. <62a06c5b-9646-17f8-b4d5-39823d3cc25a@grsecurity.net> source: <20221216121557.30714-1-minipli@grsecurity.net> * jc/spell-id-in-both-caps-in-message-id (2022-12-17) 1 commit - e-mail workflow: Message-ID is spelled with ID in both capital letters Consistently spell "Message-ID" as such, not "Message-Id". Comments? source: <xmqqsfhgnmqg.fsf@gitster.g> * lk/line-range-parsing-fix (2022-12-20) 1 commit (merged to 'next' on 2022-12-26 at 216226a6e2) + line-range: fix infinite loop bug with '$' regex When given a pattern that matches an empty string at the end of a line, the code to parse the "git diff" line-ranges fell into an infinite loop, which has been corrected. Will merge to 'master'. source: <20221219224850.2703967-1-lars@oddbit.com> * sa/git-var-sequence-editor (2022-12-18) 1 commit (merged to 'next' on 2022-12-20 at a5737674b6) + var: add GIT_SEQUENCE_EDITOR variable Just like "git var GIT_EDITOR" abstracts the complex logic to choose which editor gets used behind it, "git var" now give support to GIT_SEQUENCE_EDITOR. Will merge to 'master'. source: <pull.1424.v2.git.1671318599482.gitgitgadget@gmail.com> * ss/pull-v-recurse-fix (2022-12-18) 1 commit (merged to 'next' on 2022-12-20 at 21f91dc75d) + submodule: accept -v for the update command "git pull -v --recurse-submodules" attempted to pass "-v" down to underlying "git submodule update", which did not understand the request and barfed, which has been corrected. Will merge to 'master'. source: <f805f2da-a7e1-9fde-cc0a-04a30f79c9af@cs-ware.de> * ds/omit-trailing-hash-in-index (2022-12-17) 4 commits - features: feature.manyFiles implies fast index writes - test-lib-functions: add helper for trailing hash - read-cache: add index.skipHash config option - hashfile: allow skipping the hash function Introduce an optional configuration to allow the trailing hash that protects the index file from bit flipping. cf. <221216.86sfhf1gbc.gmgdl@evledraar.gmail.com> source: <pull.1439.v4.git.1671204678.gitgitgadget@gmail.com> * ed/fsmonitor-inotify (2022-12-13) 6 commits - fsmonitor: update doc for Linux - fsmonitor: test updates - fsmonitor: enable fsmonitor for Linux - fsmonitor: implement filesystem change listener for Linux - fsmonitor: determine if filesystem is local or remote - fsmonitor: prepare to share code between Mac OS and Linux Bundled fsmonitor for Linux using inotify API. Needs review on the updated round. source: <pull.1352.v5.git.git.1670882286.gitgitgadget@gmail.com> * es/t1509-root-fixes (2022-12-09) 3 commits - t1509: facilitate repeated script invocations - t1509: make "setup" test more robust - t1509: fix failing "root work tree" test due to owner-check source: <pull.1425.git.1668999621.gitgitgadget@gmail.com> * jt/avoid-lazy-fetch-commits (2022-12-15) 4 commits - commit: don't lazy-fetch commits - object-file: emit corruption errors when detected - object-file: refactor map_loose_object_1() - object-file: remove OBJECT_INFO_IGNORE_LOOSE Even in a repository with promissor remote, it is useless to attempt to lazily attempt fetching an object that is expected to be commit, because no "filter" mode omits commit objects. Take advantage of this assumption to fail fast on errors. Will merge to 'next'? source: <cover.1671045259.git.jonathantanmy@google.com> * jx/t1301-updates (2022-11-30) 3 commits - t1301: do not change $CWD in "shared=all" test case - t1301: use test_when_finished for cleanup - t1301: fix wrong template dir for git-init Test updates. source: <20221128130323.8914-1-worldhello.net@gmail.com> * km/send-email-with-v-reroll-count (2022-11-30) 2 commits - SQUASH??? - send-email: relay '-v N' to format-patch "git send-email -v 3" used to be expanded to "git send-email --validate 3" when the user meant to pass them down to "format-patch", which has been corrected. Seems to break CI. source: <87edtp5uws.fsf@kyleam.com> * so/diff-merges-more (2022-12-18) 5 commits - diff-merges: improve --diff-merges documentation - diff-merges: issue warning on lone '-m' option - diff-merges: support list of values for --diff-merges - diff-merges: implement log.diffMerges-m-imply-p config - diff-merges: implement [no-]hide option and log.diffMergesHide config Assorted updates to "--diff-merges=X" option. source: <20221217132955.108542-1-sorganov@gmail.com> * ab/config-multi-and-nonbool (2022-11-27) 9 commits - for-each-repo: with bad config, don't conflate <path> and <cmd> - config API: add "string" version of *_value_multi(), fix segfaults - config API users: test for *_get_value_multi() segfaults - for-each-repo: error on bad --config - config API: have *_multi() return an "int" and take a "dest" - versioncmp.c: refactor config reading next commit - config tests: add "NULL" tests for *_get_value_multi() - config tests: cover blind spots in git_die_config() tests - for-each-repo tests: test bad --config keys Assorted config API updates. Waiting for review. source: <cover-v3-0.9-00000000000-20221125T093158Z-avarab@gmail.com> * sa/cat-file-mailmap--batch-check (2022-12-20) 2 commits - cat-file: add mailmap support to --batch-check option - cat-file: add mailmap support to -s option 'cat-file' gains mailmap support for its '--batch-check' and '-s' options. Will merge to 'next'. cf. <221220.865ye6xlmo.gmgdl@evledraar.gmail.com> source: <20221220060113.51010-1-siddharthasthana31@gmail.com> * ab/tag-object-type-errors (2022-11-22) 5 commits - tag: don't emit potentially incorrect "object is a X, not a Y" - tag: don't misreport type of tagged objects in errors - object tests: add test for unexpected objects in tags - object-file.c: free the "t.tag" in check_tag() - Merge branch 'jk/parse-object-type-mismatch' into ab/tag-object-type-errors Hardening checks around mismatched object types when one of those objects is a tag. Needs review. source: <cover-0.4-00000000000-20221118T113442Z-avarab@gmail.com> * js/range-diff-mbox (2022-11-23) 1 commit - range-diff: support reading mbox files 'git range-diff' gained support for reading either side from an .mbox file instead of a revision range. Expecting review responses. source: <pull.1420.v3.git.1669108102092.gitgitgadget@gmail.com> * tr/am--no-verify (2022-12-01) 2 commits - SQUASH??? - am: allow passing --no-verify flag Conditionally skip the pre-applypatch and applypatch-msg hooks when applying patches with 'git am'. Expecting a (hopefully final) reroll. source: <20221130172833.2662751-1-thierry.reding@gmail.com> * ew/format-patch-mboxrd (2022-12-25) 1 commit (merged to 'next' on 2022-12-26 at bb6cc9e207) + format-patch: support format.mboxrd with --stdout "git format-patch" learned to honor format.mboxrd even when sending patches to the standard output stream, Will merge to 'master'. source: <20221222201619.M435985@dcvr> * js/drop-mingw-test-cmp (2022-12-13) 1 commit - tests(mingw): avoid very slow `mingw_test_cmp` Use `git diff --no-index` as a test_cmp on Windows. Will merge to 'next'? source: <pull.1309.v5.git.1670339267.gitgitgadget@gmail.com> * mc/switch-advice (2022-11-09) 1 commit - po: use `switch` over `checkout` in error message Use 'switch' instead of 'checkout' in an error message. Waiting for review response. source: <pull.1308.git.git.1668018620148.gitgitgadget@gmail.com> * ja/worktree-orphan (2022-12-20) 4 commits - worktree add: add hint to use --orphan when bad ref - worktree add: add --orphan flag - worktree add: refactor opt exclusion tests - worktree add: include -B in usage docs 'git worktree add' learned how to create a worktree based on an orphaned branch with `--orphan`. source: <20221220023637.29042-1-jacobabel@nullpo.dev> * tb/ci-concurrency (2022-11-08) 1 commit - ci: avoid unnecessary builds Avoid unnecessary builds in CI, with settings configured in ci-config. Waiting for review. source: <ff172f1de982f6f79b598e4ac6d5b2964ca4a098.1667931937.git.me@ttaylorr.com> * tl/notes--blankline (2022-11-09) 5 commits - notes.c: introduce "--no-blank-line" option - notes.c: provide tips when target and append note are both empty - notes.c: drop unreachable code in 'append_edit()' - notes.c: cleanup for "designated init" and "char ptr init" - notes.c: cleanup 'strbuf_grow' call in 'append_edit' 'git notes append' was taught '--[no-]blank-line' to conditionally add a LF between a new and existing note. Waiting for review. source: <cover.1667980450.git.dyroneteng@gmail.com> * ds/bundle-uri-4 (2022-12-25) 11 commits (merged to 'next' on 2022-12-26 at 6d7c21c44a) + clone: unbundle the advertised bundles + bundle-uri: download bundles from an advertised list + bundle-uri: allow relative URLs in bundle lists + strbuf: introduce strbuf_strip_file_from_path() + bundle-uri: serve bundle.* keys from config + bundle-uri client: add helper for testing server + transport: rename got_remote_heads + bundle-uri client: add boolean transfer.bundleURI setting + clone: request the 'bundle-uri' command when available + t: create test harness for 'bundle-uri' command + protocol v2: add server-side "bundle-uri" skeleton Bundle URIs part 4. Will merge to 'master'. source: <pull.1400.v4.git.1671722058.gitgitgadget@gmail.com> * po/pretty-hard-trunc (2022-11-13) 1 commit - pretty-formats: add hard truncation, without ellipsis, options Add a new pretty format which truncates without ellipsis. Waiting for review. source: <20221112143616.1429-1-philipoakley@iee.email> * cc/filtered-repack (2022-12-25) 3 commits - gc: add gc.repackFilter config option - repack: add --filter=<filter-spec> option - pack-objects: allow --filter without --stdout "git repack" learns to discard objects that ought to be retrievable again from the promissor remote. source: <20221221040446.2860985-1-christian.couder@gmail.com> * mc/credential-helper-auth-headers (2022-12-13) 8 commits - t5556: add HTTP authentication tests - test-http-server: add simple authentication - test-http-server: pass Git requests to http-backend - test-http-server: add HTTP request parsing - test-http-server: add HTTP error response function - test-http-server: add stub HTTP server test helper - credential: add WWW-Authenticate header to cred requests - http: read HTTP WWW-Authenticate response headers Extending credential helper protocol. Waiting for review responses (or a reroll). cf. <1dc44716-2550-47de-e666-9972b102905d@github.com> source: <pull.1352.v4.git.1670880984.gitgitgadget@gmail.com> -------------------------------------------------- [Discarded] * pw/test-todo (2022-10-06) 3 commits . test_todo: allow [verbose] test as the command . test_todo: allow [!] grep as the command . tests: add test_todo() to mark known breakages RFC for test framework improvement. Needs review. source: <pull.1374.git.1665068476.gitgitgadget@gmail.com> * cw/submodule-status-in-parallel (2022-11-08) 6 commits . diff-lib: parallelize run_diff_files for submodules . diff-lib: refactor match_stat_with_submodule . submodule: move status parsing into function . submodule: strbuf variable rename . run-command: add duplicate_output_fn to run_processes_parallel_opts . Merge branch 'ab/run-hook-api-cleanup' into cw/submodule-status-in-parallel Allow the internal "diff-files" engine to run "how has this submodule changed?" in parallel to speed up "git status". Waiting for review. source: <20221020232532.1128326-1-calvinwan@google.com> * es/doc-creation-factor-fix (2022-07-28) 2 commits . range-diff: clarify --creation-factor=<factor> . format-patch: clarify --creation-factor=<factor> Expecting a reroll by somebody more familiar with the logic cf. <xmqqo7wfix7p.fsf@gitster.g> source: <7229p500-p2r4-on87-6802-8o90s36rr3s4@tzk.qr> * ds/bundle-uri-4-fixup (2022-12-13) 3 commits . bundle-uri: remove GIT_TEST_BUNDLE_URI env variable . bundle-uri: advertise based on repo config . bundle-uri: drop unused 'uri' parameter Incremental fixes on ds/bundle-uri-4 topic. Discarded. The base topic has been updated to make these "oops" fixes unneeded. source: <pull.1443.git.1670866407.gitgitgadget@gmail.com> ^ permalink raw reply [flat|nested] 6+ messages in thread
* cmake topics & js/ci-disable-cmake-by-default (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) 2022-12-26 3:38 What's cooking in git.git (Dec 2022, #07; Mon, 26) Junio C Hamano @ 2022-12-26 10:52 ` Ævar Arnfjörð Bjarmason 2022-12-27 4:26 ` cmake topics & js/ci-disable-cmake-by-default Junio C Hamano 2022-12-28 8:36 ` built-in-submodule & in-flight dependencies (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason 1 sibling, 1 reply; 6+ messages in thread From: Ævar Arnfjörð Bjarmason @ 2022-12-26 10:52 UTC (permalink / raw) To: Junio C Hamano; +Cc: git I split up the previously merged to "next" ab/cmake-nix-and-ci and submitted the uncontroversial parts of it as: https://lore.kernel.org/git/patch-1.1-0fa41115261-20221219T102205Z-avarab@gmail.com https://lore.kernel.org/git/cover-0.2-00000000000-20221219T102813Z-avarab@gmail.com https://lore.kernel.org/git/cover-0.6-00000000000-20221219T183623Z-avarab@gmail.com I think whatever happens with js/ci-disable-cmake-by-default that it makes sense to pick up & integrate those. It's all narrow fixes for specific issues, none of which cover the area(s) that caused ab/cmake-nix-and-ci to be ejected from "next". On Mon, Dec 26 2022, Junio C Hamano wrote: > * js/ci-disable-cmake-by-default (2022-12-20) 1 commit > - ci: only run win+VS build & tests in Git for Windows' fork > > Stop running win+VS build by default. > > Will merge to 'next'? > source: <pull.1445.git.1671461414191.gitgitgadget@gmail.com> Per my feedback there, I still think it would make sense to at least split up the "should we build with MSVC?" from the "do we use cmake, and run the re-run tests we already ran with GCC with MSVC too?". But I understand that Johannes disagreed with that, and didn't think testing MSVC in addition to GCC & Clang was valuable. What I'd like clarification on if this moves forward is what the status of cmake in the tree is. The reason I submitted ab/cmake-nix-and-ci was to make testing cmake+ctest on *nix trivial, so that if I and others made Makefile changes we could change & test the cmake recipe as well. But now we won't even run that in CI, and "git-for-windows" will have ownership of it. Does that mean that for such Makefile changes we should simply leave out the cmake changes, and rely on git-for-windows to "catch up" with its cmake contrib component? Ultimately I don't mind such an arrangement, but I think that js/ci-disable-cmake-by-default brings us to a weird in-between state. Just removing it from the tree and having git-for-windows carry it would make sense. So would IMO taking ab/cmake-nix-and-ci (or the restart of it above), i.e. to have cmake fixes accepted in git/git. But now we're moving to a state where such patches aren't welcome, but we'll still carry inevitably bitrotting code in-tree, which we're intentionally not testing anymore until it's merged to git-for-windows? Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: cmake topics & js/ci-disable-cmake-by-default 2022-12-26 10:52 ` cmake topics & js/ci-disable-cmake-by-default (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason @ 2022-12-27 4:26 ` Junio C Hamano 2022-12-27 13:59 ` Ævar Arnfjörð Bjarmason 0 siblings, 1 reply; 6+ messages in thread From: Junio C Hamano @ 2022-12-27 4:26 UTC (permalink / raw) To: Ævar Arnfjörð Bjarmason; +Cc: git Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: > I split up the previously merged to "next" ab/cmake-nix-and-ci and > submitted the uncontroversial parts of it as: Not gathering any interest by folks who will be affected is different from being uncontroversial, though. It may not have seen any controversy so far, but once it reappears in my tree and sufficiently advances to cause trouble to other people, it would. In other words, I am saving time and energy of people by waiting for positive support on these changes. > I think whatever happens with js/ci-disable-cmake-by-default that it > makes sense to pick up & integrate those. I do not think so at all, at least judging by what little has been said so far on the list. Comments on two among these three are negative ones, and the other one had no traction. >> * js/ci-disable-cmake-by-default (2022-12-20) 1 commit >> - ci: only run win+VS build & tests in Git for Windows' fork >> >> Stop running win+VS build by default. >> >> Will merge to 'next'? >> source: <pull.1445.git.1671461414191.gitgitgadget@gmail.com> > > Per my feedback there, I still think it would make sense to at least > split up the "should we build with MSVC?" from the "do we use cmake, and > run the re-run tests we already ran with GCC with MSVC too?". Do you mean that in our primary CI jobs, you would, using Makefile, want to keep building the win+VS artifacts with MSVC and running the tests, even though Windows folks want to drive the same build process via CMake, and their release binaries will come from the latter? I am not sure which extra corners, which matter to us, are covered by doing so. What's the upside? > But now we won't even run that in CI, and "git-for-windows" will have > ownership of it. > > Does that mean that for such Makefile changes we should simply leave out > the cmake changes, and rely on git-for-windows to "catch up" with its > cmake contrib component? That is the natural conclusion of what has been said on the list so far. We do not even "rely on"---it is up to them who chose to use CMake to keep it up to date or lag behind. Among those who have made significant contributions and works outside Windows, we found nobody whowants to touch CMake. > Ultimately I don't mind such an arrangement, but I think that > js/ci-disable-cmake-by-default brings us to a weird in-between > state. Just removing it from the tree and having git-for-windows carry > it would make sense. That's fine by me personally, but somebody has to help coordinating such a move between two projects. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: cmake topics & js/ci-disable-cmake-by-default 2022-12-27 4:26 ` cmake topics & js/ci-disable-cmake-by-default Junio C Hamano @ 2022-12-27 13:59 ` Ævar Arnfjörð Bjarmason 2022-12-27 23:06 ` Junio C Hamano 0 siblings, 1 reply; 6+ messages in thread From: Ævar Arnfjörð Bjarmason @ 2022-12-27 13:59 UTC (permalink / raw) To: Junio C Hamano; +Cc: git, Phillip Wood, Johannes Schindelin On Tue, Dec 27 2022, Junio C Hamano wrote: > Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: > >> I split up the previously merged to "next" ab/cmake-nix-and-ci and >> submitted the uncontroversial parts of it as: > > Not gathering any interest by folks who will be affected is > different from being uncontroversial, though. It may not have seen > any controversy so far, but once it reappears in my tree and > sufficiently advances to cause trouble to other people, it would. > > In other words, I am saving time and energy of people by waiting for > positive support on these changes. These changes had made it to "next" already on the basis of the feedback the topic already got. Now, that's not justification in itself that a re-roll & split-up belongs there again, but it also strikes me as a bad use of time to just throw them away at this point now that I've submitted a version that addresses the trouble it caused others (which I think the combined v6 also fixed, but as the topic is fairly large I think this split-up was prudent). >> I think whatever happens with js/ci-disable-cmake-by-default that it >> makes sense to pick up & integrate those. > > I do not think so at all, at least judging by what little has been > said so far on the list. Comments on two among these three are > negative ones, and the other one had no traction. I'm assuming you're counting your own https://lore.kernel.org/git/xmqqcz8e29d5.fsf@gitster.g/ as negative here, I just replied at https://lore.kernel.org/git/221227.86tu1huevt.gmgdl@evledraar.gmail.com/, but maybe you're still not convinced. In any case, that 1/3 topic could be discarded without the other 2/3 being affected, even though it seems to me to be a trivially correct fix. If you mean https://lore.kernel.org/git/f67e0281-8a14-669d-0d1c-ed0b1351a64a@dunelm.org.uk/ I'm not sure what to do with that other than my reply at https://lore.kernel.org/git/221219.86h6xrzaxd.gmgdl@evledraar.gmail.com/. I don't see how I, having spent quite a bit of time untangling the platform-specific parts of the CMake recipe, would be inherently unqualified to spot and clarify the CMake docs in such a way that they make it clear what parts are the cross-platform parts, and which are the platform-specific parts. >>> * js/ci-disable-cmake-by-default (2022-12-20) 1 commit >>> - ci: only run win+VS build & tests in Git for Windows' fork >>> >>> Stop running win+VS build by default. >>> >>> Will merge to 'next'? >>> source: <pull.1445.git.1671461414191.gitgitgadget@gmail.com> >> >> Per my feedback there, I still think it would make sense to at least >> split up the "should we build with MSVC?" from the "do we use cmake, and >> run the re-run tests we already ran with GCC with MSVC too?". > > Do you mean that in our primary CI jobs, you would, using Makefile, > want to keep building the win+VS artifacts with MSVC and running the > tests, even though Windows folks want to drive the same build > process via CMake, and their release binaries will come from the > latter? I don't think "running the tests" is a good use of time, although we might want to do so if some "extended tests" ci-config option is set, which e.g. "next" and/or "master" might want to turn on. So just like we run tests on OSX for both the git built with gcc and clang (and yes, that isn't the only difference, we sneak in the "no sha1dc" there too) perhaps we'd like to run the tests for the MSVC-built git too. > I am not sure which extra corners, which matter to us, are covered > by doing so. What's the upside? Doing the *build* is relatively cheap when it comes to CI time, and would catch some portability issues. In any case, I'm not asserting here that js/ci-disable-cmake-by-default must do what I'm suggesting in the last few paragraphs. But rather that its commit message doesn't really cover what it's actually doing, and that at least adjusting that would be worthwhile. It's saying that it's disabling the "build and test with cmake + MSVC" entirely because the tests are slow, and that cmake isn't wanted in git/git. Whereas the more obvious conclusion IMO is to then undo the later change to that job to use "cmake" to build with "MSVC" (which was a later development) and/or don't run the test part. >> But now we won't even run that in CI, and "git-for-windows" will have >> ownership of it. >> >> Does that mean that for such Makefile changes we should simply leave out >> the cmake changes, and rely on git-for-windows to "catch up" with its >> cmake contrib component? > > That is the natural conclusion of what has been said on the list so > far. We do not even "rely on"---it is up to them who chose to use > CMake to keep it up to date or lag behind. Among those who have > made significant contributions and works outside Windows, we found > nobody whowants to touch CMake. Thanks, that's pretty much the clarification I was looking for. I do think it's going to result in more disruption than is strictly necessary, but *shrug*. I do think it would be good to prominently note this at the top of CMakeLists.txt, i.e. that the version in git/git can be expected to be broken or out of date, I could re-roll the "cmake doc" topic I have outstanding to add that, except... >> Ultimately I don't mind such an arrangement, but I think that >> js/ci-disable-cmake-by-default brings us to a weird in-between >> state. Just removing it from the tree and having git-for-windows carry >> it would make sense. > > That's fine by me personally, but somebody has to help coordinating > such a move between two projects. ...(continuing from above). I'd be happy to do that, even though I'm still surprised that this is the path Johannes seems to prefer going forward. As it's surely more work for git-for-windows to deal with whatever we throw over the wall at them, rather than have us take them into account (which would have been easy once we CI'd it on Ubuntu: https://lore.kernel.org/git/patch-v6-15.15-917a884eb65-20221206T001617Z-avarab@gmail.com/). But *shrug*. I think if we're going down this path it would make sense to first start by removing the support for building with MSVC and generating the VS project files from the Makefile. There's more to it than this, but it's basically this & adjacent code (e.g. in config.mak.uname): $ wc -l $(git ls-files 'contrib/buildsystems/Generators*' 'compat/vcbuild/scripts*' compat/vcbuild/find_vs_env.bat) 168 compat/vcbuild/find_vs_env.bat 133 compat/vcbuild/scripts/clink.pl 26 compat/vcbuild/scripts/lib.pl 42 contrib/buildsystems/Generators.pm 189 contrib/buildsystems/Generators/QMake.pm 579 contrib/buildsystems/Generators/Vcproj.pm 402 contrib/buildsystems/Generators/Vcxproj.pm 1539 total This has all been bitrotting since 4c2c38e800f (ci: modification of main.yml to use cmake for vs-build job, 2020-06-26), and it's been broken since sometime in 2020 for DEVELOPER=1 builds at least (started failing with new warnings, and then the C99 change in git-compat-util.h). One reason I thought keeping the CMake recipe up-to-date & in-tree is that it's much easier to maintain it than e.g. the hack that is compat/vcbuild/scripts/clink.pl, which we'd need (or something like it) if we're going to support building with MSVC with the Makefile. But I don't see how it would make sense to have CMakeLists.txt bitrotting in-tree, and for git/git to pick up the maintenance of this "build using MSVC with the Makefile", which git-for-windows itself seems to have abandoned. (The actual release versions of GfW are, as I understand it, built with GCC, so the only "cmake with MSVC" users are those using some lightweight version of MS development tools) Then, as a follow-up to that, if improvements to contrib/buildsystems/CMakeLists.txt aren't going to be accepted to git/git. Or rather, if they would in principle, but are going to stall because they might impact git-for-windows. As seems to be the case with my proposed, which have had no timely feedback from the git-for-windows folks on "upstreaming" those (or rather, fixing "upstream"). Then shouldn't we just: git rm contrib/buildsystems/CMakeLists.txt Along with the other bits in-tree which at that point would only be used for CMake, e.g. compat/vcbuild/vcpkg_install.bat? I could submit all of that if you agree. This would not be my first choice, but I don't think it's good to leave this in limbo in git/git either. The "git-for-windows" project would then just revert the part that removes contrib/buildsystems/CMakeLists.txt once it lands there, and proceed to maintain contrib/buildsystems/CMakeLists.txt only in that project, along with the diverging part of main.yml etc. What do you think? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: cmake topics & js/ci-disable-cmake-by-default 2022-12-27 13:59 ` Ævar Arnfjörð Bjarmason @ 2022-12-27 23:06 ` Junio C Hamano 0 siblings, 0 replies; 6+ messages in thread From: Junio C Hamano @ 2022-12-27 23:06 UTC (permalink / raw) To: Ævar Arnfjörð Bjarmason Cc: git, Phillip Wood, Johannes Schindelin Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: > On Tue, Dec 27 2022, Junio C Hamano wrote: > >> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: >> >>> I split up the previously merged to "next" ab/cmake-nix-and-ci and >>> submitted the uncontroversial parts of it as: >> >> Not gathering any interest by folks who will be affected is >> different from being uncontroversial, though. It may not have seen >> any controversy so far, but once it reappears in my tree and >> sufficiently advances to cause trouble to other people, it would. >> >> In other words, I am saving time and energy of people by waiting for >> positive support on these changes. > > These changes had made it to "next" already on the basis of the feedback > the topic already got. After the topic turned out to be unwanted, I am not sure how much weight we want to give to the supportive opinions that initially pushed it to 'next'. ^ permalink raw reply [flat|nested] 6+ messages in thread
* built-in-submodule & in-flight dependencies (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) 2022-12-26 3:38 What's cooking in git.git (Dec 2022, #07; Mon, 26) Junio C Hamano 2022-12-26 10:52 ` cmake topics & js/ci-disable-cmake-by-default (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason @ 2022-12-28 8:36 ` Ævar Arnfjörð Bjarmason 1 sibling, 0 replies; 6+ messages in thread From: Ævar Arnfjörð Bjarmason @ 2022-12-28 8:36 UTC (permalink / raw) To: Junio C Hamano; +Cc: git, Glen Choo, Derrick Stolee On Mon, Dec 26 2022, Junio C Hamano wrote: > [...] > [Cooking] > > * ab/no-more-git-global-super-prefix (2022-12-26) 9 commits > - read-tree: add "--super-prefix" option, eliminate global > - submodule--helper: convert "{update,clone}" to their own "--super-prefix" > - submodule--helper: convert "status" to its own "--super-prefix" > - submodule--helper: convert "sync" to its own "--super-prefix" > - submodule--helper: convert "foreach" to its own "--super-prefix" > - submodule--helper: don't use global --super-prefix in "absorbgitdirs" > - submodule.c & submodule--helper: pass along "super_prefix" param > - read-tree + fetch tests: test failing "--super-prefix" interaction > - submodule absorbgitdirs tests: add missing "Migrating git..." tests > > Stop using "git --super-prefix" and narrow the scope of its use to > the submodule--helper. > > Will merge to 'next'. > source: <cover-v5-0.9-00000000000-20221220T104519Z-avarab@gmail.com> > [...] > * ab/config-multi-and-nonbool (2022-11-27) 9 commits > - for-each-repo: with bad config, don't conflate <path> and <cmd> > - config API: add "string" version of *_value_multi(), fix segfaults > - config API users: test for *_get_value_multi() segfaults > - for-each-repo: error on bad --config > - config API: have *_multi() return an "int" and take a "dest" > - versioncmp.c: refactor config reading next commit > - config tests: add "NULL" tests for *_get_value_multi() > - config tests: cover blind spots in git_die_config() tests > - for-each-repo tests: test bad --config keys > > Assorted config API updates. > > Waiting for review. > source: <cover-v3-0.9-00000000000-20221125T093158Z-avarab@gmail.com> I see that ab/no-more-git-global-super-prefix has landed in "next", I was planning to submit the built-in submodule topic after it, i.e. the change to get rid of git-submodule.sh. But the post-cleanup in that topic of moving the relevant code from builtin/submodule--helper.c to builtin/submodule.c would conflict with ab/config-multi-and-nonbool, as some of it uses the *_multi() API. Now, it would be relatively easy to solve those conflicts, but I'd also like to make it easier for you, Junio. So more reviews on that topic would be most welcome. It fixes quite a few segfaults we currently have in-tree, so I'm a bit surprised at the relative lack of interest. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-12-28 8:44 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-12-26 3:38 What's cooking in git.git (Dec 2022, #07; Mon, 26) Junio C Hamano 2022-12-26 10:52 ` cmake topics & js/ci-disable-cmake-by-default (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason 2022-12-27 4:26 ` cmake topics & js/ci-disable-cmake-by-default Junio C Hamano 2022-12-27 13:59 ` Ævar Arnfjörð Bjarmason 2022-12-27 23:06 ` Junio C Hamano 2022-12-28 8:36 ` built-in-submodule & in-flight dependencies (was: What's cooking in git.git (Dec 2022, #07; Mon, 26)) Ævar Arnfjörð Bjarmason
Code repositories for project(s) associated with this public inbox https://80x24.org/mirrors/git.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).