{"thread":{"id":"47550","subject":"[ANNOUNCE] Git v2.16.0-rc1","startedAt":"2018-01-05T23:31:23Z","lastAt":"2018-01-10T20:09:38Z","messageCount":9,"participants":["Junio C Hamano","Johannes Schindelin","Johannes Sixt"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"336029","messageId":"xmqqbmi7ano1.fsf@gitster.mtv.corp.google.com","threadId":"47550","inReplyTo":null,"subject":"[ANNOUNCE] Git v2.16.0-rc1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-01-05T23:30:54Z","receivedAt":"2018-01-05T23:31:23Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"A release candidate Git v2.16.0-rc1 is now available for testing\nat the usual places.  It is comprised of 455 non-merge commits\nsince v2.15.0, contributed by 79 people, 23 of which are new faces.\n\nThe tarballs are found at:\n\n    https://www.kernel.org/pub/software/scm/git/testing/\n\nThe following public repositories all have a copy of the\n'v2.16.0-rc1' tag and the 'master' branch that the tag points at:\n\n  url = https://kernel.googlesource.com/pub/scm/git/git\n  url = git://repo.or.cz/alt-git.git\n  url = https://github.com/gitster/git\n\nNew contributors whose contributions weren't in v2.15.0 are as follows.\nWelcome to the Git development community!\n\n  Albert Astals Cid, Antoine Beaupré, Damien Marié, Daniel\n  Bensoussan, Florian Klink, Gennady Kupava, Guillaume Castagnino,\n  Haaris Mehmood, Hans Jerry Illikainen, Ingo Ruhnke, Jakub\n  Bereżański, Jean Carlo Machado, Julien Dusser, J Wyman,\n  Kevin, Łukasz Stelmach, Marius Paliga, Olga Telezhnaya,\n  Rafael Ascensão, Robert Abel, Robert P. J. Day, Shuyu Wei,\n  and Wei Shuyu.\n\nReturning contributors who helped this release are as follows.\nThanks for your continued support.\n\n  Adam Dinwoodie, Ævar Arnfjörð Bjarmason, Alex Vandiver,\n  Anders Kaseorg, Andrey Okoshkin, Ann T Ropea, Beat Bolli,\n  Ben Peart, Brandon Williams, brian m. carlson, Carlos Martín\n  Nieto, Charles Bailey, Christian Couder, Dave Borowitz, Dennis\n  Kaarsemaker, Derrick Stolee, Elijah Newren, Emily Xie, Eric\n  Sunshine, Eric Wong, Heiko Voigt, Jacob Keller, Jameson Miller,\n  Jean-Noel Avila, Jeff Hostetler, Jeff King, Johannes Schindelin,\n  Jonathan Nieder, Jonathan Tan, Junio C Hamano, Kaartic Sivaraam,\n  Kevin Daudt, Lars Schneider, Liam Beguin, Luke Diamand, Martin\n  Ågren, Michael Haggerty, Nicolas Morey-Chaisemartin, Phil\n  Hord, Phillip Wood, Pranit Bauva, Prathamesh Chavan, Ramsay\n  Jones, Randall S. Becker, Rasmus Villemoes, René Scharfe,\n  Simon Ruderich, Stefan Beller, Steffen Prohaska, Stephan Beyer,\n  SZEDER Gábor, Thomas Braun, Thomas Gummerer, Todd Zullinger,\n  Torsten Bögershausen, and W. Trevor King.\n\n----------------------------------------------------------------\n\nGit 2.16 Release Notes (draft)\n==============================\n\nBackward compatibility notes and other notable changes.\n\n * Use of an empty string as a pathspec element that is used for\n   'everything matches' is now an error.\n\n\nUpdates since v2.15\n-------------------\n\nUI, Workflows & Features\n\n * An empty string as a pathspec element that means \"everything\"\n   i.e. 'git add \"\"', is now illegal.  We started this by first\n   deprecating and warning a pathspec that has such an element in\n   2.11 (Nov 2016).\n\n * A hook script that is set unexecutable is simply ignored.  Git\n   notifies when such a file is ignored, unless the message is\n   squelched via advice.ignoredHook configuration.\n\n * \"git pull\" has been taught to accept \"--[no-]signoff\" option and\n   pass it down to \"git merge\".\n\n * The \"--push-option=<string>\" option to \"git push\" now defaults to a\n   list of strings configured via push.pushOption variable.\n\n * \"gitweb\" checks if a directory is searchable with Perl's \"-x\"\n   operator, which can be enhanced by using \"filetest 'access'\"\n   pragma, which now we do.\n\n * \"git stash save\" has been deprecated in favour of \"git stash push\".\n\n * The set of paths output from \"git status --ignored\" was tied\n   closely with its \"--untracked=<mode>\" option, but now it can be\n   controlled more flexibly.  Most notably, a directory that is\n   ignored because it is listed to be ignored in the ignore/exclude\n   mechanism can be handled differently from a directory that ends up\n   to be ignored only because all files in it are ignored.\n\n * The remote-helper for talking to MediaWiki has been updated to\n   truncate an overlong pagename so that \".mw\" suffix can still be\n   added.\n\n * The remote-helper for talking to MediaWiki has been updated to\n   work with mediawiki namespaces.\n\n * The \"--format=...\" option \"git for-each-ref\" takes learned to show\n   the name of the 'remote' repository and the ref at the remote side\n   that is affected for 'upstream' and 'push' via \"%(push:remotename)\"\n   and friends.\n\n * Doc and message updates to teach users \"bisect view\" is a synonym\n   for \"bisect visualize\".\n\n * \"git bisect run\" that did not specify any command to run used to go\n   ahead and treated all commits to be tested as 'good'.  This has\n   been corrected by making the command error out.\n\n * The SubmittingPatches document has been converted to produce an\n   HTML version via AsciiDoc/Asciidoctor.\n\n * We learned to talk to watchman to speed up \"git status\" and other\n   operations that need to see which paths have been modified.\n\n * The \"diff\" family of commands learned to ignore differences in\n   carriage return at the end of line.\n\n * Places that know about \"sendemail.to\", like documentation and shell\n   completion (in contrib/) have been taught about \"sendemail.tocmd\",\n   too.\n\n * \"git add --renormalize .\" is a new and safer way to record the fact\n   that you are correcting the end-of-line convention and other\n   \"convert_to_git()\" glitches in the in-repository data.\n\n * \"git branch\" and \"git checkout -b\" are now forbidden from creating\n   a branch whose name is \"HEAD\".\n\n * \"git branch --list\" learned to show its output through the pager by\n   default when the output is going to a terminal, which is controlled\n   by the pager.branch configuration variable.  This is similar to a\n   recent change to \"git tag --list\".\n\n * \"git grep -W\", \"git diff -W\" and their friends learned a heuristic\n   to extend a pre-context beyond the line that matches the \"function\n   pattern\" (aka \"diff.*.xfuncname\") to include a comment block, if\n   exists, that immediately precedes it.\n\n * \"git config --expiry-date gc.reflogexpire\" can read \"2.weeks\" from\n   the configuration and report it as a timestamp, just like \"--int\"\n   would read \"1k\" and report 1024, to help consumption by scripts.\n\n * The shell completion (in contrib/) learned that \"git pull\" can take\n   the \"--autostash\" option.\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 * \"git grep\" compiled with libpcre2 sometimes triggered a segfault,\n   which is being fixed.\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 * \"git diff\" learned, \"--anchored\", a variant of the \"--patience\"\n   algorithm, to which the user can specify which 'unique' line to be\n   used as anchoring points.\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 * 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 * With a configuration variable rebase.abbreviateCommands set,\n   \"git rebase -i\" produces the todo list with a single-letter\n   command names.\n\n * \"git worktree add\" learned to run the post-checkout hook, just like\n   \"git checkout\" does, after the initial checkout.\n\n * \"git svn\" has been updated to strip CRs in the commit messages, as\n   recent versions of Subversion rejects them.\n\n * \"git imap-send\" did not correctly quote the folder name when\n   making a request to the server, which has been corrected.\n\n * Error messages from \"git rebase\" have been somewhat cleaned up.\n\n * Git has been taught to support an https:// URL used for http.proxy\n   when using recent versions of libcurl.\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 * \"git describe\" was taught to dig trees deeper to find a\n   <commit-ish>:<path> that refers to a given blob object.\n\n\nPerformance, Internal Implementation, Development Support etc.\n\n * An earlier update made it possible to use an on-stack in-core\n   lockfile structure (as opposed to having to deliberately leak an\n   on-heap one).  Many codepaths have been updated to take advantage\n   of this new facility.\n\n * Calling cmd_foo() as if it is a general purpose helper function is\n   a no-no.  Correct two instances of such to set an example.\n\n * We try to see if somebody runs our test suite with a shell that\n   does not support \"local\" like bash/dash does.\n\n * An early part of piece-by-piece rewrite of \"git bisect\" in C.\n\n * GSoC to piece-by-piece rewrite \"git submodule\" in C.\n\n * Optimize the code to find shortest unique prefix of object names.\n\n * Pathspec-limited revision traversal was taught not to keep finding\n   unneeded differences once it knows two trees are different inside\n   given pathspec.\n\n * Conversion from uchar[20] to struct object_id continues.\n\n * Code cleanup.\n\n * A single-word \"unsigned flags\" in the diff options is being split\n   into a structure with many bitfields.\n\n * TravisCI build updates.\n\n * Parts of a test to drive the long-running content filter interface\n   has been split into its own module, hopefully to eventually become\n   reusable.\n\n * Drop (perhaps overly cautious) sanity check before using the index\n   read from the filesystem at runtime.\n\n * The build procedure has been taught to avoid some unnecessary\n   instability in the build products.\n\n * A new mechanism to upgrade the wire protocol in place is proposed\n   and demonstrated that it works with the older versions of Git\n   without harming them.\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 * The code to iterate over loose object files got optimized.\n\n * An internal function that was left for backward compatibility has\n   been removed, as there is no remaining callers.\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 * The tracing infrastructure has been optimized for cases where no\n   tracing is requested.\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 * 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 * Assorted updates for TravisCI integration.\n   (merge 4f26366679 sg/travis-fixes later to maint).\n\n * Introduce a helper to simplify code to parse a common pattern that\n   expects either \"--key\" or \"--key=<something>\".\n\n * \"git version --build-options\" learned to report the host CPU and\n   the exact commit object name the binary was built from.\n\nAlso contains various documentation updates and code clean-ups.\n\n\nFixes since v2.15\n-----------------\n\n * \"auto\" as a value for the columnar output configuration ought to\n   judge \"is the output consumed by humans?\" with the same criteria as\n   \"auto\" for coloured output configuration, i.e. either the standard\n   output stream is going to tty, or a pager is in use.  We forgot the\n   latter, which has been fixed.\n\n * The experimental \"color moved lines differently in diff output\"\n   feature was buggy around \"ignore whitespace changes\" edges, which\n   has been corrected.\n\n * Instead of using custom line comparison and hashing functions to\n   implement \"moved lines\" coloring in the diff output, use the pair\n   of these functions from lower-layer xdiff/ code.\n\n * Some codepaths did not check for errors when asking what branch the\n   HEAD points at, which have been fixed.\n\n * \"git commit\", after making a commit, did not check for errors when\n   asking on what branch it made the commit, which has been corrected.\n\n * \"git status --ignored -u\" did not stop at a working tree of a\n   separate project that is embedded in an ignored directory and\n   listed files in that other project, instead of just showing the\n   directory itself as ignored.\n\n * A broken access to object databases in recent update to \"git grep\n   --recurse-submodules\" has been fixed.\n\n * A recent regression in \"git rebase -i\" that broke execution of git\n   commands from subdirectories via \"exec\" instruction has been fixed.\n\n * A (possibly flakey) test fix.\n\n * \"git check-ref-format --branch @{-1}\" bit a \"BUG()\" when run\n   outside a repository for obvious reasons; clarify the documentation\n   and make sure we do not even try to expand the at-mark magic in\n   such a case, but still call the validation logic for branch names.\n\n * \"git fetch --recurse-submodules\" now knows that submodules can be\n   moved around in the superproject in addition to getting updated,\n   and finds the ones that need to be fetched accordingly.\n\n * Command line completion (in contrib/) update.\n\n * Description of blame.{showroot,blankboundary,showemail,date}\n   configuration variables have been added to \"git config --help\".\n\n * After an error from lstat(), diff_populate_filespec() function\n   sometimes still went ahead and used invalid data in struct stat,\n   which has been fixed.\n\n * UNC paths are also relevant in Cygwin builds and they are now\n   tested just like Mingw builds.\n\n * Correct start-up sequence so that a repository could be placed\n   immediately under the root directory again (which was broken at\n   around Git 2.13).\n\n * The credential helper for libsecret (in contrib/) has been improved\n   to allow possibly prompting the end user to unlock secrets that are\n   currently locked (otherwise the secrets may not be loaded).\n\n * MinGW updates.\n\n * Error checking in \"git imap-send\" for empty response has been\n   improved.\n\n * Recent update to the refs infrastructure implementation started\n   rewriting packed-refs file more often than before; this has been\n   optimized again for most trivial cases.\n\n * Some error messages did not quote filenames shown in it, which have\n   been fixed.\n\n * \"git rebase -i\" recently started misbehaving when a submodule that\n   is configured with 'submodule.<name>.ignore' is dirty; this has\n   been corrected.\n\n * Building with NO_LIBPCRE1_JIT did not disable it, which has been fixed.\n\n * We used to add an empty alternate object database to the system\n   that does not help anything; it has been corrected.\n\n * Doc update around use of \"format-patch --subject-prefix\" etc.\n\n * A fix for an ancient bug in \"git apply --ignore-space-change\" codepath.\n\n * Clarify and enhance documentation for \"merge-base --fork-point\", as\n   it was clear what it computed but not why/what for.\n\n * A few scripts (both in production and tests) incorrectly redirected\n   their error output.  These have been corrected.\n\n * \"git notes\" sent its error message to its standard output stream,\n   which was corrected.\n\n * The three-way merge performed by \"git cherry-pick\" was confused\n   when a new submodule was added in the meantime, which has been\n   fixed (or \"papered over\").\n\n * The sequencer machinery (used by \"git cherry-pick A..B\", and \"git\n   rebase -i\", among other things) would have lost a commit if stopped\n   due to an unlockable index file, which has been fixed.\n\n * \"git apply --inaccurate-eof\" when used with \"--ignore-space-change\"\n   triggered an internal sanity check, which has been fixed.\n\n * Command line completion (in contrib/) has been taught about the\n   \"--copy\" option of \"git branch\".\n\n * When \"git rebase\" prepared an mailbox of changes and fed it to \"git\n   am\" to replay them, it was confused when a stray \"From \" happened\n   to be in the log message of one of the replayed changes.  This has\n   been corrected.\n\n * There was a recent semantic mismerge in the codepath to write out a\n   section of a configuration section, which has been corrected.\n\n * Mentions of \"git-rebase\" and \"git-am\" (dashed form) still remained\n   in end-user visible strings emitted by the \"git rebase\" command;\n   they have been corrected.\n\n * Contrary to the documentation, \"git pull -4/-6 other-args\" did not\n   ask the underlying \"git fetch\" to go over IPv4/IPv6, which has been\n   corrected.\n\n * \"git checkout --recursive\" may overwrite and rewind the history of\n   the branch that happens to be checked out in submodule\n   repositories, which might not be desirable.  Detach the HEAD but\n   still allow the recursive checkout to succeed in such a case.\n   (merge 57f22bf997 sb/submodule-recursive-checkout-detach-head later to maint).\n\n * \"git branch --set-upstream\" has been deprecated and (sort of)\n   removed, as \"--set-upstream-to\" is the preferred one these days.\n   The documentation still had \"--set-upstream\" listed on its\n   synopsis section, which has been corrected.\n   (merge a060f3d3d8 tz/branch-doc-remove-set-upstream later to maint).\n\n * Internally we use 0{40} as a placeholder object name to signal the\n   codepath that there is no such object (e.g. the fast-forward check\n   while \"git fetch\" stores a new remote-tracking ref says \"we know\n   there is no 'old' thing pointed at by the ref, as we are creating\n   it anew\" by passing 0{40} for the 'old' side), and expect that a\n   codepath to locate an in-core object to return NULL as a sign that\n   the object does not exist.  A look-up for an object that does not\n   exist however is quite costly with a repository with large number\n   of packfiles.  This access pattern has been optimized.\n   (merge 87b5e236a1 jk/fewer-pack-rescan later to maint).\n\n * In addition to \"git stash -m message\", the command learned to\n   accept \"git stash -mmessage\" form.\n   (merge 5675473fcb ph/stash-save-m-option-fix later to maint).\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   (merge 75ce149575 ks/doc-checkout-previous later to maint).\n\n * A regression in the progress eye-candy was fixed.\n   (merge 9c5951cacf jk/progress-delay-fix later to maint).\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   (merge 4cba2b0108 en/merge-recursive-icase-removal later to maint).\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   (merge eef3df5a93 bw/pathspec-match-submodule-boundary later to maint).\n\n * Amending commits in git-gui broke the author name that is non-ascii\n   due to incorrect enconding conversion.\n\n * Recent update to the submodule configuration code broke \"diff-tree\"\n   by accidentally stopping to read from the index upfront.\n   (merge fd66bcc31f bw/submodule-config-cleanup later to maint).\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   (merge abfb04d0c7 ls/editor-waiting-message later to maint).\n\n * The \"safe crlf\" check incorrectly triggered for contents that does\n   not use CRLF as line endings, which has been corrected.\n   (merge 649f1f0948 tb/check-crlf-for-safe-crlf later to maint).\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   (merge b3b05971c1 es/clone-shared-worktree later to maint).\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   (merge 65289e9dcd rb/quick-install-doc later to maint).\n\n * Update the shell prompt script (in contrib/) to strip trailing CR\n   from strings read from various \"state\" files.\n   (merge 041fe8fc83 ra/prompt-eread-fix later to maint).\n\n * \"git merge -s recursive\" did not correctly abort when the index is\n   dirty, if the merged tree happened to be the same as the current\n   HEAD, which has been fixed.\n\n * Bytes with high-bit set were encoded incorrectly and made\n   credential helper fail.\n   (merge 4c267f2ae3 jd/fix-strbuf-add-urlencode-bytes later to maint).\n\n * Other minor doc, test and build updates and code cleanups.\n   (merge 1a1fc2d5b5 rd/man-prune-progress later to maint).\n   (merge 0ba014035a rd/man-reflog-add-n later to maint).\n   (merge e54b63359f rd/doc-notes-prune-fix later to maint).\n   (merge ff4c9b413a sp/doc-info-attributes later to maint).\n   (merge 7db2cbf4f1 jc/receive-pack-hook-doc later to maint).\n   (merge 5a0526264b tg/t-readme-updates later to maint).\n   (merge 5e83cca0b8 jk/no-optional-locks later to maint).\n   (merge 826c778f7c js/hashmap-update-sample later to maint).\n   (merge 176b2d328c sg/setup-doc-update later to maint).\n   (merge 1b09073514 rs/am-builtin-leakfix later to maint).\n   (merge addcf6cfde rs/fmt-merge-msg-string-leak-fix later to maint).\n   (merge c3ff8f6c14 rs/strbuf-read-once-reset-length later to maint).\n   (merge 6b0eb884f9 db/doc-workflows-neuter-the-maintainer later to maint).\n   (merge 8c87bdfb21 jk/cvsimport-quoting later to maint).\n   (merge 176cb979fe rs/fmt-merge-msg-leakfix later to maint).\n   (merge 5a03360e73 tb/delimit-pretty-trailers-args-with-comma later to maint).\n   (merge d0e6326026 ot/pretty later to maint).\n   (merge 44103f4197 sb/test-helper-excludes later to maint).\n   (merge 170078693f jt/transport-no-more-rsync later to maint).\n   (merge c07b3adff1 bw/path-doc later to maint).\n   (merge bf9d7df950 tz/lib-git-svn-svnserve-tests later to maint).\n   (merge dec366c9a8 sr/http-sslverify-config-doc later to maint).\n   (merge 3f824e91c8 jk/test-suite-tracing later to maint).\n   (merge 1feb061701 db/doc-config-section-names-with-bs later to maint).\n   (merge 74dea0e13c jh/memihash-opt later to maint).\n\n----------------------------------------------------------------\n\nChanges since v2.15.0 are as follows:\n\nAdam Dinwoodie (3):\n      t5580: add Cygwin support\n      rebase -i: fix comment typo\n      doc/SubmittingPatches: correct subject guidance\n\nAlbert Astals Cid (1):\n      completion: add --autostash and --no-autostash to pull\n\nAlex Vandiver (7):\n      fsmonitor: read entirety of watchman output\n      fsmonitor: set the PWD to the top of the working tree\n      fsmonitor: don't bother pretty-printing JSON from watchman\n      fsmonitor: document GIT_TRACE_FSMONITOR\n      fsmonitor: delay updating state until after split index is merged\n      fsmonitor: read from getcwd(), not the PWD environment variable\n      fsmonitor: store fsmonitor bitmap before splitting index\n\nAnders Kaseorg (1):\n      git-gui: sort entries in optimized tclIndex\n\nAndrey Okoshkin (3):\n      commit: check result of resolve_ref_unsafe\n      diff: fix lstat() error handling in diff_populate_filespec()\n      merge-recursive: check GIT_MERGE_VERBOSITY only once\n\nAnn T Ropea (7):\n      Documentation: revisions: fix typo: \"three dot\" ---> \"three-dot\" (in line with \"two-dot\").\n      Documentation: user-manual: limit usage of ellipsis\n      print_sha1_ellipsis: introduce helper\n      checkout: describe_detached_head: remove ellipsis after committish\n      t4013: prepare for upcoming \"diff --raw --abbrev\" output format change\n      diff: diff_aligned_abbrev: remove ellipsis after abbreviated SHA-1 value\n      t4013: test new output from diff --abbrev --raw\n\nAntoine Beaupré (6):\n      remote-mediawiki: limit filenames to legal\n      remote-mediawiki: show known namespace choices on failure\n      remote-mediawiki: skip virtual namespaces\n      remote-mediawiki: support fetching from (Main) namespace\n      remote-mediawiki: process namespaces in order\n      remote-mediawiki: show progress while fetching namespaces\n\nBeat Bolli (1):\n      contrib/git-jump: allow to configure the grep command\n\nBen Peart (16):\n      bswap: add 64 bit endianness helper get_be64\n      preload-index: add override to enable testing preload-index\n      update-index: add a new --force-write-index option\n      fsmonitor: teach git to optionally utilize a file system monitor to speed up detecting new or changed files.\n      fsmonitor: add documentation for the fsmonitor extension.\n      ls-files: Add support in ls-files to display the fsmonitor valid bit\n      update-index: add fsmonitor support to update-index\n      fsmonitor: add a test tool to dump the index extension\n      split-index: disable the fsmonitor extension when running the split index test\n      fsmonitor: add test cases for fsmonitor extension\n      fsmonitor: add a sample integration script for Watchman\n      fsmonitor: add a performance test\n      fsmonitor: MINGW support for watchman integration\n      fsmonitor: preserve utf8 filenames in fsmonitor-watchman log\n      read_index_from(): speed index loading by skipping verification of the entry order\n      fsmonitor: simplify determining the git worktree under Windows\n\nBrandon Williams (24):\n      pkt-line: add packet_write function\n      protocol: introduce protocol extension mechanisms\n      daemon: recognize hidden request arguments\n      upload-pack, receive-pack: introduce protocol version 1\n      connect: teach client to recognize v1 server response\n      connect: tell server that the client understands v1\n      http: tell server that the client understands v1\n      i5700: add interop test for protocol transition\n      ssh: introduce a 'simple' ssh variant\n      add, reset: use DIFF_OPT_SET macro to set a diff flag\n      diff: convert flags to be stored in bitfields\n      diff: add flag to indicate textconv was set via cmdline\n      diff: remove touched flags\n      diff: remove DIFF_OPT_TST macro\n      diff: remove DIFF_OPT_SET macro\n      diff: remove DIFF_OPT_CLR macro\n      diff: make struct diff_flags members lowercase\n      wt-status: actually ignore submodules when requested\n      pathspec: only match across submodule boundaries when requested\n      diff-tree: read the index so attribute checks work in bare repositories\n      submodule: convert stage_updated_gitmodules to take a struct index_state\n      submodule: used correct index in is_staging_gitmodules_ok\n      submodule: convert get_next_submodule to not rely on the_index\n      path: document path functions\n\nCarlos Martín Nieto (1):\n      diff: --indent-heuristic is no longer experimental\n\nCharles Bailey (2):\n      t4201: make use of abbreviation in the test more robust\n      grep: fix NO_LIBPCRE1_JIT to fully disable JIT\n\nChristian Couder (24):\n      perf/run: add '--config' option to the 'run' script\n      perf/run: add get_var_from_env_or_config()\n      perf/run: add GIT_PERF_DIRS_OR_REVS\n      perf/run: add calls to get_var_from_env_or_config()\n      perf/run: add get_subsections()\n      perf/run: update get_var_from_env_or_config() for subsections\n      perf/run: add run_subsection()\n      perf/run: show name of rev being built\n      perf: store subsection results in \"test-results/$GIT_PERF_SUBSECTION/\"\n      t0021/rot13-filter: fix list comparison\n      t0021/rot13-filter: refactor packet reading functions\n      t0021/rot13-filter: improve 'if .. elsif .. else' style\n      t0021/rot13-filter: improve error message\n      t0021/rot13-filter: add packet_initialize()\n      t0021/rot13-filter: refactor checking final lf\n      t0021/rot13-filter: add capability functions\n      Git/Packet.pm: extract parts of t0021/rot13-filter.pl for reuse\n      Git/Packet: clarify that packet_required_key_val_read allows EOF\n      Git/Packet.pm: use 'if' instead of 'unless'\n      list-objects-filter-options: fix 'keword' typo in comment\n      git-compat-util: introduce skip_to_optional_arg()\n      index-pack: use skip_to_optional_arg()\n      diff: use skip_to_optional_arg()\n      diff: use skip_to_optional_arg_default()\n\nDamien Marié (1):\n      run-command: add hint when a hook is ignored\n\nDaniel Bensoussan (1):\n      doc: reword gitworkflows.txt for neutrality\n\nDave Borowitz (1):\n      config.txt: document behavior of backslashes in subsections\n\nDennis Kaarsemaker (1):\n      credential-libsecret: unlock locked secrets\n\nDerrick Stolee (5):\n      p4211-line-log.sh: add log --online --raw --parents perf test\n      sha1_name: unroll len loop in find_unique_abbrev_r()\n      sha1_name: parse less while finding common prefix\n      sha1_name: minimize OID comparisons during disambiguation\n      sha1_file: use strbuf_add() instead of strbuf_addf()\n\nElijah Newren (10):\n      sequencer: warn when internal merge may be suboptimal due to renameLimit\n      merge-recursive: handle addition of submodule on our side of history\n      progress: fix progress meters when dealing with lots of work\n      diff: remove silent clamp of renameLimit\n      sequencer: show rename progress during cherry picks\n      merge-recursive: ignore_case shouldn't reject intentional removals\n      strbuf: remove unused stripspace function alias\n      t6044: recursive can silently incorporate dirty changes in a merge\n      move index_has_changes() from builtin/am.c to merge.c for reuse\n      merge-recursive: avoid incorporating uncommitted changes in a merge\n\nEmily Xie (1):\n      pathspec: die on empty strings as pathspec\n\nEric Sunshine (3):\n      worktree: invoke post-checkout hook (unless --no-checkout)\n      clone: support 'clone --shared' from a worktree\n      version --build-options: also report host CPU\n\nEric Wong (2):\n      rebase: use mboxrd format to avoid split errors\n      git-svn: convert CRLF to LF in commit message to SVN\n\nFlorian Klink (1):\n      git-send-email: honor $PATH for sendmail binary\n\nGennady Kupava (2):\n      trace: remove trace key normalization\n      trace: improve performance while category is disabled\n\nGuillaume Castagnino (1):\n      gitweb: use filetest to allow ACLs\n\nHaaris Mehmood (1):\n      config: add --expiry-date\n\nHans Jerry Illikainen (2):\n      merge: add config option for verifySignatures\n      t: add tests for pull --verify-signatures\n\nHeiko Voigt (3):\n      fetch: add test to make sure we stay backwards compatible\n      implement fetching of moved submodules\n      submodule: simplify decision tree whether to or not to fetch\n\nIngo Ruhnke (1):\n      remote-mediawiki: allow fetching namespaces with spaces\n\nJ Wyman (1):\n      for-each-ref: let upstream/push report the remote ref name\n\nJacob Keller (2):\n      sequencer: pass absolute GIT_DIR to exec commands\n      diff: add tests for --relative without optional prefix value\n\nJakub Bereżański (2):\n      t0302: check helper can handle empty credentials\n      wincred: handle empty username/password correctly\n\nJameson Miller (4):\n      status: add option to show ignored files differently\n      status: report matching ignored and normal untracked\n      status: document options to show matching ignored files\n      status: test ignored modes\n\nJean Carlo Machado (1):\n      fix typos in 2.15.0 release notes\n\nJean-Noel Avila (1):\n      submodule--helper.c: i18n: add a missing space in message\n\nJeff Hostetler (9):\n      dir: allow exclusions from blob in addition to file\n      oidmap: add oidmap iterator methods\n      oidset: add iterator methods to oidset\n      list-objects: filter objects in traverse_commit_list\n      rev-list: add list-objects filtering support\n      pack-objects: add list-objects filtering\n      list-objects-filter-options: support --no-filter\n      rev-list: support --no-filter argument\n      partial-clone: design doc\n\nJeff King (26):\n      revision: quit pruning diff more quickly when possible\n      t4015: refactor --color-moved whitespace test\n      t4015: check \"negative\" case for \"-w --color-moved\"\n      t4015: test the output of \"diff --color-moved -b\"\n      diff: fix whitespace-skipping with --color-moved\n      diff: handle NULs in get_string_hash()\n      test-ref-store: avoid passing NULL to printf\n      remote: handle broken symrefs\n      log: handle broken HEAD in decoration check\n      worktree: handle broken symrefs in find_shared_symref()\n      setup: avoid double slashes when looking for HEAD\n      link_alt_odb_entries: make empty input a noop\n      git-jump: give contact instructions in the README\n      p5550: factor out nonsense-pack creation\n      t/perf/lib-pack: use fast-import checkpoint to create packs\n      p5551: add a script to test fetch pack-dir rescans\n      everything_local: use \"quick\" object existence check\n      sha1_file: fast-path null sha1 as a missing object\n      git-status.txt: mention --no-optional-locks\n      progress: set default delay threshold to 100%, not 0%\n      docs/pretty-formats: mention commas in %(trailers) syntax\n      cvsimport: apply shell-quoting regex globally\n      test-lib: silence \"-x\" cleanup under bash\n      t5615: avoid re-using descriptor 4\n      test-lib: make \"-x\" work with \"--verbose-log\"\n      t/Makefile: introduce TEST_SHELL_PATH\n\nJohannes Schindelin (14):\n      for-each-ref: let upstream/push optionally report the remote name\n      status: do not get confused by submodules in excluded directories\n      mingw: include the full version information in the resources\n      mingw: add experimental feature to redirect standard handles\n      mingw: optionally redirect stderr/stdout via the same handle\n      mingw: document the standard handle redirection\n      for-each-ref: test :remotename and :remoteref\n      hashmap: adjust documentation to reflect reality\n      version --build-options: report commit, too, if possible\n      rebase: do not continue when the todo list generation failed\n      sequencer: strip bogus LF at end of error messages\n      sequencer: remove superfluous conditional\n      sequencer: report when noop has an argument\n      sequencer: do not invent whitespace when transforming OIDs\n\nJonathan Nieder (10):\n      ssh test: make copy_ssh_wrapper_as clean up after itself\n      connect: move no_fork fallback to git_tcp_connect\n      connect: split git:// setup into a separate function\n      connect: split ssh command line options into separate function\n      connect: split ssh option computation to its own function\n      ssh: 'auto' variant to select between 'ssh' and 'simple'\n      ssh: 'simple' variant does not support -4/-6\n      ssh: 'simple' variant does not support --port\n      connect: correct style of C-style comment\n      generate-cmdlist: avoid non-deterministic output\n\nJonathan Tan (10):\n      connect: in ref advertisement, shallows are last\n      Documentation: document Extra Parameters\n      Tests: clean up and document submodule helpers\n      Tests: clean up submodule recursive helpers\n      diff: support anchoring line(s)\n      diffcore-rename: make diff-tree -l0 mean -l<large>\n      decorate: clean up and document API\n      transport: remove unused \"push\" in vtable\n      clone, fetch: remove redundant transport check\n      transport: make transport vtable more private\n\nJulien Dusser (1):\n      strbuf: fix urlencode format string on signed char\n\nJunio C Hamano (38):\n      t0027: do not use an empty string as a pathspec element\n      describe: do not use cmd_*() as a subroutine\n      merge-ours: do not use cmd_*() as a subroutine\n      branch: streamline \"attr_only\" handling in validate_new_branchname()\n      branch: split validate_new_branchname() into two\n      t5601: rm the target file of cp that could still be executing\n      check-ref-format --branch: do not expand @{...} outside repository\n      check-ref-format --branch: strip refs/heads/ using skip_prefix\n      check-ref-format doc: --branch validates and expands <branch>\n      column: do not include pager.c\n      xdiff: reassign xpparm_t.flags bits\n      The first batch for 2.16\n      RelNotes: the second batch post 2.15 comes\n      diff: --ignore-cr-at-eol\n      merge-base --fork-point doc: clarify the example and failure modes\n      RelNotes: the third batch for 2.16\n      branch: correctly reject refs/heads/{-dash,HEAD}\n      Start preparation for 2.15.1\n      RelNotes: the fourth batch for 2.16\n      Almost ready for 2.15.1\n      RelNotes: the fifth batch for 2.16\n      hooks doc: clarify when receive-pack invokes its hooks\n      A bit more fixes for 2.15.1\n      RelNotes: the sixth batch for 2.16\n      Git 2.15.1\n      RelNotes: the seventh batch\n      t2020: test variations that matter\n      Prepare for 2.15.2\n      RelNotes: the eighth batch\n      diff: use skip_to_optional_arg_default() in parsing --relative\n      t4045: reindent to make helpers readable\n      RelNotes: the ninth batch\n      RelNotes: the tenth batch\n      t5573, t7612: clean up after unexpected success of 'pull' and 'merge'\n      sequencer.c: drop 'const' from function return type\n      RelNotes: the eleventh batch\n      Git 2.16-rc0\n      Git 2.16-rc1\n\nKaartic Sivaraam (12):\n      mailmap: use Kaartic Sivaraam's new address\n      builtin/branch: remove redundant check for HEAD\n      git-rebase: clean up dashed-usages in messages\n      Doc/checkout: checking out using @{-N} can lead to detached state\n      branch: improve documentation and naming of create_branch() parameters\n      branch: group related arguments of create_branch()\n      branch: update warning message shown when copying a misnamed branch\n      builtin/branch: strip refs/heads/ using skip_prefix\n      Doc/check-ref-format: clarify information about @{-N} syntax\n      rebase: consistently use branch_name variable\n      rebase: distinguish user input by quoting it\n      rebase: rebasing can also be done when HEAD is detached\n\nKevin (1):\n      remote-mediawiki: add namespace support\n\nKevin Daudt (1):\n      column: show auto columns when pager is active\n\nLars Schneider (3):\n      refactor \"dumb\" terminal determination\n      progress: drop delay-threshold code\n      launch_editor(): indicate that Git waits for user input\n\nLiam Beguin (9):\n      Documentation: move rebase.* configs to new file\n      Documentation: use preferred name for the 'todo list' script\n      rebase -i: set commit to null in exec commands\n      rebase -i: refactor transform_todo_ids\n      rebase -i: replace reference to sha1 with oid\n      rebase -i: update functions to use a flags parameter\n      rebase -i -x: add exec commands via the rebase--helper\n      rebase -i: learn to abbreviate command names\n      t3404: add test case for abbreviated commands\n\nLuke Diamand (1):\n      git-p4: update multiple shelved change lists\n\nMarius Paliga (1):\n      builtin/push.c: add push.pushOption config\n\nMartin Ågren (22):\n      sha1_file: do not leak `lock_file`\n      treewide: prefer lockfiles on the stack\n      lockfile: fix documentation on `close_lock_file_gently()`\n      tempfile: fix documentation on `delete_tempfile()`\n      checkout-index: simplify locking logic\n      cache-tree: simplify locking logic\n      apply: move lockfile into `apply_state`\n      apply: remove `newfd` from `struct apply_state`\n      cache.h: document `write_locked_index()`\n      read-cache: drop explicit `CLOSE_LOCK`-flag\n      read-cache: leave lock in right state in `write_locked_index()`\n      read_cache: roll back lock in `update_index_if_able()`\n      grep: take the read-lock when adding a submodule\n      bisect: change calling-convention of `find_bisection()`\n      bisect: fix memory leak in `find_bisection()`\n      bisect: fix off-by-one error in `best_bisection_sorted()`\n      bisect: fix memory leak when returning best element\n      builtin/merge-base: free commit lists\n      reduce_heads: fix memory leaks\n      t7006: add tests for how git branch paginates\n      branch: respect `pager.branch` in list-mode only\n      branch: change default of `pager.branch` to \"on\"\n\nMichael Haggerty (12):\n      t1409: check that `packed-refs` is not rewritten unnecessarily\n      files-backend: don't rewrite the `packed-refs` file unnecessarily\n      t0000: check whether the shell supports the \"local\" keyword\n      files_transaction_prepare(): don't leak flags to packed transaction\n      prune_ref(): call `ref_transaction_add_update()` directly\n      ref_transaction_update(): die on disallowed flags\n      ref_transaction_add_update(): remove a check\n      refs: tidy up and adjust visibility of the `ref_update` flags\n      refs: rename constant `REF_NODEREF` to `REF_NO_DEREF`\n      refs: rename constant `REF_ISPRUNING` to `REF_IS_PRUNING`\n      write_packed_entry(): take `object_id` arguments\n      refs: update some more docs to use \"oid\" rather than \"sha1\"\n\nNicolas Morey-Chaisemartin (1):\n      imap-send: URI encode server folder\n\nOlga Telezhnaya (2):\n      format: create pretty.h file\n      format: create docs for pretty.h\n\nPhil Hord (2):\n      doc: prefer 'stash push' over 'stash save'\n      stash: learn to parse -m/--message like commit does\n\nPhillip Wood (2):\n      config: avoid \"write_in_full(fd, buf, len) != len\" pattern\n      sequencer: reschedule pick if index can't be locked\n\nPranit Bauva (6):\n      bisect--helper: use OPT_CMDMODE instead of OPT_BOOL\n      bisect--helper: rewrite `check_term_format` shell function in C\n      bisect--helper: `write_terms` shell function in C\n      bisect--helper: `bisect_clean_state` shell function in C\n      t6030: explicitly test for bisection cleanup\n      bisect--helper: `is_expected_rev` & `check_expected_revs` shell function in C\n\nPrathamesh Chavan (3):\n      submodule--helper: introduce get_submodule_displaypath()\n      submodule--helper: introduce for_each_listed_submodule()\n      submodule: port submodule subcommand 'status' from shell to C\n\nRafael Ascensão (1):\n      log: add option to choose which refs to decorate\n\nRamsay Jones (1):\n      repository: fix a sparse 'using integer as NULL pointer' warning\n\nRandall S. Becker (1):\n      install-doc-quick: allow specifying what ref to install\n\nRasmus Villemoes (2):\n      Documentation/config: add sendemail.tocmd to list preceding \"See git-send-email(1)\"\n      completion: add git config sendemail.tocmd\n\nRené Scharfe (23):\n      notes: move hex_to_bytes() to hex.c and export it\n      http-push: use hex_to_bytes()\n      sha1_file: use hex_to_bytes()\n      sequencer: factor out rewrite_file()\n      sequencer: use O_TRUNC to truncate files\n      imap-send: handle NULL return of next_arg()\n      imap-send: handle missing response codes gracefully\n      apply: avoid out-of-bounds access in fuzzy_matchlines()\n      apply: update line lengths for --inaccurate-eof\n      config: flip return value of write_section()\n      t4051: add test for comments preceding function lines\n      xdiff: factor out is_func_rec()\n      xdiff: show non-empty lines before functions with -W\n      t7810: improve check of -W with user-defined function lines\n      grep: update boundary variable for pre-context\n      grep: show non-empty lines before functions with -W\n      am: release strbuf after use in split_mail_mbox()\n      fmt-merge-msg: avoid leaking strbuf in shortlog()\n      strbuf: release memory on read error in strbuf_read_once()\n      transport-helper: plug strbuf and string_list leaks\n      p7519: improve check for prerequisite WATCHMAN\n      http: use internal argv_array of struct child_process\n      send-pack: use internal argv_array of struct child_process\n\nRobert Abel (2):\n      git-prompt: make __git_eread intended use explicit\n      git-prompt: fix reading files with windows line endings\n\nRobert P. J. Day (4):\n      bisect: mention \"view\" as an alternative to \"visualize\"\n      doc: add missing \"-n\" (dry-run) option to reflog man page\n      prune: add \"--progress\" to man page and usage msg\n      notes: correct 'git notes prune' options to '[-n] [-v]'\n\nSZEDER Gábor (7):\n      travis-ci: fix running P4 and Git LFS tests in Linux build jobs\n      travis-ci: don't build Git for the static analysis job\n      setup.c: fix comment about order of .git directory discovery\n      travis-ci: introduce a $jobname variable for 'ci/*' scripts\n      travis-ci: move setting environment variables to 'ci/lib-travisci.sh'\n      travis-ci: set GIT_TEST_HTTPD in 'ci/lib-travisci.sh'\n      travis-ci: use 'set -x' in 'ci/*' scripts for extra tracing output\n\nShuyu Wei (1):\n      pull: pass -4/-6 option to 'git fetch'\n\nSimon Ruderich (3):\n      sequencer.c: check return value of close() in rewrite_file()\n      wrapper.c: consistently quote filenames in error messages\n      config: document default value of http.sslVerify\n\nStefan Beller (16):\n      recursive submodules: detach HEAD from new state\n      Documentation/checkout: clarify submodule HEADs to be detached\n      xdiff-interface: export comparing and hashing strings\n      diff.c: get rid of duplicate implementation\n      t6120: fix typo in test name\n      list-objects.c: factor out traverse_trees_and_blobs\n      config: document blame configuration\n      t/3512: demonstrate unrelated submodule/file conflict as cherry-pick failure\n      revision.h: introduce blob/tree walking in order of the commits\n      builtin/describe.c: rename `oid` to avoid variable shadowing\n      builtin/describe.c: print debug statements earlier\n      builtin/describe.c: factor out describe_commit\n      Documentation/git-clone: improve description for submodule recursing\n      t/helper: ignore everything but sources\n      builtin/describe.c: describe a blob\n      t/helper/test-lazy-name-hash: fix compilation\n\nSteffen Prohaska (1):\n      doc: Mention info/attributes in gitrepository-layout\n\nStephan Beyer (1):\n      bisect run: die if no command is given\n\nThomas Braun (1):\n      completion: add remaining flags to checkout\n\nThomas Gummerer (11):\n      stash: replace \"git stash save\" with \"git stash push\" in the documentation\n      stash: mark \"git stash save\" deprecated in the man page\n      stash: remove now superfluos help for \"stash push\"\n      checkout: factor out functions to new lib file\n      worktree: add can be created from any commit-ish\n      worktree: add --[no-]track option to the add subcommand\n      worktree: make add <path> <branch> dwim\n      t/README: remove mention of adding copyright notices\n      t/README: document test_cmp_rev\n      worktree: add --guess-remote flag to add subcommand\n      add worktree.guessRemote config option\n\nTodd Zullinger (11):\n      Replace Free Software Foundation address in license notices\n      Replace Free Software Foundation address in license notices\n      t/lib-gpg: fix gpgconf stderr redirect to /dev/null\n      rebase: fix stderr redirect in apply_autostash()\n      notes: send \"Automatic notes merge failed\" messages to stderr\n      branch doc: remove --set-upstream from synopsis\n      completion: add '--copy' option to 'git branch'\n      RelNotes: minor typo fixes in 2.15.1 draft\n      t/lib-git-svn: cleanup inconsistent tab/space usage\n      t/lib-git-svn.sh: improve svnserve tests with parallel make test\n      RelNotes: minor typo fixes in 2.16.0 draft\n\nTorsten Bögershausen (4):\n      add: introduce \"--renormalize\"\n      convert: tighten the safe autocrlf handling\n      t0027: Adapt the new MIX tests to Windows\n      check-non-portable-shell.pl: `wc -l` may have leading WS\n\nW. Trevor King (1):\n      pull: pass --signoff/--no-signoff to \"git merge\"\n\nWei Shuyu (1):\n      http: support CURLPROXY_HTTPS\n\nbrian m. carlson (31):\n      walker: convert to struct object_id\n      refs/files-backend: convert struct ref_to_prune to object_id\n      refs: convert delete_ref and refs_delete_ref to struct object_id\n      refs: convert update_ref and refs_update_ref to use struct object_id\n      refs: prevent accidental NULL dereference in write_pseudoref\n      refs: update ref transactions to use struct object_id\n      Convert check_connected to use struct object_id\n      refs: convert resolve_refdup and refs_resolve_refdup to struct object_id\n      refs: convert read_ref and read_ref_full to object_id\n      refs: convert dwim_ref and expand_ref to struct object_id\n      builtin/reflog: convert remaining unsigned char uses to object_id\n      refs: convert dwim_log to struct object_id\n      pack-bitmap: convert traverse_bitmap_commit_list to object_id\n      builtin/pack-objects: convert to struct object_id\n      refs: convert peel_ref to struct object_id\n      refs: convert read_ref_at to struct object_id\n      refs: convert reflog_expire parameter to struct object_id\n      sha1_file: convert index_path and index_fd to struct object_id\n      Convert remaining callers of resolve_gitlink_ref to object_id\n      refs: convert resolve_gitlink_ref to struct object_id\n      worktree: convert struct worktree to object_id\n      refs: convert resolve_ref_unsafe to struct object_id\n      refs: convert peel_object to struct object_id\n      refs: convert read_raw_ref backends to struct object_id\n      refs/files-backend: convert static functions to object_id\n      Documentation: enable compat-mode for Asciidoctor\n      setup: expose enumerated repo info\n      Add structure representing hash algorithm\n      Integrate hash algorithm support with repo setup\n      Switch empty tree and blob lookups to use hash abstraction\n      Documentation: convert SubmittingPatches to AsciiDoc\n\nÆvar Arnfjörð Bjarmason (2):\n      test-lib: add LIBPCRE1 & LIBPCRE2 prerequisites\n      grep: fix segfault under -P + PCRE2 <=10.30 + (*NO_JIT)\n\nŁukasz Stelmach (1):\n      git-gui: prevent double UTF-8 conversion\n\n"},{"id":"336052","messageId":"nycvar.QRO.7.76.6.1801061354430.1337@wbunaarf-fpuvaqryva.tvgsbejvaqbjf.bet","threadId":"47550","inReplyTo":"xmqqbmi7ano1.fsf@gitster.mtv.corp.google.com","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-01-06T12:59:55Z","receivedAt":"2018-01-06T13:00:05Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi team,\n\nOn Fri, 5 Jan 2018, Junio C Hamano wrote:\n\n> A release candidate Git v2.16.0-rc1 is now available for testing\n> at the usual places.  It is comprised of 455 non-merge commits\n> since v2.15.0, contributed by 79 people, 23 of which are new faces.\n\nI rebased Git for Windows' thicket of patch series on top, and I already\ngot this when running t0021:\n\n\t# failed 10 among 26 test(s)\n\t1..26\n\nIt also seems that the v2.16.0-rc1 *without* Git for Windows' patches on\ntop displays the very same problem: the test suite does *not* pass on\nWindows.\n\nAs it is, I am not willing to wreck my weekend, so this will have to wait\nuntil Monday (and Git for Windows v2.16.0-rc1 with it).\n\nCiao,\nJohannes\n"},{"id":"336070","messageId":"nycvar.QRO.7.76.6.1801062248160.1337@wbunaarf-fpuvaqryva.tvgsbejvaqbjf.bet","threadId":"47550","inReplyTo":"nycvar.QRO.7.76.6.1801061354430.1337@wbunaarf-fpuvaqryva.tvgsbejvaqbjf.bet","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-01-06T22:01:22Z","receivedAt":"2018-01-06T22:01:34Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi again,\n\nOn Sat, 6 Jan 2018, Johannes Schindelin wrote:\n\n> On Fri, 5 Jan 2018, Junio C Hamano wrote:\n> \n> > A release candidate Git v2.16.0-rc1 is now available for testing\n> > at the usual places.  It is comprised of 455 non-merge commits\n> > since v2.15.0, contributed by 79 people, 23 of which are new faces.\n> \n> I rebased Git for Windows' thicket of patch series on top, and I already\n> got this when running t0021:\n> \n> \t# failed 10 among 26 test(s)\n> \t1..26\n\nI actually was able to figure it out: it was the GITPERLLIB problem I\ndescribed earlier, where we run (once again) into trouble with Git's\nexpectations that everything behaves like Linux, including colon-separated\npath lists.\n\nThe test suite passed, so I kicked off a build which should be available\nsoon at:\n\nhttps://github.com/git-for-windows/git/releases/tag/v2.16.0-rc1.windows.1\n\nThe patch to work around the GITPERLLIB issue looks like this:\n\n-- snipsnap --\nSubject: [PATCH] mingw: handle GITPERLLIB in t0021 in a Windows-compatible\nway\n\nGit's assumption that all path lists are colon-separated is not only\nwrong on Windows, it is not even an assumption that is compatible with\nPOSIX.\n\nIn the interest of time, let's not try to fix this properly but simply\nwork around the obvious breakage on Windows, where the MSYS2 Bash used\nby Git for Windows to interpret the Git's Unix shell scripts will\nautomagically convert path lists in the environment to\nsemicolon-separated lists of Windows paths (with drive letter and the\ncorresponding colon and all that jazz).\n\nIn other words, we simply look whether there is a semicolon in\nGITPERLLIB and split by semicolons if found instead of colons. This is\nnot fool-proof, of course, as the path list could consist of a single\npath. But that is not the case in Git for Windows' test suite, there are\nalways two paths in GITPERLLIB.\n\nSigned-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>\n---\n t/t0021/rot13-filter.pl | 17 ++++++++++++++++-\n 1 file changed, 16 insertions(+), 1 deletion(-)\n\ndiff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\nindex f1678851de9..470107248eb 100644\n--- a/t/t0021/rot13-filter.pl\n+++ b/t/t0021/rot13-filter.pl\n@@ -31,7 +31,22 @@\n #\n \n use 5.008;\n-use lib (split(/:/, $ENV{GITPERLLIB}));\n+sub gitperllib {\n+\t# Git assumes that all path lists are Unix-y colon-separated ones. But\n+\t# when the Git for Windows executes the test suite, its MSYS2 Bash\n+\t# calls git.exe, and colon-separated path lists are converted into\n+\t# Windows-y semicolon-separated lists of *Windows* paths (which\n+\t# naturally contain a colon after the drive letter, so splitting by\n+\t# colons simply does not cut it).\n+\t#\n+\t# Detect semicolon-separated path list and handle them appropriately.\n+\n+\tif ($ENV{GITPERLLIB} =~ /;/) {\n+\t\treturn split(/;/, $ENV{GITPERLLIB});\n+\t}\n+\treturn split(/:/, $ENV{GITPERLLIB});\n+}\n+use lib (gitperllib());\n use strict;\n use warnings;\n use IO::File;\n-- \n2.16.0.rc0.windows.1\n\n\n"},{"id":"336320","messageId":"xmqqy3l63dzy.fsf@gitster.mtv.corp.google.com","threadId":"47550","inReplyTo":"nycvar.QRO.7.76.6.1801062248160.1337@wbunaarf-fpuvaqryva.tvgsbejvaqbjf.bet","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-01-09T21:42:57Z","receivedAt":"2018-01-09T21:43:10Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n\n> diff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\n> index f1678851de9..470107248eb 100644\n> --- a/t/t0021/rot13-filter.pl\n> +++ b/t/t0021/rot13-filter.pl\n> @@ -31,7 +31,22 @@\n>  #\n>  \n>  use 5.008;\n> -use lib (split(/:/, $ENV{GITPERLLIB}));\n> +sub gitperllib {\n> +...\n> +\tif ($ENV{GITPERLLIB} =~ /;/) {\n> +\t\treturn split(/;/, $ENV{GITPERLLIB});\n> +\t}\n> +\treturn split(/:/, $ENV{GITPERLLIB});\n> +}\n\nThis cannot be the whole story for a few reasons.\n\n - In t/test-lib.sh we see this:\n\n   GITPERLLIB=\"$GIT_BUILD_DIR\"/perl/blib/lib:\"$GIT_BUILD_DIR\"/perl/blib/arch/auto/Git\n   export GITPERLLIB\n\n   If this part wants to split with ';', then the joining needs to\n   be done with ';' to match, no?\n\n - In addition to t0021, there are similar split with colon in 0202,\n   9000 and 9700, yet I am getting the feeling that you observed the\n   issue only in0021, to which I do not think of a good explanation\n   why.\n\n"},{"id":"336331","messageId":"xmqq608a37ve.fsf@gitster.mtv.corp.google.com","threadId":"47550","inReplyTo":"xmqqy3l63dzy.fsf@gitster.mtv.corp.google.com","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-01-09T23:55:17Z","receivedAt":"2018-01-09T23:55:26Z","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> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n>\n>> diff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\n>> index f1678851de9..470107248eb 100644\n>> --- a/t/t0021/rot13-filter.pl\n>> +++ b/t/t0021/rot13-filter.pl\n>> @@ -31,7 +31,22 @@\n>>  #\n>>  \n>>  use 5.008;\n>> -use lib (split(/:/, $ENV{GITPERLLIB}));\n>> +sub gitperllib {\n>> +...\n>> +\tif ($ENV{GITPERLLIB} =~ /;/) {\n>> +\t\treturn split(/;/, $ENV{GITPERLLIB});\n>> +\t}\n>> +\treturn split(/:/, $ENV{GITPERLLIB});\n>> +}\n>\n> This cannot be the whole story for a few reasons.\n>\n>  - In t/test-lib.sh we see this:\n>\n>    GITPERLLIB=\"$GIT_BUILD_DIR\"/perl/blib/lib:\"$GIT_BUILD_DIR\"/perl/blib/arch/auto/Git\n>    export GITPERLLIB\n>\n>    If this part wants to split with ';', then the joining needs to\n>    be done with ';' to match, no?\n>\n>  - In addition to t0021, there are similar split with colon in 0202,\n>    9000 and 9700, yet I am getting the feeling that you observed the\n>    issue only in0021, to which I do not think of a good explanation\n>    why.\n\nThis somehow vaguely rang a bell, and I dug this thing up from the\narchive, [*1*] which ended like so:\n\n    >> In our C code, we have \"#define PATH_SEP ';'\", and encourage\n    >> our code to be careful and use it.  Is there something\n    >> similar for Perl scripts, I wonder.\n    >>\n    > We probably should find a better solution to allow this to\n    > work with windows style paths...? I know that python provides\n    > os.pathsep, but I haven't seen an equivalent for perl yet.\n    >\n    > The Env[1] core modules suggests using\n    > $Config::Config{path_sep}[2]..  maybe we should be using this?\n\n    I was testing this recently on the Perl included with Git for\n    Windows and it returns : for the path separator even though it's\n    on Windows, so I don't think that would work. The Perl in Git\n    for Windows seems to want UNIX-style inputs (something Dscho\n    seemed to allude to in his response earlier.). I'm not sure why\n    it's that way, but he probably knows.\n\nYour initial response in this thread made it sound as if -rc1 is the\nonly thing that changed, but looking at the differences between -rc0\nand -rc1, which does not touch t0021 or any other instances of\n\"split(/:/, $ENV{GITPERLLIB})\", I am wondering if it is possible\nthat perhaps the way Perl is built for GfW has been changed recently\nand we can safely and sanely use $Config::Config{path_sep} (contrary\nto what was found in late Oct in the message quoted above) now?\n\nIn any case, I'd prefer this issue to be resolved properly before\n-rc2; a patch to t0021/rot13-filter.pl alone does not smell like a\n\"proper solution\" that is based on the understanding of the root\ncause (and that is why I spent time digging the list archive).\n\nThanks.\n\n\n[Reference]\n\n*1* https://public-inbox.org/git/CAGyf7-EjKaHgwkN9trO4mFvba9odbWCzA9Jh0Pk6ZE6FOskOYg@mail.gmail.com/\n\n\n\n"},{"id":"336396","messageId":"nycvar.QRO.7.76.6.1801101823420.31@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz","threadId":"47550","inReplyTo":"xmqqy3l63dzy.fsf@gitster.mtv.corp.google.com","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-01-10T17:37:54Z","receivedAt":"2018-01-10T17:38:05Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Tue, 9 Jan 2018, Junio C Hamano wrote:\n\n> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n> \n> > diff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\n> > index f1678851de9..470107248eb 100644\n> > --- a/t/t0021/rot13-filter.pl\n> > +++ b/t/t0021/rot13-filter.pl\n> > @@ -31,7 +31,22 @@\n> >  #\n> >  \n> >  use 5.008;\n> > -use lib (split(/:/, $ENV{GITPERLLIB}));\n> > +sub gitperllib {\n> > +...\n> > +\tif ($ENV{GITPERLLIB} =~ /;/) {\n> > +\t\treturn split(/;/, $ENV{GITPERLLIB});\n> > +\t}\n> > +\treturn split(/:/, $ENV{GITPERLLIB});\n> > +}\n> \n> This cannot be the whole story for a few reasons.\n> \n>  - In t/test-lib.sh we see this:\n> \n>    GITPERLLIB=\"$GIT_BUILD_DIR\"/perl/blib/lib:\"$GIT_BUILD_DIR\"/perl/blib/arch/auto/Git\n>    export GITPERLLIB\n> \n>    If this part wants to split with ';', then the joining needs to\n>    be done with ';' to match, no?\n\nNo.\n\nIt is a lot more complicated than that. As you know, on Linux there is\nthis implicit assumption that path lists are colon-separated. As a\nconsequence, Cygwin does the same (because it would be too hard to port\nall those Linux/Unix projects to stop assuming colon-separated path lists,\nright?)\n\nThis is what Cygwin's Bash does (and hence the MSYS2 Bash used by Git for\nWindows, too).\n\nThen the MSYS2 Bash calls git.exe, which is *not* an MSYS2 program, hence\nthe MSYS2 runtime knows that it has to convert the path lists to Windows\npaths separated by semicolons.\n\nThe next thing happening in our case is that the Perl script is called\nfrom git.exe. Now, the MSYS2 runtime (implicitly spun up by the MSYS2 Perl\ninterpreter) does *not* convert those path lists back to Unix-like paths\nseparated by colons.\n\nAnd that's why the Unix shell script can happily construct the\ncolon-separated list, and the Perl script will *still* receive the\nsemicolon-separated version of it.\n\n>  - In addition to t0021, there are similar split with colon in 0202,\n>    9000 and 9700, yet I am getting the feeling that you observed the\n>    issue only in0021, to which I do not think of a good explanation\n>    why.\n\nHere is the good explanation: t0021 relies on a Perl package that is not\nyet installed. t0202 relies on Git::I18N, of which there is a version\ninstalled in Git for Windows' SDK. (I do not bother to slow down the test\nruns by the Subversion tests, I always skip all of them, that's why t9*\ndoes not matter to me.)\n\nGranted, that is a bug, and an old one at that: the test suite should test\nwhat is in the current revision, not what happens to be installed into the\nsystem locations.\n\nBut this late in the game, I am really uncomfortable with the idea to rush\nthrough an equivalent fix to all Perl scripts: who knows what it breaks?\n\nCiao,\nDscho\n"},{"id":"336397","messageId":"nycvar.QRO.7.76.6.1801101838360.31@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz","threadId":"47550","inReplyTo":"xmqq608a37ve.fsf@gitster.mtv.corp.google.com","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2018-01-10T17:40:18Z","receivedAt":"2018-01-10T17:40:31Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Tue, 9 Jan 2018, Junio C Hamano wrote:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n> \n> > Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n> >\n> >> diff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\n> >> index f1678851de9..470107248eb 100644\n> >> --- a/t/t0021/rot13-filter.pl\n> >> +++ b/t/t0021/rot13-filter.pl\n> >> @@ -31,7 +31,22 @@\n> >>  #\n> >>  \n> >>  use 5.008;\n> >> -use lib (split(/:/, $ENV{GITPERLLIB}));\n> >> +sub gitperllib {\n> >> +...\n> >> +\tif ($ENV{GITPERLLIB} =~ /;/) {\n> >> +\t\treturn split(/;/, $ENV{GITPERLLIB});\n> >> +\t}\n> >> +\treturn split(/:/, $ENV{GITPERLLIB});\n> >> +}\n> >\n> > This cannot be the whole story for a few reasons.\n> >\n> >  - In t/test-lib.sh we see this:\n> >\n> >    GITPERLLIB=\"$GIT_BUILD_DIR\"/perl/blib/lib:\"$GIT_BUILD_DIR\"/perl/blib/arch/auto/Git\n> >    export GITPERLLIB\n> >\n> >    If this part wants to split with ';', then the joining needs to\n> >    be done with ';' to match, no?\n> >\n> >  - In addition to t0021, there are similar split with colon in 0202,\n> >    9000 and 9700, yet I am getting the feeling that you observed the\n> >    issue only in0021, to which I do not think of a good explanation\n> >    why.\n> \n> This somehow vaguely rang a bell, and I dug this thing up from the\n> archive, [*1*] which ended like so:\n> \n>     >> In our C code, we have \"#define PATH_SEP ';'\", and encourage\n>     >> our code to be careful and use it.  Is there something\n>     >> similar for Perl scripts, I wonder.\n>     >>\n>     > We probably should find a better solution to allow this to\n>     > work with windows style paths...? I know that python provides\n>     > os.pathsep, but I haven't seen an equivalent for perl yet.\n>     >\n>     > The Env[1] core modules suggests using\n>     > $Config::Config{path_sep}[2]..  maybe we should be using this?\n> \n>     I was testing this recently on the Perl included with Git for\n>     Windows and it returns : for the path separator even though it's\n>     on Windows, so I don't think that would work. The Perl in Git\n>     for Windows seems to want UNIX-style inputs (something Dscho\n>     seemed to allude to in his response earlier.). I'm not sure why\n>     it's that way, but he probably knows.\n> \n> Your initial response in this thread made it sound as if -rc1 is the\n> only thing that changed, but looking at the differences between -rc0\n> and -rc1, which does not touch t0021 or any other instances of\n> \"split(/:/, $ENV{GITPERLLIB})\", I am wondering if it is possible\n> that perhaps the way Perl is built for GfW has been changed recently\n> and we can safely and sanely use $Config::Config{path_sep} (contrary\n> to what was found in late Oct in the message quoted above) now?\n\nNo, the only thing that changed was the introduction of Git::Packet (and\nt0021/*.perl uses that). And that Perl module is not yet installed.\n\nGranted, we tested incorrect versions of those modules, but this late in\nthe -rc phase, I would prefer to be cautious. People might be relying on\nthe current bug.\n\nOTOH I might be way too pessimistic and we should essentially replicate\nthat `sub gitperllib` trick in *all* of our Perl scripts.\n\nWhat do you think?\n\nCiao,\nDscho\n"},{"id":"336399","messageId":"2727bfc6-d789-ee45-fb3b-95b7b21e4cba@kdbg.org","threadId":"47550","inReplyTo":"nycvar.QRO.7.76.6.1801101823420.31@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz","subject":"Re: [git-for-windows] Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Johannes Sixt","fromEmail":"j6t@kdbg.org","sentAt":"2018-01-10T17:54:46Z","receivedAt":"2018-01-10T17:54:55Z","isPatch":false,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Am 10.01.2018 um 18:37 schrieb Johannes Schindelin:\n> On Tue, 9 Jan 2018, Junio C Hamano wrote:\n>> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n>>\n>>> diff --git a/t/t0021/rot13-filter.pl b/t/t0021/rot13-filter.pl\n>>> index f1678851de9..470107248eb 100644\n>>> --- a/t/t0021/rot13-filter.pl\n>>> +++ b/t/t0021/rot13-filter.pl\n>>> @@ -31,7 +31,22 @@\n>>>   #\n>>>   \n>>>   use 5.008;\n>>> -use lib (split(/:/, $ENV{GITPERLLIB}));\n>>> +sub gitperllib {\n>>> +...\n>>> +\tif ($ENV{GITPERLLIB} =~ /;/) {\n>>> +\t\treturn split(/;/, $ENV{GITPERLLIB});\n>>> +\t}\n>>> +\treturn split(/:/, $ENV{GITPERLLIB});\n>>> +}\n>>\n>> This cannot be the whole story for a few reasons.\n>>\n>>   - In t/test-lib.sh we see this:\n>>\n>>     GITPERLLIB=\"$GIT_BUILD_DIR\"/perl/blib/lib:\"$GIT_BUILD_DIR\"/perl/blib/arch/auto/Git\n>>     export GITPERLLIB\n>>\n>>     If this part wants to split with ';', then the joining needs to\n>>     be done with ';' to match, no?\n> \n> No.\n> \n> It is a lot more complicated than that. As you know, on Linux there is\n> this implicit assumption that path lists are colon-separated. As a\n> consequence, Cygwin does the same (because it would be too hard to port\n> all those Linux/Unix projects to stop assuming colon-separated path lists,\n> right?)\n> \n> This is what Cygwin's Bash does (and hence the MSYS2 Bash used by Git for\n> Windows, too).\n> \n> Then the MSYS2 Bash calls git.exe, which is *not* an MSYS2 program, hence\n> the MSYS2 runtime knows that it has to convert the path lists to Windows\n> paths separated by semicolons.\n> \n> The next thing happening in our case is that the Perl script is called\n> from git.exe. Now, the MSYS2 runtime (implicitly spun up by the MSYS2 Perl\n> interpreter) does *not* convert those path lists back to Unix-like paths\n> separated by colons.\n\nBut this is a bug in MSYS2, isn't it? The MSYS2 runtime should detect \nthat it was not invoked by some other MSYS2 process. The MSYS2 startup \nsequence should assume in this case that the environment is \nWindows-style and convert to POSIX before it calls into perl's main().\n\n> And that's why the Unix shell script can happily construct the\n> colon-separated list, and the Perl script will *still* receive the\n> semicolon-separated version of it.\n> \n>>   - In addition to t0021, there are similar split with colon in 0202,\n>>     9000 and 9700, yet I am getting the feeling that you observed the\n>>     issue only in0021, to which I do not think of a good explanation\n>>     why.\n> \n> Here is the good explanation: t0021 relies on a Perl package that is not\n> yet installed. t0202 relies on Git::I18N, of which there is a version\n> installed in Git for Windows' SDK. (I do not bother to slow down the test\n> runs by the Subversion tests, I always skip all of them, that's why t9*\n> does not matter to me.)\n\nt0202 and the t9* cases are different because perl is invoked by bash \ndirectly (AFAICS), without a non-MSYS2 process between them. There is no \ndifference when the path conversion is omitted in this case by design or \ndue to a bug.\n\n-- Hannes\n"},{"id":"336417","messageId":"xmqqk1wp1nno.fsf@gitster.mtv.corp.google.com","threadId":"47550","inReplyTo":"nycvar.QRO.7.76.6.1801101838360.31@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz","subject":"Re: [ANNOUNCE] Git v2.16.0-rc1","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2018-01-10T20:09:31Z","receivedAt":"2018-01-10T20:09:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n\n> No, the only thing that changed was the introduction of Git::Packet (and\n> t0021/*.perl uses that). And that Perl module is not yet installed.\n\nAhh, that is the difference among other users of split(/:/,\n$ENV{GITPERLLIB}).  Scripts other than 0021 may be using installed\nones and using the wrong separator does not affect them.\n\n> Granted, we tested incorrect versions of those modules, but this late in\n> the -rc phase, I would prefer to be cautious. People might be relying on\n> the current bug.\n\nPeople meaning those who run \"make test\"?\n\nIn any case, the patch to 0021 as posted sounds like the most\nconservative thing to do at this point, even though we would\ndefinitely want to revisit it and clean it up after the release.  As\nlong as the reason why only 0021 wants the special case while others\nare OK is understood, I am OK with the patch ;-)\n\nThanks.\n"}]}