{"thread":{"id":"49613","subject":"What's cooking in git.git (Oct 2018, #04; Fri, 19)","startedAt":"2018-10-19T06:02:31Z","lastAt":"2018-10-22T21:38:51Z","messageCount":8,"participants":["Junio C Hamano","SZEDER Gábor","Derrick Stolee","Johannes Sixt","Alban Gruin","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"360902","messageId":"xmqq8t2u1nkh.fsf@gitster-ct.c.googlers.com","threadId":"49613","inReplyTo":null,"subject":"What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-10-19T06:02:22Z","receivedAt":"2018-10-19T06:02:31Z","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\nTwo large set of topics on \"rebase in C\" and \"rebase -i in C\" are\nnow in 'next'.  A handful of improvements around \"git help\", \"git\ngrep\", and \"git status\" are in 'master', as well as other internal\nchanges.\n\nNote that \"cf. <msg-id>\" are not meant as \"clear all of these and\nyou are home free\".  They are primarily to prevent me from merging\ntopics that are not ready to 'next' by mistake and remind me where\nto go back to read the last state of the discussion in the archive.\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[Graduated to \"master\"]\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\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\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\n Code clean-up.\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\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\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\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\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\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\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\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\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\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\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\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\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\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\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--------------------------------------------------\n[New Topics]\n\n* ab/reject-alias-loop (2018-10-19) 1 commit\n - alias: detect loops in mixed execution mode\n\n Two (or more) aliases that mutually refer to each other can form an\n infinite loop; we now attempt to notice and stop.\n\n Will merge to 'next'.\n\n\n* cb/remove-dead-init (2018-10-19) 2 commits\n - multi-pack-index: avoid dead store for struct progress\n - unpack-trees: avoid dead store for struct progress\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* ds/ci-commit-graph-and-midx (2018-10-19) 1 commit\n - ci: add optional test variables\n\n One of our CI tests to run with \"unusual/experimental/random\"\n settings now also uses commti-graph and midx.\n\n Will merge to 'next'.\n\n\n* ds/reachable-first-parent-fix (2018-10-19) 1 commit\n - commit-reach: fix first-parent heuristic\n\n Correct performance regression in commit ancestry computation when\n generation numbers are involved.\n\n Will merge to 'next'.\n\n\n* jk/test-tool-help (2018-10-18) 1 commit\n - test-tool: show tool list on error\n\n Developer support.\n\n Will merge to 'next'.\n\n\n* jk/unused-function (2018-10-19) 1 commit\n - config.mak.dev: enable -Wunused-function\n\n Developer support.\n\n Will merge to 'next'.\n\n\n* js/diff-notice-has-drive-prefix (2018-10-19) 1 commit\n - diff: don't attempt to strip prefix from absolute Windows paths\n\n \"git diff /a/b/c /a/d/f\" noticed these are full paths with shared\n leading prefix \"/a\", but failed to notice a similar fact about \"git\n diff D:/a/b/c D:/a/d/f\", which has been corrected.\n\n Want tests.\n\n\n* js/pack-objects-mutex-init-fix (2018-10-19) 3 commits\n - pack-objects (mingw): initialize `packing_data` mutex in the correct spot\n - pack-objects (mingw): demonstrate a segmentation fault with large deltas\n - pack-objects: fix typo 'detla' -> 'delta'\n\n A mutex used in \"git pack-objects\" were not correctly initialized\n and this caused \"git repack\" to dump core on Windows.\n\n Will merge to 'next'.\n\n\n* jt/upload-pack-v2-fix-shallow (2018-10-19) 3 commits\n - upload-pack: clear flags before each v2 request\n - upload-pack: make want_obj not global\n - upload-pack: make have_obj not global\n\n \"git fetch\" over protocol v2 into a shallow repository failed to\n fetch full history behind a new tip of history that was diverged\n before the cut-off point of the history that was previously fetched\n shallowly.\n\n Will merge to 'next'.\n\n\n* pw/am-rebase-read-author-script (2018-10-19) 5 commits\n - sequencer: use read_author_script()\n - add read_author_script() to libgit\n - am: rename read_author_script()\n - am: improve author-script error reporting\n - am: don't die in read_author_script()\n\n\n* rj/header-guards (2018-10-18) 1 commit\n - headers: normalize the spelling of some header guards\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* sb/submodule-helper-remove-cruft (2018-10-18) 1 commit\n - builtin/submodule--helper: remove debugging leftover tracing\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* sb/submodule-url-to-absolute (2018-10-18) 1 commit\n - submodule helper: convert relative URL to absolute URL if needed\n\n Some codepaths failed to form a proper URL when .gitmodules record\n the URL to a submodule repository as relative to the repository of\n superproject, which has been corrected.\n\n Will merge to 'next'.\n\n\n* sg/doc-show-branch-typofix (2018-10-18) 1 commit\n - doc: fix small typo in git show-branch\n\n Docfix.\n\n Will merge to 'next'.\n\n\n* tq/branch-create-wo-branch-get (2018-10-18) 1 commit\n - builtin/branch.c: remove useless branch_get\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* tq/branch-style-fix (2018-10-18) 1 commit\n - branch: trivial style fix\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* jc/receive-deny-current-branch-fix (2018-10-19) 1 commit\n - receive: denyCurrentBranch=updateinstead should not blindly update\n\n The receive.denyCurrentBranch=updateInstead codepath kicked in even\n when the push should have been rejected due to other reasons, such\n as it does not fast-forward or the update-hook rejects it, which\n has been corrected.\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* en/merge-cleanup-more (2018-10-18) 2 commits\n - merge-recursive: avoid showing conflicts with merge branch before HEAD\n - merge-recursive: improve auto-merging messages with path collisions\n\n Further clean-up of merge-recursive machinery.\n\n Will merge to 'next'.\n\n\n* ld/p4-unshelve (2018-10-16) 3 commits\n  (merged to 'next' on 2018-10-19 at 35339798f9)\n + git-p4: fully support unshelving changelists\n + git-p4: unshelve into refs/remotes/p4-unshelved, not refs/remotes/p4/unshelved\n + git-p4: do not fail in verbose mode for missing 'fileSize' key\n\n \"git p4 unshelve\" improvements.\n\n Will merge to 'master'.\n\n\n* mg/gpg-parse-tighten (2018-10-15) 1 commit\n - gpg-interface.c: detect and reject multiple signatures on commits\n\n Detect and reject a signature block that has more than one GPG\n signature.\n\n Expecting another round of polishing, which is promising.\n cf. <1539636266.1014.6.camel@gentoo.org>\n\n\n* bc/sha-256 (2018-10-16) 14 commits\n - SQUASH???\n - commit-graph: specify OID version for SHA-256\n - hash: add an SHA-256 implementation using OpenSSL\n - sha256: add an SHA-256 implementation using libgcrypt\n - Add a base implementation of SHA-256 support\n - commit-graph: convert to using the_hash_algo\n - t/helper: add a test helper to compute hash speed\n - sha1-file: add a constant for hash block size\n - t: make the sha1 test-tool helper generic\n - t: add basic tests for our SHA-1 implementation\n - cache: make hashcmp and hasheq work with larger hashes\n - hex: introduce functions to print arbitrary hashes\n - sha1-file: provide functions to look up hash algorithms\n - sha1-file: rename algorithm to \"sha1\"\n\n Add sha-256 hash and plug it through the code to allow building Git\n with the \"NewHash\".\n\n\n* js/mingw-default-ident (2018-10-16) 3 commits\n  (merged to 'next' on 2018-10-19 at 2f710c811b)\n + mingw: use domain information for default email\n + getpwuid(mingw): provide a better default for the user name\n + getpwuid(mingw): initialize the structure only once\n\n The logic to select the default user name and e-mail on Windows has\n been improved.\n\n Will merge to 'master'.\n\n\n* js/mingw-http-ssl (2018-10-16) 3 commits\n - http: when using Secure Channel, ignore sslCAInfo by default\n - http: add support for disabling SSL revocation checks in cURL\n - http: add support for selecting SSL backends at runtime\n\n On Windows with recent enough cURL library, the configuration\n variable http.sslBackend can be used to choose between OpenSSL and\n Secure Channel at runtime as the SSL backend while talking over\n the HTTPS protocol.\n\n Almost there.\n cf. <CAPig+cQFb3S0Lm+huUZDN4aw9rWwinh0iZp12ss1zVKpJ=2MdA@mail.gmail.com>\n\n\n* js/vsts-ci (2018-10-16) 13 commits\n - travis: fix skipping tagged releases\n - README: add a build badge (status of the Azure Pipelines build)\n - tests: record more stderr with --write-junit-xml in case of failure\n - tests: include detailed trace logs with --write-junit-xml upon failure\n - git-p4: use `test_atexit` to kill the daemon\n - git-daemon: use `test_atexit` in the tests\n - tests: introduce `test_atexit`\n - ci: add a build definition for Azure DevOps\n - ci/lib.sh: add support for Azure Pipelines\n - tests: optionally write results as JUnit-style .xml\n - test-date: add a subcommand to measure times in shell scripts\n - ci/lib.sh: encapsulate Travis-specific things\n - ci: rename the library of common functions\n\n Prepare to run test suite on Azure DevOps.\n\n On hold, monitoring the discussion.\n cf. <nycvar.QRO.7.76.6.1810151657080.4546@tvgsbejvaqbjf.bet>\n\n\n* ab/gc-doc-update (2018-10-11) 1 commit\n  (merged to 'next' on 2018-10-19 at 84347d36a7)\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 Will merge to 'master'.\n\n\n* du/branch-show-current (2018-10-16) 2 commits\n - SQUASH???\n - branch: introduce --show-current display option\n\n \"git branch\" learned a new subcommand \"--show-current\".\n\n Waiting for ack(s) on SQUASH??? fixup.\n cf. <xmqqk1mizb1c.fsf@gitster-ct.c.googlers.com>\n\n\n* du/rev-parse-is-plumbing (2018-10-11) 1 commit\n  (merged to 'next' on 2018-10-16 at 50aca759e2)\n + doc: move git-rev-parse from porcelain to plumbing\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* mm/doc-no-dashed-git (2018-10-11) 1 commit\n  (merged to 'next' on 2018-10-16 at c26bcd740b)\n + doc: fix a typo and clarify a sentence\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ot/ref-filter-plug-leaks (2018-10-19) 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 Will merge to 'next'.\n\n\n* rv/send-email-cc-misc-by (2018-10-16) 3 commits\n  (merged to 'next' on 2018-10-19 at 2c1e5f75af)\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 Will merge to 'master'.\n\n\n* du/cherry-is-plumbing (2018-10-12) 1 commit\n  (merged to 'next' on 2018-10-19 at ffe6c5449e)\n + doc: move git-cherry to plumbing\n\n Doc update to mark \"git cherry\" as a plumbing command.\n\n Will merge to 'master'.\n\n\n* du/get-tar-commit-id-is-plumbing (2018-10-12) 1 commit\n  (merged to 'next' on 2018-10-16 at 23e7f018cd)\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 'master'.\n\n\n* lm/range-diff-submodule-fix (2018-10-12) 1 commit\n  (merged to 'next' on 2018-10-16 at 3e50d7a73b)\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 'master'.\n\n\n* sb/diff-emit-line-ws-markup-cleanup (2018-10-12) 1 commit\n  (merged to 'next' on 2018-10-16 at a68a6f0003)\n + diff.c: pass sign_index to emit_line_ws_markup\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* mk/use-size-t-in-zlib (2018-10-15) 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* jc/rebase-in-c-5-test-typofix (2018-10-11) 1 commit\n  (merged to 'next' on 2018-10-19 at 08c9d86ffd)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* jc/war-on-string-list (2018-10-19) 1 commit\n - fetch: replace string-list used as a look-up table with a hashmap\n\n Replace three string-list instances used as look-up tables in \"git\n fetch\" with hashmaps.\n\n\n* js/rebase-i-break (2018-10-12) 2 commits\n  (merged to 'next' on 2018-10-19 at 6db9b14495)\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 'master' 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 hold.\n cf. <20181016212008.GA249669@google.com>\n Breaks interoperability.\n\n\n* ma/t7005-bash-workaround (2018-09-28) 1 commit\n  (merged to 'next' on 2018-10-16 at 543e1e6574)\n + t7005-editor: quote filename to fix whitespace-issue\n\n Test fix.\n\n Will merge to 'master'.\n\n\n* rv/alias-help (2018-10-11) 3 commits\n  (merged to 'next' on 2018-10-19 at f16bbfb3ef)\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.\n\n This could be misleading for those who alias a command with options\n (e.g. with \"[alias] cpn = cherry-pick -n\", \"git cpn --help\" would\n show the manual of \"cherry-pick\", and the reader would not be told\n to pay close attention to the part that describes the \"--no-commit\"\n option until closing the pager that showed the contents of the\n manual, if the pager is configured to restore the original screen,\n or would not be told at all, if the pager simply makes the message\n on the standard error scroll away.\n\n Will merge to 'master'.\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 Will merge to 'next'.\n Further clean-up patches may need to follow soon before this\n change escapes to 'master'.\n\n\n* sg/split-index-racefix (2018-10-12) 7 commits\n  (merged to 'next' on 2018-10-16 at c1d6563c51)\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 '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  (merged to 'next' on 2018-10-16 at 81461ae0fd)\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 'master'.\n\n\n* bc/hash-transition-part-15 (2018-10-15) 15 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 constants\n - pack-bitmap-write: use GIT_MAX_RAWSZ for allocation\n - object_id.cocci: match only expressions of type 'struct object_id'\n\n More codepaths are moving away from hardcoded hash sizes.\n\n\n* ch/subtree-build (2018-10-18) 3 commits\n  (merged to 'next' on 2018-10-18 at f89fd5e6aa)\n + Revert \"subtree: make install targets depend on build targets\"\n  (merged to 'next' on 2018-10-16 at 919599cc37)\n + subtree: make install targets depend on build targets\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/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-15) 2 commits\n  (merged to 'next' on 2018-10-19 at 649a2bf1fb)\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 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 Expecting a reroll.\n cf. <CACsJy8DNHCxz1cL+6rFxnWvQQz3T7_j8+=5u8=CxjraRzM89mw@mail.gmail.com>\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  (merged to 'next' on 2018-10-16 at aa00813683)\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 'master'.\n\n\n* rs/subtree-fixes (2018-10-12) 5 commits\n  (merged to 'next' on 2018-10-16 at be23c9343c)\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* ds/reachable-topo-order (2018-10-18) 7 commits\n - t6012: make rev-list tests more interesting\n - revision.c: generation-based topo-order algorithm\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 cf. <pull.25.v4.git.gitgitgadget@gmail.com>\n\n\n* ds/coverage-diff (2018-10-10) 1 commit\n  (merged to 'next' on 2018-10-16 at c12b7279d9)\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 'master'.\n\n\n* sb/submodule-recursive-fetch-gets-the-tip (2018-10-18) 11 commits\n - fixup! repository: repo_submodule_init to take a submodule struct\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 - Merge branch 'ao/submodule-wo-gitmodules-checked-out' into sb/submodule-recursive-fetch-gets-the-tip\n (this branch uses ao/submodule-wo-gitmodules-checked-out.)\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\n* js/rebase-in-c-5.5-work-with-rebase-i-in-c (2018-10-11) 2 commits\n  (merged to 'next' on 2018-10-19 at b734d81006)\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 'master' 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 (this branch is used by sb/submodule-recursive-fetch-gets-the-tip.)\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-18) 10 commits\n  (merged to 'next' on 2018-10-18 at 55be2fded9)\n + list-objects: support for skipping tree traversal\n  (merged to 'next' on 2018-10-16 at 64d351d342)\n + filter-trees: code clean-up of tests\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  (merged to 'next' on 2018-10-19 at c2a0046e52)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-3-acts (2018-09-06) 7 commits\n  (merged to 'next' on 2018-10-19 at 9275677430)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-4-opts (2018-10-11) 18 commits\n  (merged to 'next' on 2018-10-19 at 2770b2f5d5)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-5-test (2018-10-11) 6 commits\n  (merged to 'next' on 2018-10-19 at 0e48bc8243)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c-6-final (2018-10-11) 1 commit\n  (merged to 'next' on 2018-10-19 at 52f1dadf69)\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 'master'.\n\n\n* ps/stash-in-c (2018-10-15) 21 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: 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: mention options in `show` synopsis\n - stash: add tests for `git stash show` config\n - stash: rename test cases to be more descriptive\n - t3903: modernize style\n - stash: improve option parsing test coverage\n - strbuf.c: add `strbuf_join_argv()`\n - sha1-name.c: add `get_oidf()` which acts like `get_oid()`\n\n \"git stash\" rewritten in C.\n\n Almost there.\n cf. <20181015221040.GD4883@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* ag/rebase-i-in-c (2018-10-09) 20 commits\n  (merged to 'next' on 2018-10-19 at 1b24712d46)\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 'master' together with pk/rebase-i-in-c-6-final.\n\n\n* pk/rebase-in-c (2018-08-06) 3 commits\n  (merged to 'next' on 2018-10-19 at 26c29ace53)\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 'master' together with pk/rebase-i-in-c-6-final.\n"},{"id":"360913","messageId":"20181019124625.GB30222@szeder.dev","threadId":"49613","inReplyTo":"xmqq8t2u1nkh.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2018-10-19T12:46:25Z","receivedAt":"2018-10-19T12:46:32Z","isPatch":false,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"On Fri, Oct 19, 2018 at 03:02:22PM +0900, Junio C Hamano wrote:\n> Two large set of topics on \"rebase in C\" and \"rebase -i in C\" are\n> now in 'next'.\n\nI see occasional failures in 't5520-pull.sh':\n\nexpecting success: \n\ttest_config rebase.autostash true &&\n\ttest_pull_autostash --rebase\n\n+ test_config rebase.autostash true\n+ config_dir=\n+ test rebase.autostash = -C\n+ test_when_finished test_unconfig  'rebase.autostash'\n+ test 0 = 0\n+ test_cleanup={ test_unconfig  'rebase.autostash'\n\t\t} && (exit \"$eval_ret\"); eval_ret=$?; :\n+ git config rebase.autostash true\n+ test_pull_autostash --rebase\n+ git reset --hard before-rebase\nHEAD is now at 12212b3 new file\n+ echo dirty\n+ git add new_file\n+ git pull --rebase . copy\nFrom .\n * branch            copy       -> FETCH_HEAD\nCreated autostash: 5417697\nHEAD is now at 12212b3 new file\nFirst, rewinding head to replay your work on top of it...\nApplying: new file\nApplying autostash resulted in conflicts.\nYour changes are safe in the stash.\nYou can run \"git stash pop\" or \"git stash drop\" at any time.\n+ test_cmp_rev HEAD^ copy\n+ git rev-parse --verify HEAD^\n+ git rev-parse --verify copy\n+ test_cmp expect.rev actual.rev\n+ diff -u expect.rev actual.rev\n+ cat new_file\ncat: new_file: No such file or directory\n+ test  = dirty\nerror: last command exited with $?=1\nnot ok 25 - pull --rebase succeeds with dirty working directory and rebase.autostash set\n\nWhen running t5520 in a loop, it tends to fail between 10-40\niterations, even when the machine is not under heavy load.\n\nIt appears that these failures started with commit 5541bd5b8f (rebase:\ndefault to using the builtin rebase, 2018-08-08), i.e. tip of\n'pk/rebase-in-c-6-final', but it's a \"flip the big switch\" commit, so\nnot very useful...\n\n"},{"id":"360914","messageId":"529dfd89-a5c9-fb14-fe9c-bb2fd78a59fd@gmail.com","threadId":"49613","inReplyTo":"xmqq8t2u1nkh.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2018-10-19T13:40:20Z","receivedAt":"2018-10-19T13:40:24Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 10/19/2018 2:02 AM, Junio C Hamano wrote:\n>\n> * ds/ci-commit-graph-and-midx (2018-10-19) 1 commit\n>   - ci: add optional test variables\n>\n>   One of our CI tests to run with \"unusual/experimental/random\"\n>   settings now also uses commti-graph and midx.\n>\n>   Will merge to 'next'.\n\ns/commti-graph/commit-graph/\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\nA reroll exists: \nhttps://public-inbox.org/git/pull.27.v2.git.gitgitgadget@gmail.com/\n\nThanks,\n-Stolee\n\n"},{"id":"360963","messageId":"2893a226-cbde-9ca6-fa97-cfc981bd57bd@kdbg.org","threadId":"49613","inReplyTo":"xmqq8t2u1nkh.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2018-10-19T17:55:09Z","receivedAt":"2018-10-19T17:55:13Z","isPatch":false,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Am 19.10.18 um 08:02 schrieb Junio C Hamano:\n> * js/diff-notice-has-drive-prefix (2018-10-19) 1 commit\n>   - diff: don't attempt to strip prefix from absolute Windows paths\n> \n>   \"git diff /a/b/c /a/d/f\" noticed these are full paths with shared\n>   leading prefix \"/a\", but failed to notice a similar fact about \"git\n>   diff D:/a/b/c D:/a/d/f\", which has been corrected.\n\nThis patch isn't about a misdetected leading prefix, but about\nincorrectly truncated absolute paths. How about:\n\n   Under certain circumstances, \"git diff D:/a/b/c D:/a/b/d\" on\n   Windows would strip initial parts from the paths because they\n   were not recognized as absolute, which has been corrected.\n\n>   Want tests.\n\nI've sent v2 with a test.\n\n-- Hannes\n"},{"id":"360965","messageId":"6e4fdcdd-6c50-a07e-74b7-1fcd15c33444@gmail.com","threadId":"49613","inReplyTo":"20181019124625.GB30222@szeder.dev","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Alban Gruin","fromEmail":"alban.gruin@gmail.com","sentAt":"2018-10-19T18:05:04Z","receivedAt":"2018-10-19T18:05:16Z","isPatch":false,"sender":{"key":"alban.gruin@gmail.com","avatar":"https://avatars.githubusercontent.com/u/6310153?v=4"},"body":"Hi,\n\nLe 19/10/2018 à 14:46, SZEDER Gábor a écrit :\n> On Fri, Oct 19, 2018 at 03:02:22PM +0900, Junio C Hamano wrote:\n>> Two large set of topics on \"rebase in C\" and \"rebase -i in C\" are\n>> now in 'next'.\n> \n> I see occasional failures in 't5520-pull.sh':\n> \n> […]\n>\n> When running t5520 in a loop, it tends to fail between 10-40\n> iterations, even when the machine is not under heavy load.\n> \n> It appears that these failures started with commit 5541bd5b8f (rebase:\n> default to using the builtin rebase, 2018-08-08), i.e. tip of\n> 'pk/rebase-in-c-6-final', but it's a \"flip the big switch\" commit, so\n> not very useful...\n> \n\nI can reproduce this.\n\nI also tried to run this specific test under valgrind, and found out\nthat some cases I did not targeted with --valgrind-only failed.  The\nsame thing happens with t3404, which did not crash with valgrind before.\n\nHere is a log:\n\n> expecting success: \n>         HEAD=$(git rev-parse HEAD) &&\n>         set_fake_editor &&\n>         git rebase -i -p HEAD^ &&\n>         git update-index --refresh &&\n>         git diff-files --quiet &&\n>         git diff-index --quiet --cached HEAD -- &&\n>         test $HEAD = $(git rev-parse HEAD)\n> \n> +++ git rev-parse HEAD\n> ++ HEAD=d2d5ba71c6d0266f26238e804f77f026984ae0d9\n> ++ set_fake_editor\n> ++ write_script fake-editor.sh\n> ++ echo '#!/bin/sh'\n> ++ cat\n> ++ chmod +x fake-editor.sh\n> +++ pwd\n> ++ test_set_editor '/tmp/git-alban/trash directory.t3404-rebase-interactive/fake-editor.sh'\n> ++ FAKE_EDITOR='/tmp/git-alban/trash directory.t3404-rebase-interactive/fake-editor.sh'\n> ++ export FAKE_EDITOR\n> ++ EDITOR='\"$FAKE_EDITOR\"'\n> ++ export EDITOR\n> ++ git rebase -i -p 'HEAD^'\n> GIT_DIR='/tmp/git-alban/trash directory.t3404-rebase-interactive/.git'; state_dir='.git/rebase-merge'; upstream_name='HEAD^'; upstream='8f99a4f1fbbd214b25a070ad34ec5a8f833522cc'; head_name='refs/heads/branch1'; orig_head='d2d5ba71c6d0266f26238e804f77f026984ae0d9'; onto='8f99a4f1fbbd214b25a070ad34ec5a8f833522cc'; onto_name='HEAD^'; unset revisions; unset restrict_revision; GIT_QUIET=''; git_am_opt=''; verbose=''; diffstat=''; force_rebase=''; action=''; signoff=''; allow_rerere_autoupdate=''; keep_empty=''; autosquash=''; unset gpg_sign_opt; unset cmd; allow_empty_message='--allow-empty-message'; rebase_merges=''; rebase_cousins=''; unset strategy; unset strategy_opts; rebase_root=''; squash_onto=''; git_format_patch_opt=''; . git-sh-setup && . git-rebase--common && . git-rebase--preserve-merges && git_rebase__preserve_merges: line 0: .: git-rebase--common: file not found\n> error: last command exited with $?=1\n> not ok 25 - -p handles \"no changes\" gracefully\n> #\n> #               HEAD=$(git rev-parse HEAD) &&\n> #               set_fake_editor &&\n> #               git rebase -i -p HEAD^ &&\n> #               git update-index --refresh &&\n> #               git diff-files --quiet &&\n> #               git diff-index --quiet --cached HEAD -- &&\n> #               test $HEAD = $(git rev-parse HEAD)\n> #\n\nThis comes from 't3404-rebase-interactive.sh', with --valgrind-only set\nto '63'.\n\nCheers,\nAlban\n\n"},{"id":"361063","messageId":"5f0fb76f-0774-3545-38fc-b86137f747fa@gmail.com","threadId":"49613","inReplyTo":"6e4fdcdd-6c50-a07e-74b7-1fcd15c33444@gmail.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Alban Gruin","fromEmail":"alban.gruin@gmail.com","sentAt":"2018-10-20T14:31:59Z","receivedAt":"2018-10-20T14:32:16Z","isPatch":false,"sender":{"key":"alban.gruin@gmail.com","avatar":"https://avatars.githubusercontent.com/u/6310153?v=4"},"body":"Le 19/10/2018 à 20:05, Alban Gruin a écrit :\n> Le 19/10/2018 à 14:46, SZEDER Gábor a écrit :\n>> On Fri, Oct 19, 2018 at 03:02:22PM +0900, Junio C Hamano wrote:\n>>> Two large set of topics on \"rebase in C\" and \"rebase -i in C\" are\n>>> now in 'next'.\n>>\n>> I see occasional failures in 't5520-pull.sh':\n>>\n>> […]\n>>\n>> When running t5520 in a loop, it tends to fail between 10-40\n>> iterations, even when the machine is not under heavy load.\n>>\n>> It appears that these failures started with commit 5541bd5b8f (rebase:\n>> default to using the builtin rebase, 2018-08-08), i.e. tip of\n>> 'pk/rebase-in-c-6-final', but it's a \"flip the big switch\" commit, so\n>> not very useful...\n>>\n\nThe bug can be bisected by prepending the command chain of 5520.25 with\n`git config --bool rebase.usebuiltin true &&`.  Commit 7debdaa4ad\n(“builtin rebase: support `--autostash` option”) is the culprit, but\nsome commits that cause this test to fail with a different error (ie.\n“TODO”…) must be marked as good.\n\nThe error comes from the call to `git stash apply $stash_id' in\nbuiltin/rebase.c:261.  When $stash_id only contains decimals and no\nletters, git-stash tries to apply stash@{$stash_id}[0][1].  Thas was not\na real problem with the shell script, because it did not abbreviate the\nobject id of the stashed commit, so it was very unlikely that the oid\nwould contain only digits.  builtin/rebase.c shortens the oid[2], making\nthis problem more likely to occur.\n\nWe could add a switch to git-stash to tell it that the parameter is an\noid, not anything else.\n\n[0] https://github.com/git/git/blob/master/git-stash.sh#L514-L520\n[1] https://github.com/git/git/blob/pu/builtin/stash.c#L167-L168\n[2] https://github.com/git/git/blob/next/builtin/rebase.c#L1373\n\nCheers,\nAlban\n\n"},{"id":"361073","messageId":"xmqqmur8yyi3.fsf@gitster-ct.c.googlers.com","threadId":"49613","inReplyTo":"5f0fb76f-0774-3545-38fc-b86137f747fa@gmail.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-10-20T23:44:20Z","receivedAt":"2018-10-20T23:47:01Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alban Gruin <alban.gruin@gmail.com> writes:\n\n> The error comes from the call to `git stash apply $stash_id' in\n> builtin/rebase.c:261.  When $stash_id only contains decimals and no\n> letters, git-stash tries to apply stash@{$stash_id}[0][1].  Thas was not\n> a real problem with the shell script, because it did not abbreviate the\n> object id of the stashed commit, so it was very unlikely that the oid\n> would contain only digits.  builtin/rebase.c shortens the oid[2], making\n> this problem more likely to occur.\n\nOK, so make \"rebase in C\" a faithful conversion of the original, the\ncode that lead to builtin/rebase.c:261 must be fixed not to pass a\nshortened oid.  I suspect that internally it has a full object name,\nso not shortening would be the right thing anyway, so regaredless of\nthis bug, it probably makes sense to do the fix.\n\nBut as you said, even the scripted version that passed the full\nobject name had a (10/16^40) chance of using a 40-hex that only has\n[0-9] and caused the same breakage, so such a change to \"rebase in\nC\" is sweeping the age old bug under the same rug, in the context of\ndiscussing this particular bug.  \n\nI agree with you that it is a better fix to the problem to allow the\ncaller to say \"I am giving an oid---it may (or may not---I do not\neven bother to check) consist of only digits, but do not treat it as\nan index to the stash reflog.\"\n"},{"id":"361205","messageId":"nycvar.QRO.7.76.6.1810222306180.4546@tvgsbejvaqbjf.bet","threadId":"49613","inReplyTo":"xmqqmur8yyi3.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (Oct 2018, #04; Fri, 19)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-10-22T21:38:47Z","receivedAt":"2018-10-22T21:38:51Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Sun, 21 Oct 2018, Junio C Hamano wrote:\n\n> Alban Gruin <alban.gruin@gmail.com> writes:\n> \n> > The error comes from the call to `git stash apply $stash_id' in\n> > builtin/rebase.c:261.  When $stash_id only contains decimals and no\n> > letters, git-stash tries to apply stash@{$stash_id}[0][1].  Thas was not\n> > a real problem with the shell script, because it did not abbreviate the\n> > object id of the stashed commit, so it was very unlikely that the oid\n> > would contain only digits.  builtin/rebase.c shortens the oid[2], making\n> > this problem more likely to occur.\n> \n> OK, so make \"rebase in C\" a faithful conversion of the original, the\n> code that lead to builtin/rebase.c:261 must be fixed not to pass a\n> shortened oid.  I suspect that internally it has a full object name,\n> so not shortening would be the right thing anyway, so regaredless of\n> this bug, it probably makes sense to do the fix.\n> \n> But as you said, even the scripted version that passed the full\n> object name had a (10/16^40) chance of using a 40-hex that only has\n> [0-9] and caused the same breakage, so such a change to \"rebase in\n> C\" is sweeping the age old bug under the same rug, in the context of\n> discussing this particular bug.  \n> \n> I agree with you that it is a better fix to the problem to allow the\n> caller to say \"I am giving an oid---it may (or may not---I do not\n> even bother to check) consist of only digits, but do not treat it as\n> an index to the stash reflog.\"\n\nWe already have a way to say \"I am giving you an oid\": append `^0` to the\nhash.\n\nI implemented a fix, pushed it up to GitGitGadget, and will submit it\ntomorrow (after a fresh look, and after the build finished):\nhttps://github.com/gitgitgadget/git/pull/52\n\nCiao,\nDscho\n"}]}