{"thread":{"id":"45063","subject":"What's cooking in git.git (Feb 2017, #02; Mon, 6)","startedAt":"2017-02-06T22:34:18Z","lastAt":"2017-02-10T21:18:04Z","messageCount":17,"participants":["Junio C Hamano","SZEDER Gábor","Jacob Keller","Stefan Beller","Jeff King","Johannes Schindelin","Michael Haggerty"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"310951","messageId":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":null,"subject":"What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-06T22:34:08Z","receivedAt":"2017-02-06T22:34: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* cw/log-updates-for-all-refs-really (2017-02-01) 4 commits\n (merged to 'next' on 2017-02-02 at 3e11673fcc)\n + doc: add note about ignoring '--no-create-reflog'\n (merged to 'next' on 2017-01-31 at 53f71d2401)\n + update-ref: add test cases for bare repository\n + refs: add option core.logAllRefUpdates = always\n + config: add markup to core.logAllRefUpdates doc\n\n The \"core.logAllRefUpdates\" that used to be boolean has been\n enhanced to take 'always' as well, to record ref updates to refs\n other than the ones that are expected to be updated (i.e. branches,\n remote-tracking branches and notes).\n\n\n* js/re-running-failed-tests (2017-01-27) 1 commit\n (merged to 'next' on 2017-01-31 at 30c3a9e0cf)\n + t/Makefile: add a rule to re-run previously-failed tests\n\n \"make -C t failed\" will now run only the tests that failed in the\n previous run. This is usable only when prove is not use, and gives\n a useless error message when run after \"make clean\", but otherwise\n is serviceable.\n\n\n* pl/complete-diff-submodule-diff (2017-01-30) 1 commit\n (merged to 'next' on 2017-01-31 at 7e668d325c)\n + Completion: Add support for --submodule=diff\n\n The command line completion (in contrib/) learned that\n \"git diff --submodule=\" can take \"diff\" as a recently added option.\n\n\n* rs/object-id (2017-01-30) 3 commits\n (merged to 'next' on 2017-01-31 at c442e4780c)\n + checkout: convert post_checkout_hook() to struct object_id\n + use oidcpy() for copying hashes between instances of struct object_id\n + use oid_to_hex_r() for converting struct object_id hashes to hex strings\n\n \"uchar [40]\" to \"struct object_id\" conversion continues.\n\n\n* sb/submodule-recursive-absorb (2017-01-26) 3 commits\n (merged to 'next' on 2017-01-31 at 0a24cfd06b)\n + submodule absorbing: fix worktree/gitdir pointers recursively for non-moves\n + cache.h: expose the dying procedure for reading gitlinks\n + setup: add gentle version of resolve_git_dir\n\n When a submodule \"A\", which has another submodule \"B\" nested within\n it, is \"absorbed\" into the top-level superproject, the inner\n submodule \"B\" used to be left in a strange state. The logic to\n adjust the .git pointers in these submodules has been corrected.\n\n\n* sb/submodule-update-initial-runs-custom-script (2017-01-26) 1 commit\n (merged to 'next' on 2017-01-31 at d794f894c6)\n + submodule update: run custom update script for initial populating as well\n\n The user can specify a custom update method that is run when\n \"submodule update\" updates an already checked out submodule. This\n was ignored when checking the submodule out for the first time and\n we instead always just checked out the commit that is bound to the\n path in the superproject's index.\n\n\n* sb/unpack-trees-super-prefix (2017-01-25) 4 commits\n (merged to 'next' on 2017-01-31 at dabe6ca2b1)\n + unpack-trees: support super-prefix option\n + t1001: modernize style\n + t1000: modernize style\n + read-tree: use OPT_BOOL instead of OPT_SET_INT\n\n \"git read-tree\" and its underlying unpack_trees() machinery learned\n to report problematic paths prefixed with the --super-prefix option.\n\n--------------------------------------------------\n[New Topics]\n\n* bw/push-submodule-only (2017-02-01) 2 commits\n (merged to 'next' on 2017-02-06 at 851edafb14)\n + completion: add completion for --recurse-submodules=only\n + doc: add doc for git-push --recurse-submodules=only\n\n Add missing documentation update to a recent topic.\n\n Will merge to 'master'.\n\n\n* cw/completion (2017-02-03) 7 commits\n - completion: recognize more long-options\n - completion: teach remote subcommands to complete options\n - completion: teach replace to complete options\n - completion: teach ls-remote to complete options\n - completion: improve bash completion for git-add\n - completion: add subcommand completion for rerere\n - completion: teach submodule subcommands to complete options\n\n More command line completion (in contrib/) for recent additions.\n\n Needs review.\n\n\n* cw/tag-reflog-message (2017-02-06) 1 commit\n - tag: generate useful reflog message\n\n \"git tag\", because refs/tags/* doesn't keep reflog by default, did\n not leave useful message when adding a new entry to reflog.\n\n Waiting for review to conclude.\n An uneven error handling is somewhat disturbing.\n\n\n* dl/document-that-difftool-takes-no-gui-option (2017-02-06) 1 commit\n - Document the --no-gui option in difftool\n\n A minor doc update.\n\n Waiting for review to conclude.\n A minor inconsistency is somewhat disturbing.\n\n\n* ew/complete-svn-authorship-options (2017-02-06) 1 commit\n (merged to 'next' on 2017-02-06 at dca324db7c)\n + completion: fix git svn authorship switches\n\n Correct command line completion (in contrib/) on \"git svn\"\n\n Will merge to 'master'.\n\n\n* jk/reset-to-break-a-commit-doc (2017-02-03) 1 commit\n (merged to 'next' on 2017-02-06 at 7f571e62e9)\n + reset: add an example of how to split a commit into two\n\n A minor doc update.\n\n Will merge to 'master'.\n\n\n* js/difftool-builtin (2017-02-06) 1 commit\n (merged to 'next' on 2017-02-06 at 6a90549f38)\n + difftool: fix bug when printing usage\n\n A hot-fix to C-rewrite of \"git difftool\".\n\n Will merge to 'master'.\n\n\n* ps/worktree-prune-help-fix (2017-02-06) 1 commit\n (merged to 'next' on 2017-02-06 at eeb96f1677)\n + worktree: fix option descriptions for `prune`\n\n Incorrect usage help message for \"git worktree prune\" has been fixed.\n\n Will merge to 'master'.\n\n\n* rs/p5302-create-repositories-before-tests (2017-02-06) 1 commit\n (merged to 'next' on 2017-02-06 at f93bd2ed47)\n + p5302: create repositories for index-pack results explicitly\n\n Adjust a perf test to new world order where commands that do\n require a repository are really strict about having a repository.\n\n Will merge to 'master'.\n\n\n* sg/completion (2017-02-03) 21 commits\n - completion: cache the path to the repository\n - completion: extract repository discovery from __gitdir()\n - completion: don't guard git executions with __gitdir()\n - completion: consolidate silencing errors from git commands\n - completion: don't use __gitdir() for git commands\n - completion: respect 'git -C '\n - rev-parse: add '--absolute-git-dir' option\n - completion: fix completion after 'git -C '\n - completion: don't offer commands when 'git --opt' needs an argument\n - completion: list short refs from a remote given as a URL\n - completion: don't list 'HEAD' when trying refs completion outside of a repo\n - completion: list refs from remote when remote's name matches a directory\n - completion: respect 'git --git-dir=' when listing remote refs\n - completion: fix most spots not respecting 'git --git-dir='\n - completion: ensure that the repository path given on the command line exists\n - completion tests: add tests for the __git_refs() helper function\n - completion tests: check __gitdir()'s output in the error cases\n - completion tests: consolidate getting path of current working directory\n - completion tests: make the $cur variable local to the test helper functions\n - completion tests: don't add test cruft to the test repository\n - completion: improve __git_refs()'s in-code documentation\n (this branch is used by sg/completion-refs-speedup.)\n\n Clean-up and updates to command line completion (in contrib/).\n\n Will merge to 'next'.\n\n\n* sg/completion-refs-speedup (2017-02-06) 13 commits\n - squash! completion: fill COMPREPLY directly when completing refs\n - completion: fill COMPREPLY directly when completing refs\n - completion: list only matching symbolic and pseudorefs when completing refs\n - completion: let 'for-each-ref' sort remote branches for 'checkout' DWIMery\n - completion: let 'for-each-ref' filter remote branches for 'checkout' DWIMery\n - completion: let 'for-each-ref' strip the remote name from remote branches\n - completion: let 'for-each-ref' and 'ls-remote' filter matching refs\n - completion: don't disambiguate short refs\n - completion: don't disambiguate tags and branches\n - completion: support excluding full refs\n - completion: support completing full refs after '--option=refs/'\n - completion: wrap __git_refs() for better option parsing\n - completion: remove redundant __gitcomp_nl() options from _git_commit()\n (this branch uses sg/completion.)\n\n Will hold.\n This seems to break 9902 when merged to 'pu'.\n\n\n* sk/parse-remote-cleanup (2017-02-06) 1 commit\n (merged to 'next' on 2017-02-06 at 6ec89f72d5)\n + parse-remote: remove reference to unused op_prep\n\n Code clean-up.\n\n Will merge to 'master'.\n\n--------------------------------------------------\n[Stalled]\n\n* pb/bisect (2016-10-18) 27 commits\n - bisect--helper: remove the dequote in bisect_start()\n - bisect--helper: retire `--bisect-auto-next` subcommand\n - bisect--helper: retire `--bisect-autostart` subcommand\n - bisect--helper: retire `--bisect-write` subcommand\n - bisect--helper: `bisect_replay` shell function in C\n - bisect--helper: `bisect_log` shell function in C\n - bisect--helper: retire `--write-terms` subcommand\n - bisect--helper: retire `--check-expected-revs` subcommand\n - bisect--helper: `bisect_state` & `bisect_head` shell function in C\n - bisect--helper: `bisect_autostart` shell function in C\n - bisect--helper: retire `--next-all` subcommand\n - bisect--helper: retire `--bisect-clean-state` subcommand\n - bisect--helper: `bisect_next` and `bisect_auto_next` shell function in C\n - t6030: no cleanup with bad merge base\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` & bisect_voc shell function in C\n - bisect--helper: `check_and_set_terms` shell function in C\n - bisect--helper: `bisect_write` shell function in C\n - bisect--helper: `is_expected_rev` & `check_expected_revs` shell function in C\n - bisect--helper: `bisect_reset` shell function in C\n - wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()\n - t6030: explicitly test for bisection cleanup\n - bisect--helper: `bisect_clean_state` shell function in C\n - bisect--helper: `write_terms` shell function in C\n - bisect: rewrite `check_term_format` shell function in C\n - bisect--helper: use OPT_CMDMODE instead of OPT_BOOL\n\n Move more parts of \"git bisect\" to C.\n\n Expecting a reroll.\n cf. \n cf. \n\n\n* ls/filter-process-delayed (2017-01-08) 1 commit\n . convert: add \"status=delayed\" to filter process protocol\n\n Ejected, as does not build when merged to 'pu'.\n\n\n* ls/p4-path-encoding (2016-12-18) 1 commit\n - git-p4: fix git-p4.pathEncoding for removed files\n\n When \"git p4\" imports changelist that removes paths, it failed to\n convert pathnames when the p4 used encoding different from the one\n used on the Git side. This has been corrected.\n\n Will be rerolled.\n cf. <7E1C7387-4F37-423F-803D-3B5690B49D40@gmail.com>\n\n\n* sh/grep-tree-obj-tweak-output (2017-01-20) 2 commits\n - grep: use '/' delimiter for paths\n - grep: only add delimiter if there isn't one already\n\n \"git grep\", when fed a tree-ish as an input, shows each hit\n prefixed with \":::\". As is\n almost always either a commit or a tag that points at a commit, the\n early part of the output \":\" can be used as the\n name of the blob and given to \"git show\". When is a\n tree given in the extended SHA-1 syntax (e.g. \":\", or\n \":\"), however, this results in a string that does not\n name a blob (e.g. \"::\" or \"::\").\n \"git grep\" has been taught to be a bit more intelligent about these\n cases and omit a colon (in the former case) or use slash (in the\n latter case) to produce \":\" and\n \":/\" that can be used as the name of a blob.\n\n Waiting for the review discussion to settle, followed by a reroll.\n\n\n* mh/ref-remove-empty-directory (2017-01-07) 23 commits\n - files_transaction_commit(): clean up empty directories\n - try_remove_empty_parents(): teach to remove parents of reflogs, too\n - try_remove_empty_parents(): don't trash argument contents\n - try_remove_empty_parents(): rename parameter \"name\" -> \"refname\"\n - delete_ref_loose(): inline function\n - delete_ref_loose(): derive loose reference path from lock\n - log_ref_write_1(): inline function\n - log_ref_setup(): manage the name of the reflog file internally\n - log_ref_write_1(): don't depend on logfile argument\n - log_ref_setup(): pass the open file descriptor back to the caller\n - log_ref_setup(): improve robustness against races\n - log_ref_setup(): separate code for create vs non-create\n - log_ref_write(): inline function\n - rename_tmp_log(): improve error reporting\n - rename_tmp_log(): use raceproof_create_file()\n - lock_ref_sha1_basic(): use raceproof_create_file()\n - lock_ref_sha1_basic(): inline constant\n - raceproof_create_file(): new function\n - safe_create_leading_directories(): set errno on SCLD_EXISTS\n - safe_create_leading_directories_const(): preserve errno\n - t5505: use \"for-each-ref\" to test for the non-existence of references\n - refname_is_safe(): correct docstring\n - files_rename_ref(): tidy up whitespace\n\n Deletion of a branch \"foo/bar\" could remove .git/refs/heads/foo\n once there no longer is any other branch whose name begins with\n \"foo/\", but we didn't do so so far. Now we do.\n\n Expecting a reroll.\n cf. <5051c78e-51f9-becd-e1a6-9c0b781d6912@alum.mit.edu>\n\n\n* jc/diff-b-m (2015-02-23) 5 commits\n . WIPWIP\n . WIP: diff-b-m\n - diffcore-rename: allow easier debugging\n - diffcore-rename.c: add locate_rename_src()\n - diffcore-break: allow debugging\n\n \"git diff -B -M\" produced incorrect patch when the postimage of a\n completely rewritten file is similar to the preimage of a removed\n file; such a resulting file must not be expressed as a rename from\n other place.\n\n The fix in this patch is broken, unfortunately.\n\n Will discard.\n\n--------------------------------------------------\n[Cooking]\n\n* jk/delta-chain-limit (2017-01-27) 2 commits\n (merged to 'next' on 2017-02-06 at 9ff36ae9b2)\n + pack-objects: convert recursion to iteration in break_delta_chain()\n + pack-objects: enforce --depth limit in reused deltas\n\n \"git repack --depth=\" for a long time busted the specified depth\n when reusing delta from existing packs. This has been corrected.\n\n Will cook in 'next'.\n\n\n* mm/merge-rename-delete-message (2017-01-30) 1 commit\n - merge-recursive: make \"CONFLICT (rename/delete)\" message show both paths\n\n When \"git merge\" detects a path that is renamed in one history\n while the other history deleted (or modified) it, it now reports\n both paths to help the user understand what is going on in the two\n histories being merged.\n\n Will merge to and then cook in 'next'.\n\n\n* rs/swap (2017-01-30) 5 commits\n - graph: use SWAP macro\n - diff: use SWAP macro\n - use SWAP macro\n - apply: use SWAP macro\n - add SWAP macro\n\n Code clean-up.\n\n Will merge to 'next'.\n ... unless there is an objection with better alternative. The\n discussion seems to have quieted down.\n\n\n* ps/urlmatch-wildcard (2017-02-01) 5 commits\n - urlmatch: allow globbing for the URL host part\n - urlmatch: include host in urlmatch ranking\n - urlmatch: split host and port fields in `struct url_info`\n - urlmatch: enable normalization of URLs with globs\n - mailmap: add Patrick Steinhardt's work address\n\n The part in \"http..\" configuration variable\n can now be spelled with '*' that serves as wildcard.\n E.g. \"http.https://*.example.com.proxy\" can be used to specify the\n proxy used for https://a.example.com, https://b.example.com, etc.,\n i.e. any host in the example.com domain.\n\n Will merge to and then cook in 'next'.\n\n\n* sf/putty-w-args (2017-02-01) 5 commits\n - SQUASH???\n - connect: Add the envvar GIT_SSH_VARIANT and ssh.variant config\n - git_connect(): factor out SSH variant handling\n - connect: rename tortoiseplink and putty variables\n - connect: handle putty/plink also in GIT_SSH_COMMAND\n\n The command line options for ssh invocation needs to be tweaked for\n some implementations of SSH (e.g. PuTTY plink wants \"-P \"\n while OpenSSH wants \"-p \" to specify port to connect to), and\n the variant was guessed when GIT_SSH environment variable is used\n to specify it. Extend the guess to the command specified by the\n newer GIT_SSH_COMMAND and also core.sshcommand configuration\n variable, and give an escape hatch for users to deal with\n misdetected cases.\n\n Stalled?\n cf. \n\n\n* jk/describe-omit-some-refs (2017-01-23) 5 commits\n (merged to 'next' on 2017-01-23 at f8a14b4996)\n + describe: teach describe negative pattern matches\n + describe: teach --match to accept multiple patterns\n + name-rev: add support to exclude refs by pattern match\n + name-rev: extend --refs to accept multiple patterns\n + doc: add documentation for OPT_STRING_LIST\n\n \"git describe\" and \"git name-rev\" have been taught to take more\n than one refname patterns to restrict the set of refs to base their\n naming output on, and also learned to take negative patterns to\n name refs not to be used for naming via their \"--exclude\" option.\n\n Will cook in 'next'.\n\n\n* sb/submodule-doc (2017-01-12) 3 commits\n - submodules: add a background story\n - submodule update documentation: don't repeat ourselves\n - submodule documentation: add options to the subcommand\n\n Needs review.\n\n\n* bw/attr (2017-02-01) 27 commits\n - attr: reformat git_attr_set_direction() function\n - attr: push the bare repo check into read_attr()\n - attr: store attribute stack in attr_check structure\n - attr: tighten const correctness with git_attr and match_attr\n - attr: remove maybe-real, maybe-macro from git_attr\n - attr: eliminate global check_all_attr array\n - attr: use hashmap for attribute dictionary\n - attr: change validity check for attribute names to use positive logic\n - attr: pass struct attr_check to collect_some_attrs\n - attr: retire git_check_attrs() API\n - attr: convert git_check_attrs() callers to use the new API\n - attr: convert git_all_attrs() to use \"struct attr_check\"\n - attr: (re)introduce git_check_attr() and struct attr_check\n - attr: rename function and struct related to checking attributes\n - attr.c: outline the future plans by heavily commenting\n - Documentation: fix a typo\n - attr.c: add push_stack() helper\n - attr: support quoting pathname patterns in C style\n - attr.c: plug small leak in parse_attr_line()\n - attr.c: tighten constness around \"git_attr\" structure\n - attr.c: simplify macroexpand_one()\n - attr.c: mark where #if DEBUG ends more clearly\n - attr.c: complete a sentence in a comment\n - attr.c: explain the lack of attr-name syntax check in parse_attr()\n - attr.c: update a stale comment on \"struct match_attr\"\n - attr.c: use strchrnul() to scan for one line\n - commit.c: use strchrnul() to scan for one line\n\n The gitattributes machinery is being taught to work better in a\n multi-threaded environment.\n\n Will merge to and then cook in 'next'.\n\n\n* vn/xdiff-func-context (2017-01-15) 1 commit\n - xdiff -W: relax end-of-file function detection\n\n \"git diff -W\" has been taught to handle the case where a new\n function is added at the end of the file better.\n\n Will hold.\n An follow-up change to go back from the line that matches the\n funcline to show comments before the function definition is still\n being discussed.\n\n\n* nd/worktree-move (2017-01-27) 7 commits\n - fixup! worktree move: new command\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 Needs review.\n\n\n* cc/split-index-config (2016-12-26) 21 commits\n - Documentation/git-update-index: explain splitIndex.*\n - Documentation/config: add splitIndex.sharedIndexExpire\n - read-cache: use freshen_shared_index() in read_index_from()\n - read-cache: refactor read_index_from()\n - t1700: test shared index file expiration\n - read-cache: unlink old sharedindex files\n - config: add git_config_get_expiry() from gc.c\n - read-cache: touch shared index files when used\n - sha1_file: make check_and_freshen_file() non static\n - Documentation/config: add splitIndex.maxPercentChange\n - t1700: add tests for splitIndex.maxPercentChange\n - read-cache: regenerate shared index if necessary\n - config: add git_config_get_max_percent_split_change()\n - Documentation/git-update-index: talk about core.splitIndex config var\n - Documentation/config: add information for core.splitIndex\n - t1700: add tests for core.splitIndex\n - update-index: warn in case of split-index incoherency\n - read-cache: add and then use tweak_split_index()\n - split-index: add {add,remove}_split_index() functions\n - config: add git_config_get_split_index()\n - config: mark an error message up for translation\n\n The experimental \"split index\" feature has gained a few\n configuration variables to make it easier to use.\n\n Waiting for review comments to be addressed.\n cf. <20161226102222.17150-1-chriscool@tuxfamily.org>\n cf. \n\n\n* kn/ref-filter-branch-list (2017-01-31) 20 commits\n (merged to 'next' on 2017-01-31 at e7592a5461)\n + branch: implement '--format' option\n + branch: use ref-filter printing APIs\n + branch, tag: use porcelain output\n + ref-filter: allow porcelain to translate messages in the output\n + ref-filter: add an 'rstrip=' option to atoms which deal with refnames\n + ref-filter: modify the 'lstrip=' option to work with negative ''\n + ref-filter: Do not abruptly die when using the 'lstrip=' option\n + ref-filter: rename the 'strip' option to 'lstrip'\n + ref-filter: make remote_ref_atom_parser() use refname_atom_parser_internal()\n + ref-filter: introduce refname_atom_parser()\n + ref-filter: introduce refname_atom_parser_internal()\n + ref-filter: make \"%(symref)\" atom work with the ':short' modifier\n + ref-filter: add support for %(upstream:track,nobracket)\n + ref-filter: make %(upstream:track) prints \"[gone]\" for invalid upstreams\n + ref-filter: introduce format_ref_array_item()\n + ref-filter: move get_head_description() from branch.c\n + ref-filter: modify \"%(objectname:short)\" to take length\n + ref-filter: implement %(if:equals=) and %(if:notequals=)\n + ref-filter: include reference to 'used_atom' within 'atom_value'\n + ref-filter: implement %(if), %(then), and %(else) atoms\n\n The code to list branches in \"git branch\" has been consolidated\n with the more generic ref-filter API.\n\n Will cook in 'next'.\n\n\n* jk/no-looking-at-dotgit-outside-repo-final (2016-10-26) 1 commit\n (merged to 'next' on 2016-12-05 at 0c77e39cd5)\n + setup_git_env: avoid blind fall-back to \".git\"\n\n Originally merged to 'next' on 2016-10-26\n\n This is the endgame of the topic to avoid blindly falling back to\n \".git\" when the setup sequence said we are _not_ in Git repository.\n A corner case that happens to work right now may be broken by a\n call to die(\"BUG\").\n\n Will cook in 'next'.\n\n\n* jc/merge-drop-old-syntax (2015-04-29) 1 commit\n (merged to 'next' on 2016-12-05 at 041946dae0)\n + merge: drop 'git merge HEAD ' syntax\n\n Originally merged to 'next' on 2016-10-11\n\n Stop supporting \"git merge HEAD \" syntax that has\n been deprecated since October 2007, and issues a deprecation\n warning message since v2.5.0.\n\n Will cook in 'next'.\n\n\n* jc/bundle (2016-03-03) 6 commits\n . index-pack: --clone-bundle option\n . Merge branch 'jc/index-pack' into jc/bundle\n . bundle v3: the beginning\n . bundle: keep a copy of bundle file name in the in-core bundle header\n . bundle: plug resource leak\n . bundle doc: 'verify' is not about verifying the bundle\n\n The beginning of \"split bundle\", which could be one of the\n ingredients to allow \"git clone\" traffic off of the core server\n network to CDN.\n\n While I think it would make it easier for people to experiment and\n build on if the topic is merged to 'next', I am at the same time a\n bit reluctant to merge an unproven new topic that introduces a new\n file format, which we may end up having to support til the end of\n time. It is likely that to support a \"prime clone from CDN\", it\n would need a lot more than just \"these are the heads and the pack\n data is over there\", so this may not be sufficient.\n\n--------------------------------------------------\n[Discarded]\n\n* jk/nofollow-attr-ignore (2016-11-02) 5 commits\n . exclude: do not respect symlinks for in-tree .gitignore\n . attr: do not respect symlinks for in-tree .gitattributes\n . exclude: convert \"check_index\" into a flags field\n . attr: convert \"macro_ok\" into a flags field\n . add open_nofollow() helper\n\n As we do not follow symbolic links when reading control files like\n .gitignore and .gitattributes from the index, match the behaviour\n and not follow symbolic links when reading them from the working\n tree. This also tightens security a bit by not leaking contents of\n an unrelated file in the error messages when it is pointed at by\n one of these files that is a symbolic link.\n\n Perhaps we want to cover .gitmodules too with the same mechanism?\n\n\n* sb/push-make-submodule-check-the-default (2017-01-26) 2 commits\n (merged to 'next' on 2017-01-26 at 5f4715cea6)\n + Revert \"push: change submodule default to check when submodules exist\"\n (merged to 'next' on 2016-12-12 at 1863e05af5)\n + push: change submodule default to check when submodules exist\n\n Turn the default of \"push.recurseSubmodules\" to \"check\" when\n submodules seem to be in use.\n\n Retracted.\n\n\n"},{"id":"310957","messageId":"CAM0VKjmhO9NQLz9TDv5M3OhxSBt-JdjaouVT0pTA-a6mGaF4_A@mail.gmail.com","threadId":"45063","inReplyTo":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2017-02-07T00:24:09Z","receivedAt":"2017-02-07T00:24:15Z","isPatch":false,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"> * sg/completion-refs-speedup (2017-02-06) 13 commits\n> - squash! completion: fill COMPREPLY directly when completing refs\n> - completion: fill COMPREPLY directly when completing refs\n> - completion: list only matching symbolic and pseudorefs when completing refs\n> - completion: let 'for-each-ref' sort remote branches for 'checkout' DWIMery\n> - completion: let 'for-each-ref' filter remote branches for 'checkout' DWIMery\n> - completion: let 'for-each-ref' strip the remote name from remote branches\n> - completion: let 'for-each-ref' and 'ls-remote' filter matching refs\n> - completion: don't disambiguate short refs\n> - completion: don't disambiguate tags and branches\n> - completion: support excluding full refs\n> - completion: support completing full refs after '--option=refs/'\n> - completion: wrap __git_refs() for better option parsing\n> - completion: remove redundant __gitcomp_nl() options from _git_commit()\n> (this branch uses sg/completion.)\n>\n> Will hold.\n> This seems to break 9902 when merged to 'pu'.\n\nAll failing tests fail with the same error:\n\n fatal: unrecognized %(refname:strip=2) argument: strip=2\n\nThat's because of this topic:\n\n> * kn/ref-filter-branch-list (2017-01-31) 20 commits\n> (merged to 'next' on 2017-01-31 at e7592a5461)\n> + branch: implement '--format' option\n> + branch: use ref-filter printing APIs\n> + branch, tag: use porcelain output\n> + ref-filter: allow porcelain to translate messages in the output\n> + ref-filter: add an 'rstrip=' option to atoms which deal with refnames\n> + ref-filter: modify the 'lstrip=' option to work with negative ''\n> + ref-filter: Do not abruptly die when using the 'lstrip=' option\n> + ref-filter: rename the 'strip' option to 'lstrip'\n\nAnd in particular this commit, which, well, does what it's subject\nsays it's doing, thus breaking backwards compatibility.\n\n\n> + ref-filter: make remote_ref_atom_parser() use refname_atom_parser_internal()\n> + ref-filter: introduce refname_atom_parser()\n> + ref-filter: introduce refname_atom_parser_internal()\n> + ref-filter: make \"%(symref)\" atom work with the ':short' modifier\n> + ref-filter: add support for %(upstream:track,nobracket)\n> + ref-filter: make %(upstream:track) prints \"[gone]\" for invalid upstreams\n> + ref-filter: introduce format_ref_array_item()\n> + ref-filter: move get_head_description() from branch.c\n> + ref-filter: modify \"%(objectname:short)\" to take length\n> + ref-filter: implement %(if:equals=) and %(if:notequals=)\n> + ref-filter: include reference to 'used_atom' within 'atom_value'\n> + ref-filter: implement %(if), %(then), and %(else) atoms\n>\n> The code to list branches in \"git branch\" has been consolidated\n> with the more generic ref-filter API.\n>\n> Will cook in 'next'.\n"},{"id":"310964","messageId":"CA+P7+xpdhYG77LA6QkHrJwP0_3t4_0vB11Y0jyw6=w9_w=mY9Q@mail.gmail.com","threadId":"45063","inReplyTo":"CAM0VKjmhO9NQLz9TDv5M3OhxSBt-JdjaouVT0pTA-a6mGaF4_A@mail.gmail.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Jacob Keller","fromEmail":"jacob.keller@gmail.com","sentAt":"2017-02-07T01:17:09Z","receivedAt":"2017-02-07T01:17:37Z","isPatch":false,"sender":{"key":"jacob.keller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/874719?v=4"},"body":"On Mon, Feb 6, 2017 at 4:24 PM, SZEDER Gábor wrote:\n>> * sg/completion-refs-speedup (2017-02-06) 13 commits\n>> - squash! completion: fill COMPREPLY directly when completing refs\n>> - completion: fill COMPREPLY directly when completing refs\n>> - completion: list only matching symbolic and pseudorefs when completing refs\n>> - completion: let 'for-each-ref' sort remote branches for 'checkout' DWIMery\n>> - completion: let 'for-each-ref' filter remote branches for 'checkout' DWIMery\n>> - completion: let 'for-each-ref' strip the remote name from remote branches\n>> - completion: let 'for-each-ref' and 'ls-remote' filter matching refs\n>> - completion: don't disambiguate short refs\n>> - completion: don't disambiguate tags and branches\n>> - completion: support excluding full refs\n>> - completion: support completing full refs after '--option=refs/'\n>> - completion: wrap __git_refs() for better option parsing\n>> - completion: remove redundant __gitcomp_nl() options from _git_commit()\n>> (this branch uses sg/completion.)\n>>\n>> Will hold.\n>> This seems to break 9902 when merged to 'pu'.\n>\n> All failing tests fail with the same error:\n>\n> fatal: unrecognized %(refname:strip=2) argument: strip=2\n>\n> That's because of this topic:\n>\n>> * kn/ref-filter-branch-list (2017-01-31) 20 commits\n>> (merged to 'next' on 2017-01-31 at e7592a5461)\n>> + branch: implement '--format' option\n>> + branch: use ref-filter printing APIs\n>> + branch, tag: use porcelain output\n>> + ref-filter: allow porcelain to translate messages in the output\n>> + ref-filter: add an 'rstrip=' option to atoms which deal with refnames\n>> + ref-filter: modify the 'lstrip=' option to work with negative ''\n>> + ref-filter: Do not abruptly die when using the 'lstrip=' option\n>> + ref-filter: rename the 'strip' option to 'lstrip'\n>\n> And in particular this commit, which, well, does what it's subject\n> says it's doing, thus breaking backwards compatibility.\n>\n\nWhat about making strip a deprecated variant of lstrip?\n\nThanks,\nJake\n"},{"id":"311001","messageId":"xmqqlgthzs6a.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"CAM0VKjmhO9NQLz9TDv5M3OhxSBt-JdjaouVT0pTA-a6mGaF4_A@mail.gmail.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-07T20:01:33Z","receivedAt":"2017-02-07T20:01:42Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"SZEDER Gábor writes:\n\n> All failing tests fail with the same error:\n>\n> fatal: unrecognized %(refname:strip=2) argument: strip=2\n>\n> That's because of this topic:\n>\n>> * kn/ref-filter-branch-list (2017-01-31) 20 commits\n\nAhh, of course.\n\nLet's make sure the series won't escape to 'master' before the\n\"strip\" breakage is fixed. How about queuing this on top of the\nref-filter topic? \n\nIt seems to unblock your completion-refs-speedup topic and makes the\ntest pass ;-)\n\nThanks.\n\n-- >8 --\nSubject: [PATCH] ref-filter: resurrect \"strip\" as a synonym to \"lstrip\"\n\nWe forgot that \"strip\" was introduced at 0571979bd6 (\"tag: do not\nshow ambiguous tag names as \"tags/foo\"\", 2016-01-25) as part of Git\n2.8 (and 2.7.1), yet in the update to ref-filter, we started calling\nit \"lstrip\" to make it easier to explain the new \"rstrip\" operation.\n\nWe shouldn't have renamed the existing one; \"lstrip\" should have\nbeen a new synonym that means the same thing as \"strip\". Scripts\nin the wild are surely using the original form already.\n\nSigned-off-by: Junio C Hamano \n---\n Documentation/git-for-each-ref.txt | 2 ++\n ref-filter.c | 3 ++-\n t/t6300-for-each-ref.sh | 12 ++++++++++++\n 3 files changed, 16 insertions(+), 1 deletion(-)\n\ndiff --git a/Documentation/git-for-each-ref.txt b/Documentation/git-for-each-ref.txt\nindex 2008600e7e..111e1be6f5 100644\n--- a/Documentation/git-for-each-ref.txt\n+++ b/Documentation/git-for-each-ref.txt\n@@ -107,6 +107,8 @@ refname::\n \tenough components, the result becomes an empty string if\n \tstripping with positive , or it becomes the full refname if\n \tstripping with negative . Neither is an error.\n++\n+`strip` can be used as a synomym to `lstrip`.\n \n objecttype::\n \tThe type of the object (`blob`, `tree`, `commit`, `tag`).\ndiff --git a/ref-filter.c b/ref-filter.c\nindex 01b5c18ef0..2a94d6da98 100644\n--- a/ref-filter.c\n+++ b/ref-filter.c\n@@ -112,7 +112,8 @@ static void refname_atom_parser_internal(struct refname_atom *atom,\n \t\tatom->option = R_NORMAL;\n \telse if (!strcmp(arg, \"short\"))\n \t\tatom->option = R_SHORT;\n-\telse if (skip_prefix(arg, \"lstrip=\", &arg)) {\n+\telse if (skip_prefix(arg, \"lstrip=\", &arg) ||\n+\t\t skip_prefix(arg, \"strip=\", &arg)) {\n \t\tatom->option = R_LSTRIP;\n \t\tif (strtol_i(arg, 10, &atom->lstrip))\n \t\t\tdie(_(\"Integer value expected refname:lstrip=%s\"), arg);\ndiff --git a/t/t6300-for-each-ref.sh b/t/t6300-for-each-ref.sh\nindex 25a9973ce9..c87dc1f8bc 100755\n--- a/t/t6300-for-each-ref.sh\n+++ b/t/t6300-for-each-ref.sh\n@@ -59,18 +59,26 @@ test_atom head refname:rstrip=1 refs/heads\n test_atom head refname:rstrip=2 refs\n test_atom head refname:rstrip=-1 refs\n test_atom head refname:rstrip=-2 refs/heads\n+test_atom head refname:strip=1 heads/master\n+test_atom head refname:strip=2 master\n+test_atom head refname:strip=-1 master\n+test_atom head refname:strip=-2 heads/master\n test_atom head upstream refs/remotes/origin/master\n test_atom head upstream:short origin/master\n test_atom head upstream:lstrip=2 origin/master\n test_atom head upstream:lstrip=-2 origin/master\n test_atom head upstream:rstrip=2 refs/remotes\n test_atom head upstream:rstrip=-2 refs/remotes\n+test_atom head upstream:strip=2 origin/master\n+test_atom head upstream:strip=-2 origin/master\n test_atom head push refs/remotes/myfork/master\n test_atom head push:short myfork/master\n test_atom head push:lstrip=1 remotes/myfork/master\n test_atom head push:lstrip=-1 master\n test_atom head push:rstrip=1 refs/remotes/myfork\n test_atom head push:rstrip=-1 refs\n+test_atom head push:strip=1 remotes/myfork/master\n+test_atom head push:strip=-1 master\n test_atom head objecttype commit\n test_atom head objectsize 171\n test_atom head objectname $(git rev-parse refs/heads/master)\n@@ -636,6 +644,10 @@ EOF\n test_expect_success 'Verify usage of %(symref:lstrip) atom' '\n \tgit for-each-ref --format=\"%(symref:lstrip=2)\" refs/heads/sym > actual &&\n \tgit for-each-ref --format=\"%(symref:lstrip=-2)\" refs/heads/sym >> actual &&\n+\ttest_cmp expected actual &&\n+\n+\tgit for-each-ref --format=\"%(symref:strip=2)\" refs/heads/sym > actual &&\n+\tgit for-each-ref --format=\"%(symref:strip=-2)\" refs/heads/sym >> actual &&\n \ttest_cmp expected actual\n '\n \n-- \n2.12.0-rc0-144-g99fe1a5456\n\n"},{"id":"311131","messageId":"CAGZ79kZ6m1NGz9Lhhd5BDxayLW-UC8MqexMyhHBhHKrmg-tB+w@mail.gmail.com","threadId":"45063","inReplyTo":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2017-02-09T00:25:20Z","receivedAt":"2017-02-09T00:25:54Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"> * sb/submodule-doc (2017-01-12) 3 commits\n> - submodules: add a background story\n> - submodule update documentation: don't repeat ourselves\n> - submodule documentation: add options to the subcommand\n>\n> Needs review.\n>\n\nThe first two commits are good to go IIRC as you seemed to be\npositive about them at the time. Though I have a hard time finding\nevidence of such.\n\nI am currently reworking the 3/3 \"add a background story\" patch as it is\nRFC-ish, so no need to review that any more.\n\nMaybe we can get 1&2 merged and then 3 comes on its own?\n\nThanks,\nStefan\n"},{"id":"311133","messageId":"xmqq37fotbwq.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"CAGZ79kZ6m1NGz9Lhhd5BDxayLW-UC8MqexMyhHBhHKrmg-tB+w@mail.gmail.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T01:01:41Z","receivedAt":"2017-02-09T01:01:48Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Stefan Beller writes:\n\n>> * sb/submodule-doc (2017-01-12) 3 commits\n>> - submodules: add a background story\n>> - submodule update documentation: don't repeat ourselves\n>> - submodule documentation: add options to the subcommand\n>\n> The first two commits are good to go IIRC as you seemed to be\n> positive about them at the time. Though I have a hard time finding\n> evidence of such.\n>\n> I am currently reworking the 3/3 \"add a background story\" patch as it is\n> RFC-ish, so no need to review that any more.\n>\n> Maybe we can get 1&2 merged and then 3 comes on its own?\n\nYeah, sorry, I keep forgetting this topic, it seems. Let's advance\nthe earlier two. Tentatively I'd drop the third one as you plan to\nredo it separately.\n\nThanks for prodding.\n\n"},{"id":"311140","messageId":"20170209034657.qbkzbbzuvjpxl422@sigill.intra.peff.net","threadId":"45063","inReplyTo":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-02-09T03:46:58Z","receivedAt":"2017-02-09T03:47:05Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Feb 06, 2017 at 02:34:08PM -0800, Junio C Hamano wrote:\n\n> * sk/parse-remote-cleanup (2017-02-06) 1 commit\n> (merged to 'next' on 2017-02-06 at 6ec89f72d5)\n> + parse-remote: remove reference to unused op_prep\n> \n> Code clean-up.\n> \n> Will merge to 'master'.\n\nHrm. Are the functions in git-parse-remote.sh part of the public API?\nThat is, do we expect third-party scripts to do:\n\n . \"$(git rev-parse --exec)/git-parse-remote.sh\n error_on_missing_default_upstream \"$a\" \"$b\" \"$c\" \"$d\"\n\n? If so, then they may be surprised by the change in function signature.\n\nI generally think of git-sh-setup as the one that external scripts would\nuse. There _is_ a manpage for git-parse-remote, but it doesn't list any\nfunctions. So maybe they're all fair game for changing?\n\nI just didn't see any discussion of this in the original patch thread,\nso I wanted to make sure we were making that decision consciously, and\nnot accidentally. :)\n\n-Peff\n"},{"id":"311146","messageId":"xmqqy3xgrlvh.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"20170209034657.qbkzbbzuvjpxl422@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T05:09:22Z","receivedAt":"2017-02-09T05:09:30Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> On Mon, Feb 06, 2017 at 02:34:08PM -0800, Junio C Hamano wrote:\n>\n>> * sk/parse-remote-cleanup (2017-02-06) 1 commit\n>> (merged to 'next' on 2017-02-06 at 6ec89f72d5)\n>> + parse-remote: remove reference to unused op_prep\n>> \n>> Code clean-up.\n>> \n>> Will merge to 'master'.\n>\n> Hrm. Are the functions in git-parse-remote.sh part of the public API?\n> That is, do we expect third-party scripts to do:\n>\n> . \"$(git rev-parse --exec)/git-parse-remote.sh\n> error_on_missing_default_upstream \"$a\" \"$b\" \"$c\" \"$d\"\n>\n> ? If so, then they may be surprised by the change in function signature.\n>\n> I generally think of git-sh-setup as the one that external scripts would\n> use. There _is_ a manpage for git-parse-remote, but it doesn't list any\n> functions. So maybe they're all fair game for changing?\n>\n> I just didn't see any discussion of this in the original patch thread,\n> so I wanted to make sure we were making that decision consciously, and\n> not accidentally. :)\n\nUmmm, yes, I admit that this was accidental. I didn't really think\nof parse-remote as an externally visible and supported interface,\nbut users have tendency to break our expectations, so, I dunno.\n"},{"id":"311165","messageId":"alpine.DEB.2.20.1702091319350.3496@virtualbox","threadId":"45063","inReplyTo":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-02-09T12:29:33Z","receivedAt":"2017-02-09T12:31:24Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Mon, 6 Feb 2017, Junio C Hamano wrote:\n\n> * sf/putty-w-args (2017-02-01) 5 commits\n> - SQUASH???\n> - connect: Add the envvar GIT_SSH_VARIANT and ssh.variant config\n> - git_connect(): factor out SSH variant handling\n> - connect: rename tortoiseplink and putty variables\n> - connect: handle putty/plink also in GIT_SSH_COMMAND\n> \n> The command line options for ssh invocation needs to be tweaked for\n> some implementations of SSH (e.g. PuTTY plink wants \"-P \"\n> while OpenSSH wants \"-p \" to specify port to connect to), and\n> the variant was guessed when GIT_SSH environment variable is used\n> to specify it. Extend the guess to the command specified by the\n> newer GIT_SSH_COMMAND and also core.sshcommand configuration\n> variable, and give an escape hatch for users to deal with\n> misdetected cases.\n> \n> Stalled?\n> cf. \n\nThe latest messages in that thread are\n\n- your claim that you never said correctness is pused to a back seat (when\n an earlier, detailed mail listed four priorities of your patch review,\n none of which is said correctness, so I did not bother to answer), and\n\n- my answer that suggested to take a break because the conversation turned\n less rational: I had to point out that your objection was not really\n valid in this case.\n\nI now see that you added a SQUASH commit (that was news to me, thank you\nvery much), and that you seem to still insist that the code should prepare\nfor possible future changes in the config settings that may actually never\nmaterialize. (And that would have to be handled at a different point, as I\nhad pointed out, so that suggested preparation would most likely not help\nat all.)\n\nIn short: unless I read any convincing argument in favor of said SQUASH\ncommit, I will remain convinced that v3, as submitted, is actually the\nbest way forward.\n\nThank you for your attention,\nJohannes\n"},{"id":"311185","messageId":"2f67fc21-92f9-a03e-1b09-a237af6dbc46@alum.mit.edu","threadId":"45063","inReplyTo":"xmqqzihzymn3.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2017-02-09T16:08:13Z","receivedAt":"2017-02-09T16:11:38Z","isPatch":false,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"On 02/06/2017 11:34 PM, Junio C Hamano wrote:\n> [...]\n> --------------------------------------------------\n> [Stalled]\n> [...]\n> * mh/ref-remove-empty-directory (2017-01-07) 23 commits\n> - files_transaction_commit(): clean up empty directories\n> - try_remove_empty_parents(): teach to remove parents of reflogs, too\n> - try_remove_empty_parents(): don't trash argument contents\n> - try_remove_empty_parents(): rename parameter \"name\" -> \"refname\"\n> - delete_ref_loose(): inline function\n> - delete_ref_loose(): derive loose reference path from lock\n> - log_ref_write_1(): inline function\n> - log_ref_setup(): manage the name of the reflog file internally\n> - log_ref_write_1(): don't depend on logfile argument\n> - log_ref_setup(): pass the open file descriptor back to the caller\n> - log_ref_setup(): improve robustness against races\n> - log_ref_setup(): separate code for create vs non-create\n> - log_ref_write(): inline function\n> - rename_tmp_log(): improve error reporting\n> - rename_tmp_log(): use raceproof_create_file()\n> - lock_ref_sha1_basic(): use raceproof_create_file()\n> - lock_ref_sha1_basic(): inline constant\n> - raceproof_create_file(): new function\n> - safe_create_leading_directories(): set errno on SCLD_EXISTS\n> - safe_create_leading_directories_const(): preserve errno\n> - t5505: use \"for-each-ref\" to test for the non-existence of references\n> - refname_is_safe(): correct docstring\n> - files_rename_ref(): tidy up whitespace\n> \n> Deletion of a branch \"foo/bar\" could remove .git/refs/heads/foo\n> once there no longer is any other branch whose name begins with\n> \"foo/\", but we didn't do so so far. Now we do.\n> \n> Expecting a reroll.\n> cf. <5051c78e-51f9-becd-e1a6-9c0b781d6912@alum.mit.edu>\n\nI think you missed v4 of this patch series [1], which is the re-roll\nthat you were waiting for. And I missed that you missed it...\n\nMichael\n\n[1] http://public-inbox.org/git/cover.1483719289.git.mhagger@alum.mit.edu/\n\n"},{"id":"311192","messageId":"xmqqfujns2li.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"alpine.DEB.2.20.1702091319350.3496@virtualbox","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T17:20:25Z","receivedAt":"2017-02-09T17:28:34Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin writes:\n\n> (And that would have to be handled at a different point, as I\n> had pointed out, so that suggested preparation would most likely not help\n> at all.)\n\nI did not think \"that would have to be handled at a different point\"\nis correct at all, if by \"a point\" you meant \"a location in the\ncode\" [*1*]. If we want to make it configurable in a more detailed\nway by directly allowing to override port_option and needs_batch\nseparately, you would do that in override_ssh_variant(), without\ntouching handle_ssh_variant() in the refactored code. That way, you\ndo not have to worry about breaking the auto detection based on the\ncommand name.\n\n\n[Footnote]\n\n*1* Or did you mean \"point in time\", aka \"let's do it outside the\n scope of this patch series\"?\n\nLet's not keep it as a SQUASH??? proposal, but as a separate hot-fix\nfollow-up patch.\n\n-- >8 --\nSubject: connect.c: stop conflating ssh command names and overrides\n\ndd33e07766 (\"connect: Add the envvar GIT_SSH_VARIANT and ssh.variant\nconfig\", 2017-02-01) attempted to add support for configuration and\nenvironment variable to override the different handling of\nport_option and needs_batch settings suitable for variants of the\nssh implementation that was autodetected by looking at the ssh\ncommand name. Because it piggybacked on the code that turns command\nname to specific override (e.g. \"plink.exe\" and \"plink\" means\nport_option needs to be set to 'P' instead of the default 'p'), yet\nit defined a separate namespace for these overrides (e.g. \"putty\"\ncan be usable to signal that port_option needs to be 'P'), however,\nit made the auto-detection based on the command name less robust\n(e.g. the code now accepts \"putty\" as a SSH command name and applies\nthe same override).\n\nSeparate the code that interprets the override that was read from\nthe configuration & environment from the original code that handles\nthe command names, as they are in separate namespaces, to fix this\nconfusion.\n\nThis incidentally also makes it easier for future enhancement of the\noverride syntax (e.g. \"port_option=p,needs_batch=1\" may want to be\naccepted as a more explicit syntax) without affecting the code for\nauto-detection based on the command name.\n\nWhile at it, update the return type of the handle_ssh_variant()\nhelper function to void; the caller does not use it, and the\nfunction does not return any meaningful value.\n\nSigned-off-by: Junio C Hamano \n---\n connect.c | 45 ++++++++++++++++++++++++++++++++-------------\n 1 file changed, 32 insertions(+), 13 deletions(-)\n\ndiff --git a/connect.c b/connect.c\nindex 7f1f802396..7d65c1c736 100644\n--- a/connect.c\n+++ b/connect.c\n@@ -691,17 +691,39 @@ static const char *get_ssh_command(void)\n \treturn NULL;\n }\n \n-static int handle_ssh_variant(const char *ssh_command, int is_cmdline,\n-\t\t\t int *port_option, int *needs_batch)\n+static int override_ssh_variant(int *port_option, int *needs_batch)\n {\n-\tconst char *variant = getenv(\"GIT_SSH_VARIANT\");\n+\tchar *variant;\n+\n+\tvariant = xstrdup_or_null(getenv(\"GIT_SSH_VARIANT\"));\n+\tif (!variant &&\n+\t git_config_get_string(\"ssh.variant\", &variant))\n+\t\treturn 0;\n+\n+\tif (!strcmp(variant, \"plink\") || !strcmp(variant, \"putty\")) {\n+\t\t*port_option = 'P';\n+\t\t*needs_batch = 0;\n+\t} else if (!strcmp(variant, \"tortoiseplink\")) {\n+\t\t*port_option = 'P';\n+\t\t*needs_batch = 1;\n+\t} else {\n+\t\t*port_option = 'p';\n+\t\t*needs_batch = 0;\n+\t}\n+\tfree(variant);\n+\treturn 1;\n+}\n+\n+static void handle_ssh_variant(const char *ssh_command, int is_cmdline,\n+\t\t\t int *port_option, int *needs_batch)\n+{\n+\tconst char *variant;\n \tchar *p = NULL;\n \n-\tif (variant)\n-\t\t; /* okay, fall through */\n-\telse if (!git_config_get_string(\"ssh.variant\", &p))\n-\t\tvariant = p;\n-\telse if (!is_cmdline) {\n+\tif (override_ssh_variant(port_option, needs_batch))\n+\t\treturn;\n+\n+\tif (!is_cmdline) {\n \t\tp = xstrdup(ssh_command);\n \t\tvariant = basename(p);\n \t} else {\n@@ -717,12 +739,11 @@ static int handle_ssh_variant(const char *ssh_command, int is_cmdline,\n \t\t\t */\n \t\t\tfree(ssh_argv);\n \t\t} else\n-\t\t\treturn 0;\n+\t\t\treturn;\n \t}\n \n \tif (!strcasecmp(variant, \"plink\") ||\n-\t !strcasecmp(variant, \"plink.exe\") ||\n-\t !strcasecmp(variant, \"putty\"))\n+\t !strcasecmp(variant, \"plink.exe\"))\n \t\t*port_option = 'P';\n \telse if (!strcasecmp(variant, \"tortoiseplink\") ||\n \t\t !strcasecmp(variant, \"tortoiseplink.exe\")) {\n@@ -730,8 +751,6 @@ static int handle_ssh_variant(const char *ssh_command, int is_cmdline,\n \t\t*needs_batch = 1;\n \t}\n \tfree(p);\n-\n-\treturn 1;\n }\n \n /*\n-- \n2.12.0-rc0-221-g3e954cf1aa\n\n"},{"id":"311200","messageId":"xmqq37fnqikg.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"2f67fc21-92f9-a03e-1b09-a237af6dbc46@alum.mit.edu","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T19:18:23Z","receivedAt":"2017-02-09T19:30:19Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Michael Haggerty writes:\n\n> On 02/06/2017 11:34 PM, Junio C Hamano wrote:\n>> [...]\n>> --------------------------------------------------\n>> [Stalled]\n>> [...]\n>> * mh/ref-remove-empty-directory (2017-01-07) 23 commits\n>> \n>> Deletion of a branch \"foo/bar\" could remove .git/refs/heads/foo\n>> once there no longer is any other branch whose name begins with\n>> \"foo/\", but we didn't do so so far. Now we do.\n>> \n>> Expecting a reroll.\n>> cf. <5051c78e-51f9-becd-e1a6-9c0b781d6912@alum.mit.edu>\n>\n> I think you missed v4 of this patch series [1], which is the re-roll\n> that you were waiting for. And I missed that you missed it...\n>\n> Michael\n>\n> [1] http://public-inbox.org/git/cover.1483719289.git.mhagger@alum.mit.edu/\n\nGreat. Thanks.\n"},{"id":"311206","messageId":"xmqqy3xfp3pk.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"2f67fc21-92f9-a03e-1b09-a237af6dbc46@alum.mit.edu","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T19:24:39Z","receivedAt":"2017-02-09T19:58:24Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Michael Haggerty writes:\n\n> On 02/06/2017 11:34 PM, Junio C Hamano wrote:\n>> [...]\n>> --------------------------------------------------\n>> [Stalled]\n>> [...]\n>> * mh/ref-remove-empty-directory (2017-01-07) 23 commits\n>> - files_transaction_commit(): clean up empty directories\n>> - try_remove_empty_parents(): teach to remove parents of reflogs, too\n>> - try_remove_empty_parents(): don't trash argument contents\n>> - try_remove_empty_parents(): rename parameter \"name\" -> \"refname\"\n>> - delete_ref_loose(): inline function\n>> - delete_ref_loose(): derive loose reference path from lock\n>> - log_ref_write_1(): inline function\n>> - log_ref_setup(): manage the name of the reflog file internally\n>> - log_ref_write_1(): don't depend on logfile argument\n>> - log_ref_setup(): pass the open file descriptor back to the caller\n>> - log_ref_setup(): improve robustness against races\n>> - log_ref_setup(): separate code for create vs non-create\n>> - log_ref_write(): inline function\n>> - rename_tmp_log(): improve error reporting\n>> - rename_tmp_log(): use raceproof_create_file()\n>> - lock_ref_sha1_basic(): use raceproof_create_file()\n>> - lock_ref_sha1_basic(): inline constant\n>> - raceproof_create_file(): new function\n>> - safe_create_leading_directories(): set errno on SCLD_EXISTS\n>> - safe_create_leading_directories_const(): preserve errno\n>> - t5505: use \"for-each-ref\" to test for the non-existence of references\n>> - refname_is_safe(): correct docstring\n>> - files_rename_ref(): tidy up whitespace\n>> \n>> Deletion of a branch \"foo/bar\" could remove .git/refs/heads/foo\n>> once there no longer is any other branch whose name begins with\n>> \"foo/\", but we didn't do so so far. Now we do.\n>> \n>> Expecting a reroll.\n>> cf. <5051c78e-51f9-becd-e1a6-9c0b781d6912@alum.mit.edu>\n>\n> I think you missed v4 of this patch series [1], which is the re-roll\n> that you were waiting for. And I missed that you missed it...\n>\n> Michael\n>\n> [1] http://public-inbox.org/git/cover.1483719289.git.mhagger@alum.mit.edu/\n\nActually it was worse than that. What the above lists *is* v4; I\njust failed to update \"Expecting a reroll\" note when I updated the\ntopic with your rerolled patches, and left it there trusting the\nnow-stale note of mine.\n\nSorry, and a HUGE thanks for noticing the mistake.\n"},{"id":"311207","messageId":"alpine.DEB.2.20.1702092135050.3496@virtualbox","threadId":"45063","inReplyTo":"xmqqfujns2li.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-02-09T20:37:04Z","receivedAt":"2017-02-09T20:38:13Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Thu, 9 Feb 2017, Junio C Hamano wrote:\n\n> Johannes Schindelin writes:\n> \n> > (And that would have to be handled at a different point, as I had\n> > pointed out, so that suggested preparation would most likely not help\n> > at all.)\n> \n> I did not think \"that would have to be handled at a different point\"\n> is correct at all, if by \"a point\" you meant \"a location in the\n> code\" [*1*].\n\nYes, I mean the location in the code.\n\nBut since you keep insisting that you are right and I am wrong, and even\ngo so far as calling your patch reverting my refactoring a hot-fix, why\ndon't you just go ahead and merge the result over my objections?\n\nCiao,\nJohannes\n"},{"id":"311222","messageId":"xmqqk28znhtx.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"alpine.DEB.2.20.1702092135050.3496@virtualbox","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-09T22:02:34Z","receivedAt":"2017-02-09T22:02:52Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin writes:\n\n>> > (And that would have to be handled at a different point, as I had\n>> > pointed out, so that suggested preparation would most likely not help\n>> > at all.)\n>> \n>> I did not think \"that would have to be handled at a different point\"\n>> is correct at all, if by \"a point\" you meant \"a location in the\n>> code\" [*1*].\n>\n> Yes, I mean the location in the code.\n>\n> But since you keep insisting that you are right and I am wrong,...\n\nThere is no \"insisting\". Didn't we just see how wrong you were\nabout \"different point\"? An extended syntax of override would be\nhandled in the new helper to handle override, the same point in the\ncode as other overrides are handled.\n\n> ... and even\n> go so far as calling your patch reverting my refactoring a hot-fix, why\n> don't you just go ahead and merge the result over my objections?\n\nAt this point, you are simply being silly.\n\nIsn't \"Putty is not a command but is also handled as if it is a\nvalid implementation of SSH\" a bug? Isn't making the code not to be\nconfused like so a fix?\n\nA different approach to fix the issue would be to declare that the\ncommand names and overrides are not in separate namespaces.\n\nIf you prefer to go that route, the documentation can use an update\nto make it not mention \"putty\" as a valid override (the users have\nto spell \"plink\"), mention \"plink.exe\" is also accepted, etc. and\nmake it clear that the override environment and configuration\nvariables are the way to tell Git: \"The ssh implementation I have\nbehaves the same way as this well-known implementation, so treat it\nas such without actually looking at the path to the command in the\nssh.command string\".\n\nThat may limit the freedom for future enhancement of overrides, but\nis an acceptable short-cut. After all, the overrides are merely an\nescape hatch and we expect them to be used only rarely.\n"},{"id":"311233","messageId":"alpine.DEB.2.20.1702092329440.3496@virtualbox","threadId":"45063","inReplyTo":"xmqqk28znhtx.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-02-09T22:32:25Z","receivedAt":"2017-02-09T22:40:16Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\n\nOn Thu, 9 Feb 2017, Junio C Hamano wrote:\n\n> Johannes Schindelin writes:\n> \n> > ... and even go so far as calling your patch reverting my refactoring\n> > a hot-fix, why don't you just go ahead and merge the result over my\n> > objections?\n> \n> At this point, you are simply being silly.\n\nHow is it that this patch cannot be applied when, and if, that\nhypothetical config setting is introduced?\n\nMaybe I am dense here, but I would really like to know why this\npreparatory patch must be applied *now*, when there is nothing to prepare\nfor.\n\n> Isn't \"Putty is not a command but is also handled as if it is a valid\n> implementation of SSH\" a bug?\n\nIf you think it is a bug to handle an ssh command called \"putty\" as if it\nwere plink, sure. I do not think there is a valid use case, but hey.\n\nCiao,\nJohannes\n"},{"id":"311312","messageId":"xmqqh941kanv.fsf@gitster.mtv.corp.google.com","threadId":"45063","inReplyTo":"xmqqy3xgrlvh.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Feb 2017, #02; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-02-10T21:17:56Z","receivedAt":"2017-02-10T21:18:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> Jeff King writes:\n>\n>> On Mon, Feb 06, 2017 at 02:34:08PM -0800, Junio C Hamano wrote:\n>>\n>>> * sk/parse-remote-cleanup (2017-02-06) 1 commit\n>>> (merged to 'next' on 2017-02-06 at 6ec89f72d5)\n>>> + parse-remote: remove reference to unused op_prep\n>>> \n>>> Code clean-up.\n>>> \n>>> Will merge to 'master'.\n>>\n>> Hrm. Are the functions in git-parse-remote.sh part of the public API?\n>> That is, do we expect third-party scripts to do:\n>>\n>> . \"$(git rev-parse --exec)/git-parse-remote.sh\n>> error_on_missing_default_upstream \"$a\" \"$b\" \"$c\" \"$d\"\n>>\n>> ? If so, then they may be surprised by the change in function signature.\n>>\n>> I generally think of git-sh-setup as the one that external scripts would\n>> use. There _is_ a manpage for git-parse-remote, but it doesn't list any\n>> functions. So maybe they're all fair game for changing?\n>>\n>> I just didn't see any discussion of this in the original patch thread,\n>> so I wanted to make sure we were making that decision consciously, and\n>> not accidentally. :)\n>\n> Ummm, yes, I admit that this was accidental. I didn't really think\n> of parse-remote as an externally visible and supported interface,\n> but users have tendency to break our expectations, so, I dunno.\n\nAfter sleeping on this, I doubt that the value of this \"code\nclean-up\" is worth the trouble of waiting to see if a third-party\nwho dot sources parse-remote steps up, which may never materialize\nwhile the topic is cooking in 'next' and more importantly risking\nbreakage on such a third-party.\n\nLet's drop the topic and excise it from 'next' at the next version\nboundary.\n"}]}