{"thread":{"id":"49549","subject":"What's cooking in git.git (Oct 2018, #02; Sat, 13)","startedAt":"2018-10-12T14:54:03Z","lastAt":"2018-10-16T23:11:51Z","messageCount":8,"participants":["Junio C Hamano","Ævar Arnfjörð Bjarmason","Stefan Beller","Josh Steadmon"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"360316","messageId":"xmqqh8hr9pxb.fsf@gitster-ct.c.googlers.com","threadId":"49549","inReplyTo":null,"subject":"What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-10-12T14:53:52Z","receivedAt":"2018-10-12T14:54:03Z","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\nQuite a lot of topics have been merged to 'next'.  Let's see how\nwell they fare.\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/gc-doc-update (2018-10-11) 1 commit\n - gc doc: mention the commit-graph in the intro\n\n The documentation of \"git gc\" has been updated to mention that it\n is no longer limited to \"pruning away crufts\" but also updates\n ancillary files like commit-graph as a part of repository\n optimization.\n\n Waiting for reactions.\n cf. <20181010193818.20399-1-avarab@gmail.com>\n The author seems to feel that this might be controversial.\n\n\n* du/branch-show-current (2018-10-12) 2 commits\n - SQUASH???\n - branch: introduce --show-current display option\n\n \"git branch\" learned a new subcommand \"--show-current\".\n\n\n* du/rev-parse-is-plumbing (2018-10-11) 1 commit\n - doc: move git-rev-parse from porcelain to plumbing\n\n Doc update.\n\n Will merge to 'next'.\n\n\n* mm/doc-no-dashed-git (2018-10-11) 1 commit\n - doc: fix a typo and clarify a sentence\n\n Doc update.\n\n Will merge to 'next'.\n\n\n* ot/ref-filter-plug-leaks (2018-10-11) 3 commits\n . ref-filter: free item->value and item->value->s\n . ls-remote: release memory instead of UNLEAK\n . ref-filter: free memory from used_atom\n\n Plugging a handful of memory leaks in the ref-filter codepath.\n\n Seems to break t6300 standalone and when merged to 'pu'.\n\n\n* rv/send-email-cc-misc-by (2018-10-11) 3 commits\n - send-email: also pick up cc addresses from -by trailers\n - send-email: only consider lines containing @ or <> for automatic Cc'ing\n - Documentation/git-send-email.txt: style fixes\n\n \"git send-email\" learned to grab address-looking string on any\n trailer whose name ends with \"-by\"; --suppress-cc=misc-by on the\n command line, or setting sendemail.suppresscc configuration\n variable to \"misc-by\", can be used to disable this behaviour.\n\n This is a backward-incompatible change that may surprise existing\n users.\n\n\n* du/cherry-is-plumbing (2018-10-12) 1 commit\n - doc: move git-cherry to plumbing\n\n Doc update to mark \"git cherry\" as a plumbing command.\n\n\n* du/get-tar-commit-id-is-plumbing (2018-10-12) 1 commit\n - doc: move git-get-tar-commit-id to plumbing\n\n Doc update to mark \"git get-tar-commit-id\" as a plumbing command.\n\n Will merge to 'next'.\n\n\n* lm/range-diff-submodule-fix (2018-10-12) 1 commit\n - range-diff: allow to diff files regardless submodule\n\n \"git range-diff\" did not work well when the compared ranges had\n changes in submodules and the \"--submodule=log\" was used.\n\n Will merge to 'next'.\n\n\n* sb/diff-emit-line-ws-markup-cleanup (2018-10-12) 1 commit\n - diff.c: pass sign_index to emit_line_ws_markup\n\n Code clean-up.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\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[Cooking]\n\n* mk/use-size-t-in-zlib (2018-10-12) 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\n* ds/reachable-final-cleanup (2018-09-25) 1 commit\n  (merged to 'next' on 2018-10-12 at 32c6d5f55a)\n + commit-reach: cleanups in can_all_from_reach...\n\n Code already in 'master' is further cleaned-up by this patch.\n\n Will merge to 'master'.\n\n\n* dz/credential-doc-url-matching-rules (2018-09-27) 1 commit\n  (merged to 'next' on 2018-10-12 at 4547952530)\n + doc: clarify gitcredentials path component matching\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* en/status-multiple-renames-to-the-same-target-fix (2018-09-27) 1 commit\n  (merged to 'next' on 2018-10-12 at 4976fc61a0)\n + commit: fix erroneous BUG, 'multiple renames on the same target? how?'\n\n The code in \"git status\" sometimes hit an assertion failure.  This\n was caused by a structure that was reused without cleaning the data\n used for the first run, which has been corrected.\n\n Will merge to 'master'.\n\n\n* jc/how-to-document-api (2018-09-29) 1 commit\n  (merged to 'next' on 2018-10-12 at 7c9bd82285)\n + CodingGuidelines: document the API in *.h files\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* jc/rebase-in-c-5-test-typofix (2018-10-11) 1 commit\n - rebase: fix typoes in error messages\n (this branch uses 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; is tangled with js/rebase-in-c-5.5-work-with-rebase-i-in-c and pk/rebase-in-c-6-final.)\n\n Typofix.\n\n Will merge to 'next' together with pk/rebase-i-in-c-6-final.\n\n\n* jc/war-on-string-list (2018-09-28) 1 commit\n - fetch: replace string-list used as a look-up table with a hashmap\n\n Replace two string-list instances used as look-up tables in \"git\n fetch\" with a pair of hashmaps.  WIP as there is another such use\n of string-list nearby that should be converted at the same time.\n\n\n* jk/check-everything-connected-is-long-gone (2018-09-25) 1 commit\n  (merged to 'next' on 2018-10-12 at 4ce30c9a30)\n + receive-pack: update comment with check_everything_connected\n\n Comment fix.\n\n Will merge to 'master'.\n\n\n* jk/oideq-hasheq-cleanup (2018-10-04) 1 commit\n  (merged to 'next' on 2018-10-12 at 7c9b5681da)\n + more oideq/hasheq conversions\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* js/mingw-wants-vista-or-above (2018-10-04) 3 commits\n  (merged to 'next' on 2018-10-12 at 3a174c3d43)\n + mingw: bump the minimum Windows version to Vista\n + mingw: set _WIN32_WINNT explicitly for Git for Windows\n + compat/poll: prepare for targeting Windows Vista\n\n The minimum version of Windows supported by Windows port fo Git is\n now set to Vista.\n\n Will merge to 'master'.\n\n\n* js/rebase-i-break (2018-10-12) 2 commits\n - rebase -i: introduce the 'break' command\n - rebase -i: clarify what happens on a failed `exec`\n (this branch uses ag/rebase-i-in-c; is tangled with ag/sequencer-reduce-rewriting-todo, js/rebase-in-c-5.5-work-with-rebase-i-in-c and pk/rebase-in-c-6-final.)\n\n \"git rebase -i\" learned a new insn, 'break', that the user can\n insert in the to-do list.  Upon hitting it, the command returns\n control back to the user.\n\n Will merge to 'next' after 'pk/rebase-in-c-6-final'.\n\n\n* js/remote-archive-v2 (2018-09-28) 4 commits\n  (merged to 'next' on 2018-10-12 at 5f34377f60)\n + archive: allow archive over HTTP(S) with proto v2\n + archive: implement protocol v2 archive command\n + archive: use packet_reader for communications\n + archive: follow test standards around assertions\n\n The original implementation of \"git archive --remote\" more or less\n bypassed the transport layer and did not work over http(s).  The\n version 2 of the protocol is defined to allow going over http(s) as\n well as Git native transport.\n\n Will merge to 'master'.\n\n\n* jt/non-blob-lazy-fetch (2018-10-04) 2 commits\n  (merged to 'next' on 2018-10-12 at 7466c6bd7d)\n + fetch-pack: exclude blobs when lazy-fetching trees\n + fetch-pack: avoid object flags if no_dependents\n\n A partial clone that is configured to lazily fetch missing objects\n will on-demand issue a \"git fetch\" request to the originating\n repository to fill not-yet-obtained objects.  The request has been\n optimized for requesting a tree object (and not the leaf blob\n objects contained in it) by telling the originating repository that\n no blobs are needed.\n\n Will merge to 'master'.\n\n\n* ma/commit-graph-docs (2018-09-27) 4 commits\n  (merged to 'next' on 2018-10-12 at eafaf79b5a)\n + Doc: refer to the \"commit-graph file\" with dash\n + git-commit-graph.txt: refer to \"*commit*-graph file\"\n + git-commit-graph.txt: typeset more in monospace\n + git-commit-graph.txt: fix bullet lists\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ma/mailing-list-address-in-git-help (2018-09-29) 1 commit\n  (merged to 'next' on 2018-10-12 at 4c3737d234)\n + git doc: direct bug reporters to mailing list archive\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ma/t1400-undebug-test (2018-09-28) 1 commit\n  (merged to 'next' on 2018-10-12 at d736f83435)\n + t1400: drop debug `echo` to actually execute `test`\n\n Test fix.\n\n Will merge to 'master'.\n\n\n* ma/t7005-bash-workaround (2018-09-28) 1 commit\n - t7005-editor: quote filename to fix whitespace-issue\n\n Test fix.\n\n Will merge to 'next'.\n\n\n* nd/help-commands-verbose-by-default (2018-10-03) 1 commit\n  (merged to 'next' on 2018-10-12 at 32de8f53e0)\n + help -a: improve and make --verbose default\n\n \"git help -a\" and \"git help -av\" give different pieces of\n information, and generally the \"verbose\" version is more friendly\n to the new users.  \"git help -a\" by default now uses the more\n verbose output (with \"--no-verbose\", you can go back to the\n original).  Also \"git help -av\" now lists aliases and external\n commands, which it did not used to.\n\n Will merge to 'master'.\n\n\n* nd/packobjectshook-doc-fix (2018-09-29) 1 commit\n  (merged to 'next' on 2018-10-12 at 85d75f5219)\n + config.txt: correct the note about uploadpack.packObjectsHook\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* pw/diff-color-moved-ws-fix (2018-10-04) 5 commits\n  (merged to 'next' on 2018-10-12 at 73badc83fe)\n + diff --color-moved: fix a memory leak\n + diff --color-moved-ws: fix another memory leak\n + diff --color-moved-ws: fix a memory leak\n + diff --color-moved-ws: fix out of bounds string access\n + diff --color-moved-ws: fix double free crash\n\n Various fixes to \"diff --color-moved-ws\".\n\n Will merge to 'master'.\n\n\n* rs/grep-no-recursive (2018-10-03) 1 commit\n  (merged to 'next' on 2018-10-12 at 1499dc98ba)\n + grep: add -r/--[no-]recursive\n\n Unlike \"grep\", \"git grep\" by default recurses to the whole tree.\n The command learned \"git grep --recursive\" option, so that \"git\n grep --no-recursive\" can serve as a synonym to setting the\n max-depth to 0.\n\n Will merge to 'master'.\n\n\n* rs/oidset-on-khash (2018-10-04) 5 commits\n  (merged to 'next' on 2018-10-12 at 8bb0152b3f)\n + oidset: uninline oidset_init()\n + oidset: use khash\n + khash: factor out kh_release_*\n + fetch-pack: load tip_oids eagerly iff needed\n + fetch-pack: factor out is_unmatched_ref()\n\n The oidset API was built on top of the oidmap API which in turn is\n on the hashmap API.  Replace the implementation to build on top of\n the khash API and gain performance.\n\n Will merge to 'master'.\n\n\n* rs/sequencer-oidset-insert-avoids-dups (2018-10-03) 1 commit\n  (merged to 'next' on 2018-10-12 at b294394711)\n + sequencer: use return value of oidset_insert()\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* rt/rebase-typofix (2018-09-28) 1 commit\n  (merged to 'next' on 2018-10-12 at 58691d20df)\n + git-rebase.sh: fix typos in error messages\n\n Typofix.\n\n Will merge to 'master'.\n\n\n* rv/alias-help (2018-10-11) 3 commits\n - git-help.txt: document \"git help cmd\" vs \"git cmd --help\" for aliases\n - git.c: handle_alias: prepend alias info when first argument is -h\n - help: redirect to aliased commands for \"git cmd --help\"\n\n \"git cmd --help\" when \"cmd\" is aliased used to only say \"cmd is\n aliased to ...\".  Now it shows that to the standard error stream\n and runs \"git $cmd --help\" where $cmd is the first word of the\n alias expansion.  This could be misleading for those who alias a\n command with options (e.g. with \"[alias] cpn = cherry-pick -n\",\n \"git cpn --help\" would show the manual of \"cherry-pick\", and the\n reader would not be told to pay close attention to the part that\n describes the \"--no-commit\" option until closing the pager that\n showed the contents of the manual, if the pager is configured to\n restore the original screen, or would not be told at all, if the\n pager simply makes the message on the standard error scroll away.\n\n Will merge to 'next'.\n\n\n* sb/strbuf-h-update (2018-09-29) 1 commit\n - strbuf.h: format according to coding guidelines\n\n Code clean-up to serve as a BCP example.\n\n What's the status of this one after the discussion thread stopped here?\n cf. <CAGZ79kbV6QjsFKcD2uG_P9j1AvzSNQSi-_jXGQ9w0YU9fjhEGg@mail.gmail.com>\n\n\n* sg/split-index-racefix (2018-10-12) 7 commits\n - split-index: BUG() when cache entry refers to non-existing shared entry\n - split-index: smudge and add racily clean cache entries to split index\n - split-index: don't compare cached data of entries already marked for split index\n - split-index: count the number of deleted entries\n - t1700-split-index: date back files to avoid racy situations\n - split-index: add tests to demonstrate the racy split index problem\n - t1700-split-index: document why FSMONITOR is disabled in this test script\n\n The codepath to support the experimental split-index mode had\n remaining \"racily clean\" issues fixed.\n\n Will merge to 'next'.\n\n\n* sm/show-superproject-while-conflicted (2018-09-28) 1 commit\n  (merged to 'next' on 2018-10-12 at 0334353131)\n + rev-parse: --show-superproject-working-tree should work during a merge\n\n A corner-case bugfix.\n\n Will merge to 'master'.\n\n\n* ag/sequencer-reduce-rewriting-todo (2018-10-09) 16 commits\n - rebase--interactive: move transform_todo_file() to rebase--interactive.c\n - sequencer: fix a call to error() in transform_todo_file()\n - sequencer: use edit_todo_list() in complete_action()\n - rebase-interactive: rewrite edit_todo_list() to handle the initial edit\n - rebase-interactive: append_todo_help() changes\n - rebase-interactive: use todo_list_transform() in edit_todo_list()\n - sequencer: refactor skip_unnecessary_picks() to work on a todo_list\n - sequencer: change complete_action() to use the refactored functions\n - sequencer: make sequencer_make_script() write its script to a strbuf\n - sequencer: refactor transform_todos() to work on a todo_list\n - sequencer: refactor rearrange_squash() to work on a todo_list\n - sequencer: refactor sequencer_add_exec_commands() to work on a todo_list\n - sequencer: refactor check_todo_list() to work on a todo_list\n - sequencer: make the todo_list structure public\n - sequencer: clear the number of items of a todo_list before parsing\n - Merge branch 'ag/rebase-i-in-c' into ag/sequencer-reduce-rewriting-todo\n (this branch uses ag/rebase-i-in-c; is tangled with js/rebase-i-break, js/rebase-in-c-5.5-work-with-rebase-i-in-c and pk/rebase-in-c-6-final.)\n\n The scripted version of \"git rebase -i\" wrote and rewrote the todo\n list many times during a single step of its operation, and the\n recent C-rewrite made a faithful conversion of the logic to C.  The\n implementation has been updated to carry necessary information\n around in-core to avoid rewriting the same file over and over\n unnecessarily.\n\n Will hold, waiting for the \"rebase-in-c\" and \"rebase-i-in-c\" topics.\n\n\n* bc/editorconfig (2018-10-09) 2 commits\n - editorconfig: indicate settings should be kept in sync\n - editorconfig: provide editor settings for Git developers\n\n To help developers, an EditorConfig file that attempts to follow\n the project convention has been added.\n\n Will merge to 'next'.\n\n\n* bc/hash-transition-part-15 (2018-10-09) 14 commits\n - rerere: convert to use the_hash_algo\n - submodule: make zero-oid comparison hash function agnostic\n - apply: rename new_sha1_prefix and old_sha1_prefix\n - apply: replace hard-coded constants\n - tag: express constant in terms of the_hash_algo\n - transport: use parse_oid_hex instead of a constant\n - upload-pack: express constants in terms of the_hash_algo\n - refs/packed-backend: express constants using the_hash_algo\n - packfile: express constants in terms of the_hash_algo\n - pack-revindex: express constants in terms of the_hash_algo\n - builtin/fetch-pack: remove constants with parse_oid_hex\n - builtin/mktree: remove hard-coded constant\n - builtin/repack: replace hard-coded constant\n - pack-bitmap-write: use GIT_MAX_RAWSZ for allocation\n\n More codepaths are moving away from hardcoded hash sizes.\n\n\n* ch/subtree-build (2018-10-10) 1 commit\n  (merged to 'next' on 2018-10-12 at 4ed9ff6300)\n + subtree: add build targets 'man' and 'html'\n\n Build update for \"git subtree\" (in contrib/) documentation pages.\n\n Will merge to 'master'.\n\n\n* ds/commit-graph-leakfix (2018-10-07) 3 commits\n  (merged to 'next' on 2018-10-12 at 8cc7f2f1e9)\n + commit-graph: reduce initial oid allocation\n + builtin/commit-graph.c: UNLEAK variables\n + commit-graph: clean up leaked memory during write\n (this branch uses ab/commit-graph-progress.)\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* ds/test-multi-pack-index (2018-10-09) 3 commits\n - multi-pack-index: define GIT_TEST_MULTI_PACK_INDEX\n - midx: close multi-pack-index on repack\n - midx: fix broken free() in close_midx()\n\n Tests for the recently introduced multi-pack index machinery.\n\n Expecting a reroll.\n cf. <8b5dbe3d-b382-bf48-b524-d9e8a074ac4d@gmail.com>\n\n\n* js/fuzzer (2018-10-10) 2 commits\n - fuzz: add fuzz testing for packfile indices\n - fuzz: add basic fuzz testing target\n\n An experiment to fuzz test a few areas, hopefully we can gain more\n coverage to various areas.\n\n\n* jt/avoid-ls-refs (2018-10-07) 4 commits\n  (merged to 'next' on 2018-10-12 at 5775aabbc1)\n + fetch: do not list refs if fetching only hashes\n + transport: list refs before fetch if necessary\n + transport: do not list refs if possible\n + transport: allow skipping of ref listing\n\n Over some transports, fetching objects with an exact commit object\n name can be done without first seeing the ref advertisements.  The\n code has been optimized to exploit this.\n\n Will merge to 'master'.\n\n\n* jt/cache-tree-allow-missing-object-in-partial-clone (2018-10-10) 1 commit\n  (merged to 'next' on 2018-10-12 at 152ad8e336)\n + cache-tree: skip some blob checks in partial clone\n\n In a partial clone that will lazily be hydrated from the\n originating repository, we generally want to avoid \"does this\n object exist (locally)?\" on objects that we deliberately omitted\n when we created the clone.  The cache-tree codepath (which is used\n to write a tree object out of the index) however insisted that the\n object exists, even for paths that are outside of the partial\n checkout area.  The code has been updated to avoid such a check.\n\n Will merge to 'master'.\n\n\n* mw/doc-typofixes (2018-10-07) 3 commits\n  (merged to 'next' on 2018-10-12 at 97297b8ce9)\n + docs: typo: s/isimilar/similar/\n + docs: graph: remove unnecessary `graph_update()' call\n + docs: typo: s/go/to/\n\n Typofixes.\n\n Will merge to 'master'.\n\n\n* nd/per-worktree-ref-iteration (2018-10-07) 9 commits\n . SQUASH???\n . reflog expire: cover reflog from all worktrees\n . fsck: check HEAD and reflog from other worktrees\n . fsck: Move fsck_head_link() to get_default_heads() to avoid some globals\n . revision.c: better error reporting on ref from different worktrees\n . revision.c: correct a parameter name\n . refs: new ref types to make per-worktree refs visible to all worktrees\n . Add a place for (not) sharing stuff between worktrees\n . refs.c: indent with tabs, not spaces\n\n Ejected for now, as it seems to break t2025 when in 'pu'.\n\n\n* np/log-graph-octopus-fix (2018-10-12) 1 commit\n - log: fix coloring of certain octopus merge shapes\n\n \"git log --graph\" showing an octopus merge sometimes miscounted the\n number of display columns it is consuming to show the merge and its\n parent commits, which has been corrected.\n\n Will merge to 'next'.\n\n\n* rs/subtree-fixes (2018-10-12) 5 commits\n - subtree: performance improvement for finding unexpected parent commits\n  (merged to 'next' on 2018-10-12 at 2b47258e04)\n + subtree: improve decision on merges kept in split\n + subtree: use commits before rejoins for splits\n + subtree: make --ignore-joins pay attention to adds\n + subtree: refactor split of a commit into standalone method\n\n Various subtree fixes.\n\n Will merge to 'master'.\n Unless somebody objects, that is.\n\n\n* sf/complete-stash-list (2018-10-07) 1 commit\n  (merged to 'next' on 2018-10-12 at ff0e9a0805)\n + git-completion.bash: add completion for stash list\n\n The completion script (in contrib/) lerned to complete a handful of\n options \"git stash list\" command takes.\n\n Will merge to 'master'.\n\n\n* tb/filter-alternate-refs (2018-10-09) 4 commits\n  (merged to 'next' on 2018-10-12 at 21c14722f1)\n + transport.c: introduce core.alternateRefsPrefixes\n + transport.c: introduce core.alternateRefsCommand\n + transport.c: extract 'fill_alternate_refs_command'\n + transport: drop refnames from for_each_alternate_ref\n\n When pushing into a repository that borrows its objects from an\n alternate object store, \"git receive-pack\" that responds to the\n push request on the other side lists the tips of refs in the\n alternate to reduce the amount of objects transferred.  This\n sometimes is detrimental when the number of refs in the alternate\n is absurdly large, in which case the bandwidth saved in potentially\n fewer objects transferred is wasted in excessively large ref\n advertisement.  The alternate refs that are advertised are now\n configurable with a pair of configuration variables.\n\n Will merge to 'master'.\n\n\n* fe/doc-updates (2018-09-21) 3 commits\n  (merged to 'next' on 2018-10-10 at 2eea3a88bc)\n + git-describe.1: clarify that \"human readable\" is also git-readable\n + git-column.1: clarify initial description, provide examples\n + git-archimport.1: specify what kind of Arch we're talking about\n\n Doc updates.\n\n Will merge to 'master'.\n\n\n* md/test-cleanup (2018-10-07) 7 commits\n  (merged to 'next' on 2018-10-10 at 7e0bf1b573)\n + tests: order arguments to git-rev-list properly\n + t9109: don't swallow Git errors upstream of pipes\n + tests: don't swallow Git errors upstream of pipes\n + t/*: fix ordering of expected/observed arguments\n + tests: standardize pipe placement\n + Documentation: add shell guidelines\n + t/README: reformat Do, Don't, Keep in mind lists\n\n Various test scripts have been updated for style and also correct\n handling of exit status of various commands.\n\n Will merge to 'master'.\n\n\n* nd/complete-fetch-multiple-args (2018-09-21) 1 commit\n  (merged to 'next' on 2018-10-10 at f78e14123c)\n + completion: support \"git fetch --multiple\"\n\n Teach bash completion that \"git fetch --multiple\" only takes remote\n names as arguments and no refspecs.\n\n Will merge to 'master'.\n\n\n* jt/fetch-tips-in-partial-clone (2018-09-21) 2 commits\n  (merged to 'next' on 2018-10-12 at 521b3fb44d)\n + fetch: in partial clone, check presence of targets\n + connected: document connectivity in partial clones\n\n \"git fetch $repo $object\" in a partial clone did not correctly\n fetch the asked-for object that is referenced by an object in\n promisor packfile, which has been fixed.\n\n Will merge to 'master'.\n\n\n* tg/t5551-with-curl-7.61.1 (2018-09-24) 2 commits\n  (merged to 'next' on 2018-10-10 at 5ada84ed7a)\n + t5551: compare sorted cookies files\n + t5551: move setup code inside test_expect blocks\n\n Test update.\n\n Will merge to 'master'.\n Supersedes tz/t5551-with-curl-7.61.1 topic\n\n\n* jn/gc-auto-prep (2018-07-17) 2 commits\n  (merged to 'next' on 2018-10-10 at 4ab6a62f62)\n + gc: exit with status 128 on failure\n + gc: improve handling of errors reading gc.log\n (this branch is used by jn/gc-auto.)\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* nd/status-refresh-progress (2018-09-17) 1 commit\n  (merged to 'next' on 2018-10-12 at 9240c05add)\n + status: show progress bar if refreshing the index takes too long\n\n \"git status\" learns to show progress bar when refreshing the index\n takes a long time.\n\n Will merge to 'master'.\n\n\n* nd/the-index (2018-09-21) 23 commits\n  (merged to 'next' on 2018-10-10 at 16e2e2e947)\n + revision.c: reduce implicit dependency the_repository\n + revision.c: remove implicit dependency on the_index\n + ws.c: remove implicit dependency on the_index\n + tree-diff.c: remove implicit dependency on the_index\n + submodule.c: remove implicit dependency on the_index\n + line-range.c: remove implicit dependency on the_index\n + userdiff.c: remove implicit dependency on the_index\n + rerere.c: remove implicit dependency on the_index\n + sha1-file.c: remove implicit dependency on the_index\n + patch-ids.c: remove implicit dependency on the_index\n + merge.c: remove implicit dependency on the_index\n + merge-blobs.c: remove implicit dependency on the_index\n + ll-merge.c: remove implicit dependency on the_index\n + diff-lib.c: remove implicit dependency on the_index\n + read-cache.c: remove implicit dependency on the_index\n + diff.c: remove implicit dependency on the_index\n + grep.c: remove implicit dependency on the_index\n + diff.c: remove the_index dependency in textconv() functions\n + blame.c: rename \"repo\" argument to \"r\"\n + combine-diff.c: remove implicit dependency on the_index\n + diff.c: reduce implicit dependency on the_index\n + read-cache.c: remove 'const' from index_has_changes()\n + archive.c: remove implicit dependency the_repository\n (this branch is used by sb/more-repo-in-api.)\n\n Various codepaths in the core-ish part learn to work on an\n arbitrary in-core index structure, not necessarily the default\n instance \"the_index\".\n\n Will merge to 'master'.\n\n\n* tq/refs-internal-comment-fix (2018-09-17) 1 commit\n  (merged to 'next' on 2018-10-09 at 422313bbd0)\n + refs: docstring typo\n\n Fix for typo in a sample code in comment.\n\n Will merge to 'master'.\n\n\n* ts/alias-of-alias (2018-09-17) 3 commits\n  (merged to 'next' on 2018-10-09 at ac19b4730b)\n + t0014: introduce an alias testing suite\n + alias: show the call history when an alias is looping\n + alias: add support for aliases of an alias\n\n An alias that expands to another alias has so far been forbidden,\n but now it is allowed to create such an alias.\n\n Will merge to 'master'.\n\n\n* ds/reachable-topo-order (2018-09-21) 7 commits\n - revision.c: refactor basic topo-order logic\n - revision.h: add whitespace in flag definitions\n - commit/revisions: bookkeeping before refactoring\n - revision.c: begin refactoring --topo-order logic\n - test-reach: add rev-list tests\n - test-reach: add run_three_modes method\n - prio-queue: add 'peek' operation\n\n The revision walker machinery learned to take advantage of the\n commit generation numbers stored in the commit-graph file.\n\n What's the status of this topic?\n\n\n* en/merge-cleanup (2018-09-20) 4 commits\n  (merged to 'next' on 2018-10-09 at f3a00b506f)\n + merge-recursive: rename merge_file_1() and merge_content()\n + merge-recursive: remove final remaining caller of merge_file_one()\n + merge-recursive: avoid wrapper function when unnecessary and wasteful\n + merge-recursive: set paths correctly when three-way merging content\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* jk/delta-islands-with-bitmap-reuse-delta-fix (2018-09-19) 1 commit\n  (merged to 'next' on 2018-10-09 at 10e58be2af)\n + pack-objects: handle island check for \"external\" delta base\n\n Fix interactions between two recent topics.\n\n Will merge to 'master'.\n\n\n* jn/mailmap-update (2018-09-25) 1 commit\n  (merged to 'next' on 2018-10-10 at fa2b394bd5)\n + mailmap: consistently normalize brian m. carlson's name\n\n The mailmap file update.\n\n Will merge to 'master'.\n\n\n* ma/config-doc-update (2018-09-20) 2 commits\n  (merged to 'next' on 2018-10-09 at 312a873a2a)\n + git-config.txt: fix 'see: above' note\n + Doc: use `--type=bool` instead of `--bool`\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* rj/header-check (2018-09-20) 8 commits\n  (merged to 'next' on 2018-10-09 at 7fa9c68ef0)\n + delta-islands.h: add missing forward declarations (hdr-check)\n + midx.h: add missing forward declarations (hdr-check)\n + refs/refs-internal.h: add missing declarations (hdr-check)\n + refs/packed-backend.h: add missing declaration (hdr-check)\n + refs/ref-cache.h: add missing declarations (hdr-check)\n + ewah/ewok_rlw.h: add missing include (hdr-check)\n + json-writer.h: add missing include (hdr-check)\n + Makefile: add a hdr-check target\n\n Header files clean-up.\n\n Will merge to 'master'.\n\n\n* bp/read-cache-parallel (2018-10-11) 7 commits\n  (merged to 'next' on 2018-10-12 at ed6edde799)\n + read-cache: load cache entries on worker threads\n + ieot: add Index Entry Offset Table (IEOT) extension\n + read-cache: load cache extensions on a worker thread\n + config: add new index.threads config setting\n + eoie: add End of Index Entry (EOIE) extension\n + read-cache: clean up casting and byte decoding\n + read-cache.c: optimize reading index format v4\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 'master'.\n\n\n* ds/coverage-diff (2018-10-10) 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 Will merge to 'next'.\n\n\n* sb/submodule-recursive-fetch-gets-the-tip (2018-10-11) 9 commits\n . builtin/fetch: check for submodule updates for non branch fetches\n . fetch: retry fetching submodules if needed objects were not fetched\n . submodule: fetch in submodules git directory instead of in worktree\n . repository: repo_submodule_init to take a submodule struct\n . submodule.c: do not copy around submodule list\n . submodule.c: 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\n \"git fetch --recurse-submodules\" may not fetch the necessary commit\n that is bound to the superproject, which is getting corrected.\n\n Ejected for now, as it has fallouts in places like t/helper/.\n\n\n* bp/rename-test-env-var (2018-09-28) 6 commits\n  (merged to 'next' on 2018-10-12 at 201e451d20)\n + t0000: do not get self-test disrupted by environment warnings\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: use git_env_bool() not getenv() for customization\n + t/README: correct spelling of \"uncommon\"\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 'master'.\n\n\n* ab/commit-graph-progress (2018-09-20) 3 commits\n  (merged to 'next' on 2018-09-24 at 76f2f5c1e3)\n + gc: fix regression in 7b0f229222 impacting --quiet\n + commit-graph verify: add progress output\n + commit-graph write: add progress output\n (this branch is used by ds/commit-graph-leakfix.)\n\n (Originally merged to 'next' on 2018-09-20 at 24ca94b1d4)\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 'master'.\n\n\n* ss/wt-status-committable (2018-10-03) 5 commits\n  (merged to 'next' on 2018-10-10 at ea30d8819d)\n + roll wt_status_state into wt_status and populate in the collect phase\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\n Code clean-up in the internal machinery used by \"git status\" and\n \"git commit --dry-run\".\n\n Will merge to 'master'.\n\n\n* js/rebase-in-c-5.5-work-with-rebase-i-in-c (2018-10-11) 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; is tangled with ag/sequencer-reduce-rewriting-todo, jc/rebase-in-c-5-test-typofix and js/rebase-i-break.)\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 Will merge to 'next' together with pk/rebase-i-in-c-6-final.\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* ao/submodule-wo-gitmodules-checked-out (2018-10-09) 10 commits\n - t/helper: add test-submodule-nested-repo-config\n - submodule: support reading .gitmodules when it's not in the working tree\n - submodule: add a helper to check if it is safe to write to .gitmodules\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 - t7411: merge tests 5 and 6\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 cf. <20181010205645.e1529eff9099805029b1d6ef@ao2.it>\n\n\n* md/filter-trees (2018-10-07) 8 commits\n  (merged to 'next' on 2018-10-12 at c66e49d4ca)\n + list-objects-filter: implement filter tree:0\n + list-objects-filter-options: do not over-strbuf_init\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 'master'.\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 jc/rebase-in-c-5-test-typofix, 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 Rewrite \"git rebase\" in C.\n\n Will merge to 'next' together with pk/rebase-i-in-c-6-final.\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 jc/rebase-in-c-5-test-typofix, 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 Rewrite \"git rebase\" in C.\n\n Will merge to 'next' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-4-opts (2018-10-11) 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 jc/rebase-in-c-5-test-typofix, 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 Rewrite \"git rebase\" in C.\n\n Will merge to 'next' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-5-test (2018-10-11) 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 jc/rebase-in-c-5-test-typofix, 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 Rewrite \"git rebase\" in C.\n\n Will merge to 'next' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-6-final (2018-10-11) 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; is tangled with ag/sequencer-reduce-rewriting-todo, jc/rebase-in-c-5-test-typofix and js/rebase-i-break.)\n\n The final step of rewriting \"rebase -i\" in C.\n\n Will merge to 'next'.\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 \"git stash\" rewritten in C.\n\n Expecting a reroll.\n cf. <20181010141841.GA17445@hank.intra.tgummerer.com>\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 discard.\n No further feedbacks on the topic for quite some time.\n\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  (merged to 'next' on 2018-10-09 at 851a457102)\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 Will merge to 'master'.\n\n\n* jn/gc-auto (2018-07-17) 1 commit\n  (merged to 'next' on 2018-10-10 at 9f0f1f770e)\n + gc: do not return error for prior errors in daemonized mode\n (this branch uses jn/gc-auto-prep.)\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 Will merge to 'master'.\n cf. <20180917182639.GB140909@aiede.svl.corp.google.com>\n cf. <20181009234502.oxzfwirjcew2sxrm@dcvr>\n\n\n* ag/rebase-i-in-c (2018-10-09) 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 ag/sequencer-reduce-rewriting-todo, js/rebase-i-break, 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 Will merge to 'next' together with pk/rebase-i-in-c-6-final.\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 jc/rebase-in-c-5-test-typofix, 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 Will merge to 'next' together with pk/rebase-i-in-c-6-final.\n\n--------------------------------------------------\n[Discarded]\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 Discarded for a cleaned-up rewrite.\n cf. <CAGZ79kYJGMOtroLhPwOdhWewXf3X_b0QOgiUb65pyJ7xJ-whQw@mail.gmail.com>\n\n\n* sb/grep-submodule-cleanup (2018-10-10) 1 commit\n . builtin/grep.c: remove superfluous submodule code\n\n Code clean-up.\n\n Discarded, expecting a future reroll.\n cf. <CAGZ79kYJGMOtroLhPwOdhWewXf3X_b0QOgiUb65pyJ7xJ-whQw@mail.gmail.com>\n cf. <CAGZ79kbZ35OEh=2JiZuOHgG-P3a3PeSP5hgrXX-tQh9feOQsjQ@mail.gmail.com>\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\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\n* bw/submodule-name-to-dir (2018-08-10) 2 commits\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 Discarded, 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 Discarded after hearing no strong support.\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\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 cf. <0102015f5e5ee171-f30f4868-886f-47a1-a4e4-b4936afc545d-000000@eu-west-1.amazonses.com>\n ... there is no update to address any of the review comments\n in the thread above.\n"},{"id":"360317","messageId":"878t33dx8i.fsf@evledraar.gmail.com","threadId":"49549","inReplyTo":"xmqqh8hr9pxb.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2018-10-12T15:02:21Z","receivedAt":"2018-10-12T15:02:27Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Fri, Oct 12 2018, Junio C Hamano wrote:\n\n> * ab/gc-doc-update (2018-10-11) 1 commit\n>  - gc doc: mention the commit-graph in the intro\n>\n>  The documentation of \"git gc\" has been updated to mention that it\n>  is no longer limited to \"pruning away crufts\" but also updates\n>  ancillary files like commit-graph as a part of repository\n>  optimization.\n>\n>  Waiting for reactions.\n>  cf. <20181010193818.20399-1-avarab@gmail.com>\n>  The author seems to feel that this might be controversial.\n\nProbably shouldn't have mentioned that. SZEDER I think had more issues\nwith the \"that's not GC\" parts of what git-gc is doing, so I'll let him\nchime in. I'm working on a more general update to gc.c which might bring\nit more in the \"not really GC anymore\" direction.\n\nBut this patch is just describing the status quo, so I think it should\nbe uncontroversial to merge it, or at least we can have the discussion\nabout what we should do in a different venue than a doc fixup for\naccurately describing what we're doing now.\n\nAnother topic: As noted in <20181010193235.17359-1-avarab@gmail.com> and\nthis follow-up\nhttps://public-inbox.org/git/87h8hsexdm.fsf@evledraar.gmail.com/ it\nwould be great if you could pick up the \"gc: remove redundant check for\ngc_auto_threshold\" patch. A small change that makes later refactoring a\ntiny bit smaller and easier to understand.\n\n"},{"id":"360339","messageId":"CAGZ79kZGEMWpr7aqeqXbC4bkmsCCiW+1pxbEV4T0vfsYG+_3iA@mail.gmail.com","threadId":"49549","inReplyTo":"xmqqh8hr9pxb.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2018-10-12T19:44:23Z","receivedAt":"2018-10-12T19:44:38Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":">\n> * sb/strbuf-h-update (2018-09-29) 1 commit\n>  - strbuf.h: format according to coding guidelines\n>\n>  Code clean-up to serve as a BCP example.\n>\n>  What's the status of this one after the discussion thread stopped here?\n>  cf. <CAGZ79kbV6QjsFKcD2uG_P9j1AvzSNQSi-_jXGQ9w0YU9fjhEGg@mail.gmail.com>\n\nI started rewriting the documentation according to your proposal of having\nparameters with name, then referred to as NAME in the docs.\n\nAfter a few examples, I must admit I do not like that style,\nso I would just want to do the minimal part that would get this patch landed,\ni.e.\n(a) convince you that the patch is good as-is or\n(b) resend with fewer parameters made explicit if\n    we desire to be concise instead.\n\n> * nd/the-index (2018-09-21) 23 commits\n>   (merged to 'next' on 2018-10-10 at 16e2e2e947)\n[...]\n>  (this branch is used by sb/more-repo-in-api.)\n>\n>  Various codepaths in the core-ish part learn to work on an\n>  arbitrary in-core index structure, not necessarily the default\n>  instance \"the_index\".\n>\n>  Will merge to 'master'.\n\nCool!\n\nsb/more-repo-in-api is not part of this message,\nbut it requires at least one resend, so I'll do that.\n\n\n> * sb/submodule-recursive-fetch-gets-the-tip (2018-10-11) 9 commits\n>  . builtin/fetch: check for submodule updates for non branch fetches\n>  . fetch: retry fetching submodules if needed objects were not fetched\n>  . submodule: fetch in submodules git directory instead of in worktree\n>  . repository: repo_submodule_init to take a submodule struct\n>  . submodule.c: do not copy around submodule list\n>  . submodule.c: 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>\n>  \"git fetch --recurse-submodules\" may not fetch the necessary commit\n>  that is bound to the superproject, which is getting corrected.\n>\n>  Ejected for now, as it has fallouts in places like t/helper/.\n\nThis is the first time I hear about that, I'll look into that.\nThe tipmost commit there is also shoddy, I'll redo that.\n"},{"id":"360364","messageId":"CAGZ79kavEzEJwYBtKSrEnraqZbYmYzQk9Nsaj+agPKzDZOjafA@mail.gmail.com","threadId":"49549","inReplyTo":"CAGZ79kZGEMWpr7aqeqXbC4bkmsCCiW+1pxbEV4T0vfsYG+_3iA@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2018-10-12T23:37:09Z","receivedAt":"2018-10-12T23:37:23Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Fri, Oct 12, 2018 at 12:44 PM Stefan Beller <sbeller@google.com> wrote:\n\n> > * sb/submodule-recursive-fetch-gets-the-tip (2018-10-11) 9 commits\n> >  . builtin/fetch: check for submodule updates for non branch fetches\n> >  . fetch: retry fetching submodules if needed objects were not fetched\n> >  . submodule: fetch in submodules git directory instead of in worktree\n> >  . repository: repo_submodule_init to take a submodule struct\n> >  . submodule.c: do not copy around submodule list\n> >  . submodule.c: 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> >\n> >  \"git fetch --recurse-submodules\" may not fetch the necessary commit\n> >  that is bound to the superproject, which is getting corrected.\n> >\n> >  Ejected for now, as it has fallouts in places like t/helper/.\n>\n> This is the first time I hear about that, I'll look into that.\n\nI looked into that, and merging with origin/next only\nhad one merge conflict in submodule.c, easy to resolve.\nIt builds and tests cleanly after that.\n\nWhat fallouts did you observe?\n\n> The tipmost commit there is also shoddy, I'll redo that.\n\nI confused this series with sb/submodule-update-in-C\nwhich got merged already, I may send a fixup commit there.\n\nSo it seems to me that this series is still good.\n\nThanks,\nStefan\n"},{"id":"360375","messageId":"xmqqd0seac9y.fsf@gitster-ct.c.googlers.com","threadId":"49549","inReplyTo":"CAGZ79kZGEMWpr7aqeqXbC4bkmsCCiW+1pxbEV4T0vfsYG+_3iA@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-10-13T01:03:21Z","receivedAt":"2018-10-13T01:03:26Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller <sbeller@google.com> writes:\n\n>> * sb/submodule-recursive-fetch-gets-the-tip (2018-10-11) 9 commits\n>>  . builtin/fetch: check for submodule updates for non branch fetches\n>>  . fetch: retry fetching submodules if needed objects were not fetched\n>>  . submodule: fetch in submodules git directory instead of in worktree\n>>  . repository: repo_submodule_init to take a submodule struct\n>>  . submodule.c: do not copy around submodule list\n>>  . submodule.c: 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>>\n>>  \"git fetch --recurse-submodules\" may not fetch the necessary commit\n>>  that is bound to the superproject, which is getting corrected.\n>>\n>>  Ejected for now, as it has fallouts in places like t/helper/.\n>\n> This is the first time I hear about that, I'll look into that.\n> The tipmost commit there is also shoddy, I'll redo that.\n\nThis is the first time I saw the breakage with this series, but I\nwould not be suprised, as this was rerolled recently.  Who knows\nwhat got changed in this series and in other topics---any new\ninteraction can arise and that is a normal part of distributed\ndevelopment.\n\nThe xx/sb-submodule-recursive-fetch-gets-the-tip-in-pu branch at\ngit://github.com/gitster/git.git has a merge of this into 'pu', with\ntextual conflicts all resolved.  \n\nAt least t/helper/test-submodule-nested-repo-config.c fails to\nbuild; I didn't check if there are other breakages.\n\n"},{"id":"360547","messageId":"CAGZ79kYQfSHkk-cxxL5zz3sj6gYigPFw4zccA46poOYY-fBeSA@mail.gmail.com","threadId":"49549","inReplyTo":"xmqqd0seac9y.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2018-10-15T17:32:30Z","receivedAt":"2018-10-15T17:32:45Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"On Fri, Oct 12, 2018 at 6:03 PM Junio C Hamano <gitster@pobox.com> wrote:\n>\n> Stefan Beller <sbeller@google.com> writes:\n>\n> >> * sb/submodule-recursive-fetch-gets-the-tip (2018-10-11) 9 commits\n> >>  . builtin/fetch: check for submodule updates for non branch fetches\n> >>  . fetch: retry fetching submodules if needed objects were not fetched\n> >>  . submodule: fetch in submodules git directory instead of in worktree\n> >>  . repository: repo_submodule_init to take a submodule struct\n> >>  . submodule.c: do not copy around submodule list\n> >>  . submodule.c: 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> >>\n> >>  \"git fetch --recurse-submodules\" may not fetch the necessary commit\n> >>  that is bound to the superproject, which is getting corrected.\n> >>\n> >>  Ejected for now, as it has fallouts in places like t/helper/.\n> >\n> > This is the first time I hear about that, I'll look into that.\n> > The tipmost commit there is also shoddy, I'll redo that.\n>\n> This is the first time I saw the breakage with this series, but I\n> would not be suprised, as this was rerolled recently.  Who knows\n> what got changed in this series and in other topics---any new\n> interaction can arise and that is a normal part of distributed\n> development.\n\nI performed the same merge last week, and the range-diff indicates,\nthat your version of next was further ahead than mine.\n\nThe breakage itself comes from\n\nt/helper/test-submodule-nested-repo-config.c: In function\n‘cmd__submodule_nested_repo_config’:\nt/helper/test-submodule-nested-repo-config.c:20:54: warning: passing\nargument 3 of ‘repo_submodule_init’ from incompatible pointer type\n[-Wincompatible-pointer-types]\n  if (repo_submodule_init(&submodule, the_repository, argv[1])) {\n                                                      ^~~~\nIn file included from ./cache.h:17:0,\n                 from ./submodule-config.h:4,\n                 from t/helper/test-submodule-nested-repo-config.c:2:\n./repository.h:126:5: note: expected ‘const struct submodule *’ but\nargument is of type ‘const char *’\n int repo_submodule_init(struct repository *subrepo,\n     ^~~~~~~~~~~~~~~~~~~\n\nwhich is introduced by\ncommit c369da44610acc5e56213b8784d4250ae619fdb9\n  (origin/ao/submodule-wo-gitmodules-checked-out)\nAuthor: Antonio Ospite <ao2@ao2.it>\nDate:   2018-10-05 15:06\n\n    t/helper: add test-submodule-nested-repo-config\n\n    Add a test tool to exercise config_from_gitmodules(), in particular for\n    the case of nested submodules.\n\n    Add also a test to document that reading the submoudles config of nested\n    submodules does not work yet when the .gitmodules file is not in the\n    working tree but it still in the index.\n\n    This is because the git API does not always make it possible access the\n    object store  of an arbitrary repository (see get_oid() usage in\n    config_from_gitmodules()).\n\n    When this git limitation gets fixed the aforementioned use case will be\n    supported too.\n\n    Signed-off-by: Antonio Ospite <ao2@ao2.it>\n    Signed-off-by: Junio C Hamano <gitster@pobox.com>\n\nMy resend will take that into account, building on Antonios series.\n\nThanks,\nStefan\n"},{"id":"360672","messageId":"20181016212008.GA249669@google.com","threadId":"49549","inReplyTo":"xmqqh8hr9pxb.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Josh Steadmon","fromEmail":"steadmon@google.com","sentAt":"2018-10-16T21:20:08Z","receivedAt":"2018-10-16T21:20:17Z","isPatch":false,"sender":{"key":"steadmon@google.com","avatar":"https://avatars.githubusercontent.com/u/2654920?v=4"},"body":"On 2018.10.12 23:53, Junio C Hamano wrote:\n> * js/remote-archive-v2 (2018-09-28) 4 commits\n>   (merged to 'next' on 2018-10-12 at 5f34377f60)\n>  + archive: allow archive over HTTP(S) with proto v2\n>  + archive: implement protocol v2 archive command\n>  + archive: use packet_reader for communications\n>  + archive: follow test standards around assertions\n> \n>  The original implementation of \"git archive --remote\" more or less\n>  bypassed the transport layer and did not work over http(s).  The\n>  version 2 of the protocol is defined to allow going over http(s) as\n>  well as Git native transport.\n> \n>  Will merge to 'master'.\n\nThe first two patches (test cleanup and packet_reader refactor) are OK,\nbut the latter two will break the archive command when an old client\nattempts to talk to a new server (due to the version advertisement\nproblem noted in [1]). Sorry that I didn't catch that these had made it\ninto next.\n\n[1]: https://public-inbox.org/git/20180927223314.GA230445@google.com/#t\n"},{"id":"360693","messageId":"xmqqftx5xz9p.fsf@gitster-ct.c.googlers.com","threadId":"49549","inReplyTo":"20181016212008.GA249669@google.com","subject":"Re: What's cooking in git.git (Oct 2018, #02; Sat, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-10-16T23:11:46Z","receivedAt":"2018-10-16T23:11:51Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Josh Steadmon <steadmon@google.com> writes:\n\n> The first two patches (test cleanup and packet_reader refactor) are OK,\n> but the latter two will break the archive command when an old client\n> attempts to talk to a new server (due to the version advertisement\n> problem noted in [1]). Sorry that I didn't catch that these had made it\n> into next.\n\nThanks.  Will hld.\n"}]}