{"thread":{"id":"47439","subject":"What's cooking in git.git (Dec 2017, #03; Wed, 13)","startedAt":"2017-12-13T23:00:18Z","lastAt":"2018-01-09T18:59:55Z","messageCount":15,"participants":["Junio C Hamano","Kaartic Sivaraam","Ævar Arnfjörð Bjarmason","Jonathan Nieder","Johannes Schindelin","Lars Schneider"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"334782","messageId":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":null,"subject":"What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-13T23:00:03Z","receivedAt":"2017-12-13T23:00:18Z","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\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* ab/pcre2-grep (2017-11-24) 2 commits\n  (merged to 'next' on 2017-12-05 at 88f1927207)\n + grep: fix segfault under -P + PCRE2 <=10.30 + (*NO_JIT)\n + test-lib: add LIBPCRE1 & LIBPCRE2 prerequisites\n\n \"git grep\" compiled with libpcre2 sometimes triggered a segfault,\n which is being fixed.\n\n\n* bc/hash-algo (2017-11-28) 5 commits\n  (merged to 'next' on 2017-11-28 at 5c74632345)\n + repository: fix a sparse 'using integer as NULL pointer' warning\n  (merged to 'next' on 2017-11-27 at 98cc9ef2a8)\n + Switch empty tree and blob lookups to use hash abstraction\n + Integrate hash algorithm support with repo setup\n + Add structure representing hash algorithm\n + setup: expose enumerated repo info\n\n An infrastructure to define what hash function is used in Git is\n introduced, and an effort to plumb that throughout various\n codepaths has been started.\n\n\n* ds/for-each-file-in-obj-micro-optim (2017-12-04) 1 commit\n  (merged to 'next' on 2017-12-05 at 55ba487ccd)\n + sha1_file: use strbuf_add() instead of strbuf_addf()\n\n The code to iterate over loose object files got optimized.\n\n\n* en/remove-stripspace (2017-12-05) 1 commit\n  (merged to 'next' on 2017-12-06 at c926373a49)\n + strbuf: remove unused stripspace function alias\n\n An internal function that was left for backward compatibility has\n been removed, as there is no remaining callers.\n\n\n* fk/sendmail-from-path (2017-11-28) 1 commit\n  (merged to 'next' on 2017-12-05 at deb7a5f4a8)\n + git-send-email: honor $PATH for sendmail binary\n\n \"git send-email\" tries to see if the sendmail program is available\n in /usr/lib and /usr/sbin; extend the list of locations to be\n checked to also include directories on $PATH.\n\n\n* jc/receive-pack-hook-doc (2017-11-24) 1 commit\n  (merged to 'next' on 2017-12-05 at ffa14b1d48)\n + hooks doc: clarify when receive-pack invokes its hooks\n\n Doc update.\n\n\n* jk/no-optional-locks (2017-11-27) 1 commit\n  (merged to 'next' on 2017-12-06 at e642dde605)\n + git-status.txt: mention --no-optional-locks\n\n Doc update for a feature available in Git v2.14 and upwards.\n\n\n* jk/progress-delay-fix (2017-12-04) 2 commits\n  (merged to 'next' on 2017-12-05 at 8e62c2b18b)\n + progress: drop delay-threshold code\n + progress: set default delay threshold to 100%, not 0%\n\n A regression in the progress eye-candy was fixed.\n\n\n* js/hashmap-update-sample (2017-12-05) 1 commit\n  (merged to 'next' on 2017-12-06 at 40ceee9a18)\n + hashmap: adjust documentation to reflect reality\n\n Code comment update.\n\n\n* ks/doc-checkout-previous (2017-11-28) 1 commit\n  (merged to 'next' on 2017-12-05 at 02f17e3c55)\n + Doc/checkout: checking out using @{-N} can lead to detached state\n\n @{-N} in \"git checkout @{-N}\" may refer to a detached HEAD state,\n but the documentation was not clear about it, which has been fixed.\n\n\n* pc/submodule-helper (2017-11-26) 1 commit\n  (merged to 'next' on 2017-12-05 at fdf56787be)\n + submodule--helper.c: i18n: add a missing space in message\n\n A message fix.\n\n\n* ra/decorate-limit-refs (2017-11-22) 1 commit\n  (merged to 'next' on 2017-12-05 at 02c66aa4e0)\n + log: add option to choose which refs to decorate\n\n The tagnames \"git log --decorate\" uses to annotate the commits can\n now be limited to subset of available refs with the two additional\n options, --decorate-refs[-exclude]=<pattern>.\n\n\n* tg/t-readme-updates (2017-11-27) 2 commits\n  (merged to 'next' on 2017-12-05 at c0b0e2d65b)\n + t/README: document test_cmp_rev\n + t/README: remove mention of adding copyright notices\n\n Developer doc updates.\n\n--------------------------------------------------\n[New Topics]\n\n* es/worktree-checkout-hook (2017-12-07) 1 commit\n - worktree: invoke post-checkout hook (unless --no-checkout)\n\n \"git worktree add\" learned to run the post-checkout hook, just like\n \"git checkout\" does, after the initial checkout.\n\n Will merge to 'next'.\n\n\n* rs/am-builtin-leakfix (2017-12-07) 1 commit\n - am: release strbuf after use in split_mail_mbox()\n\n Leakfix.\n\n Will merge to 'next'.\n\n\n* rs/fmt-merge-msg-string-leak-fix (2017-12-07) 1 commit\n - fmt-merge-msg: avoid leaking strbuf in shortlog()\n\n Leakfix.\n\n Will merge to 'next'.\n\n\n* rs/strbuf-read-once-reset-length (2017-12-07) 1 commit\n - strbuf: release memory on read error in strbuf_read_once()\n\n Leakfix.\n\n Will merge to 'next'.\n\n\n* db/doc-workflows-neuter-the-maintainer (2017-12-08) 1 commit\n - doc: reword gitworkflows.txt for neutrality\n\n Docfix.\n\n Will merge to 'next'.\n\n\n* es/clone-shared-worktree (2017-12-11) 1 commit\n - clone: support 'clone --shared' from a worktree\n\n \"git clone --shared\" to borrow from a (secondary) worktree did not\n work, even though \"git clone --local\" did.  Both are now accepted.\n\n Will merge to 'next'.\n\n\n* jh/partial-clone-doc (2017-12-08) 1 commit\n - partial-clone: design doc\n\n Will merge to 'next'.\n\n\n* jk/cvsimport-quoting (2017-12-08) 1 commit\n - cvsimport: apply shell-quoting regex globally\n\n Typo/Logico fix.\n\n Will merge to 'next'.\n\n\n* jk/test-suite-tracing (2017-12-08) 4 commits\n - t/Makefile: introduce TEST_SHELL_PATH\n - test-lib: make \"-x\" work with \"--verbose-log\"\n - t5615: avoid re-using descriptor 4\n - test-lib: silence \"-x\" cleanup under bash\n\n Assorted fixes around running tests with \"-x\" tracing option.\n\n\n* jt/decorate-api (2017-12-08) 1 commit\n - decorate: clean up and document API\n\n A few structures and variables that are implementation details of\n the decorate API have been renamed and then the API got documented\n better.\n\n Will merge to 'next'.\n\n\n* ks/branch-cleanup (2017-12-07) 4 commits\n - builtin/branch: strip refs/heads/ using skip_prefix\n - branch: update warning message shown when copying a misnamed branch\n - branch: group related arguments of create_branch()\n - branch: improve documentation and naming of create_branch() parameters\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* rs/fmt-merge-msg-leakfix (2017-12-08) 1 commit\n - transport-helper: plug strbuf and string_list leaks\n\n Leakfix.\n\n Will merge to 'next'.\n\n\n* tb/delimit-pretty-trailers-args-with-comma (2017-12-08) 1 commit\n - docs/pretty-formats: mention commas in %(trailers) syntax\n\n Doc updates.\n\n Will merge to 'next'.\n\n\n* bw/submodule-sans-cache-compat (2017-12-12) 3 commits\n - submodule: convert get_next_submodule to not rely on the_index\n - submodule: used correct index in is_staging_gitmodules_ok\n - submodule: convert stage_updated_gitmodules to take a struct index_state\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* hi/merge-verify-sig-config (2017-12-12) 2 commits\n - t: add tests for pull --verify-signatures\n - merge: add config option for verifySignatures\n\n \"git merge\" learned to pay attention to merge.verifySignatures\n configuration variable and pretend as if '--verify-signatures'\n option was given from the command line.\n\n Will merge to 'next'.\n\n\n* ot/pretty (2017-12-12) 2 commits\n - format: create docs for pretty.h\n - format: create pretty.h file\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* sb/test-helper-excludes (2017-12-12) 1 commit\n - t/helper: ignore everything but sources\n\n Simplify the ignore rules for t/helper directory.\n\n Will merge to 'next'.\n\n\n* sg/travis-fixes (2017-12-12) 4 commits\n - travis-ci: use 'set -x' in 'ci/*' scripts for extra tracing output\n - travis-ci: set GIT_TEST_HTTPD in 'ci/lib-travisci.sh'\n - travis-ci: move setting environment variables to 'ci/lib-travisci.sh'\n - travis-ci: introduce a $jobname variable for 'ci/*' scripts\n\n Assorted updates for TravisCI integration.\n\n Will merge to 'next'.\n\n\n* bw/path-doc (2017-12-13) 1 commit\n - path: document path functions\n\n Doc updates.\n\n Will merge to 'next'.\n\n\n* jt/transport-no-more-rsync (2017-12-12) 1 commit\n - transport: remove unused \"push\" in vtable\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* rb/quick-install-doc (2017-12-12) 1 commit\n - install-doc-quick: allow specifying what ref to install\n\n The build procedure now allows not just the repositories but also\n the refs to be used to take pre-formatted manpages and html\n documents to install.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* jc/merge-symlink-ours-theirs (2017-09-26) 1 commit\n - merge: teach -Xours/-Xtheirs to symbolic link merge\n\n \"git merge -Xours/-Xtheirs\" learned to use our/their version when\n resolving a conflicting updates to a symbolic link.\n\n Needs review.\n\n\n* mg/merge-base-fork-point (2017-09-17) 3 commits\n - merge-base: find fork-point outside partial reflog\n - merge-base: return fork-point outside reflog\n - t6010: test actual test output\n\n \"merge-base --fork-point $branch $commit\" is used to guess on which\n commit among the commits that were once at the tip of the $branch the\n $commit was built on top of, and it learns these historical tips from\n the reflog of the $branch.  When the true fork-point is lost due to\n pruning of old reflog entries, the command does not give any output,\n because it has no way to guess correctly and does not want to mislead\n the user with a wrong guess.\n\n The command has been updated to give the best but not known to be\n correct guess, based on a hope that a merge-base between $commit and a\n virtual merge across all the reflog entries that still are available\n for $branch may still be a closer to the true fork-point than the\n merge-base between $commit and the current tip of the $branch.\n\n This may have to be offered by an additional option, to allow the\n users that are prepared to see a potentially incorrect guess to opt\n into the feature, without affecting the current callers that may not\n be prepared to accept a guess that is not known to be correct.\n\n What's the doneness of this one?\n\n\n* jk/drop-ancient-curl (2017-08-09) 5 commits\n - http: #error on too-old curl\n - curl: remove ifdef'd code never used with curl >=7.19.4\n - http: drop support for curl < 7.19.4\n - http: drop support for curl < 7.16.0\n - http: drop support for curl < 7.11.1\n\n Some code in http.c that has bitrot is being removed.\n\n Expecting a reroll.\n\n\n* mk/use-size-t-in-zlib (2017-08-10) 1 commit\n . zlib.c: use size_t for size\n\n The wrapper to call into zlib followed our long tradition to use\n \"unsigned long\" for sizes of regions in memory, which have been\n updated to use \"size_t\".\n\n Needs resurrecting by making sure the fix is good and still applies\n (or adjusted to today's codebase).\n\n\n* mg/status-in-progress-info (2017-05-10) 2 commits\n - status --short --inprogress: spell it as --in-progress\n - status: show in-progress info for short status\n\n \"git status\" learns an option to report various operations\n (e.g. \"merging\") that the user is in the middle of.\n\n cf. <xmqqmvakcdqw.fsf@gitster.mtv.corp.google.com>\n\n\n* nd/worktree-move (2017-04-20) 6 commits\n - worktree remove: new command\n - worktree move: refuse to move worktrees with submodules\n - worktree move: accept destination as directory\n - worktree move: new command\n - worktree.c: add update_worktree_location()\n - worktree.c: add validate_worktree()\n\n \"git worktree\" learned move and remove subcommands.\n\n Expecting a reroll.\n cf. <20170420101024.7593-1-pclouds@gmail.com>\n cf. <20170421145916.mknekgqzhxffu7di@sigill.intra.peff.net>\n cf. <d0e81b1e-5869-299e-f462-4d43dc997bd1@ramsayjones.plus.com>\n\n--------------------------------------------------\n[Cooking]\n\n* bw/pathspec-match-submodule-boundary (2017-12-05) 1 commit\n  (merged to 'next' on 2017-12-13 at e256d292a4)\n + pathspec: only match across submodule boundaries when requested\n\n An v2.12-era regression in pathspec match logic, which made it look\n into submodule tree even when it is not desired, has been fixed.\n\n Will merge to 'master'.\n\n\n* ls/git-gui-no-double-utf8-author-name (2017-12-05) 2 commits\n  (merged to 'next' on 2017-12-13 at be577d6e1b)\n + Merge branch 'ls/no-double-utf8-author-name' of ../git-gui into ls/git-gui-no-double-utf8-author-name\n + git-gui: prevent double UTF-8 conversion\n\n Amending commits in git-gui broke the author name that is non-ascii\n due to incorrect enconding conversion.\n\n Will merge to 'master'.\n\n\n* sb/clone-recursive-submodule-doc (2017-12-05) 1 commit\n  (merged to 'next' on 2017-12-13 at abfed699db)\n + Documentation/git-clone: improve description for submodule recursing\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ab/sha1dc-build (2017-12-12) 4 commits\n - Makefile: use the sha1collisiondetection submodule by default\n - sha1dc_git.h: re-arrange an ifdef chain for a subsequent change\n - Makefile: under \"make dist\", include the sha1collisiondetection submodule\n - Makefile: don't error out under DC_SHA1_EXTERNAL if DC_SHA1_SUBMODULE=auto\n\n\n* bw/submodule-config-cleanup (2017-12-06) 1 commit\n  (merged to 'next' on 2017-12-13 at c952bf1b84)\n + diff-tree: read the index so attribute checks work in bare repositories\n\n Recent update to the submodule configuration code broke \"diff-tree\"\n by accidentally stopping to read from the index upfront.\n\n Will merge to 'master'.\n\n\n* sg/setup-doc-update (2017-12-07) 1 commit\n  (merged to 'next' on 2017-12-13 at 4355c6e0ef)\n + setup.c: fix comment about order of .git directory discovery\n\n Comment update.\n\n Will merge to 'master'.\n\n\n* ab/simplify-perl-makefile (2017-12-11) 1 commit\n  (merged to 'next' on 2017-12-13 at 1b791d2503)\n + Makefile: replace perl/Makefile.PL with simple make rules\n\n The build procedure for perl/ part has been greatly simplified by\n weaning ourselves off of MakeMaker.\n\n Will merge to 'master'.\n\n\n* cc/skip-to-optional-val (2017-12-11) 7 commits\n - t4045: reindent to make helpers readable\n - diff: add tests for --relative without optional prefix value\n - diff: use skip_to_optional_arg_default() in parsing --relative\n - diff: use skip_to_optional_arg_default()\n - diff: use skip_to_optional_arg()\n - index-pack: use skip_to_optional_arg()\n - git-compat-util: introduce skip_to_optional_arg()\n\n Introduce a helper to simplify code to parse a common pattern that\n expects either \"--key\" or \"--key=<something>\".\n\n\n* lb/rebase-i-short-command-names (2017-12-05) 9 commits\n - t3404: add test case for abbreviated commands\n - rebase -i: learn to abbreviate command names\n - rebase -i -x: add exec commands via the rebase--helper\n - rebase -i: update functions to use a flags parameter\n - rebase -i: replace reference to sha1 with oid\n - rebase -i: refactor transform_todo_ids\n - rebase -i: set commit to null in exec commands\n - Documentation: use preferred name for the 'todo list' script\n - Documentation: move rebase.* configs to new file\n\n Allow a single-letter command name in the \"rebase -i\" todo list.\n\n Will merge to 'next'.\n\n\n* ra/prompt-eread-fix (2017-12-06) 2 commits\n - git-prompt: fix reading files with windows line endings\n - git-prompt: make __git_eread intended use explicit\n\n Update the shell prompt script (in contrib/) to strip trailing CR\n from strings read from various \"state\" files.\n\n\n* en/merge-recursive-icase-removal (2017-11-27) 1 commit\n  (merged to 'next' on 2017-12-13 at 85c6538a2a)\n + merge-recursive: ignore_case shouldn't reject intentional removals\n\n The code internal to the recursive merge strategy was not fully\n prepared to see a path that is renamed to try overwriting another\n path that is only different in case on case insensitive systems.\n This does not matter in the current code, but will start to matter\n once the rename detection logic starts taking hints from nearby\n paths moving to some directory and moves a new path along with them.\n\n Will merge to 'master'.\n\n\n* gk/tracing-optimization (2017-12-06) 2 commits\n  (merged to 'next' on 2017-12-13 at d6bfac03ad)\n + trace: improve performance while category is disabled\n + trace: remove trace key normalization\n\n The tracing infrastructure has been optimized for cases where no\n tracing is requested.\n\n Will merge to 'master'.\n\n\n* tb/check-crlf-for-safe-crlf (2017-12-08) 2 commits\n  (merged to 'next' on 2017-12-08 at 5279b80103)\n + t0027: Adapt the new MIX tests to Windows\n  (merged to 'next' on 2017-12-05 at 7adaa1fe01)\n + convert: tighten the safe autocrlf handling\n\n The \"safe crlf\" check incorrectly triggered for contents that does\n not use CRLF as line endings, which has been corrected.\n\n\n* cc/require-tcl-tk-for-build (2017-11-29) 2 commits\n - travis-ci: avoid new tcl/tk build requirement\n - Makefile: check that tcl/tk is installed\n\n A first-time builder of Git may have installed neither tclsh nor\n msgfmt, in which case git-gui and gitk part will fail and break the\n build.  As a workaround, refuse to run a build when tclsh is not\n installed and NO_TCLTK is not set.\n\n Undecided.\n I still feel that requring tclsh to be installed, with or without\n \"escape hatch\" for experts, may be too heavy-handed.\n\n\n* mk/http-backend-content-length (2017-11-27) 4 commits\n - SQUASH???\n - t5560-http-backend-noserver.sh: add CONTENT_LENGTH cases\n - SQUASH???\n - http-backend: respect CONTENT_LENGTH as specified by rfc3875\n\n The http-backend (used for smart-http transport) used to slurp the\n whole input until EOF, without paying attention to CONTENT_LENGTH\n that is supplied in the environment and instead expecting the Web\n server to close the input stream.  This has been fixed.\n\n Expecting a reroll.\n Suggested fixes to be used when rerolling is queued, but I'd\n prefer _not_ squashing them myself.\n\n\n* jt/diff-anchored-patience (2017-11-28) 1 commit\n  (merged to 'next' on 2017-12-13 at 5f4843d7a0)\n + diff: support anchoring line(s)\n\n \"git diff\" learned a variant of the \"--patience\" algorithm, to\n which the user can specify which 'unique' line to be used as\n anchoring points.\n\n Will merge to 'master'.\n\n\n* en/rename-directory-detection (2017-12-13) 35 commits\n - SQUASH???\n - merge-recursive: ensure we write updates for directory-renamed file\n - merge-recursive: avoid spurious rename/rename conflict from dir renames\n - directory rename detection: new testcases showcasing a pair of bugs\n - merge-recursive: fix remaining directory rename + dirty overwrite cases\n - merge-recursive: fix overwriting dirty files involved in renames\n - merge-recursive: avoid clobbering untracked files with directory renames\n - merge-recursive: apply necessary modifications for directory renames\n - merge-recursive: when comparing files, don't include trees\n - merge-recursive: check for file level conflicts then get new name\n - merge-recursive: add computation of collisions due to dir rename & merging\n - merge-recursive: add a new hashmap for storing file collisions\n - merge-recursive: check for directory level conflicts\n - merge-recursive: add get_directory_renames()\n - merge-recursive: make a helper function for cleanup for handle_renames\n - merge-recursive: add a new hashmap for storing directory renames\n - merge-recursive: split out code for determining diff_filepairs\n - merge-recursive: make !o->detect_rename codepath more obvious\n - merge-recursive: fix leaks of allocated renames and diff_filepairs\n - merge-recursive: introduce new functions to handle rename logic\n - merge-recursive: move the get_renames() function\n - directory rename detection: tests for handling overwriting dirty files\n - directory rename detection: tests for handling overwriting untracked files\n - directory rename detection: miscellaneous testcases to complete coverage\n - directory rename detection: testcases exploring possibly suboptimal merges\n - directory rename detection: more involved edge/corner testcases\n - directory rename detection: testcases checking which side did the rename\n - directory rename detection: files/directories in the way of some renames\n - directory rename detection: partially renamed directory testcase/discussion\n - directory rename detection: testcases to avoid taking detection too far\n - directory rename detection: directory splitting testcases\n - directory rename detection: basic testcases\n - merge-recursive: add explanation for src_entry and dst_entry\n - merge-recursive: fix logic ordering issue\n - Tighten and correct a few testcases for merging and cherry-picking\n\n Rename detection logic in \"diff\" family that is used in \"merge\" has\n learned to guess when all of x/a, x/b and x/c have moved to z/a,\n z/b and z/c, it is likely that x/d added in the meantime would also\n want to move to z/d by taking the hint that the entire directory\n 'x' moved to 'z'.\n\n Needs review.\n\n\n* sb/diff-blobfind (2017-12-12) 1 commit\n  (merged to 'next' on 2017-12-13 at 9a27a20c5f)\n + diffcore: add a filter to find a specific blob\n\n \"git diff\" family of commands learned --blobfind=<object-name> that\n allows you to limit the output only to a change that involves the\n named blob object (either changing the contents from or to it).\n\n Will merge to 'master'.\n\n\n* tg/worktree-create-tracking (2017-12-06) 6 commits\n  (merged to 'next' on 2017-12-13 at 830b68c660)\n + add worktree.guessRemote config option\n + worktree: add --guess-remote flag to add subcommand\n + worktree: make add <path> <branch> dwim\n + worktree: add --[no-]track option to the add subcommand\n + worktree: add can be created from any commit-ish\n + checkout: factor out functions to new lib file\n\n The way \"git worktree add\" determines what branch to create from\n where and checkout in the new worktree has been updated a bit.\n\n Will merge to 'master'.\n\n\n* pw/sequencer-in-process-commit (2017-12-13) 10 commits\n  (merged to 'next' on 2017-12-13 at ec4d2b9c84)\n + sequencer: improve config handling\n  (merged to 'next' on 2017-12-06 at a4212f7ebd)\n + t3512/t3513: remove KNOWN_FAILURE_CHERRY_PICK_SEES_EMPTY_COMMIT=1\n + sequencer: try to commit without forking 'git commit'\n + sequencer: load commit related config\n + sequencer: simplify adding Signed-off-by: trailer\n + commit: move print_commit_summary() to libgit\n + commit: move post-rewrite code to libgit\n + Add a function to update HEAD after creating a commit\n + commit: move empty message checks to libgit\n + t3404: check intermediate squash messages\n\n The sequencer infrastructure is shared across \"git cherry-pick\",\n \"git rebase -i\", etc., and has always spawned \"git commit\" when it\n needs to create a commit.  It has been taught to do so internally,\n when able, by reusing the codepath \"git commit\" itself uses, which\n gives performance boost for a few tens of percents in some sample\n scenarios.\n\n Will cook in 'next'.\n\n\n* dj/runtime-prefix (2017-12-05) 4 commits\n . exec_cmd: RUNTIME_PREFIX on some POSIX systems\n . Makefile: add Perl runtime prefix support\n . Makefile: add support for \"perllibdir\"\n . Makefile: generate Perl header from template file\n\n A build-time option has been added to allow Git to be told to refer\n to its associated files relative to the main binary, in the same\n way that has been possible on Windows for quite some time, for\n Linux, BSDs and Darwin.\n\n Tentatively kicked out of 'next' to see how well another topic\n ab/simplify-perl-makefile that heavily conflicts with this fares.\n\n\n* ls/editor-waiting-message (2017-12-07) 2 commits\n  (merged to 'next' on 2017-12-13 at 494b5b41e3)\n + launch_editor(): indicate that Git waits for user input\n + refactor \"dumb\" terminal determination\n\n Git shows a message to tell the user that it is waiting for the\n user to finish editing when spawning an editor, in case the editor\n opens to a hidden window or somewhere obscure and the user gets\n lost.\n\n Will merge to 'master'.\n\n\n* en/rename-progress (2017-12-02) 5 commits\n  (merged to 'next' on 2017-12-04 at 49b39d2297)\n + diffcore-rename: make diff-tree -l0 mean -l<large>\n  (merged to 'next' on 2017-11-20 at 77a2e0ddd9)\n + sequencer: show rename progress during cherry picks\n + diff: remove silent clamp of renameLimit\n + progress: fix progress meters when dealing with lots of work\n + sequencer: warn when internal merge may be suboptimal due to renameLimit\n\n Historically, the diff machinery for rename detection had a\n hardcoded limit of 32k paths; this is being lifted to allow users\n trade cycles with a (possibly) easier to read result.\n\n Will merge to 'master'.\n\n\n* ar/unconfuse-three-dots (2017-12-06) 8 commits\n  (merged to 'next' on 2017-12-13 at 33bd0b67c0)\n + t2020: test variations that matter\n + t4013: test new output from diff --abbrev --raw\n + diff: diff_aligned_abbrev: remove ellipsis after abbreviated SHA-1 value\n + t4013: prepare for upcoming \"diff --raw --abbrev\" output format change\n + checkout: describe_detached_head: remove ellipsis after committish\n + print_sha1_ellipsis: introduce helper\n + Documentation: user-manual: limit usage of ellipsis\n + Documentation: revisions: fix typo: \"three dot\" ---> \"three-dot\" (in line with \"two-dot\").\n\n Ancient part of codebase still shows dots after an abbreviated\n object name just to show that it is not a full object name, but\n these ellipses are confusing to people who newly discovered Git\n who are used to seeing abbreviated object names and find them\n confusing with the range syntax.\n\n Will merge to 'master'.\n\n\n* jh/object-filtering (2017-12-05) 9 commits\n  (merged to 'next' on 2017-12-05 at 3a56b51085)\n + rev-list: support --no-filter argument\n + list-objects-filter-options: support --no-filter\n + list-objects-filter-options: fix 'keword' typo in comment\n  (merged to 'next' on 2017-11-27 at e5008c3b28)\n + pack-objects: add list-objects filtering\n + rev-list: add list-objects filtering support\n + list-objects: filter objects in traverse_commit_list\n + oidset: add iterator methods to oidset\n + oidmap: add oidmap iterator methods\n + dir: allow exclusions from blob in addition to file\n (this branch is used by jh/fsck-promisors and jh/partial-clone.)\n\n In preparation for implementing narrow/partial clone, the object\n walking machinery has been taught a way to tell it to \"filter\" some\n objects from enumeration.\n\n\n* jh/fsck-promisors (2017-12-08) 10 commits\n - gc: do not repack promisor packfiles\n - rev-list: support termination at promisor objects\n - sha1_file: support lazily fetching missing objects\n - introduce fetch-object: fetch one promisor object\n - index-pack: refactor writing of .keep files\n - fsck: support promisor objects as CLI argument\n - fsck: support referenced promisor objects\n - fsck: support refs pointing to promisor objects\n - fsck: introduce partialclone extension\n - extension.partialclone: introduce partial clone extension\n (this branch is used by jh/partial-clone; uses jh/object-filtering.)\n\n In preparation for implementing narrow/partial clone, the machinery\n for checking object connectivity used by gc and fsck has been\n taught that a missing object is OK when it is referenced by a\n packfile specially marked as coming from trusted repository that\n promises to make them available on-demand and lazily.\n\n\n* jh/partial-clone (2017-12-08) 13 commits\n - t5616: test bulk prefetch after partial fetch\n - fetch: inherit filter-spec from partial clone\n - t5616: end-to-end tests for partial clone\n - fetch-pack: restore save_commit_buffer after use\n - unpack-trees: batch fetching of missing blobs\n - clone: partial clone\n - partial-clone: define partial clone settings in config\n - fetch: support filters\n - fetch: refactor calculation of remote list\n - fetch-pack: test support excluding large blobs\n - fetch-pack: add --no-filter\n - fetch-pack, index-pack, transport: partial clone\n - upload-pack: add object filtering for partial clone\n (this branch uses jh/fsck-promisors and jh/object-filtering.)\n\n The machinery to clone & fetch, which in turn involves packing and\n unpacking objects, have been told how to omit certain objects using\n the filtering mechanism introduced by the jh/object-filtering\n topic, and also mark the resulting pack as a promisor pack to\n tolerate missing objects, taking advantage of the mechanism\n introduced by the jh/fsck-promisors topic.\n\n\n* sb/describe-blob (2017-12-08) 8 commits\n - SQUASH???\n - builtin/describe.c: describe a blob\n - builtin/describe.c: factor out describe_commit\n - builtin/describe.c: print debug statements earlier\n - builtin/describe.c: rename `oid` to avoid variable shadowing\n - revision.h: introduce blob/tree walking in order of the commits\n - list-objects.c: factor out traverse_trees_and_blobs\n - t6120: fix typo in test name\n\n \"git describe\" was taught to dig trees deeper to find a\n <commit-ish>:<path> that refers to a given blob object.\n\n\n* pb/bisect-helper-2 (2017-10-28) 8 commits\n - t6030: make various test to pass GETTEXT_POISON tests\n - bisect--helper: `bisect_start` shell function partially in C\n - bisect--helper: `get_terms` & `bisect_terms` shell function in C\n - bisect--helper: `bisect_next_check` shell function in C\n - bisect--helper: `check_and_set_terms` shell function in C\n - wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()\n - bisect--helper: `bisect_write` shell function in C\n - bisect--helper: `bisect_reset` shell function in C\n\n Expecting a reroll.\n cf. <0102015f5e5ee171-f30f4868-886f-47a1-a4e4-b4936afc545d-000000@eu-west-1.amazonses.com>\n\n\n* ot/mru-on-list (2017-10-01) 1 commit\n - mru: use double-linked list from list.h\n\n The first step to getting rid of mru API and using the\n doubly-linked list API directly instead.\n\n Is there any remaining issues on this one?\n\n\n* jm/svn-pushmergeinfo-fix (2017-09-17) 1 commit\n - git-svn: fix svn.pushmergeinfo handling of svn+ssh usernames.\n\n \"git svn dcommit\" did not take into account the fact that a\n svn+ssh:// URL with a username@ (typically used for pushing) refers\n to the same SVN repository without the username@ and failed when\n svn.pushmergeinfo option is set.\n\n Needs a test, perhaps as a follow-up patch.\n\n Will merge to 'next'.\n"},{"id":"334803","messageId":"1513255038.6351.2.camel@gmail.com","threadId":"47439","inReplyTo":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Kaartic Sivaraam","fromEmail":"kaartic.sivaraam@gmail.com","sentAt":"2017-12-14T12:37:18Z","receivedAt":"2017-12-14T12:37:31Z","isPatch":false,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"It seems my series that fixes an error message in 'git-rebase'[1]\n(apart from a little cleanups) is missing. I guess I addressed the\nissue that was raised in 3/3 as a v3 for that patch[2]. Are any more\nchanges needed?\n\n[1]: <20171127172104.5796-1-kaartic.sivaraam@gmail.com>\n[2]: <20171201060935.19749-1-kaartic.sivaraam@gmail.com>\n\n\n-- \nKaartic\n"},{"id":"334809","messageId":"xmqqk1xpusyh.fsf@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"1513255038.6351.2.camel@gmail.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-14T17:20:38Z","receivedAt":"2017-12-14T17:20:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Kaartic Sivaraam <kaartic.sivaraam@gmail.com> writes:\n\n> It seems my series that fixes an error message in 'git-rebase'[1]\n> (apart from a little cleanups) is missing. I guess I addressed the\n> issue that was raised in 3/3 as a v3 for that patch[2]. Are any more\n> changes needed?\n>\n> [1]: <20171127172104.5796-1-kaartic.sivaraam@gmail.com>\n> [2]: <20171201060935.19749-1-kaartic.sivaraam@gmail.com>\n\nI think you only sent 3/3 in newer rounds, which made it not to\napply to my tree.  If you meant to drop changes in 1/3 and 2/3,\nperhaps because they were needless churn, then 3/3 needs to be\nupdated not to depend on the changes these two made.\n\nThanks.\n\n\n"},{"id":"334811","messageId":"87zi6lb424.fsf@evledraar.gmail.com","threadId":"47439","inReplyTo":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2017-12-14T17:41:07Z","receivedAt":"2017-12-14T17:41:17Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Wed, Dec 13 2017, Junio C. Hamano jotted:\n\n>\n> * ab/simplify-perl-makefile (2017-12-11) 1 commit\n>   (merged to 'next' on 2017-12-13 at 1b791d2503)\n>  + Makefile: replace perl/Makefile.PL with simple make rules\n>\n>  The build procedure for perl/ part has been greatly simplified by\n>  weaning ourselves off of MakeMaker.\n>\n>  Will merge to 'master'.\n\nI noticed this tiny grammar error in the commit message which you may\nwant to amend (or maybe it's not worth it since it's in next already):\n\n    As a side-effect of these general changes the perl documentation\n    now only installed by install-{doc,man}, not a mere \"install\" as\n    before\n\nThat should be:\n\n    As a side-effect of these general changes the perl documentation is\n    now only installed by install-{doc,man}, not a mere \"install\" as\n    before\n\nI.e. it's missing an \"is\" between \"documentation\" and \"now\".\n"},{"id":"334833","messageId":"20171214210453.GB32842@aiede.mtv.corp.google.com","threadId":"47439","inReplyTo":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2017-12-14T21:04:53Z","receivedAt":"2017-12-14T21:05:02Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Hi,\n\nJunio C Hamano wrote:\n\n> * sb/diff-blobfind (2017-12-12) 1 commit\n>   (merged to 'next' on 2017-12-13 at 9a27a20c5f)\n>  + diffcore: add a filter to find a specific blob\n>\n>  \"git diff\" family of commands learned --blobfind=<object-name> that\n>  allows you to limit the output only to a change that involves the\n>  named blob object (either changing the contents from or to it).\n>\n>  Will merge to 'master'.\n\nSorry, I should have replied about this a long time ago: I love this\noption but I am not sure that --blobfind is the right name for it.\nIf we can think of a better name quickly then it would be nice to\nchange it before people start relying on it, so I'd rather hold off\non merging this to 'master' for the moment.\n\nThat said, if we don't have any better ideas for the option name\nwithin a few days then my objection goes away.\n\nI'll reply in the patch thread.\n\nThanks,\nJonathan\n"},{"id":"334834","messageId":"xmqq6099t3xi.fsf@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"20171214210453.GB32842@aiede.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-14T21:06:33Z","receivedAt":"2017-12-14T21:06:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Nieder <jrnieder@gmail.com> writes:\n\n>> * sb/diff-blobfind (2017-12-12) 1 commit\n>>   (merged to 'next' on 2017-12-13 at 9a27a20c5f)\n>>  + diffcore: add a filter to find a specific blob\n>>\n>>  \"git diff\" family of commands learned --blobfind=<object-name> that\n>>  allows you to limit the output only to a change that involves the\n>>  named blob object (either changing the contents from or to it).\n>>\n>>  Will merge to 'master'.\n>\n> Sorry, I should have replied about this a long time ago: I love this\n> option but I am not sure that --blobfind is the right name for it.\n\nSorry.  I should have updated the description when the option name\nwas updated in the latest round.\nJonathan Nieder <jrnieder@gmail.com> writes:\n\n"},{"id":"334836","messageId":"xmqq1sjxt3tz.fsf@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"xmqqk1xpusyh.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-14T21:08:40Z","receivedAt":"2017-12-14T21:08:47Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> I think you only sent 3/3 in newer rounds, which made it not to\n> apply to my tree.  If you meant to drop changes in 1/3 and 2/3,\n> perhaps because they were needless churn, then 3/3 needs to be\n> updated not to depend on the changes these two made.\n\nHere is what I reconstructed to suit my taste better ;-)\n\n-- >8 --\nFrom: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\nDate: Mon, 27 Nov 2017 22:51:02 +0530\nSubject: [PATCH v4 1/3] rebase: consistently use branch_name variable\n\nThe variable \"branch_name\" holds the <branch> parameter in \"git\nrebase <upstream> <branch>\", but one codepath did not use it after\nassigning $1 to it (instead it kept using $1).  Make it use the\nvariable consistently.\n\nAlso, update an error message to say there is no such branch or\ncommit, as we are expecting either of them, and not limiting\nourselves to a branch name.\n\nSigned-off-by: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\n---\n git-rebase.sh | 15 +++++++++------\n 1 file changed, 9 insertions(+), 6 deletions(-)\n\ndiff --git a/git-rebase.sh b/git-rebase.sh\nindex 60b70f3def..5526b17a36 100755\n--- a/git-rebase.sh\n+++ b/git-rebase.sh\n@@ -528,15 +528,18 @@ case \"$#\" in\n \tbranch_name=\"$1\"\n \tswitch_to=\"$1\"\n \n-\tif git show-ref --verify --quiet -- \"refs/heads/$1\" &&\n-\t   orig_head=$(git rev-parse -q --verify \"refs/heads/$1\")\n+\t# Is it a local branch?\n+\tif git show-ref --verify --quiet -- \"refs/heads/$branch_name\" &&\n+\t   orig_head=$(git rev-parse -q --verify \"refs/heads/$branch_name\")\n \tthen\n-\t\thead_name=\"refs/heads/$1\"\n-\telif orig_head=$(git rev-parse -q --verify \"$1\")\n+\t\thead_name=\"refs/heads/$branch_name\"\n+\t# If not is it a valid ref (branch or commit)?\n+\telif orig_head=$(git rev-parse -q --verify \"$branch_name\")\n \tthen\n \t\thead_name=\"detached HEAD\"\n+\n \telse\n-\t\tdie \"$(eval_gettext \"fatal: no such branch: \\$branch_name\")\"\n+\t\tdie \"$(eval_gettext \"fatal: no such branch/commit: \\$branch_name\")\"\n \tfi\n \t;;\n 0)\n@@ -547,7 +550,7 @@ case \"$#\" in\n \t\tbranch_name=$(expr \"z$branch_name\" : 'zrefs/heads/\\(.*\\)')\n \telse\n \t\thead_name=\"detached HEAD\"\n-\t\tbranch_name=HEAD ;# detached\n+\t\tbranch_name=HEAD\n \tfi\n \torig_head=$(git rev-parse --verify HEAD) || exit\n \t;;\n-- \n2.15.1-554-g7ec1e7e2b9\n\n"},{"id":"334837","messageId":"xmqqwp1prp8r.fsf_-_@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"xmqqk1xpusyh.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v4 2/3] rebase: distinguish user input by quoting it","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-14T21:09:08Z","receivedAt":"2017-12-14T21:09:15Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"From: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\nDate: Mon, 27 Nov 2017 22:51:03 +0530\n\nSigned-off-by: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n git-rebase.sh | 4 ++--\n 1 file changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-rebase.sh b/git-rebase.sh\nindex 5526b17a36..e5adb596a0 100755\n--- a/git-rebase.sh\n+++ b/git-rebase.sh\n@@ -477,7 +477,7 @@ then\n \t\t;;\n \tesac\n \tupstream=$(peel_committish \"${upstream_name}\") ||\n-\tdie \"$(eval_gettext \"invalid upstream \\$upstream_name\")\"\n+\tdie \"$(eval_gettext \"invalid upstream '\\$upstream_name'\")\"\n \tupstream_arg=\"$upstream_name\"\n else\n \tif test -z \"$onto\"\n@@ -539,7 +539,7 @@ case \"$#\" in\n \t\thead_name=\"detached HEAD\"\n \n \telse\n-\t\tdie \"$(eval_gettext \"fatal: no such branch/commit: \\$branch_name\")\"\n+\t\tdie \"$(eval_gettext \"fatal: no such branch/commit '\\$branch_name'\")\"\n \tfi\n \t;;\n 0)\n-- \n2.15.1-554-g7ec1e7e2b9\n\n"},{"id":"334838","messageId":"xmqqshcdrp83.fsf_-_@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"xmqqk1xpusyh.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v4 3/3] rebase: rebasing can also be done when HEAD is detached","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-14T21:09:32Z","receivedAt":"2017-12-14T21:09:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"From: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\nDate: Mon, 27 Nov 2017 22:51:04 +0530\n\nAttempting to rebase when the HEAD is detached and is already\nup to date with upstream (so there's nothing to do), the\nfollowing message is shown\n\n        Current branch HEAD is up to date.\n\nwhich is clearly wrong as HEAD is not a branch.\n\nHandle the special case of HEAD correctly to give a more precise\nerror message.\n\nSigned-off-by: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n git-rebase.sh | 16 ++++++++++++++--\n 1 file changed, 14 insertions(+), 2 deletions(-)\n\ndiff --git a/git-rebase.sh b/git-rebase.sh\nindex e5adb596a0..f3dd864437 100755\n--- a/git-rebase.sh\n+++ b/git-rebase.sh\n@@ -601,11 +601,23 @@ then\n \t\ttest -z \"$switch_to\" ||\n \t\tGIT_REFLOG_ACTION=\"$GIT_REFLOG_ACTION: checkout $switch_to\" \\\n \t\t\tgit checkout -q \"$switch_to\" --\n-\t\tsay \"$(eval_gettext \"Current branch \\$branch_name is up to date.\")\"\n+\t\tif test \"$branch_name\" = \"HEAD\" &&\n+\t\t\t !(git symbolic-ref -q HEAD)\n+\t\tthen\n+\t\t\tsay \"$(eval_gettext \"HEAD is up to date.\")\"\n+\t\telse\n+\t\t\tsay \"$(eval_gettext \"Current branch \\$branch_name is up to date.\")\"\n+\t\tfi\n \t\tfinish_rebase\n \t\texit 0\n \telse\n-\t\tsay \"$(eval_gettext \"Current branch \\$branch_name is up to date, rebase forced.\")\"\n+\t\tif test \"$branch_name\" = \"HEAD\" &&\n+\t\t\t !(git symbolic-ref -q HEAD)\n+\t\tthen\n+\t\t\tsay \"$(eval_gettext \"HEAD is up to date, rebase forced.\")\"\n+\t\telse\n+\t\t\tsay \"$(eval_gettext \"Current branch \\$branch_name is up to date, rebase forced.\")\"\n+\t\tfi\n \tfi\n fi\n \n-- \n2.15.1-554-g7ec1e7e2b9\n\n"},{"id":"334844","messageId":"20171214212609.GD32842@aiede.mtv.corp.google.com","threadId":"47439","inReplyTo":"xmqq6099t3xi.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2017-12-14T21:26:09Z","receivedAt":"2017-12-14T21:26:17Z","isPatch":false,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Junio C Hamano wrote:\n> Jonathan Nieder <jrnieder@gmail.com> writes:\n\n>>> * sb/diff-blobfind (2017-12-12) 1 commit\n>>>   (merged to 'next' on 2017-12-13 at 9a27a20c5f)\n>>>  + diffcore: add a filter to find a specific blob\n>>>\n>>>  \"git diff\" family of commands learned --blobfind=<object-name> that\n>>>  allows you to limit the output only to a change that involves the\n>>>  named blob object (either changing the contents from or to it).\n>>>\n>>>  Will merge to 'master'.\n>>\n>> Sorry, I should have replied about this a long time ago: I love this\n>> option but I am not sure that --blobfind is the right name for it.\n>\n> Sorry.  I should have updated the description when the option name\n> was updated in the latest round.\n\nSure.  My worry still applies:\nhttps://public-inbox.org/git/20171214212234.GC32842@aiede.mtv.corp.google.com/\n\nBut as I said, if nothing comes of it within a few days then I'm happy\nto conclude that we did our best.\n\nThanks,\nJonathan\n"},{"id":"334876","messageId":"alpine.DEB.2.21.1.1712151339110.406@MININT-6BKU6QN.europe.corp.microsoft.com","threadId":"47439","inReplyTo":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-12-15T12:39:58Z","receivedAt":"2017-12-15T12:40:09Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Wed, 13 Dec 2017, Junio C Hamano wrote:\n\n> * db/doc-workflows-neuter-the-maintainer (2017-12-08) 1 commit\n\nI hope you do not mean this literally. There is no need to bring family\nplanning into a technical document.\n\nCiao,\nDscho\n"},{"id":"334884","messageId":"fc2d03d1-ad76-21ae-2880-8a74644b8b80@gmail.com","threadId":"47439","inReplyTo":"xmqqk1xpusyh.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Kaartic Sivaraam","fromEmail":"kaartic.sivaraam@gmail.com","sentAt":"2017-12-15T15:29:32Z","receivedAt":"2017-12-15T15:29:45Z","isPatch":false,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Thursday 14 December 2017 10:50 PM, Junio C Hamano wrote:\n> Kaartic Sivaraam <kaartic.sivaraam@gmail.com> writes:\n> \n>> It seems my series that fixes an error message in 'git-rebase'[1]\n>> (apart from a little cleanups) is missing. I guess I addressed the\n>> issue that was raised in 3/3 as a v3 for that patch[2]. Are any more\n>> changes needed?\n>>\n>> [1]: <20171127172104.5796-1-kaartic.sivaraam@gmail.com>\n>> [2]: <20171201060935.19749-1-kaartic.sivaraam@gmail.com>\n> \n> I think you only sent 3/3 in newer rounds, which made it not to\n> apply to my tree.  \n\nThat's surprising, let me check.\n\n\n> If you meant to drop changes in 1/3 and 2/3,\n> perhaps because they were needless churn, then 3/3 needs to be\n> updated not to depend on the changes these two made.\n> \n\n2/3 is for sure not needless. But 1/3 might be. Though I think it \nimproves the communicativity of the variable name. In case you think \nit's completely useless, then I can drop it as we won't lose much anyway ??\n\n\n---\nKaartic\n"},{"id":"334913","messageId":"dbed9ef3-0bf6-01e2-13d2-4c92a243c321@gmail.com","threadId":"47439","inReplyTo":"xmqq1sjxt3tz.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Kaartic Sivaraam","fromEmail":"kaartic.sivaraam@gmail.com","sentAt":"2017-12-16T06:06:30Z","receivedAt":"2017-12-16T06:06:46Z","isPatch":false,"sender":{"key":"kaartic.sivaraam@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12448084?v=4"},"body":"On Friday 15 December 2017 02:38 AM, Junio C Hamano wrote:\n> Junio C Hamano <gitster@pobox.com> writes:\n> \n>> I think you only sent 3/3 in newer rounds, which made it not to\n>> apply to my tree.  If you meant to drop changes in 1/3 and 2/3,\n>> perhaps because they were needless churn, then 3/3 needs to be\n>> updated not to depend on the changes these two made.\n> \n> Here is what I reconstructed to suit my taste better ;-)\n> \n\nLooks good to me except that your v4 3/3 seems to be spawning a subshell \nis an oversight, I guess? Also, I guess it would be better if there was \ncomment in the code saying 'branch_nam' might hold a commit/branch name. \nNo worries I'll send a v5 to that thread based on your v4.\n\nThanks,\nKaartic\n"},{"id":"336228","messageId":"ABE531C2-3173-441D-8710-B9B75C97B4F5@gmail.com","threadId":"47439","inReplyTo":"xmqqzi6mutcc.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2018-01-09T12:59:43Z","receivedAt":"2018-01-09T12:59:54Z","isPatch":false,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 14 Dec 2017, at 00:00, Junio C Hamano <gitster@pobox.com> wrote:\n> \n> 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\n> the integration branches, but I am still holding onto them.\n> \n> You can find the changes described here in the integration branches\n> of the repositories listed at\n> \n>    http://git-blame.blogspot.com/p/git-public-repositories.html\n> ...\n> \n> * jk/progress-delay-fix (2017-12-04) 2 commits\n>  (merged to 'next' on 2017-12-05 at 8e62c2b18b)\n> + progress: drop delay-threshold code\n> + progress: set default delay threshold to 100%, not 0%\n> \n> A regression in the progress eye-candy was fixed.\n\nHi Junio,\n\nthis fixes a bug that affects the Git LFS community (not only\neye-candy). Would it be possible to get this into Git 2.15.2?\n\nThanks,\nLars\n"},{"id":"336292","messageId":"xmqqmv1m6eor.fsf@gitster.mtv.corp.google.com","threadId":"47439","inReplyTo":"ABE531C2-3173-441D-8710-B9B75C97B4F5@gmail.com","subject":"Re: What's cooking in git.git (Dec 2017, #03; Wed, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-01-09T18:59:48Z","receivedAt":"2018-01-09T18:59:55Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Schneider <larsxschneider@gmail.com> writes:\n\n>> On 14 Dec 2017, at 00:00, Junio C Hamano <gitster@pobox.com> wrote:\n>> \n>> 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\n>> the integration branches, but I am still holding onto them.\n>> \n>> You can find the changes described here in the integration branches\n>> of the repositories listed at\n>> \n>>    http://git-blame.blogspot.com/p/git-public-repositories.html\n>> ...\n>> \n>> * jk/progress-delay-fix (2017-12-04) 2 commits\n>>  (merged to 'next' on 2017-12-05 at 8e62c2b18b)\n>> + progress: drop delay-threshold code\n>> + progress: set default delay threshold to 100%, not 0%\n>> \n>> A regression in the progress eye-candy was fixed.\n>\n> Hi Junio,\n>\n> this fixes a bug that affects the Git LFS community (not only\n> eye-candy). Would it be possible to get this into Git 2.15.2?\n\nThe topic is in 'master' and is a candidate to eventually hit the\n'maint' track.  I do not know if 2.15.2 is warranted, though, as we\nare getting closer to 2.16 and supposed to be concentrating more on\nregression fixes.\n\nThanks.\n"}]}