{"thread":{"id":"49349","subject":"What's cooking in git.git (Sep 2018, #03; Fri, 14)","startedAt":"2018-09-14T21:56:45Z","lastAt":"2018-09-19T18:43:46Z","messageCount":17,"participants":["Junio C Hamano","Antonio Ospite","Duy Nguyen","brian m. carlson","Ævar Arnfjörð Bjarmason","Jeff King","Jonathan Nieder","Derrick Stolee","Martin Ågren"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"358165","messageId":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":null,"subject":"What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-14T21:56:36Z","receivedAt":"2018-09-14T21:56:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'.  The ones marked with '.' do not appear in any of\nthe integration branches, but I am still holding onto them.\n\nThe tip of 'next' hasn't been rewound yet, but the states of many\ntopics that were marked to \"Cook in 'next'\" read \"Will merge to\n'master'\" now.  I'll probably start merging a handful of topics that\nhave been in 'next' down to 'master' tonight (they appear at the\nbottom of \"log --first-parent master..pu\" output).\n\nThe three GSoC \"rewrite in C\" topics are still unclassified in this\n\"What's cooking\" report, but I am hoping that we can have them in\n'next' sooner rather than later.  I got an impression that Dscho\nwanted a chance for the final clean-up on some of them, so I am not\ndoing anything hasty yet at this moment, though.\n\nYou can find the changes described here in the integration branches\nof the repositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[New Topics]\n\n* ab/fsck-skiplist (2018-09-12) 10 commits\n - fsck: support comments & empty lines in skipList\n - fsck: use oidset instead of oid_array for skipList\n - fsck: use strbuf_getline() to read skiplist file\n - fsck: add a performance test for skipList\n - fsck: add a performance test\n - fsck: document that skipList input must be unabbreviated\n - fsck: document and test commented & empty line skipList input\n - fsck: document and test sorted skipList input\n - fsck tests: add a test for no skipList input\n - fsck tests: setup of bogus commit object\n\n Update fsck.skipList implementation and documentation.\n\n Will merge to 'next'.\n\n\n* bc/hash-independent-tests (2018-09-13) 12 commits\n - t5318: use test_oid for HASH_LEN\n - t1407: make hash size independent\n - t1406: make hash-size independent\n - t1405: make hash size independent\n - t1400: switch hard-coded object ID to variable\n - t1006: make hash size independent\n - t0064: make hash size independent\n - t0027: make hash size independent\n - t0002: abstract away SHA-1 specific constants\n - t0000: update tests for SHA-256\n - t0000: use hash translation table\n - t: add test functions to translate hash-related values\n\n Various tests have been updated to make it easier to swap the\n hash function used for object identification.\n\n Will merge to 'next'.\n\n\n* bp/mv-submodules-with-fsmonitor (2018-09-12) 1 commit\n - git-mv: allow submodules and fsmonitor to work together\n\n When fsmonitor is in use, after operation on submodules updates\n .gitmodules, we lost track of the fact that we did so and relied on\n stale fsmonitor data.\n\n Will merge to 'next'.\n\n\n* bp/read-cache-parallel (2018-09-13) 6 commits\n - SQUASH???\n - read-cache: clean up casting and byte decoding\n - read-cache.c: optimize reading index format v4\n - read-cache: load cache entries on worker threads\n - read-cache: load cache extensions on a worker thread\n - eoie: add End of Index Entry (EOIE) extension\n\n A new extension to the index file has been introduced, which allows\n the file to be read in parallel.\n\n Will merge to 'next' after squashing the fix in.\n\n\n* ds/coverage-diff (2018-09-12) 1 commit\n - contrib: add coverage-diff script\n\n The result of coverage test can be combined with \"git blame\" to\n check the test coverage of code introduced recently with a new\n 'coverage-diff' tool (in contrib/).\n\n Expecting a reroll.\n\n\n* ds/format-patch-range-diff-test (2018-09-12) 1 commit\n - t3206-range-diff.sh: cover single-patch case\n (this branch uses es/format-patch-interdiff and es/format-patch-rangediff.)\n\n Will merge to 'next'.\n\n\n* ds/multi-pack-verify (2018-09-13) 11 commits\n - fsck: verify multi-pack-index\n - multi-pack-index: report progress during 'verify'\n - multi-pack-index: verify object offsets\n - multi-pack-index: fix 32-bit vs 64-bit size check\n - multi-pack-index: verify oid lookup order\n - multi-pack-index: verify oid fanout order\n - multi-pack-index: verify missing pack\n - multi-pack-index: verify packname order\n - multi-pack-index: verify corrupt chunk lookup table\n - multi-pack-index: verify bad header\n - multi-pack-index: add 'verify' verb\n (this branch uses ds/multi-pack-index.)\n\n \"git multi-pack-index\" learned to detect corruption in the .midx\n file it uses, and this feature has been integrated into \"git fsck\".\n\n Will merge to 'next'.\n\n\n* en/sequencer-empty-edit-result-aborts (2018-09-13) 1 commit\n - sequencer: fix --allow-empty-message behavior, make it smarter\n\n \"git rebase\" etc. in Git 2.19 fails to abort when given an empty\n commit log message as result of editing, which has been corrected.\n\n Will merge to 'next'.\n\n\n* en/update-ref-no-deref-stdin (2018-09-12) 2 commits\n - update-ref: allow --no-deref with --stdin\n - update-ref: fix type of update_flags variable to match its usage\n\n \"git update-ref\" learned to make both \"--no-deref\" and \"--stdin\"\n work at the same time.\n\n Will merge to 'next'.\n\n\n* jt/lazy-object-fetch-fix (2018-09-13) 2 commits\n - fetch-object: set exact_oid when fetching\n - fetch-object: unify fetch_object[s] functions\n\n The code to backfill objects in lazily cloned repository did not\n work correctly, which has been corrected.\n\n Will merge to 'next'.\n\n\n* ms/remote-error-message-update (2018-09-14) 1 commit\n - builtin/remote: quote remote name on error to display empty name\n\n Update error messages given by \"git remote\" and make them consistent.\n\n Will merge to 'next'.\n\n\n* nd/config-split (2018-09-12) 11 commits\n - config.txt: move submodule part out to a separate file\n - config.txt: move sequence.editor out of \"core\" part\n - config.txt: move sendemail part out to a separate file\n - config.txt: move receive part out to a separate file\n - config.txt: move push part out to a separate file\n - config.txt: move pull part out to a separate file\n - config.txt: move gui part out to a separate file\n - config.txt: move gitcvs part out to a separate file\n - config.txt: move format part out to a separate file\n - config.txt: move fetch part out to a separate file\n - config.txt: follow camelCase naming\n\n Split Documentation/config.txt for easier maintenance.\n\n Will merge to 'next'.\n\n\n* sb/submodule-recursive-fetch-gets-the-tip (2018-09-12) 9 commits\n - builtin/fetch: check for submodule updates for non branch fetches\n - fetch: retry fetching submodules if sha1 were not fetched\n - submodule: fetch in submodules git directory instead of in worktree\n - submodule.c: do not copy around submodule list\n - submodule: move global changed_submodule_names into fetch submodule struct\n - submodule.c: sort changed_submodule_names before searching it\n - submodule.c: fix indentation\n - sha1-array: provide oid_array_filter\n - string-list: add string_list_{pop, last} functions\n\n \"git fetch --recurse-submodules\" may not fetch the necessary commit\n that is bound to the superproject, which is getting corrected.\n\n Expecting a reroll.\n cf. <CAGZ79kbavjVbTqXsmtjW6=jhkq47_p3mc6=92xOp4_mfhqDtvw@mail.gmail.com>\n cf. <b16af8c0-0435-0de4-ed6c-53888d6190af@ramsayjones.plus.com>\n cf. <CAGZ79kZKKf9N8yx9EuCRZhrZS_mA2218PouEG7aHDhK2bJGEdA@mail.gmail.com>\n\n\n* sg/split-index-test (2018-09-12) 2 commits\n - t0090: disable GIT_TEST_SPLIT_INDEX for the test checking split index\n - t1700-split-index: drop unnecessary 'grep'\n\n Test updates.\n\n Will merge to 'next'.\n\n\n* tb/void-check-attr (2018-09-12) 1 commit\n - Make git_check_attr() a void function\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* tg/range-diff-corner-case-fix (2018-09-14) 1 commit\n - linear-assignment: fix potential out of bounds memory access\n\n Recently added \"range-diff\" had a corner-case bug to cause it\n segfault, which has been corrected.\n\n Will merge to 'next'.\n\n\n* bp/rename-test-env-var (2018-09-14) 5 commits\n - preload-index: update GIT_FORCE_PRELOAD_TEST support\n - read-cache: update TEST_GIT_INDEX_VERSION support\n - fsmonitor: update GIT_TEST_FSMONITOR support\n - preload-index: teach GIT_FORCE_PRELOAD_TEST to take a boolean\n - correct typo/spelling error in t/README\n\n Some environment variables that control the runtime options of Git\n used during tests are getting renamed for consistency.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* bw/submodule-name-to-dir (2018-08-10) 2 commits\n  (merged to 'next' on 2018-08-22 at c17f83be24)\n + submodule: munge paths to submodule git directories\n + submodule: create helper to build paths to submodule gitdirs\n\n In modern repository layout, the real body of a cloned submodule\n repository is held in .git/modules/ of the superproject, indexed by\n the submodule name.  URLencode the submodule name before computing\n the name of the directory to make sure they form a flat namespace.\n\n Will eject out of 'next', expecting further work on the topic.\n cf. <CAGZ79kYnbjaPoWdda0SM_-_X77mVyYC7JO61OV8nm2yj3Q1OvQ@mail.gmail.com>\n\n\n* ng/status-i-short-for-ignored (2018-08-09) 1 commit\n - status: -i shorthand for --ignored command line option\n\n \"git status --ignored\" gained a shorthand \"git status -i\".\n\n Will discard, after hearing no strong support.\n What's the list opinion on this one?  It is Meh to me, but\n obviously the author cared enough to write a patch, so...\n\n\n* sb/submodule-move-head-with-corruption (2018-08-28) 2 commits\n - submodule.c: warn about missing submodule git directories\n - t2013: add test for missing but active submodule\n\n Will discard and wait for a cleaned-up rewrite.\n cf. <20180907195349.GA103699@aiede.svl.corp.google.com>\n\n\n* sl/commit-dry-run-with-short-output-fix (2018-07-30) 4 commits\n . commit: fix exit code when doing a dry run\n . wt-status: teach wt_status_collect about merges in progress\n . wt-status: rename commitable to committable\n . t7501: add coverage for flags which imply dry runs\n\n \"git commit --dry-run\" gave a correct exit status even during a\n conflict resolution toward a merge, but it did not with the\n \"--short\" option, which has been corrected.\n\n Seems to break 7512, 3404 and 7060 in 'pu'.\n\n\n* ma/wrapped-info (2018-05-28) 2 commits\n - usage: prefix all lines in `vreportf()`, not just the first\n - usage: extract `prefix_suffix_lines()` from `advise()`\n\n An attempt to help making multi-line messages fed to warning(),\n error(), and friends more easily translatable.\n\n Will discard and wait for a cleaned-up rewrite.\n cf. <20180529213957.GF7964@sigill.intra.peff.net>\n\n\n* hn/bisect-first-parent (2018-04-21) 1 commit\n - bisect: create 'bisect_flags' parameter in find_bisection()\n (this branch is used by tb/bisect-first-parent.)\n\n Preliminary code update to allow passing more flags down the\n bisection codepath in the future.\n\n We do not add random code that does not have real users to our\n codebase, so let's have it wait until such a real code materializes\n before too long.\n\n\n* pb/bisect-helper-2 (2018-07-23) 8 commits\n - t6030: make various test to pass GETTEXT_POISON tests\n - bisect--helper: `bisect_start` shell function partially in C\n - bisect--helper: `get_terms` & `bisect_terms` shell function in C\n - bisect--helper: `bisect_next_check` shell function in C\n - bisect--helper: `check_and_set_terms` shell function in C\n - wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()\n - bisect--helper: `bisect_write` shell function in C\n - bisect--helper: `bisect_reset` shell function in C\n\n Expecting a reroll.\n cf. <0102015f5e5ee171-f30f4868-886f-47a1-a4e4-b4936afc545d-000000@eu-west-1.amazonses.com>\n\n I just rebased the topic to a newer base as it did not build\n standalone with the base I originally queued the topic on, but\n otherwise there is no update to address any of the review comments\n in the thread above---we are still waiting for a reroll.\n\n\n* jk/drop-ancient-curl (2017-08-09) 5 commits\n - http: #error on too-old curl\n - curl: remove ifdef'd code never used with curl >=7.19.4\n - http: drop support for curl < 7.19.4\n - http: drop support for curl < 7.16.0\n - http: drop support for curl < 7.11.1\n\n Some code in http.c that has bitrot is being removed.\n\n Expecting a reroll.\n\n\n* mk/use-size-t-in-zlib (2017-08-10) 1 commit\n . zlib.c: use size_t for size\n\n The wrapper to call into zlib followed our long tradition to use\n \"unsigned long\" for sizes of regions in memory, which have been\n updated to use \"size_t\".\n\n Needs resurrecting by making sure the fix is good and still applies\n (or adjusted to today's codebase).\n\n--------------------------------------------------\n[Cooking]\n\n* ab/commit-graph-progress (2018-09-11) 2 commits\n - commit-graph verify: add progress output\n - commit-graph write: add progress output\n\n Generation of (expermental) commit-graph files have so far been\n fairly silent, even though it takes noticeable amount of time in a\n meaningfully large repository.  The users will now see progress\n output.\n\n Will merge to 'next'.\n\n\n* bw/protocol-v2 (2018-09-10) 1 commit\n - config: document value 2 for protocol.version\n\n Doc fix.\n\n Will merge to 'next'.\n\n\n* en/double-semicolon-fix (2018-09-05) 1 commit\n - Remove superfluous trailing semicolons\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* en/rerere-multi-stage-1-fix (2018-09-11) 2 commits\n - rerere: avoid buffer overrun\n - t4200: demonstrate rerere segfault on specially crafted merge\n\n A corner case bugfix in \"git rerere\" code.\n\n Will merge to 'next'.\n\n\n* jk/dev-build-format-security (2018-09-11) 1 commit\n - config.mak.dev: add -Wformat-security\n\n Build tweak to help developers.\n\n Will merge to 'next'.\n\n\n* jk/reopen-tempfile-truncate (2018-09-05) 1 commit\n - reopen_tempfile(): truncate opened file\n\n Fix for a long-standing bug that leaves the index file corrupt when\n it shrinks during a partial commit.\n\n Will merge to 'next'.\n\n\n* js/mingw-o-append (2018-09-11) 2 commits\n - mingw: fix mingw_open_append to work with named pipes\n - t0051: test GIT_TRACE to a windows named pipe\n\n Further fix for O_APPEND emulation on Windows\n\n Will merge to 'next'.\n\n\n* mk/http-backend-content-length (2018-09-11) 1 commit\n  (merged to 'next' on 2018-09-11 at e8095fc635)\n + http-backend test: make empty CONTENT_LENGTH test more realistic\n\n Test update.\n\n Will merge to 'master'.\n\n\n* nd/test-tool (2018-09-11) 6 commits\n - Makefile: add a hint about TEST_BUILTINS_OBJS\n - t/helper: merge test-dump-fsmonitor into test-tool\n - t/helper: merge test-parse-options into test-tool\n - t/helper: merge test-pkt-line into test-tool\n - t/helper: merge test-dump-untracked-cache into test-tool\n - t/helper: keep test-tool command list sorted\n\n Test helper binaries clean-up.\n\n Will merge to 'next'.\n\n\n* sb/diff-color-move-more (2018-09-11) 1 commit\n - diff: fix --color-moved-ws=allow-indentation-change\n\n Bugfix.\n\n Will merge to 'next'.\n\n\n* sb/string-list-remove-unused (2018-09-11) 1 commit\n - string-list: remove unused function print_string_list\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* sg/t3701-tighten-trace (2018-09-11) 1 commit\n - t3701-add-interactive: tighten the check of trace output\n\n Test update.\n\n Will merge to 'next'.\n\n\n* ss/wt-status-committable (2018-09-07) 4 commits\n - wt-status.c: set the committable flag in the collect phase\n - t7501: add test of \"commit --dry-run --short\"\n - wt-status: rename commitable to committable\n - wt-status.c: move has_unmerged earlier in the file\n (this branch is used by jc/wt-status-state-cleanup.)\n\n Code clean-up in the internal machinery used by \"git status\" and\n \"git commit --dry-run\".\n\n Will merge to 'next'.\n\n\n* jc/wt-status-state-cleanup (2018-09-07) 1 commit\n - WIP: roll wt_status_state into wt_status and populate in the collect phase\n (this branch uses ss/wt-status-committable.)\n\n\n\n* tz/t5551-with-curl-7.61.1 (2018-09-11) 1 commit\n - t5551-http-fetch-smart.sh: sort cookies before comparing\n\n Test fix.\n\n Will merge to 'next'.\n\n\n* ab/fetch-tags-noclobber (2018-08-31) 9 commits\n  (merged to 'next' on 2018-09-14 at 0384a7a8b4)\n + fetch: stop clobbering existing tags without --force\n + fetch: document local ref updates with/without --force\n + push doc: correct lies about how push refspecs work\n + push doc: move mention of \"tag <tag>\" later in the prose\n + push doc: remove confusing mention of remote merger\n + fetch tests: add a test for clobbering tag behavior\n + push tests: use spaces in interpolated string\n + push tests: make use of unused $1 in test description\n + fetch: change \"branch\" to \"reference\" in --force -h output\n\n The rules used by \"git push\" and \"git fetch\" to determine if a ref\n can or cannot be updated were inconsistent; specifically, fetching\n to update existing tags were allowed even though tags are supposed\n to be unmoving anchoring points.  \"git fetch\" was taught to forbid\n updates to existing tags without the \"--force\" option.\n\n Will merge to 'master'.\n This is a backward incompatible change but in a good way; it may\n still need to be treated carefully.\n\n\n* es/worktree-forced-ops-fix (2018-09-05) 10 commits\n  (merged to 'next' on 2018-09-14 at 1a0cc3204d)\n + doc-diff: force worktree add\n + worktree: delete .git/worktrees if empty after 'remove'\n + worktree: teach 'remove' to override lock when --force given twice\n + worktree: teach 'move' to override lock when --force given twice\n + worktree: teach 'add' to respect --force for registered but missing path\n + worktree: disallow adding same path multiple times\n + worktree: prepare for more checks of whether path can become worktree\n + worktree: generalize delete_git_dir() to reduce code duplication\n + worktree: move delete_git_dir() earlier in file for upcoming new callers\n + worktree: don't die() in library function find_worktree()\n\n Fix a bug in which the same path could be registered under multiple\n worktree entries if the path was missing (for instance, was removed\n manually).  Also, as a convenience, expand the number of cases in\n which --force is applicable.\n\n Will merge to 'master'.\n\n\n* jk/patch-corrupted-delta-fix (2018-08-30) 6 commits\n  (merged to 'next' on 2018-09-14 at 7517309cb0)\n + t5303: use printf to generate delta bases\n + patch-delta: handle truncated copy parameters\n + patch-delta: consistently report corruption\n + patch-delta: fix oob read\n + t5303: test some corrupt deltas\n + test-delta: read input into a heap buffer\n\n Malformed or crafted data in packstream can make our code attempt\n to read or write past the allocated buffer and abort, instead of\n reporting an error, which has been fixed.\n\n Will merge to 'master'.\n\n\n* jk/pack-objects-with-bitmap-fix (2018-09-04) 4 commits\n  (merged to 'next' on 2018-09-14 at 392eb2abb1)\n + pack-bitmap: drop \"loaded\" flag\n + traverse_bitmap_commit_list(): don't free result\n + t5310: test delta reuse with bitmaps\n + bitmap_has_sha1_in_uninteresting(): drop BUG check\n (this branch uses jk/pack-delta-reuse-with-bitmap.)\n\n Hotfix of the base topic.\n\n Will merge to 'master'.\n\n\n* js/rebase-i-autosquash-fix (2018-09-04) 2 commits\n - rebase -i: be careful to wrap up fixup/squash chains\n - rebase -i --autosquash: demonstrate a problem skipping the last squash\n\n \"git rebase -i\" did not clear the state files correctly when a run\n of \"squash/fixup\" is aborted and then the user manually amended the\n commit instead, which has been corrected.\n\n Will merge to 'next'.\n\n\n* nd/bisect-show-list-fix (2018-09-04) 1 commit\n  (merged to 'next' on 2018-09-14 at 18242da7ef)\n + bisect.c: make show_list() build again\n\n Debugging aid update.\n\n Will merge to 'master'.\n\n\n* sg/doc-trace-appends (2018-09-04) 1 commit\n  (merged to 'next' on 2018-09-14 at 6d82abb8bf)\n + Documentation/git.txt: clarify that GIT_TRACE=/path appends\n\n Docfix.\n\n Will merge to 'master'.\n\n\n* ds/format-commit-graph-docs (2018-08-21) 2 commits\n - commit-graph.txt: improve formatting for asciidoc\n - Docs: Add commit-graph tech docs to Makefile\n\n Design docs for the commit-graph machinery is now made into HTML as\n well as text.\n\n Will discard.\n I am inclined to drop these, as I do not see much clarity in HTML\n output over the text source.  Opinions?\n\n\n* jk/diff-rendered-docs (2018-08-31) 5 commits\n  (merged to 'next' on 2018-09-14 at 9b43d5a568)\n + doc/Makefile: drop doc-diff worktree and temporary files on \"make clean\"\n + doc-diff: add --clean mode to remove temporary working gunk\n + doc-diff: fix non-portable 'man' invocation\n + doc-diff: always use oids inside worktree\n  (merged to 'next' on 2018-08-22 at dd7a2b71cd)\n + SubmittingPatches: mention doc-diff\n\n Dev doc update.\n\n Will merge to 'master'.\n\n\n* jk/pack-delta-reuse-with-bitmap (2018-08-21) 6 commits\n  (merged to 'next' on 2018-08-22 at fc50b59dab)\n + pack-objects: reuse on-disk deltas for thin \"have\" objects\n + pack-bitmap: save \"have\" bitmap from walk\n + t/perf: add perf tests for fetches from a bitmapped server\n + t/perf: add infrastructure for measuring sizes\n + t/perf: factor out percent calculations\n + t/perf: factor boilerplate out of test_perf\n (this branch is used by jk/pack-objects-with-bitmap-fix.)\n\n When creating a thin pack, which allows objects to be made into a\n delta against another object that is not in the resulting pack but\n is known to be present on the receiving end, the code learned to\n take advantage of the reachability bitmap; this allows the server\n to send a delta against a base beyond the \"boundary\" commit.\n\n Will merge to 'master'.\n\n\n* jk/rev-list-stdin-noop-is-ok (2018-08-22) 1 commit\n  (merged to 'next' on 2018-08-27 at d5916f7bc1)\n + rev-list: make empty --stdin not an error\n\n \"git rev-list --stdin </dev/null\" used to be an error; it now shows\n no output without an error.  \"git rev-list --stdin --default HEAD\"\n still falls back to the given default when nothing is given on the\n standard input.\n\n Will merge to 'master'.\n\n\n* js/rebase-in-c-5.5-work-with-rebase-i-in-c (2018-09-06) 2 commits\n - builtin rebase: prepare for builtin rebase -i\n - Merge branch 'ag/rebase-i-in-c' into js/rebase-in-c-5.5-work-with-rebase-i-in-c\n (this branch is used by pk/rebase-in-c-6-final; uses ag/rebase-i-in-c, pk/rebase-in-c, pk/rebase-in-c-2-basic, pk/rebase-in-c-3-acts, pk/rebase-in-c-4-opts and pk/rebase-in-c-5-test.)\n\n \"rebase\" that has been rewritten learns the new calling convention\n used by \"rebase -i\" that was rewritten in C, tying the loose end\n between two GSoC topics that stomped on each other's toes.\n\n\n* jk/trailer-fixes (2018-08-23) 8 commits\n  (merged to 'next' on 2018-08-27 at 93b671b8c6)\n + append_signoff: use size_t for string offsets\n + sequencer: ignore \"---\" divider when parsing trailers\n + pretty, ref-filter: format %(trailers) with no_divider option\n + interpret-trailers: allow suppressing \"---\" divider\n + interpret-trailers: tighten check for \"---\" patch boundary\n + trailer: pass process_trailer_opts to trailer_info_get()\n + trailer: use size_t for iterating trailer list\n + trailer: use size_t for string offsets\n\n \"git interpret-trailers\" and its underlying machinery had a buggy\n code that attempted to ignore patch text after commit log message,\n which triggered in various codepaths that will always get the log\n message alone and never get such an input.\n\n Will merge to 'master'.\n\n\n* tg/rerere-doc-updates (2018-08-29) 2 commits\n  (merged to 'next' on 2018-08-31 at ce4fef1a97)\n + rerere: add note about files with existing conflict markers\n + rerere: mention caveat about unmatched conflict markers\n (this branch uses tg/rerere.)\n\n Clarify a part of technical documentation for rerere.\n\n Will merge to 'master'.\n\n\n* ds/commit-graph-tests (2018-08-29) 1 commit\n  (merged to 'next' on 2018-09-14 at d072a0ee3e)\n + commit-graph: define GIT_TEST_COMMIT_GRAPH\n\n We can now optionally run tests with commit-graph enabled.\n\n Will merge to 'master'.\n\n\n* jk/cocci (2018-08-29) 9 commits\n  (merged to 'next' on 2018-08-31 at 914b4f17ce)\n + show_dirstat: simplify same-content check\n + read-cache: use oideq() in ce_compare functions\n + convert hashmap comparison functions to oideq()\n + convert \"hashcmp() != 0\" to \"!hasheq()\"\n + convert \"oidcmp() != 0\" to \"!oideq()\"\n + convert \"hashcmp() == 0\" to hasheq()\n + convert \"oidcmp() == 0\" to oideq()\n + introduce hasheq() and oideq()\n + coccinelle: use <...> for function exclusion\n\n spatch transformation to replace boolean uses of !hashcmp() to\n newly introduced oideq() is added, and applied, to regain\n performance lost due to support of multiple hash algorithms.\n\n Will merge to 'master'.\n\n\n* js/add-i-coalesce-after-editing-hunk (2018-08-28) 1 commit\n - add -p: coalesce hunks before testing applicability\n\n Applicability check after a patch is edited in a \"git add -i/p\"\n session has been improved.\n\n Will hold.\n cf. <e5b2900a-0558-d3bf-8ea1-d526b078bbc2@talktalk.net>\n\n\n* rs/mailinfo-format-flowed (2018-08-29) 1 commit\n  (merged to 'next' on 2018-08-31 at 9e8b92176c)\n + mailinfo: support format=flowed\n\n \"git mailinfo\" used in \"git am\" learned to make a best-effort\n recovery of a patch corrupted by MUA that sends text/plain with\n format=flawed option.\n\n Will merge to 'master'.\n\n\n* tg/conflict-marker-size (2018-08-29) 1 commit\n  (merged to 'next' on 2018-08-31 at 12099161f0)\n + .gitattributes: add conflict-marker-size for relevant files\n\n Developer aid.\n\n Will merge to 'master'.\n\n\n* ts/doc-build-manpage-xsl-quietly (2018-08-29) 1 commit\n  (merged to 'next' on 2018-08-31 at 7527e0f8d3)\n + Documentation/Makefile: make manpage-base-url.xsl generation quieter\n\n Build tweak.\n\n Will merge to 'master'.\n\n\n* bp/checkout-new-branch-optim (2018-08-16) 1 commit\n  (merged to 'next' on 2018-08-27 at e69bfd115f)\n + checkout: optimize \"git checkout -b <new_branch>\"\n\n \"git checkout -b newbranch [HEAD]\" should not have to do as much as\n checking out a commit different from HEAD.  An attempt is made to\n optimize this special case.\n\n Will merge to 'master'.\n\n\n* ao/submodule-wo-gitmodules-checked-out (2018-08-14) 7 commits\n - submodule: support reading .gitmodules even when it's not checked out\n - t7506: clean up .gitmodules properly before setting up new scenario\n - submodule: use the 'submodule--helper config' command\n - submodule--helper: add a new 'config' subcommand\n - t7411: be nicer to future tests and really clean things up\n - submodule: factor out a config_set_in_gitmodules_file_gently function\n - submodule: add a print_config_from_gitmodules() helper\n\n The submodule support has been updated to read from the blob at\n HEAD:.gitmodules when the .gitmodules file is missing from the\n working tree.\n\n Expecting a reroll.\n\n I find the design a bit iffy in that our usual \"missing in the\n working tree?  let's use the latest blob\" fallback is to take it\n from the index, not from the HEAD.\n\n\n* cc/delta-islands (2018-08-16) 7 commits\n  (merged to 'next' on 2018-08-27 at cf3d7bd93f)\n + pack-objects: move 'layer' into 'struct packing_data'\n + pack-objects: move tree_depth into 'struct packing_data'\n + t5320: tests for delta islands\n + repack: add delta-islands support\n + pack-objects: add delta-islands support\n + pack-objects: refactor code into compute_layer_order()\n + Add delta-islands.{c,h}\n\n Lift code from GitHub to restrict delta computation so that an\n object that exists in one fork is not made into a delta against\n another object that does not appear in the same forked repository.\n\n Will merge to 'master'.\n\n\n* md/filter-trees (2018-09-14) 7 commits\n - list-objects-filter: implement filter tree:0\n - list-objects-filter: use BUG rather than die\n - revision: mark non-user-given objects instead\n - rev-list: handle missing tree objects properly\n - list-objects: always parse trees gently\n - list-objects: refactor to process_tree_contents\n - list-objects: store common func args in struct\n\n The \"rev-list --filter\" feature learned to exclude all trees via\n \"tree:0\" filter.\n\n Will merge to 'next'.\n The test scripts need to be cleaned up.\n\n\n* pk/rebase-in-c-2-basic (2018-09-06) 11 commits\n - builtin rebase: support `git rebase <upstream> <switch-to>`\n - builtin rebase: only store fully-qualified refs in `options.head_name`\n - builtin rebase: start a new rebase only if none is in progress\n - builtin rebase: support --force-rebase\n - builtin rebase: try to fast forward when possible\n - builtin rebase: require a clean worktree\n - builtin rebase: support the `verbose` and `diffstat` options\n - builtin rebase: support --quiet\n - builtin rebase: handle the pre-rebase hook and --no-verify\n - builtin rebase: support `git rebase --onto A...B`\n - builtin rebase: support --onto\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c-3-acts, pk/rebase-in-c-4-opts, pk/rebase-in-c-5-test and pk/rebase-in-c-6-final; uses pk/rebase-in-c.)\n\n\n* pk/rebase-in-c-3-acts (2018-09-06) 7 commits\n - builtin rebase: stop if `git am` is in progress\n - builtin rebase: actions require a rebase in progress\n - builtin rebase: support --edit-todo and --show-current-patch\n - builtin rebase: support --quit\n - builtin rebase: support --abort\n - builtin rebase: support --skip\n - builtin rebase: support --continue\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c-4-opts, pk/rebase-in-c-5-test and pk/rebase-in-c-6-final; uses pk/rebase-in-c and pk/rebase-in-c-2-basic.)\n\n\n* pk/rebase-in-c-4-opts (2018-09-06) 18 commits\n - builtin rebase: support --root\n - builtin rebase: add support for custom merge strategies\n - builtin rebase: support `fork-point` option\n - merge-base --fork-point: extract libified function\n - builtin rebase: support --rebase-merges[=[no-]rebase-cousins]\n - builtin rebase: support `--allow-empty-message` option\n - builtin rebase: support `--exec`\n - builtin rebase: support `--autostash` option\n - builtin rebase: support `-C` and `--whitespace=<type>`\n - builtin rebase: support `--gpg-sign` option\n - builtin rebase: support `--autosquash`\n - builtin rebase: support `keep-empty` option\n - builtin rebase: support `ignore-date` option\n - builtin rebase: support `ignore-whitespace` option\n - builtin rebase: support --committer-date-is-author-date\n - builtin rebase: support --rerere-autoupdate\n - builtin rebase: support --signoff\n - builtin rebase: allow selecting the rebase \"backend\"\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c-5-test and pk/rebase-in-c-6-final; uses pk/rebase-in-c, pk/rebase-in-c-2-basic and pk/rebase-in-c-3-acts.)\n\n\n* pk/rebase-in-c-5-test (2018-09-06) 6 commits\n - builtin rebase: error out on incompatible option/mode combinations\n - builtin rebase: use no-op editor when interactive is \"implied\"\n - builtin rebase: show progress when connected to a terminal\n - builtin rebase: fast-forward to onto if it is a proper descendant\n - builtin rebase: optionally pass custom reflogs to reset_head()\n - builtin rebase: optionally auto-detect the upstream\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c and pk/rebase-in-c-6-final; uses pk/rebase-in-c, pk/rebase-in-c-2-basic, pk/rebase-in-c-3-acts and pk/rebase-in-c-4-opts.)\n\n\n* pk/rebase-in-c-6-final (2018-09-06) 1 commit\n - rebase: default to using the builtin rebase\n (this branch uses ag/rebase-i-in-c, js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c, pk/rebase-in-c-2-basic, pk/rebase-in-c-3-acts, pk/rebase-in-c-4-opts and pk/rebase-in-c-5-test.)\n\n\n* ps/stash-in-c (2018-08-31) 20 commits\n - stash: replace all `write-tree` child processes with API calls\n - stash: optimize `get_untracked_files()` and `check_changes()`\n - stash: convert `stash--helper.c` into `stash.c`\n - stash: convert save to builtin\n - stash: make push -q quiet\n - stash: convert push to builtin\n - stash: convert create to builtin\n - stash: convert store to builtin\n - stash: mention options in `show` synopsis\n - stash: convert show to builtin\n - stash: convert list to builtin\n - stash: convert pop to builtin\n - stash: convert branch to builtin\n - stash: convert drop and clear to builtin\n - stash: convert apply to builtin\n - stash: add tests for `git stash show` config\n - stash: rename test cases to be more descriptive\n - stash: update test cases conform to coding guidelines\n - stash: improve option parsing test coverage\n - sha1-name.c: add `get_oidf()` which acts like `get_oid()`\n\n\n* nd/clone-case-smashing-warning (2018-08-17) 1 commit\n  (merged to 'next' on 2018-08-22 at eedae40a8d)\n + clone: report duplicate entries on case-insensitive filesystems\n\n Running \"git clone\" against a project that contain two files with\n pathnames that differ only in cases on a case insensitive\n filesystem would result in one of the files lost because the\n underlying filesystem is incapable of holding both at the same\n time.  An attempt is made to detect such a case and warn.\n\n Will merge to 'master'.\n\n\n* nd/unpack-trees-with-cache-tree (2018-08-27) 8 commits\n  (merged to 'next' on 2018-08-27 at b1d841d034)\n + Document update for nd/unpack-trees-with-cache-tree\n  (merged to 'next' on 2018-08-22 at 138b902673)\n + cache-tree: verify valid cache-tree in the test suite\n + unpack-trees: add missing cache invalidation\n + unpack-trees: reuse (still valid) cache-tree from src_index\n + unpack-trees: reduce malloc in cache-tree walk\n + unpack-trees: optimize walking same trees with cache-tree\n + unpack-trees: add performance tracing\n + trace.h: support nested performance tracing\n\n The unpack_trees() API used in checking out a branch and merging\n walks one or more trees along with the index.  When the cache-tree\n in the index tells us that we are walking a tree whose flattened\n contents is known (i.e. matches a span in the index), as linearly\n scanning a span in the index is much more efficient than having to\n open tree objects recursively and listing their entries, the walk\n can be optimized, which is done in this topic.\n\n Will merge to 'master'.\n\n\n* sb/range-diff-colors (2018-08-20) 11 commits\n  (merged to 'next' on 2018-08-22 at eb7ed4fca3)\n + range-diff: indent special lines as context\n + range-diff: make use of different output indicators\n + diff.c: add --output-indicator-{new, old, context}\n + diff.c: rewrite emit_line_0 more understandably\n + diff.c: omit check for line prefix in emit_line_0\n + diff: use emit_line_0 once per line\n + diff.c: add set_sign to emit_line_0\n + diff.c: reorder arguments for emit_line_ws_markup\n + diff.c: simplify caller of emit_line_0\n + t3206: add color test for range-diff --dual-color\n + test_decode_color: understand FAINT and ITALIC\n\n The color output support for recently introduced \"range-diff\"\n command got tweaked a bit.\n\n Will merge to 'master'.\n\n\n* sg/t1404-update-ref-test-timeout (2018-08-01) 1 commit\n  (merged to 'next' on 2018-08-22 at f3cd37b5ea)\n + t1404: increase core.packedRefsTimeout to avoid occasional test failure\n\n An attempt to unflake a test a bit.\n\n Will merge to 'master'.\n\n\n* pw/add-p-select (2018-07-26) 4 commits\n - add -p: optimize line selection for short hunks\n - add -p: allow line selection to be inverted\n - add -p: select modified lines correctly\n - add -p: select individual hunk lines\n\n \"git add -p\" interactive interface learned to let users choose\n individual added/removed lines to be used in the operation, instead\n of accepting or rejecting a whole hunk.\n\n Will hold.\n cf. <d622a95b-7302-43d4-4ec9-b2cf3388c653@talktalk.net>\n I found the feature to be hard to explain, and may result in more\n end-user complaints, but let's see.\n\n\n* ds/commit-graph-with-grafts (2018-08-21) 8 commits\n - commit-graph: close_commit_graph before shallow walk\n - commit-graph: not compatible with uninitialized repo\n - commit-graph: not compatible with grafts\n - commit-graph: not compatible with replace objects\n - test-repository: properly init repo\n - commit-graph: update design document\n - refs.c: upgrade for_each_replace_ref to be a each_repo_ref_fn callback\n - refs.c: migrate internal ref iteration to pass thru repository argument\n\n The recently introduced commit-graph auxiliary data is incompatible\n with mechanisms such as replace & grafts that \"breaks\" immutable\n nature of the object reference relationship.  Disable optimizations\n based on its use (and updating existing commit-graph) when these\n incompatible features are in use in the repository.\n\n Replaced with a newer version.\n\n\n* ds/reachable (2018-08-28) 19 commits\n  (merged to 'next' on 2018-08-28 at b1634b371d)\n + commit-reach: correct accidental #include of C file\n  (merged to 'next' on 2018-08-22 at 17f3275afb)\n + commit-reach: use can_all_from_reach\n + commit-reach: make can_all_from_reach... linear\n + commit-reach: replace ref_newer logic\n + test-reach: test commit_contains\n + test-reach: test can_all_from_reach_with_flags\n + test-reach: test reduce_heads\n + test-reach: test get_merge_bases_many\n + test-reach: test is_descendant_of\n + test-reach: test in_merge_bases\n + test-reach: create new test tool for ref_newer\n + commit-reach: move can_all_from_reach_with_flags\n + upload-pack: generalize commit date cutoff\n + upload-pack: refactor ok_to_give_up()\n + upload-pack: make reachable() more generic\n + commit-reach: move commit_contains from ref-filter\n + commit-reach: move ref_newer from remote.c\n + commit.h: remove method declarations\n + commit-reach: move walk methods from commit.c\n\n The code for computing history reachability has been shuffled,\n obtained a bunch of new tests to cover them, and then being\n improved.\n\n Will merge to 'master'.\n\n\n* es/format-patch-interdiff (2018-07-23) 6 commits\n  (merged to 'next' on 2018-08-31 at 63927e0227)\n + format-patch: allow --interdiff to apply to a lone-patch\n + log-tree: show_log: make commentary block delimiting reusable\n + interdiff: teach show_interdiff() to indent interdiff\n + format-patch: teach --interdiff to respect -v/--reroll-count\n + format-patch: add --interdiff option to embed diff in cover letter\n + format-patch: allow additional generated content in make_cover_letter()\n (this branch is used by ds/format-patch-range-diff-test and es/format-patch-rangediff.)\n\n \"git format-patch\" learned a new \"--interdiff\" option to explain\n the difference between this version and the previous atttempt in\n the cover letter (or after the tree-dashes as a comment).\n\n Will merge to 'master'.\n\n\n* es/format-patch-rangediff (2018-08-14) 10 commits\n  (merged to 'next' on 2018-08-31 at 65627afece)\n + format-patch: allow --range-diff to apply to a lone-patch\n + format-patch: add --creation-factor tweak for --range-diff\n + format-patch: teach --range-diff to respect -v/--reroll-count\n + format-patch: extend --range-diff to accept revision range\n + format-patch: add --range-diff option to embed diff in cover letter\n + range-diff: relieve callers of low-level configuration burden\n + range-diff: publish default creation factor\n + range-diff: respect diff_option.file rather than assuming 'stdout'\n + Merge branch 'es/format-patch-interdiff' into es/format-patch-rangediff\n + Merge branch 'js/range-diff' into es/format-patch-rangediff\n (this branch is used by ds/format-patch-range-diff-test; uses es/format-patch-interdiff.)\n\n \"git format-patch\" learned a new \"--range-diff\" option to explain\n the difference between this version and the previous attempt in\n the cover letter (or after the tree-dashes as a comment).\n\n Will merge to 'master'.\n\n\n* jn/gc-auto (2018-07-17) 3 commits\n - gc: do not return error for prior errors in daemonized mode\n - gc: exit with status 128 on failure\n - gc: improve handling of errors reading gc.log\n\n \"gc --auto\" ended up calling exit(-1) upon error, which has been\n corrected to use exit(1).  Also the error reporting behaviour when\n daemonized has been updated to exit with zero status when stopping\n due to a previously discovered error (which implies there is no\n point running gc to improve the situation); we used to exit with\n failure in such a case.\n\n What's the donness of this one?\n cf. <20180717201348.GD26218@sigill.intra.peff.net>\n\n\n* sb/submodule-update-in-c (2018-08-14) 7 commits\n  (merged to 'next' on 2018-08-17 at 23c81e5ff7)\n + submodule--helper: introduce new update-module-mode helper\n + submodule--helper: replace connect-gitdir-workingtree by ensure-core-worktree\n + builtin/submodule--helper: factor out method to update a single submodule\n + builtin/submodule--helper: store update_clone information in a struct\n + builtin/submodule--helper: factor out submodule updating\n + git-submodule.sh: rename unused variables\n + git-submodule.sh: align error reporting for update mode to use path\n\n \"git submodule update\" is getting rewritten piece-by-piece into C.\n\n Will merge to 'master'.\n\n\n* tg/rerere (2018-08-06) 11 commits\n  (merged to 'next' on 2018-08-17 at 919a958cdc)\n + rerere: recalculate conflict ID when unresolved conflict is committed\n + rerere: teach rerere to handle nested conflicts\n + rerere: return strbuf from handle path\n + rerere: factor out handle_conflict function\n + rerere: only return whether a path has conflicts or not\n + rerere: fix crash with files rerere can't handle\n + rerere: add documentation for conflict normalization\n + rerere: mark strings for translation\n + rerere: wrap paths in output in sq\n + rerere: lowercase error messages\n + rerere: unify error messages when read_cache fails\n (this branch is used by tg/rerere-doc-updates.)\n\n Fixes to \"git rerere\" corner cases, especially when conflict\n markers cannot be parsed in the file.\n\n Will merge to 'master'.\n\n\n* ag/rebase-i-in-c (2018-08-29) 20 commits\n - rebase -i: move rebase--helper modes to rebase--interactive\n - rebase -i: remove git-rebase--interactive.sh\n - rebase--interactive2: rewrite the submodes of interactive rebase in C\n - rebase -i: implement the main part of interactive rebase as a builtin\n - rebase -i: rewrite init_basic_state() in C\n - rebase -i: rewrite write_basic_state() in C\n - rebase -i: rewrite the rest of init_revisions_and_shortrevisions() in C\n - rebase -i: implement the logic to initialize $revisions in C\n - rebase -i: remove unused modes and functions\n - rebase -i: rewrite complete_action() in C\n - t3404: todo list with commented-out commands only aborts\n - sequencer: change the way skip_unnecessary_picks() returns its result\n - sequencer: refactor append_todo_help() to write its message to a buffer\n - rebase -i: rewrite checkout_onto() in C\n - rebase -i: rewrite setup_reflog_action() in C\n - sequencer: add a new function to silence a command, except if it fails\n - rebase -i: rewrite the edit-todo functionality in C\n - editor: add a function to launch the sequence editor\n - rebase -i: rewrite append_todo_help() in C\n - sequencer: make three functions and an enum from sequencer.c public\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c and pk/rebase-in-c-6-final.)\n\n Rewrite of the remaining \"rebase -i\" machinery in C.\n\n\n* lt/date-human (2018-07-09) 1 commit\n - Add 'human' date format\n\n A new date format \"--date=human\" that morphs its output depending\n on how far the time is from the current time has been introduced.\n \"--date=auto\" can be used to use this new format when the output is\n goint to the pager or to the terminal and otherwise the default\n format.\n\n\n* pk/rebase-in-c (2018-08-06) 3 commits\n - builtin/rebase: support running \"git rebase <upstream>\"\n - rebase: refactor common shell functions into their own file\n - rebase: start implementing it as a builtin\n (this branch is used by js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c-2-basic, pk/rebase-in-c-3-acts, pk/rebase-in-c-4-opts, pk/rebase-in-c-5-test and pk/rebase-in-c-6-final.)\n\n Rewrite of the \"rebase\" machinery in C.\n\n\n* jk/branch-l-1-repurpose (2018-08-30) 2 commits\n  (merged to 'next' on 2018-08-31 at cfa73bbfcb)\n + doc/git-branch: remove obsolete \"-l\" references\n  (merged to 'next' on 2018-08-08 at d2a08dd08e)\n + branch: make \"-l\" a synonym for \"--list\"\n\n Updated plan to repurpose the \"-l\" option to \"git branch\".\n\n Will merge to 'master'.\n\n\n* ds/multi-pack-index (2018-08-20) 33 commits\n  (merged to 'next' on 2018-08-21 at d15e8cadd4)\n + pack-objects: consider packs in multi-pack-index\n + midx: test a few commands that use get_all_packs\n + treewide: use get_all_packs\n + packfile: add all_packs list\n + midx: fix bug that skips midx with alternates\n + midx: stop reporting garbage\n + midx: mark bad packed objects\n + multi-pack-index: store local property\n + multi-pack-index: provide more helpful usage info\n + Sync 'ds/multi-pack-index' to v2.19.0-rc0\n  (merged to 'next' on 2018-08-08 at 1a56c52967)\n + midx: clear midx on repack\n + packfile: skip loading index if in multi-pack-index\n + midx: prevent duplicate packfile loads\n + midx: use midx in approximate_object_count\n + midx: use existing midx when writing new one\n + midx: use midx in abbreviation calculations\n + midx: read objects from multi-pack-index\n + config: create core.multiPackIndex setting\n + midx: write object offsets\n + midx: write object id fanout chunk\n + midx: write object ids in a chunk\n + midx: sort and deduplicate objects from packfiles\n + midx: read pack names into array\n + multi-pack-index: write pack names in chunk\n + multi-pack-index: read packfile list\n + packfile: generalize pack directory list\n + t5319: expand test data\n + multi-pack-index: load into memory\n + midx: write header information to lockfile\n + multi-pack-index: add 'write' verb\n + multi-pack-index: add builtin\n + multi-pack-index: add format details\n + multi-pack-index: add design document\n (this branch is used by ds/multi-pack-verify.)\n\n When there are too many packfiles in a repository (which is not\n recommended), looking up an object in these would require\n consulting many pack .idx files; a new mechanism to have a single\n file that consolidates all of these .idx files is introduced.\n\n Will merge to 'master'.\n\n--------------------------------------------------\n[Discarded]\n\n* am/sequencer-author-script-fix (2018-07-18) 1 commit\n . sequencer.c: terminate the last line of author-script properly\n\n The author-script that records the author information created by\n the sequencer machinery lacked the closing single quote on the last\n entry.\n\n Superseded by another topic.\n\n\n* jc/push-cas-opt-comment (2018-08-01) 1 commit\n . push: comment on a funny unbalanced option help\n\n Code clarification.\n\n Superseded by another topic.\n\n\n* cc/remote-odb (2018-08-02) 9 commits\n . Documentation/config: add odb.<name>.promisorRemote\n . t0410: test fetching from many promisor remotes\n . Use odb.origin.partialclonefilter instead of core.partialclonefilter\n . Use remote_odb_get_direct() and has_remote_odb()\n . remote-odb: add remote_odb_reinit()\n . remote-odb: implement remote_odb_get_many_direct()\n . remote-odb: implement remote_odb_get_direct()\n . Add initial remote odb support\n . fetch-object: make functions return an error code\n\n Implement lazy fetches of missing objects to complement the\n experimental partial clone feature.\n\n Ejected; seems to break existing repositories that use partialclone\n repository extension.\n\n I haven't seen much interest in this topic on list.  What's the\n doneness of this thing?\n\n I do not particularly mind adding code to support a niche feature\n as long as it is cleanly made and it is clear that the feature\n won't negatively affect those who do not use it, so a review from\n that point of view may also be appropriate.\n\n\n* jh/structured-logging (2018-08-28) 26 commits\n . SQUASH??? spatch fix\n . structured-logging: add config data facility\n . structured-logging: t0420 tests for interacitve child_summary\n . structured-logging: t0420 tests for child process detail events\n . structured-logging: add child process classification\n . structured-logging: add detail-events for child processes\n . structured-logging: add structured logging to remote-curl\n . structured-logging: t0420 tests for aux-data\n . structured-logging: add aux-data for size of sparse-checkout file\n . structured-logging: add aux-data for index size\n . structured-logging: add aux-data facility\n . structured-logging: t0420 tests for timers\n . structured-logging: add timer around preload_index\n . structured-logging: add timer around wt-status functions\n . structured-logging: add timer around do_write_index\n . structured-logging: add timer around do_read_index\n . structured-logging: add timer facility\n . structured-logging: add detail-event for lazy_init_name_hash\n . structured-logging: add detail-event facility\n . structured-logging: t0420 basic tests\n . structured-logging: set sub_command field for checkout command\n . structured-logging: set sub_command field for branch command\n . structured-logging: add session-id to log events\n . structured-logging: add structured logging framework\n . structured-logging: add STRUCTURED_LOGGING=1 to Makefile\n . structured-logging: design document\n\n Being rerolled with an updated tracing API.\n\n\n* av/fsmonitor-updates (2018-01-04) 6 commits\n . fsmonitor: use fsmonitor data in `git diff`\n . fsmonitor: remove debugging lines from t/t7519-status-fsmonitor.sh\n . fsmonitor: make output of test-dump-fsmonitor more concise\n . fsmonitor: update helper tool, now that flags are filled later\n . fsmonitor: stop inline'ing mark_fsmonitor_valid / _invalid\n . dir.c: update comments to match argument name\n\n Code clean-up on fsmonitor integration, plus optional utilization\n of the fsmonitor data in diff-files.\n\n Tired of waiting for an update.\n cf. <alpine.DEB.2.21.1.1801042335130.32@MININT-6BKU6QN.europe.corp.microsoft.com>\n\n Also there were backward incompatible API changes brought by other\n topics in flight; having to keep up with those disruptive changes\n is not worth the maintenance effort for a stale topic.\n\n\n* jc/rebase-in-c-9-fixes (2018-09-04) 1 commit\n . rebase: re-add forgotten -k that stands for --keep-empty\n (this branch uses ag/rebase-i-in-c, js/rebase-in-c-5.5-work-with-rebase-i-in-c, pk/rebase-in-c, pk/rebase-in-c-2-basic, pk/rebase-in-c-3-acts, pk/rebase-in-c-4-opts, pk/rebase-in-c-5-test and pk/rebase-in-c-6-final.)\n\n The fix has been rolled into the original topic that introduced the\n issue, hence this topic is no longer necessary.\n\n\n* jn/http-backend-content-length (2018-09-11) 2 commits\n . SQUASH???? perhaps after reflowing\n . http-backend: treat empty CONTENT_LENGTH as zero\n\n\n* nd/optim-reading-index-v4 (2018-09-04) 1 commit\n . read-cache.c: optimize reading index format v4\n\n The v4 format of the index file uses prefix compression to store\n the pathnames to save file size.  The codepath to read such a file\n has been optimized.\n\n Absorbed in bp/read-cache-parallel topic.\n"},{"id":"358218","messageId":"20180915221743.a018386eb34f6ac792c6e694@ao2.it","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Antonio Ospite","fromEmail":"ao2@ao2.it","sentAt":"2018-09-15T20:17:43Z","receivedAt":"2018-09-15T20:17:49Z","isPatch":false,"sender":{"key":"ao2@ao2.it","avatar":"https://avatars.githubusercontent.com/u/1249395?v=4"},"body":"On Fri, 14 Sep 2018 14:56:36 -0700\nJunio C Hamano <gitster@pobox.com> wrote:\n\n[...]\n> * ao/submodule-wo-gitmodules-checked-out (2018-08-14) 7 commits\n>  - submodule: support reading .gitmodules even when it's not checked out\n>  - t7506: clean up .gitmodules properly before setting up new scenario\n>  - submodule: use the 'submodule--helper config' command\n>  - submodule--helper: add a new 'config' subcommand\n>  - t7411: be nicer to future tests and really clean things up\n>  - submodule: factor out a config_set_in_gitmodules_file_gently function\n>  - submodule: add a print_config_from_gitmodules() helper\n> \n>  The submodule support has been updated to read from the blob at\n>  HEAD:.gitmodules when the .gitmodules file is missing from the\n>  working tree.\n> \n>  Expecting a reroll.\n> \n>  I find the design a bit iffy in that our usual \"missing in the\n>  working tree?  let's use the latest blob\" fallback is to take it\n>  from the index, not from the HEAD.\n> \n\nHi Junio,\n\nI did send a v4 addressing this:\nhttps://public-inbox.org/git/20180824132951.8000-1-ao2@ao2.it/\n\nIn case you missed it I might as well send a v5 with some minor\ncosmetic touches suggested by Ævar:\nhttps://public-inbox.org/git/87wosfesxl.fsf@evledraar.gmail.com/\n\nCiao,\n   Antonio\n\n-- \nAntonio Ospite\nhttps://ao2.it\nhttps://twitter.com/ao2it\n\nA: Because it messes up the order in which people normally read text.\n   See http://en.wikipedia.org/wiki/Posting_style\nQ: Why is top-posting such a bad thing?\n"},{"id":"358222","messageId":"CACsJy8DXT8rF_WYE3C0ak7E8_8j=kRS+Yhi1pZjKd5Rr4OsM_A@mail.gmail.com","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2018-09-16T06:39:03Z","receivedAt":"2018-09-16T06:40:12Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Fri, Sep 14, 2018 at 11:56 PM Junio C Hamano <gitster@pobox.com> wrote:\n> * jn/gc-auto (2018-07-17) 3 commits\n>  - gc: do not return error for prior errors in daemonized mode\n>  - gc: exit with status 128 on failure\n>  - gc: improve handling of errors reading gc.log\n>\n>  \"gc --auto\" ended up calling exit(-1) upon error, which has been\n>  corrected to use exit(1).  Also the error reporting behaviour when\n>  daemonized has been updated to exit with zero status when stopping\n>  due to a previously discovered error (which implies there is no\n>  point running gc to improve the situation); we used to exit with\n>  failure in such a case.\n>\n>  What's the donness of this one?\n>  cf. <20180717201348.GD26218@sigill.intra.peff.net>\n\nThis topic has stayed in 'pu' for a long time. I thought it was\nconcluded that this was a good change? Jeff, Jonathan?\n-- \nDuy\n"},{"id":"358233","messageId":"20180916120114.GV432229@genre.crustytoothpaste.net","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2018-09-16T12:01:14Z","receivedAt":"2018-09-16T12:01:24Z","isPatch":false,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"On Fri, Sep 14, 2018 at 02:56:36PM -0700, Junio C Hamano wrote:\n> * bc/hash-independent-tests (2018-09-13) 12 commits\n>  - t5318: use test_oid for HASH_LEN\n>  - t1407: make hash size independent\n>  - t1406: make hash-size independent\n>  - t1405: make hash size independent\n>  - t1400: switch hard-coded object ID to variable\n>  - t1006: make hash size independent\n>  - t0064: make hash size independent\n>  - t0027: make hash size independent\n\nCould you drop this particular patch (the t0027 fix)?  I found an issue\nwith it that I'd like to address, and it would be easier for me to just\nsend a completely fixed version in the future since it doesn't have any\ndependencies with the rest of the series.\n-- \nbrian m. carlson: Houston, Texas, US\nOpenPGP: https://keybase.io/bk2204\n"},{"id":"358235","messageId":"877ejllbs9.fsf@evledraar.gmail.com","threadId":"49349","inReplyTo":"CACsJy8DXT8rF_WYE3C0ak7E8_8j=kRS+Yhi1pZjKd5Rr4OsM_A@mail.gmail.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2018-09-16T15:09:10Z","receivedAt":"2018-09-16T15:09:17Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Sun, Sep 16 2018, Duy Nguyen wrote:\n\n> On Fri, Sep 14, 2018 at 11:56 PM Junio C Hamano <gitster@pobox.com> wrote:\n>> * jn/gc-auto (2018-07-17) 3 commits\n>>  - gc: do not return error for prior errors in daemonized mode\n>>  - gc: exit with status 128 on failure\n>>  - gc: improve handling of errors reading gc.log\n>>\n>>  \"gc --auto\" ended up calling exit(-1) upon error, which has been\n>>  corrected to use exit(1).  Also the error reporting behaviour when\n>>  daemonized has been updated to exit with zero status when stopping\n>>  due to a previously discovered error (which implies there is no\n>>  point running gc to improve the situation); we used to exit with\n>>  failure in such a case.\n>>\n>>  What's the donness of this one?\n>>  cf. <20180717201348.GD26218@sigill.intra.peff.net>\n>\n> This topic has stayed in 'pu' for a long time. I thought it was\n> concluded that this was a good change? Jeff, Jonathan?\n\nThere's still outstanding feedback on it, including my:\nhttps://public-inbox.org/git/878t69dgvx.fsf@evledraar.gmail.com/\n\nI think it would be great to merge [12]/3 down, and have a re-submission\nof 3/3 stand-alone, once that's merged down, so we can untangle\nobviously correct bugfixes ([12]/3) with meaningful changes in exit code\nbehavior (3/3).\n"},{"id":"358244","messageId":"20180917023912.GC22024@sigill.intra.peff.net","threadId":"49349","inReplyTo":"CACsJy8DXT8rF_WYE3C0ak7E8_8j=kRS+Yhi1pZjKd5Rr4OsM_A@mail.gmail.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2018-09-17T02:39:13Z","receivedAt":"2018-09-17T02:46:51Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Sep 16, 2018 at 08:39:03AM +0200, Duy Nguyen wrote:\n\n> On Fri, Sep 14, 2018 at 11:56 PM Junio C Hamano <gitster@pobox.com> wrote:\n> > * jn/gc-auto (2018-07-17) 3 commits\n> >  - gc: do not return error for prior errors in daemonized mode\n> >  - gc: exit with status 128 on failure\n> >  - gc: improve handling of errors reading gc.log\n> >\n> >  \"gc --auto\" ended up calling exit(-1) upon error, which has been\n> >  corrected to use exit(1).  Also the error reporting behaviour when\n> >  daemonized has been updated to exit with zero status when stopping\n> >  due to a previously discovered error (which implies there is no\n> >  point running gc to improve the situation); we used to exit with\n> >  failure in such a case.\n> >\n> >  What's the donness of this one?\n> >  cf. <20180717201348.GD26218@sigill.intra.peff.net>\n> \n> This topic has stayed in 'pu' for a long time. I thought it was\n> concluded that this was a good change? Jeff, Jonathan?\n\nI read over the thread again. I don't think I actually have any\ncomplaints about the patches as-is. There was some discussion from Junio\nand Ævar about the third one. I don't have a strong opinion. My\nexperience has been that \"gc --auto\" is garbage anyway on the server\nside, but I think Ævar's experience is that it's reasonable for small to\nmedium sites (which seems plausible to me).\n\nThe message-id quoted there is my \"this looks good\". I mentioned a few\npossible nits, but I think it would be OK with or without them\naddressed.\n\n-Peff\n"},{"id":"358274","messageId":"xmqq4leo9n24.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":"20180916120114.GV432229@genre.crustytoothpaste.net","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-17T15:10:59Z","receivedAt":"2018-09-17T15:11:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"brian m. carlson\" <sandals@crustytoothpaste.net> writes:\n\n> On Fri, Sep 14, 2018 at 02:56:36PM -0700, Junio C Hamano wrote:\n>> * bc/hash-independent-tests (2018-09-13) 12 commits\n>>  - t5318: use test_oid for HASH_LEN\n>>  - t1407: make hash size independent\n>>  - t1406: make hash-size independent\n>>  - t1405: make hash size independent\n>>  - t1400: switch hard-coded object ID to variable\n>>  - t1006: make hash size independent\n>>  - t0064: make hash size independent\n>>  - t0027: make hash size independent\n>\n> Could you drop this particular patch (the t0027 fix)?  I found an issue\n> with it that I'd like to address, and it would be easier for me to just\n> send a completely fixed version in the future since it doesn't have any\n> dependencies with the rest of the series.\n\nThanks, will do.\n"},{"id":"358275","messageId":"xmqqzhwg88go.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":"20180915221743.a018386eb34f6ac792c6e694@ao2.it","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-17T15:11:35Z","receivedAt":"2018-09-17T15:11:40Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Antonio Ospite <ao2@ao2.it> writes:\n\n> I did send a v4 addressing this:\n> https://public-inbox.org/git/20180824132951.8000-1-ao2@ao2.it/\n>\n> In case you missed it I might as well send a v5 with some minor\n> cosmetic touches suggested by Ævar:\n> https://public-inbox.org/git/87wosfesxl.fsf@evledraar.gmail.com/\n\nThanks, will take a look.\n"},{"id":"358304","messageId":"xmqqo9cw6mhk.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":"20180917023912.GC22024@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-17T17:51:35Z","receivedAt":"2018-09-17T17:51:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n>> >  What's the donness of this one?\n>> >  cf. <20180717201348.GD26218@sigill.intra.peff.net>\n>> \n>> This topic has stayed in 'pu' for a long time. I thought it was\n>> concluded that this was a good change? Jeff, Jonathan?\n>\n> I read over the thread again. I don't think I actually have any\n> complaints about the patches as-is. There was some discussion from Junio\n> and Ævar about the third one. I don't have a strong opinion. My\n> experience has been that \"gc --auto\" is garbage anyway on the server\n> side, but I think Ævar's experience is that it's reasonable for small to\n> medium sites (which seems plausible to me).\n>\n> The message-id quoted there is my \"this looks good\". I mentioned a few\n> possible nits, but I think it would be OK with or without them\n> addressed.\n\nThat matches my reading of your position.  I tend to agree with\nÆvar's recommendation to postpone 3/3 and use 1 & 2 for now.\n\nBy the way, people shouldn't read too much into the messages\nreferred to in \"What's cooking\"; they are not \"these messages point\nout all the issues that block the topic\".  It's just \"I am aware of\nthis thread, which readers would hopefully find a good starting\npoint to form their opinions\".\n\nThanks.\n\n"},{"id":"358308","messageId":"20180917182210.GB3894@sigill.intra.peff.net","threadId":"49349","inReplyTo":"xmqqo9cw6mhk.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2018-09-17T18:22:11Z","receivedAt":"2018-09-17T18:22:14Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Sep 17, 2018 at 10:51:35AM -0700, Junio C Hamano wrote:\n\n> Jeff King <peff@peff.net> writes:\n> \n> >> >  What's the donness of this one?\n> >> >  cf. <20180717201348.GD26218@sigill.intra.peff.net>\n> >> \n> >> This topic has stayed in 'pu' for a long time. I thought it was\n> >> concluded that this was a good change? Jeff, Jonathan?\n> >\n> > I read over the thread again. I don't think I actually have any\n> > complaints about the patches as-is. There was some discussion from Junio\n> > and Ævar about the third one. I don't have a strong opinion. My\n> > experience has been that \"gc --auto\" is garbage anyway on the server\n> > side, but I think Ævar's experience is that it's reasonable for small to\n> > medium sites (which seems plausible to me).\n> >\n> > The message-id quoted there is my \"this looks good\". I mentioned a few\n> > possible nits, but I think it would be OK with or without them\n> > addressed.\n> \n> That matches my reading of your position.  I tend to agree with\n> Ævar's recommendation to postpone 3/3 and use 1 & 2 for now.\n\nLet me inject some more uncertainty, then. ;)\n\nIf we are not going to do 3/3, then should 2/3 simply avoid passing \"-1\"\nback via return from main? I guess I don't have a strong opinion, but\none of the things I noted was that we converted those die() calls\nintroduced in 2/3 back into returns in 3/3. Do we want to leave it in\nthe state where we are calling die() a lot more?\n\n-Peff\n"},{"id":"358309","messageId":"20180917182639.GB140909@aiede.svl.corp.google.com","threadId":"49349","inReplyTo":"20180917182210.GB3894@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2018-09-17T18:26:39Z","receivedAt":"2018-09-17T18:26:44Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Jeff King wrote:\n\n> Let me inject some more uncertainty, then. ;)\n>\n> If we are not going to do 3/3, then should 2/3 simply avoid passing \"-1\"\n> back via return from main? I guess I don't have a strong opinion, but\n> one of the things I noted was that we converted those die() calls\n> introduced in 2/3 back into returns in 3/3. Do we want to leave it in\n> the state where we are calling die() a lot more?\n\nWould you mind replying in the patch thread instead of this what's\ncooking email?\n\nThat way, I can understand your suggestion better in context, I can\nfind it more easily later, I would feel less bad about adding noise by\nreplying, etc.\n\nThanks,\nJonathan\n"},{"id":"358312","messageId":"43e8d5aa-5418-2e2d-2097-9e21e7a74825@gmail.com","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2018-09-17T18:35:49Z","receivedAt":"2018-09-17T18:35:57Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 9/14/2018 5:56 PM, Junio C Hamano wrote:\n> * ds/format-commit-graph-docs (2018-08-21) 2 commits\n>   - commit-graph.txt: improve formatting for asciidoc\n>   - Docs: Add commit-graph tech docs to Makefile\n>\n>   Design docs for the commit-graph machinery is now made into HTML as\n>   well as text.\n>\n>   Will discard.\n>   I am inclined to drop these, as I do not see much clarity in HTML\n>   output over the text source.  Opinions?\n\nDiscarding is fine. I originally created it because I thought we were \nsupposed to mark all documents for HTML generation. You're right, the \nHTML doesn't add anything.\n\n> * ds/commit-graph-with-grafts (2018-08-21) 8 commits\n>   - commit-graph: close_commit_graph before shallow walk\n>   - commit-graph: not compatible with uninitialized repo\n>   - commit-graph: not compatible with grafts\n>   - commit-graph: not compatible with replace objects\n>   - test-repository: properly init repo\n>   - commit-graph: update design document\n>   - refs.c: upgrade for_each_replace_ref to be a each_repo_ref_fn callback\n>   - refs.c: migrate internal ref iteration to pass thru repository argument\n>\n>   The recently introduced commit-graph auxiliary data is incompatible\n>   with mechanisms such as replace & grafts that \"breaks\" immutable\n>   nature of the object reference relationship.  Disable optimizations\n>   based on its use (and updating existing commit-graph) when these\n>   incompatible features are in use in the repository.\n>\n>   Replaced with a newer version.\n\nI think this has been the same note for a few weeks now. What does it \nmean? Did I send a new version out that you haven't picked up?\n\nAlso, my \"git log --topo-order\" series was never picked up, but I see it \nhas conflicts with 'next' now, so I'll rebase to resolve conflicts and \nsend a v2.\n\nThanks,\n-Stolee\n\n"},{"id":"358317","messageId":"xmqqk1nk6j47.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":"43e8d5aa-5418-2e2d-2097-9e21e7a74825@gmail.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-17T19:04:24Z","receivedAt":"2018-09-17T19:04:29Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee <stolee@gmail.com> writes:\n\n>>   Replaced with a newer version.\n>\n> I think this has been the same note for a few weeks now. What does it\n> mean? Did I send a new version out that you haven't picked up?\n\nIt just means that I didn't bother to look at the current status\nmessage for the topic that had no activity X-<.\n\n> Also, my \"git log --topo-order\" series was never picked up, but I see\n> it has conflicts with 'next' now, so I'll rebase to resolve conflicts\n> and send a v2.\n\nGood idea, as I do not recall anything about that series.\n"},{"id":"358369","messageId":"xmqq36u768hb.fsf@gitster-ct.c.googlers.com","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Sep 2018, #03; Fri, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-09-17T22:54:08Z","receivedAt":"2018-09-17T22:54:13Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> The tip of 'next' hasn't been rewound yet, but the states of many\n> topics that were marked to \"Cook in 'next'\" read \"Will merge to\n> 'master'\" now.  I'll probably start merging a handful of topics that\n> have been in 'next' down to 'master' tonight (they appear at the\n> bottom of \"log --first-parent master..pu\" output).\n\nA bunch of topics have been merged to 'master' and also to 'next'\nwith today's update.  The tip of 'next' hasn't been rewound.  I am\nhoping that we can thin down the difference between 'master' and\n'next' by merging a bit more topic from 'next' to 'master' without\nadding more topics to 'next' for a week or so, and then rebuild\n'next' at the end of that period, and then start to accept more\ntopics to 'next'.\n"},{"id":"358441","messageId":"20180919034907.GA7626@sigill.intra.peff.net","threadId":"49349","inReplyTo":"xmqqy3c3agkr.fsf@gitster-ct.c.googlers.com","subject":"[PATCH] pack-objects: handle island check for \"external\" delta base","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2018-09-19T03:49:07Z","receivedAt":"2018-09-19T03:49:11Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Sep 14, 2018 at 02:56:36PM -0700, Junio C Hamano wrote:\n\n> * cc/delta-islands (2018-08-16) 7 commits\n>   (merged to 'next' on 2018-08-27 at cf3d7bd93f)\n>  + pack-objects: move 'layer' into 'struct packing_data'\n>  + pack-objects: move tree_depth into 'struct packing_data'\n>  + t5320: tests for delta islands\n>  + repack: add delta-islands support\n>  + pack-objects: add delta-islands support\n>  + pack-objects: refactor code into compute_layer_order()\n>  + Add delta-islands.{c,h}\n> \n>  Lift code from GitHub to restrict delta computation so that an\n>  object that exists in one fork is not made into a delta against\n>  another object that does not appear in the same forked repository.\n> \n>  Will merge to 'master'.\n\nThis needed some conflict resolution with my pack-bitmap-reuse-delta\ntopic, but there's a subtle bug in the result that went to 'master'.\nDetails and a fix below.\n\nAs a side note, I did this same resolution myself at least twice (for my\npersonal build and for porting the refreshed delta-reuse series to our\nGitHub build), and I wrote the exact same resolution you did both times.\nSo I think it was an easy mistake to make. :)\n\n-Peff\n\n-- >8 --\nSubject: pack-objects: handle island check for \"external\" delta base\n\nTwo recent topics, jk/pack-delta-reuse-with-bitmap and\ncc/delta-islands, can have a funny interaction. When\nchecking if we can reuse an on-disk delta, the first topic\nallows base_entry to be NULL when we find an object that's\nnot in the packing list. But the latter topic introduces a\ncall to in_same_island(), which needs to look at\nbase_entry->idx.oid. When these two features are used\ntogether, we might try to dereference a NULL base_entry.\n\nIn practice, this doesn't really happen. We'd generally only\nuse delta islands when packing to disk, since the whole\npoint is to optimize the pack for serving fetches later. And\nthe new delta-reuse code relies on having used reachability\nbitmaps to determine the set of objects, which we would\ntypically only do when serving an actual fetch.\n\nHowever, it is technically possible to combine these\nfeatures. And even without doing so, building with\n\"SANITIZE=address,undefined\" will cause t5310.46 to\ncomplain.  Even though that test does not have delta islands\nenabled, we still take the address of the NULL entry to pass\nto in_same_island(). That function then promptly returns\nwithout dereferencing the value when it sees that islands\nare not enabled, but it's enough to trigger a sanitizer\nerror.\n\nThe solution is straight-forward: when both features are\nused together, we should pass the oid of the found base to\nin_same_island().\n\nThis is tricky to do inside a single \"if\" statement. And\nafter the merge in f3504ea3dd (Merge branch\n'cc/delta-islands', 2018-09-17), that \"if\" condition is\nalready getting pretty unwieldy. So this patch moves the\nlogic into a helper function, where we can easily use\nmultiple return paths. The result is a bit longer, but the\nlogic should be much easier to follow.\n\nSigned-off-by: Jeff King <peff@peff.net>\n---\n builtin/pack-objects.c | 68 ++++++++++++++++++++++++++++++++----------\n 1 file changed, 52 insertions(+), 16 deletions(-)\n\ndiff --git a/builtin/pack-objects.c b/builtin/pack-objects.c\nindex 5041818ddf..27cb674124 100644\n--- a/builtin/pack-objects.c\n+++ b/builtin/pack-objects.c\n@@ -1470,6 +1470,57 @@ static void cleanup_preferred_base(void)\n \tdone_pbase_paths_num = done_pbase_paths_alloc = 0;\n }\n \n+/*\n+ * Return 1 iff the object specified by \"delta\" can be sent\n+ * literally as a delta against the base in \"base_sha1\". If\n+ * so, then *base_out will point to the entry in our packing\n+ * list, or NULL if we must use the external-base list.\n+ *\n+ * Depth value does not matter - find_deltas() will\n+ * never consider reused delta as the base object to\n+ * deltify other objects against, in order to avoid\n+ * circular deltas.\n+ */\n+static int can_reuse_delta(const unsigned char *base_sha1,\n+\t\t\t   struct object_entry *delta,\n+\t\t\t   struct object_entry **base_out)\n+{\n+\tstruct object_entry *base;\n+\n+\tif (!base_sha1)\n+\t\treturn 0;\n+\n+\t/*\n+\t * First see if we're already sending the base (or it's explicitly in\n+\t * our \"excluded\" list.\n+\t */\n+\tbase = packlist_find(&to_pack, base_sha1, NULL);\n+\tif (base) {\n+\t\tif (!in_same_island(&delta->idx.oid, &base->idx.oid))\n+\t\t\treturn 0;\n+\t\t*base_out = base;\n+\t\treturn 1;\n+\t}\n+\n+\t/*\n+\t * Otherwise, reachability bitmaps may tell us if the receiver has it,\n+\t * even if it was buried too deep in history to make it into the\n+\t * packing list.\n+\t */\n+\tif (thin && bitmap_has_sha1_in_uninteresting(bitmap_git, base_sha1)) {\n+\t\tif (use_delta_islands) {\n+\t\t\tstruct object_id base_oid;\n+\t\t\thashcpy(base_oid.hash, base_sha1);\n+\t\t\tif (!in_same_island(&delta->idx.oid, &base_oid))\n+\t\t\t\treturn 0;\n+\t\t}\n+\t\t*base_out = NULL;\n+\t\treturn 1;\n+\t}\n+\n+\treturn 0;\n+}\n+\n static void check_object(struct object_entry *entry)\n {\n \tunsigned long canonical_size;\n@@ -1556,22 +1607,7 @@ static void check_object(struct object_entry *entry)\n \t\t\tbreak;\n \t\t}\n \n-\t\tif (base_ref && (\n-\t\t    (base_entry = packlist_find(&to_pack, base_ref, NULL)) ||\n-\t\t    (thin &&\n-\t\t     bitmap_has_sha1_in_uninteresting(bitmap_git, base_ref))) &&\n-\t\t    in_same_island(&entry->idx.oid, &base_entry->idx.oid)) {\n-\t\t\t/*\n-\t\t\t * If base_ref was set above that means we wish to\n-\t\t\t * reuse delta data, and either we found that object in\n-\t\t\t * the list of objects we want to pack, or it's one we\n-\t\t\t * know the receiver has.\n-\t\t\t *\n-\t\t\t * Depth value does not matter - find_deltas() will\n-\t\t\t * never consider reused delta as the base object to\n-\t\t\t * deltify other objects against, in order to avoid\n-\t\t\t * circular deltas.\n-\t\t\t */\n+\t\tif (can_reuse_delta(base_ref, entry, &base_entry)) {\n \t\t\toe_set_type(entry, entry->in_pack_type);\n \t\t\tSET_SIZE(entry, in_pack_size); /* delta size */\n \t\t\tSET_DELTA_SIZE(entry, in_pack_size);\n-- \n2.19.0.745.g75ede3edf3\n\n"},{"id":"358473","messageId":"CAN0heSpMRfDd7=E_kYL6ucL9bL2VWMGLTAkT72BUrVwR6YTezQ@mail.gmail.com","threadId":"49349","inReplyTo":"20180919034907.GA7626@sigill.intra.peff.net","subject":"Re: [PATCH] pack-objects: handle island check for \"external\" delta base","fromName":"Martin Ågren","fromEmail":"martin.agren@gmail.com","sentAt":"2018-09-19T18:34:05Z","receivedAt":"2018-09-19T18:34:19Z","isPatch":true,"sender":{"key":"martin.agren@gmail.com","avatar":null},"body":"On Wed, 19 Sep 2018 at 05:49, Jeff King <peff@peff.net> wrote:\n> This is tricky to do inside a single \"if\" statement. And\n> after the merge in f3504ea3dd (Merge branch\n> 'cc/delta-islands', 2018-09-17), that \"if\" condition is\n> already getting pretty unwieldy. So this patch moves the\n> logic into a helper function, where we can easily use\n> multiple return paths. The result is a bit longer, but the\n> logic should be much easier to follow.\n\n> +static int can_reuse_delta(const unsigned char *base_sha1,\n> +                          struct object_entry *delta,\n> +                          struct object_entry **base_out)\n> +{\n> +       struct object_entry *base;\n> +\n> +       if (!base_sha1)\n> +               return 0;\n\nSo this corresponds to \"if (base_ref &&\".\n\n> +       /*\n> +        * First see if we're already sending the base (or it's explicitly in\n> +        * our \"excluded\" list.\n> +        */\n\nMissing ')'.\n\n> +       base = packlist_find(&to_pack, base_sha1, NULL);\n> +       if (base) {\n> +               if (!in_same_island(&delta->idx.oid, &base->idx.oid))\n> +                       return 0;\n\nThis logic matches the removed code...\n\n> +               *base_out = base;\n> +               return 1;\n> +       }\n> +\n> +       /*\n> +        * Otherwise, reachability bitmaps may tell us if the receiver has it,\n> +        * even if it was buried too deep in history to make it into the\n> +        * packing list.\n> +        */\n> +       if (thin && bitmap_has_sha1_in_uninteresting(bitmap_git, base_sha1)) {\n\nThis matches...\n\n> +               if (use_delta_islands) {\n> +                       struct object_id base_oid;\n> +                       hashcpy(base_oid.hash, base_sha1);\n> +                       if (!in_same_island(&delta->idx.oid, &base_oid))\n> +                               return 0;\n\nThis does some extra juggling to avoid using `base->idx.oid`, which\nwould have been the moral equivalent of the original code, but which\nwon't fly since `base` is NULL.\n\n> +               }\n> +               *base_out = NULL;\n> +               return 1;\n> +       }\n> +\n> +       return 0;\n> +}\n> +\n>  static void check_object(struct object_entry *entry)\n>  {\n>         unsigned long canonical_size;\n> @@ -1556,22 +1607,7 @@ static void check_object(struct object_entry *entry)\n>                         break;\n>                 }\n>\n> -               if (base_ref && (\n> -                   (base_entry = packlist_find(&to_pack, base_ref, NULL)) ||\n> -                   (thin &&\n> -                    bitmap_has_sha1_in_uninteresting(bitmap_git, base_ref))) &&\n> -                   in_same_island(&entry->idx.oid, &base_entry->idx.oid)) {\n\nYeah, the new function looks much simpler than this. We have\n\n  if (A && (B1 || B2) && C) {.\n\nKnowing what to look for, it can be seen that we can -- under the right\ncircumstances -- have A and B2, but not B1, and try to evalute C by\ndereferencing `base_entry` which will be NULL.\n\n> +               if (can_reuse_delta(base_ref, entry, &base_entry)) {\n>                         oe_set_type(entry, entry->in_pack_type);\n>                         SET_SIZE(entry, in_pack_size); /* delta size */\n>                         SET_DELTA_SIZE(entry, in_pack_size);\n\nWithout being at all familiar with this code, this looks sane to me.\nJust had a small nit about the missing closing ')'.\n\nMartin\n"},{"id":"358475","messageId":"20180919184343.GA13014@sigill.intra.peff.net","threadId":"49349","inReplyTo":"CAN0heSpMRfDd7=E_kYL6ucL9bL2VWMGLTAkT72BUrVwR6YTezQ@mail.gmail.com","subject":"Re: [PATCH] pack-objects: handle island check for \"external\" delta base","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2018-09-19T18:43:43Z","receivedAt":"2018-09-19T18:43:46Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Sep 19, 2018 at 08:34:05PM +0200, Martin Ågren wrote:\n\n> > +       /*\n> > +        * First see if we're already sending the base (or it's explicitly in\n> > +        * our \"excluded\" list.\n> > +        */\n> \n> Missing ')'.\n\nOops, yes.\n\n> > +               if (use_delta_islands) {\n> > +                       struct object_id base_oid;\n> > +                       hashcpy(base_oid.hash, base_sha1);\n> > +                       if (!in_same_island(&delta->idx.oid, &base_oid))\n> > +                               return 0;\n> \n> This does some extra juggling to avoid using `base->idx.oid`, which\n> would have been the moral equivalent of the original code, but which\n> won't fly since `base` is NULL.\n\nYeah, this is the actual bug-fix.\n\nI wasn't happy about having to write in_same_island() twice, but writing\nit the other way ended up pretty nasty, too. Something like:\n\n  struct object_id ext_oid;\n  struct object_id *base_oid;\n\n  base = packlist_find(&to_pack, base_sha1, NULL);\n  if (base) {\n\tbase_oid = &base->idx.oid;\n\t*base_out = base;\n  }\n  else if (thin && bitmap_has_sha1_in_uninteresting(bitmap_git, base_sha1)) {\n\thashcpy(ext_oid.hash, base_sha1);\n\tbase_oid = &ext_oid;\n\t*base_out = NULL;\n  } else {\n\treturn 0;\n  }\n\n  if (use_island_marks && !in_same_island(&delta->idx.oid, base_oid))\n\treturn 0;\n\n  return 1;\n\nThat's less repetitive, but I feel like it's harder to follow which\nvariables are valid when.\n\n> > +               if (can_reuse_delta(base_ref, entry, &base_entry)) {\n> >                         oe_set_type(entry, entry->in_pack_type);\n> >                         SET_SIZE(entry, in_pack_size); /* delta size */\n> >                         SET_DELTA_SIZE(entry, in_pack_size);\n> \n> Without being at all familiar with this code, this looks sane to me.\n> Just had a small nit about the missing closing ')'.\n\nThanks for the review!\n\n-Peff\n"}]}