{"thread":{"id":"33626","subject":"[ANNOUNCE] Git v1.8.3-rc0","startedAt":"2013-04-27T00:22:22Z","lastAt":"2013-05-01T17:55:54Z","messageCount":16,"participants":["Junio C Hamano","shawn wilson","Michael Haggerty","John Keeping","Marc Branchaud","Eric Sunshine","Phil Hord","Lukas Fleischer"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"215668","messageId":"7vvc78u8jl.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":null,"subject":"[ANNOUNCE] Git v1.8.3-rc0","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-27T00:22:22Z","receivedAt":"2013-04-27T00:22:22Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"A release candidate preview Git v1.8.3-rc0 is now available for\ntesting at the usual places.\n\nThe release tarballs are found at:\n\n    http://code.google.com/p/git-core/downloads/list\n\nand their SHA-1 checksums are:\n\nf0b0b415f0c693865895c1918859b12c4a7d5b17  git-1.8.3.rc0.tar.gz\n089efc61b3d45504cb53003719d1bb3d953a41a8  git-htmldocs-1.8.3.rc0.tar.gz\n57a6e889667597f2699e4d6fb97c8cf6e595cb4a  git-manpages-1.8.3.rc0.tar.gz\n\nAlso the following public repositories all have a copy of the v1.8.3-rc0\ntag 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://code.google.com/p/git-core/\n  url = git://git.sourceforge.jp/gitroot/git-core/git.git\n  url = git://git-core.git.sourceforge.net/gitroot/git-core/git-core\n  url = https://github.com/gitster/git\n\nGit v1.8.3 Release Notes (draft)\n========================\n\nBackward compatibility notes (for Git 2.0)\n------------------------------------------\n\nWhen \"git push [$there]\" does not say what to push, we have used the\ntraditional \"matching\" semantics so far (all your branches were sent\nto the remote as long as there already are branches of the same name\nover there).  In Git 2.0, the default will change to the \"simple\"\nsemantics that pushes the current branch to the branch with the same\nname, only when the current branch is set to integrate with that\nremote branch.  There is a user preference configuration variable\n\"push.default\" to change this.  If you are an old-timer who is used\nto the \"matching\" semantics, you can set it to \"matching\" to keep the\ntraditional behaviour.  If you want to live in the future early,\nyou can set it to \"simple\" today without waiting for Git 2.0.\n\nWhen \"git add -u\" and \"git add -A\", that does not specify what paths\nto add on the command line is run from inside a subdirectory, these\ncommands will operate on the entire tree in Git 2.0 for consistency\nwith \"git commit -a\" and other commands. Because there will be no\nmechanism to make \"git add -u\" behave as if \"git add -u .\", it is\nimportant for those who are used to \"git add -u\" (without pathspec)\nupdating the index only for paths in the current subdirectory to start\ntraining their fingers to explicitly say \"git add -u .\" when they mean\nit before Git 2.0 comes.  A warning is issued when these commands are\nrun without a pathspec and when you have local changes outside the\ncurrent directory, because the behaviour in Git 2.0 will be different\nfrom today's version in such a situation.\n\nIn Git 2.0, \"git add <path>\" will behave as \"git add -A <path>\", so\nthat \"git add dir/\" will notice paths you removed from the directory\nand record the removal.  Versions before Git 2.0, including this\nrelease, will keep ignoring removals, but the users who rely on this\nbehaviour is encouraged to use \"git add --ignore-removal <path>\" and\nget used to it.\n\n\nUpdates since v1.8.2\n--------------------\n\nForeign interface\n\n * remote-hg and remote-bzr helpers (in contrib/) have been updated.\n\n\nUI, Workflows & Features\n\n * \"git branch --vv\" learned to paint the name of the branch it\n   integrates with in a different color (color.branch.upstream,\n   which defaults to blue).\n\n * In a sparsely populated working tree, \"git checkout <pathspec>\" no\n   longer unmarks paths that match the given pathspec that were\n   originally ignored with \"--sparse\" (use --ignore-skip-worktree-bits\n   option to resurrect these paths out of the index if you really want\n   to).\n\n * \"git log --format\" specifier learned %C(auto) token that tells Git\n   to use color when interpolating %d (decoration), %h (short commit\n   object name), etc. for terminal output.\n\n * \"git bisect\" leaves the final outcome as a comment in its bisect\n   log file.\n\n * \"git clone --reference\" can now refer to a gitfile \"textual symlink\"\n   that points at the real location of the repository.\n\n * \"git count-objects\" learned \"--human-readable\" aka \"-H\" option to\n   show various large numbers in Ki/Mi/GiB scaled as necessary.\n\n * \"git cherry-pick $blob\" and \"git cherry-pick $tree\" are nonsense,\n   and a more readable error message e.g. \"can't cherry-pick a tree\"\n   is given (we used to say \"expected exactly one commit\").\n\n * The \"--annotate\" option to \"git send-email\" can be turned on (or\n   off) by default with sendemail.annotate configuration variable (you\n   can use --no-annotate from the command line to override it).\n\n * The \"--cover-letter\" option to \"git format-patch\" can be turned on\n   (or off) by default with format.coverLetter configuration\n   variable. By setting it to 'auto', you can turn it on only for a\n   series with two or more patches.\n\n * The bash completion support (in contrib/) learned that cherry-pick\n   takes a few more options than it already knew about.\n\n * \"git help\" learned \"-g\" option to show the list of guides just like\n   list of commands are given with \"-a\".\n\n * A triangular \"pull from one place, push to another place\" workflow\n   is supported better by new remote.pushdefault (overrides the\n   \"origin\" thing) and branch.*.pushremote (overrides the\n   branch.*.remote) configuration variables.\n\n * \"git status\" learned to report that you are in the middle of a\n   revert session, just like it does for a cherry-pick and a bisect\n   session.\n\n * The handling by \"git branch --set-upstream-to\" against various forms\n   of erroneous inputs was suboptimal and has been improved.\n\n * When the interactive access to git-shell is not enabled, it issues\n   a message meant to help the system administrator to enable it.\n   An explicit way to help the end users who connect to the service by\n   issuing custom messages to refuse such an access has been added.\n\n * In addition to the case where the user edits the log message with\n   the \"e)dit\" option of \"am -i\", replace the \"Applying: this patch\"\n   message with the final log message contents after applymsg hook\n   munges it.\n\n * \"git status\" suggests users to look into using --untracked=no option\n   when it takes too long.\n\n * \"git status\" shows a bit more information to \"git status\" during a\n   rebase/bisect session.\n\n * \"git fetch\" learned to fetch a commit at the tip of an unadvertised\n   ref by specifying a raw object name from the command line when the\n   server side supports this feature.\n\n * Output from \"git log --graph\" works better with submodule log\n   output now.\n\n * \"git count-objects -v\" learned to report leftover temporary\n   packfiles and other garbage in the object store.\n\n * A new read-only credential helper (in contrib/) to interact with\n   the .netrc/.authinfo files has been added.\n\n * \"git send-email\" can be used with the credential helper system.\n\n * There was no Porcelain way to say \"I no longer am interested in\n   this submodule\", once you express your interest in a submodule with\n   \"submodule init\".  \"submodule deinit\" is the way to do so.\n\n * \"git pull --rebase\" learned to pass \"-v/-q\" options to underlying\n   \"git rebase\".\n\n * The new \"--follow-tags\" option tells \"git push\" to push relevant\n   annotated tags when pushing branches out.\n\n * \"git merge\" and \"git pull\" can optionally be told to inspect and\n   reject when merging a commit that does not carry a trusted GPG\n   signature.\n\n * \"git mergetool\" now feeds files to the \"p4merge\" backend in the\n   order that matches the p4 convention, where \"theirs\" is usually\n   shown on the left side, which is the opposite from other backend\n   expects.\n\n * \"show/log\" now honors gpg.program configuration just like other\n   parts of the code that use GnuPG.\n\n * \"git log\" that shows the difference between the parent and the\n   child has been optimized somewhat.\n\n * \"git difftool\" allows the user to write into the temporary files\n   being shown; if the user makes changes to the working tree at the\n   same time, one of the changes has to be lost in such a case, but it\n   tells the user what happened and refrains from overwriting the copy\n   in the working tree.\n\n * There was no good way to ask \"I have a random string that came from\n   outside world. I want to turn it into a 40-hex object name while\n   making sure such an object exists\".  A new peeling suffix ^{object}\n   can be used for that purpose, together with \"rev-parse --verify\".\n\n\nPerformance, Internal Implementation, etc.\n\n * Updates for building under msvc.\n\n * A handful of issues in the code to traverse working tree to find\n   untracked and/or ignored files have been fixed, and the general\n   codepath involved in \"status -u\" and \"clean\" have been cleaned up\n   and optimized.\n\n * The stack footprint of some codepaths that access an object from a\n   pack has been shrunk.\n\n * The logic to coalesce the same lines removed from the parents in\n   the output from \"diff -c/--cc\" has been updated, but with an O(n^2)\n   complexity, so this might turn out to be undesirable.\n\n * The code to enforce permission bits on files in $GIT_DIR/ for\n   shared repositories have been simplified.\n\n * A few codepaths knew how much data they need to put in the\n   hashtables they use upfront, but still started from a small table\n   repeatedly growing and rehashing.\n\n * The API to walk reflog entries from the latest to older, which was\n   necessary for operations such as \"git checkout -\", was cumbersome\n   to use correctly and also inefficient.\n\n * Codepaths that inspect log-message-to-be and decide when to add a\n   new Signed-off-by line in various commands have been consolidated.\n\n * The pkt-line API, implementation and its callers have been cleaned\n   up to make them more robust.\n\n * Cygwin port has a faster-but-lying lstat(2) emulation whose\n   incorrectness does not matter in practice except for a few\n   codepaths, and setting permission bits to directories is a codepath\n   that needs to use a more correct one.\n\n * \"git checkout\" had repeated pathspec matches on the same paths,\n   which have been consolidated.  Also a bug in \"git checkout dir/\"\n   that is started from an unmerged index has been fixed.\n\n * A few bugfixes to \"git rerere\" working on corner case merge\n   conflicts have been applied.\n\n\nAlso contains various documentation updates and code clean-ups.\n\n\nFixes since v1.8.2\n------------------\n\nUnless otherwise noted, all the fixes since v1.8.2 in the maintenance\ntrack are contained in this release (see release notes to them for\ndetails).\n\n * When receive-pack detects error in the pack header it received in\n   order to decide which of unpack-objects or index-pack to run, it\n   returned without closing the error stream, which led to a hang\n   sideband thread.\n\n * Zsh completion forgot that '%' character used to signal untracked\n   files needs to be escaped with another '%'.\n\n * A commit object whose author or committer ident are malformed\n   crashed some code that trusted that a name, an email and an\n   timestamp can always be found in it.\n\n * When \"upload-pack\" fails while generating a pack in response to\n   \"git fetch\" (or \"git clone\"), the receiving side mistakenly said\n   there was a programming error to trigger the die handler\n   recursively.\n\n * \"rev-list --stdin\" and friends kept bogus pointers into input\n   buffer around as human readble object names.  This was not a huge\n   problem but was exposed by a new change that uses these names in\n   error output.\n   (merge 70d26c6 tr/copy-revisions-from-stdin later to maint).\n\n * Smart-capable HTTP servers were not restricted via the\n   GIT_NAMESPACE mechanism when talking with commit-walker clients,\n   like they do when talking with smart HTTP clients.\n   (merge 6130f86 jk/http-dumb-namespaces later to maint).\n\n * \"git merge-tree\" did not omit a merge result that is identical to\n   \"our\" side in certain cases.\n   (merge aacecc3 jk/merge-tree-added-identically later to maint).\n\n * Perl scripts like \"git-svn\" closed (not redirecting to /dev/null)\n   the standard error stream, which is not a very smart thing to do.\n   Later open may return file descriptor #2 for unrelated purpose, and\n   error reporting code may write into them.\n\n * \"git show-branch\" was not prepared to show a very long run of\n   ancestor operators e.g. foobar^2~2^2^2^2...^2~4 correctly.\n\n * \"git diff --diff-algorithm algo\" is also understood as \"git diff\n   --diff-algorithm=algo\".\n\n * The new core.commentchar configuration was not applied to a few\n   places.\n\n * \"git bundle\" did not like a bundle created using a commit without\n   any message as its one of the prerequistes.\n\n * \"git log -S/-G\" started paying attention to textconv filter, but\n   there was no way to disable this.  Make it honor --no-textconv\n   option.\n\n * When used with \"-d temporary-directory\" option, \"git filter-branch\"\n   failed to come back to the original working tree to perform the\n   final clean-up procedure.\n\n * \"git merge $(git rev-parse v1.8.2)\" behaved quite differently from\n   \"git merge v1.8.2\", as if v1.8.2 were written as v1.8.2^0 and did\n   not pay much attention to the annotated tag payload.  Make the code\n   notice the type of the tag object, in addition to the dwim_ref()\n   based classification the current code uses (i.e. the name appears\n   in refs/tags/) to decide when to special case merging of tags.\n\n * Fix 1.8.1.x regression that stopped matching \"dir\" (without\n   trailing slash) to a directory \"dir\".\n   (merge efa5f82 jc/directory-attrs-regression-fix later to maint-1.8.1).\n\n * \"git apply --whitespace=fix\" was not prepared to see a line getting\n   longer after fixing whitespaces (e.g. tab-in-indent aka Python).\n   (merge 329b26e jc/apply-ws-fix-tab-in-indent later to maint-1.8.1).\n\n * The prompt string generator (in contrib/completion/) did not notice\n   when we are in a middle of a \"git revert\" session.\n\n * \"submodule summary --summary-limit\" option did not support\n   \"--option=value\" form.\n\n * \"index-pack --fix-thin\" used an uninitialized value to compute\n   delta depths of objects it appends to the resulting pack.\n\n * \"index-pack --verify-stat\" used a few counters outside protection\n   of mutex, possibly showing incorrect numbers.\n\n * The code to keep track of what directory names are known to Git on\n   platforms with case insensitive filesystems can get confused upon a\n   hash collision between these pathnames and looped forever.\n\n * Annotated tags outside refs/tags/ hierarchy were not advertised\n   correctly to the ls-remote and fetch with recent version of Git.\n\n * Recent optimization broke shallow clones.\n\n * \"git cmd -- ':(top'\" was not diagnosed as an invalid syntax, and\n   instead the parser kept reading beyond the end of the string.\n\n * \"git tag -f <tag>\" always said \"Updated tag '<tag>'\" even when\n   creating a new tag (i.e. not overwriting nor updating).\n\n * \"git p4\" did not behave well when the path to the root of the P4\n   client was not its real path.\n   (merge bbd8486 pw/p4-symlinked-root later to maint).\n\n * \"git archive\" reports a failure when asked to create an archive out\n   of an empty tree.  It would be more intuitive to give an empty\n   archive back in such a case.\n\n * When \"format-patch\" quoted a non-ascii strings on the header files,\n   it incorrectly applied rfc2047 and chopped a single character in\n   the middle of it.\n\n * An aliased command spawned from a bare repository that does not say\n   it is bare with \"core.bare = yes\" is treated as non-bare by mistake.\n\n * In \"git reflog expire\", REACHABLE bit was not cleared from the\n   correct objects.\n\n * The logic used by \"git diff -M --stat\" to shorten the names of\n   files before and after a rename did not work correctly when the\n   common prefix and suffix between the two filenames overlapped.\n\n * The \"--match=<pattern>\" option of \"git describe\", when used with\n   \"--all\" to allow refs that are not annotated tags to be used as a\n   base of description, did not restrict the output from the command\n   to those that match the given pattern.\n\n * Clarify in the documentation \"what\" gets pushed to \"where\" when the\n   command line to \"git push\" does not say these explicitly.\n\n * The \"--color=<when>\" argument to the commands in the diff family\n   was described poorly.\n\n * The arguments given to pre-rebase hook were not documented.\n\n * The v4 index format was not documented.\n\n * The \"--match=<pattern>\" argument \"git describe\" takes uses glob\n   pattern but it wasn't obvious from the documentation.\n\n * Some sources failed to compile on systems that lack NI_MAXHOST in\n   their system header (e.g. z/OS).\n\n * Add an example use of \"--env-filter\" in \"filter-branch\"\n   documentation.\n\n * \"git bundle verify\" did not say \"records a complete history\" for a\n   bundle that does not have any prerequisites.\n\n * In the v1.8.0 era, we changed symbols that do not have to be global\n   to file scope static, but a few functions in graph.c were used by\n   CGit from sideways bypassing the entry points of the API the\n   in-tree users use.\n\n * \"git update-index -h\" did not do the usual \"-h(elp)\" thing.\n\n * \"git index-pack\" had a buffer-overflow while preparing an\n   informational message when the translated version of it was too\n   long.\n\n * 'git commit -m \"$msg\"' used to add an extra newline even when\n   $msg already ended with one.\n\n * The SSL peer verification done by \"git imap-send\" did not ask for\n   Server Name Indication (RFC 4366), failing to connect SSL/TLS\n   sites that serve multiple hostnames on a single IP.\n\n * perl/Git.pm::cat_blob slurped everything in core only to write it\n   out to a file descriptor, which was not a very smart thing to do.\n\n * \"git branch\" did not bother to check nonsense command line\n   parameters and issue errors in many cases.\n\n * Verification of signed tags were not done correctly when not in C\n   or en/US locale.\n\n * Some platforms and users spell UTF-8 differently; retry with the\n   most official \"UTF-8\" when the system does not understand the\n   user-supplied encoding name that are the common alternative\n   spellings of UTF-8.\n\n * When export-subst is used, \"zip\" output recorded incorrect\n   size of the file.\n\n * \"git am $maildir/\" applied messages in an unexpected order; sort\n   filenames read from the maildir/ in a way that is more likely to\n   sort messages in the order the writing MUA meant to, by sorting\n   numeric segment in numeric order and non-numeric segment in\n   alphabetical order.\n\n * \"git submodule update\", when recursed into sub-submodules, did not\n   accumulate the prefix paths.\n"},{"id":"215673","messageId":"CAH_OBievcf-_z_AX9UrmWL_HVFT2vSQTu+wXAjAFeQBM8iFSGw@mail.gmail.com","threadId":"33626","inReplyTo":"7vvc78u8jl.fsf@alter.siamese.dyndns.org","subject":"Re: [ANNOUNCE] Git v1.8.3-rc0","fromName":"shawn wilson","fromEmail":"ag4ve.us@gmail.com","sentAt":"2013-04-27T02:24:07Z","receivedAt":"2013-04-27T02:24:07Z","isPatch":false,"sender":{"key":"ag4ve.us@gmail.com","avatar":null},"body":"On Fri, Apr 26, 2013 at 8:22 PM, Junio C Hamano <gitster@pobox.com> wrote:\n\n>  * There was no good way to ask \"I have a random string that came from\n>    outside world. I want to turn it into a 40-hex object name while\n>    making sure such an object exists\".  A new peeling suffix ^{object}\n>    can be used for that purpose, together with \"rev-parse --verify\".\n>\n\nWhat does this mean / what is the reason behind this? I can only think\nit might be useful in a test suite to make sure git isn't doing\nanything stupid with hashes...?\n"},{"id":"215678","messageId":"517B62D8.5020006@alum.mit.edu","threadId":"33626","inReplyTo":"CAH_OBievcf-_z_AX9UrmWL_HVFT2vSQTu+wXAjAFeQBM8iFSGw@mail.gmail.com","subject":"Re: [ANNOUNCE] Git v1.8.3-rc0","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2013-04-27T05:32:08Z","receivedAt":"2013-04-27T05:32:08Z","isPatch":false,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"On 04/27/2013 04:24 AM, shawn wilson wrote:\n> On Fri, Apr 26, 2013 at 8:22 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> \n>>  * There was no good way to ask \"I have a random string that came from\n>>    outside world. I want to turn it into a 40-hex object name while\n>>    making sure such an object exists\".  A new peeling suffix ^{object}\n>>    can be used for that purpose, together with \"rev-parse --verify\".\n>>\n> \n> What does this mean / what is the reason behind this? I can only think\n> it might be useful in a test suite to make sure git isn't doing\n> anything stupid with hashes...?\n\nThe topic is discussed here:\n\nhttp://git.661346.n2.nabble.com/Bug-in-quot-git-rev-parse-verify-quot-td7580929.html\n\nAs discussed in the thread, when verifying that an argument names an\nexisting object, it is usually also appropriate to verify that the named\nobject is of a particular type (or can be converted to a particular\ntype), which could already be done with syntax like\n\"$userstring^{commit}\".  But if, for example, you want to avoid\nunwrapping tags but also want to verify that the named object really\nexists, \"$userstring^{object}\" now provides a way.\n\nAnd what do you have against test suites? :-)\n\nMichael\n\n-- \nMichael Haggerty\nmhagger@alum.mit.edu\nhttp://softwareswirl.blogspot.com/\n"},{"id":"215681","messageId":"20130427091833.GB472@serenity.lan","threadId":"33626","inReplyTo":"7vvc78u8jl.fsf@alter.siamese.dyndns.org","subject":"Re: [ANNOUNCE] Git v1.8.3-rc0","fromName":"John Keeping","fromEmail":"john@keeping.me.uk","sentAt":"2013-04-27T09:18:33Z","receivedAt":"2013-04-27T09:18:33Z","isPatch":false,"sender":{"key":"john@keeping.me.uk","avatar":"https://avatars.githubusercontent.com/u/1702081?v=4"},"body":"On Fri, Apr 26, 2013 at 05:22:22PM -0700, Junio C Hamano wrote:\n>  * \"git difftool\" allows the user to write into the temporary files\n>    being shown; if the user makes changes to the working tree at the\n>    same time, one of the changes has to be lost in such a case, but it\n>    tells the user what happened and refrains from overwriting the copy\n>    in the working tree.\n\nThis feels slightly misleading to me, perhaps something like this would\nbe clearer?\n\n   \"git difftool\" allows the user to write into the temporary files\n   being shown; if the user makes changes to the working tree at the\n   same time, it now refrains from overwriting the copy in the working\n   tree and leaves the temporary file so that changes can be merged\n   manually.\n"},{"id":"215811","messageId":"7vvc76pdfc.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"517B62D8.5020006@alum.mit.edu","subject":"Re: [ANNOUNCE] Git v1.8.3-rc0","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-28T21:12:39Z","receivedAt":"2013-04-28T21:12:39Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Michael Haggerty <mhagger@alum.mit.edu> writes:\n\n> On 04/27/2013 04:24 AM, shawn wilson wrote:\n>> On Fri, Apr 26, 2013 at 8:22 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> \n>>>  * There was no good way to ask \"I have a random string that came from\n>>>    outside world. I want to turn it into a 40-hex object name while\n>>>    making sure such an object exists\".  A new peeling suffix ^{object}\n>>>    can be used for that purpose, together with \"rev-parse --verify\".\n>>>\n>> \n>> What does this mean / what is the reason behind this? I can only think\n>> it might be useful in a test suite to make sure git isn't doing\n>> anything stupid with hashes...?\n>\n> The topic is discussed here:\n>\n> http://git.661346.n2.nabble.com/Bug-in-quot-git-rev-parse-verify-quot-td7580929.html\n>\n> As discussed in the thread, when verifying that an argument names an\n> existing object, it is usually also appropriate to verify that the named\n> object is of a particular type (or can be converted to a particular\n> type), which could already be done with syntax like\n> \"$userstring^{commit}\".  But if, for example, you want to avoid\n> unwrapping tags but also want to verify that the named object really\n> exists, \"$userstring^{object}\" now provides a way.\n>\n> And what do you have against test suites? :-)\n\nAnd it is not about test in the first place.  Git is designed to be\nscriptable, and it is not unreasonable for a scripted Porcelain to\nwant to learn the full object name of the object that is referred to\nby a string that it suspects may be an object name.  Perhaps you are\nfeeding the entire git mailing list archive to a script that picks\nup any object name in the messages and tallying the number of times\neach object is mentioned.  Then you would want to key the table that\ncounts the number of appearances for each object with the object\nname, because different message may spell the name of the same\nobject differently, e.g. f9fc12cf3, v1.8.3-rc0, etc.  With a helper\nfunction \"found_one_more_instance\" that records the fact you saw\nanother mention of an object, such a program may do something like\nthis:\n\n\ttokenize_git_mailing_list_message |\n        while read userstring\n        do\n\t\tcanonical=$(git rev-parse $userstring^{object}) &&\n\t        found_one_more_instance \"$canonical\"\n\tdone\n"},{"id":"215817","messageId":"7vzjwinhav.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"20130427091833.GB472@serenity.lan","subject":"Re: [ANNOUNCE] Git v1.8.3-rc0","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-29T03:31:52Z","receivedAt":"2013-04-29T03:31:52Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"John Keeping <john@keeping.me.uk> writes:\n\n> On Fri, Apr 26, 2013 at 05:22:22PM -0700, Junio C Hamano wrote:\n>>  * \"git difftool\" allows the user to write into the temporary files\n>>    being shown; if the user makes changes to the working tree at the\n>>    same time, one of the changes has to be lost in such a case, but it\n>>    tells the user what happened and refrains from overwriting the copy\n>>    in the working tree.\n>\n> This feels slightly misleading to me, perhaps something like this would\n> be clearer?\n>\n>    \"git difftool\" allows the user to write into the temporary files\n>    being shown; if the user makes changes to the working tree at the\n>    same time, it now refrains from overwriting the copy in the working\n>    tree and leaves the temporary file so that changes can be merged\n>    manually.\n\nProbably.  I'll keep the above in my stash and roll it in by -rc1.\n\nThanks.\n"},{"id":"215905","messageId":"1367262904-9885-1-git-send-email-marcnarc@xiplink.com","threadId":"33626","inReplyTo":"7vvc78u8jl.fsf@alter.siamese.dyndns.org","subject":"[PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2013-04-29T19:15:04Z","receivedAt":"2013-04-29T19:15:04Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"Signed-off-by: Marc Branchaud <marcnarc@xiplink.com>\n\n---\n\nThis started out as an attempt to make the backward compatibility notes\nmore parsable, but then I just kept going...\n\n\t\tM.\n\n\n Documentation/RelNotes/1.8.3.txt | 145 +++++++++++++++++++--------------------\n 1 file changed, 72 insertions(+), 73 deletions(-)\n\ndiff --git a/Documentation/RelNotes/1.8.3.txt b/Documentation/RelNotes/1.8.3.txt\nindex 6d25165..06bc831 100644\n--- a/Documentation/RelNotes/1.8.3.txt\n+++ b/Documentation/RelNotes/1.8.3.txt\n@@ -8,23 +8,22 @@ When \"git push [$there]\" does not say what to push, we have used the\n traditional \"matching\" semantics so far (all your branches were sent\n to the remote as long as there already are branches of the same name\n over there).  In Git 2.0, the default will change to the \"simple\"\n-semantics that pushes the current branch to the branch with the same\n-name, only when the current branch is set to integrate with that\n-remote branch.  There is a user preference configuration variable\n+semantics that pushes only the current branch to the branch with the same\n+name, and only when the current branch is set to integrate with that\n+remote branch.  Use the user preference configuration variable\n \"push.default\" to change this.  If you are an old-timer who is used\n-to the \"matching\" semantics, you can set it to \"matching\" to keep the\n+to the \"matching\" semantics, you can set the varaible to \"matching\" to keep the\n traditional behaviour.  If you want to live in the future early,\n you can set it to \"simple\" today without waiting for Git 2.0.\n \n-When \"git add -u\" and \"git add -A\", that does not specify what paths\n-to add on the command line is run from inside a subdirectory, these\n-commands will operate on the entire tree in Git 2.0 for consistency\n-with \"git commit -a\" and other commands. Because there will be no\n-mechanism to make \"git add -u\" behave as if \"git add -u .\", it is\n-important for those who are used to \"git add -u\" (without pathspec)\n-updating the index only for paths in the current subdirectory to start\n-training their fingers to explicitly say \"git add -u .\" when they mean\n-it before Git 2.0 comes.  A warning is issued when these commands are\n+When \"git add -u\" (and \"git add -A\") is run inside a subdirectory and\n+does not specify which paths to add on the command line, it\n+will operate on the entire tree in Git 2.0 for consistency\n+with \"git commit -a\" and other commands.  There will be no\n+mechanism to make plain \"git add -u\" behave like \"git add -u .\".\n+Current users of \"git add -u\" (without a pathspec) should start\n+training their fingers to explicitly say \"git add -u .\"\n+before Git 2.0 comes.  A warning is issued when these commands are\n run without a pathspec and when you have local changes outside the\n current directory, because the behaviour in Git 2.0 will be different\n from today's version in such a situation.\n@@ -33,8 +32,8 @@ In Git 2.0, \"git add <path>\" will behave as \"git add -A <path>\", so\n that \"git add dir/\" will notice paths you removed from the directory\n and record the removal.  Versions before Git 2.0, including this\n release, will keep ignoring removals, but the users who rely on this\n-behaviour is encouraged to use \"git add --ignore-removal <path>\" and\n-get used to it.\n+behaviour are encouraged to start using \"git add --ignore-removal <path>\"\n+now before 2.0 is released.\n \n \n Updates since v1.8.2\n@@ -114,7 +113,7 @@ UI, Workflows & Features\n  * \"git status\" suggests users to look into using --untracked=no option\n    when it takes too long.\n \n- * \"git status\" shows a bit more information to \"git status\" during a\n+ * \"git status\" shows a bit more information during a\n    rebase/bisect session.\n \n  * \"git fetch\" learned to fetch a commit at the tip of an unadvertised\n@@ -148,8 +147,8 @@ UI, Workflows & Features\n \n  * \"git mergetool\" now feeds files to the \"p4merge\" backend in the\n    order that matches the p4 convention, where \"theirs\" is usually\n-   shown on the left side, which is the opposite from other backend\n-   expects.\n+   shown on the left side, which is the opposite from what other backends\n+   expect.\n \n  * \"show/log\" now honors gpg.program configuration just like other\n    parts of the code that use GnuPG.\n@@ -173,7 +172,7 @@ Performance, Internal Implementation, etc.\n \n  * Updates for building under msvc.\n \n- * A handful of issues in the code to traverse working tree to find\n+ * A handful of issues in the code that traverses the working tree to find\n    untracked and/or ignored files have been fixed, and the general\n    codepath involved in \"status -u\" and \"clean\" have been cleaned up\n    and optimized.\n@@ -182,15 +181,15 @@ Performance, Internal Implementation, etc.\n    pack has been shrunk.\n \n  * The logic to coalesce the same lines removed from the parents in\n-   the output from \"diff -c/--cc\" has been updated, but with an O(n^2)\n+   the output from \"diff -c/--cc\" has been updated, but with O(n^2)\n    complexity, so this might turn out to be undesirable.\n \n  * The code to enforce permission bits on files in $GIT_DIR/ for\n-   shared repositories have been simplified.\n+   shared repositories has been simplified.\n \n- * A few codepaths knew how much data they need to put in the\n-   hashtables they use upfront, but still started from a small table\n-   repeatedly growing and rehashing.\n+ * A few codepaths know how much data they need to put in the\n+   hashtables they use when they start, but still began with small tables\n+   and repeatedly grew and rehashed them.\n \n  * The API to walk reflog entries from the latest to older, which was\n    necessary for operations such as \"git checkout -\", was cumbersome\n@@ -202,9 +201,9 @@ Performance, Internal Implementation, etc.\n  * The pkt-line API, implementation and its callers have been cleaned\n    up to make them more robust.\n \n- * Cygwin port has a faster-but-lying lstat(2) emulation whose\n+ * The Cygwin port has a faster-but-lying lstat(2) emulation whose\n    incorrectness does not matter in practice except for a few\n-   codepaths, and setting permission bits to directories is a codepath\n+   codepaths, and setting permission bits on directories is a codepath\n    that needs to use a more correct one.\n \n  * \"git checkout\" had repeated pathspec matches on the same paths,\n@@ -225,42 +224,42 @@ Unless otherwise noted, all the fixes since v1.8.2 in the maintenance\n track are contained in this release (see release notes to them for\n details).\n \n- * When receive-pack detects error in the pack header it received in\n+ * When receive-pack detects an error in the pack header it received in\n    order to decide which of unpack-objects or index-pack to run, it\n-   returned without closing the error stream, which led to a hang\n+   returned without closing the error stream, which led to a hung\n    sideband thread.\n \n- * Zsh completion forgot that '%' character used to signal untracked\n+ * Zsh completion forgot that the '%' character used to signal untracked\n    files needs to be escaped with another '%'.\n \n  * A commit object whose author or committer ident are malformed\n-   crashed some code that trusted that a name, an email and an\n+   crashed some code that trusted that a name, an email and a\n    timestamp can always be found in it.\n \n  * When \"upload-pack\" fails while generating a pack in response to\n-   \"git fetch\" (or \"git clone\"), the receiving side mistakenly said\n-   there was a programming error to trigger the die handler\n+   \"git fetch\" (or \"git clone\"), the receiving side had\n+   a programming error that triggered the die handler\n    recursively.\n \n- * \"rev-list --stdin\" and friends kept bogus pointers into input\n+ * \"rev-list --stdin\" and friends kept bogus pointers into the input\n    buffer around as human readble object names.  This was not a huge\n    problem but was exposed by a new change that uses these names in\n    error output.\n    (merge 70d26c6 tr/copy-revisions-from-stdin later to maint).\n \n  * Smart-capable HTTP servers were not restricted via the\n-   GIT_NAMESPACE mechanism when talking with commit-walker clients,\n-   like they do when talking with smart HTTP clients.\n+   GIT_NAMESPACE mechanism when talking with commit-walking clients,\n+   like they are when talking with smart HTTP clients.\n    (merge 6130f86 jk/http-dumb-namespaces later to maint).\n \n  * \"git merge-tree\" did not omit a merge result that is identical to\n-   \"our\" side in certain cases.\n+   the \"our\" side in certain cases.\n    (merge aacecc3 jk/merge-tree-added-identically later to maint).\n \n- * Perl scripts like \"git-svn\" closed (not redirecting to /dev/null)\n+ * Perl scripts like \"git-svn\" closed (instead of redirecting to /dev/null)\n    the standard error stream, which is not a very smart thing to do.\n-   Later open may return file descriptor #2 for unrelated purpose, and\n-   error reporting code may write into them.\n+   A later open may return file descriptor #2 for an unrelated purpose, and\n+   error reporting code may write into it.\n \n  * \"git show-branch\" was not prepared to show a very long run of\n    ancestor operators e.g. foobar^2~2^2^2^2...^2~4 correctly.\n@@ -268,17 +267,17 @@ details).\n  * \"git diff --diff-algorithm algo\" is also understood as \"git diff\n    --diff-algorithm=algo\".\n \n- * The new core.commentchar configuration was not applied to a few\n+ * The new core.commentchar configuration was not applied in a few\n    places.\n \n  * \"git bundle\" did not like a bundle created using a commit without\n-   any message as its one of the prerequistes.\n+   any message, as it is one of the prerequistes.\n \n  * \"git log -S/-G\" started paying attention to textconv filter, but\n-   there was no way to disable this.  Make it honor --no-textconv\n+   there was no way to disable this.  Make it honor the --no-textconv\n    option.\n \n- * When used with \"-d temporary-directory\" option, \"git filter-branch\"\n+ * When used with the \"-d temporary-directory\" option, \"git filter-branch\"\n    failed to come back to the original working tree to perform the\n    final clean-up procedure.\n \n@@ -287,9 +286,9 @@ details).\n    not pay much attention to the annotated tag payload.  Make the code\n    notice the type of the tag object, in addition to the dwim_ref()\n    based classification the current code uses (i.e. the name appears\n-   in refs/tags/) to decide when to special case merging of tags.\n+   in refs/tags/) to decide when to special-case tag merging.\n \n- * Fix 1.8.1.x regression that stopped matching \"dir\" (without\n+ * Fix a 1.8.1.x regression that stopped matching \"dir\" (without a\n    trailing slash) to a directory \"dir\".\n    (merge efa5f82 jc/directory-attrs-regression-fix later to maint-1.8.1).\n \n@@ -300,46 +299,46 @@ details).\n  * The prompt string generator (in contrib/completion/) did not notice\n    when we are in a middle of a \"git revert\" session.\n \n- * \"submodule summary --summary-limit\" option did not support\n+ * \"submodule summary --summary-limit\" option did not support the\n    \"--option=value\" form.\n \n  * \"index-pack --fix-thin\" used an uninitialized value to compute\n-   delta depths of objects it appends to the resulting pack.\n+   the delta depths of objects it appends to the resulting pack.\n \n- * \"index-pack --verify-stat\" used a few counters outside protection\n-   of mutex, possibly showing incorrect numbers.\n+ * \"index-pack --verify-stat\" used a few counters outside the protection\n+   of a mutex, possibly showing incorrect numbers.\n \n  * The code to keep track of what directory names are known to Git on\n-   platforms with case insensitive filesystems can get confused upon a\n-   hash collision between these pathnames and looped forever.\n+   platforms with case insensitive filesystems could get confused upon a\n+   hash collision between these pathnames and would loop forever.\n \n- * Annotated tags outside refs/tags/ hierarchy were not advertised\n-   correctly to the ls-remote and fetch with recent version of Git.\n+ * Annotated tags outside the refs/tags/ hierarchy were not advertised\n+   correctly to ls-remote and fetch with recent versions of Git.\n \n- * Recent optimization broke shallow clones.\n+ * Recent optimizations broke shallow clones.\n \n  * \"git cmd -- ':(top'\" was not diagnosed as an invalid syntax, and\n    instead the parser kept reading beyond the end of the string.\n \n  * \"git tag -f <tag>\" always said \"Updated tag '<tag>'\" even when\n-   creating a new tag (i.e. not overwriting nor updating).\n+   creating a new tag (i.e. neither overwriting nor updating).\n \n  * \"git p4\" did not behave well when the path to the root of the P4\n    client was not its real path.\n    (merge bbd8486 pw/p4-symlinked-root later to maint).\n \n- * \"git archive\" reports a failure when asked to create an archive out\n-   of an empty tree.  It would be more intuitive to give an empty\n+ * \"git archive\" reported a failure when asked to create an archive out\n+   of an empty tree.  It is more intuitive to give an empty\n    archive back in such a case.\n \n- * When \"format-patch\" quoted a non-ascii strings on the header files,\n+ * When \"format-patch\" quoted a non-ascii string in header files,\n    it incorrectly applied rfc2047 and chopped a single character in\n-   the middle of it.\n+   the middle of the string.\n \n  * An aliased command spawned from a bare repository that does not say\n-   it is bare with \"core.bare = yes\" is treated as non-bare by mistake.\n+   it is bare with \"core.bare = yes\" was treated as non-bare by mistake.\n \n- * In \"git reflog expire\", REACHABLE bit was not cleared from the\n+ * In \"git reflog expire\", the REACHABLE bit was not cleared from the\n    correct objects.\n \n  * The logic used by \"git diff -M --stat\" to shorten the names of\n@@ -347,9 +346,9 @@ details).\n    common prefix and suffix between the two filenames overlapped.\n \n  * The \"--match=<pattern>\" option of \"git describe\", when used with\n-   \"--all\" to allow refs that are not annotated tags to be used as a\n+   \"--all\" to allow refs that are not annotated tags to be a\n    base of description, did not restrict the output from the command\n-   to those that match the given pattern.\n+   to those refs that match the given pattern.\n \n  * Clarify in the documentation \"what\" gets pushed to \"where\" when the\n    command line to \"git push\" does not say these explicitly.\n@@ -357,7 +356,7 @@ details).\n  * The \"--color=<when>\" argument to the commands in the diff family\n    was described poorly.\n \n- * The arguments given to pre-rebase hook were not documented.\n+ * The arguments given to the pre-rebase hook were not documented.\n \n  * The v4 index format was not documented.\n \n@@ -375,7 +374,7 @@ details).\n \n  * In the v1.8.0 era, we changed symbols that do not have to be global\n    to file scope static, but a few functions in graph.c were used by\n-   CGit from sideways bypassing the entry points of the API the\n+   CGit sideways, bypassing the entry points of the API the\n    in-tree users use.\n \n  * \"git update-index -h\" did not do the usual \"-h(elp)\" thing.\n@@ -388,30 +387,30 @@ details).\n    $msg already ended with one.\n \n  * The SSL peer verification done by \"git imap-send\" did not ask for\n-   Server Name Indication (RFC 4366), failing to connect SSL/TLS\n+   Server Name Indication (RFC 4366), failing to connect to SSL/TLS\n    sites that serve multiple hostnames on a single IP.\n \n  * perl/Git.pm::cat_blob slurped everything in core only to write it\n    out to a file descriptor, which was not a very smart thing to do.\n \n  * \"git branch\" did not bother to check nonsense command line\n-   parameters and issue errors in many cases.\n+   parameters.  It now issues errors in many cases.\n \n- * Verification of signed tags were not done correctly when not in C\n+ * Verification of signed tags was not done correctly when not in C\n    or en/US locale.\n \n  * Some platforms and users spell UTF-8 differently; retry with the\n    most official \"UTF-8\" when the system does not understand the\n-   user-supplied encoding name that are the common alternative\n-   spellings of UTF-8.\n+   user-supplied encoding name that is a common alternative\n+   spelling of UTF-8.\n \n- * When export-subst is used, \"zip\" output recorded incorrect\n+ * When export-subst is used, \"zip\" output recorded an incorrect\n    size of the file.\n \n  * \"git am $maildir/\" applied messages in an unexpected order; sort\n    filenames read from the maildir/ in a way that is more likely to\n-   sort messages in the order the writing MUA meant to, by sorting\n-   numeric segment in numeric order and non-numeric segment in\n+   sort the messages in the order the writing MUA meant to, by sorting\n+   numeric segments in numeric order and non-numeric segments in\n    alphabetical order.\n \n  * \"git submodule update\", when recursed into sub-submodules, did not\n-- \n1.8.2\n"},{"id":"215920","messageId":"7vli81ghsf.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"1367262904-9885-1-git-send-email-marcnarc@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-29T21:15:28Z","receivedAt":"2013-04-29T21:15:28Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marc Branchaud <marcnarc@xiplink.com> writes:\n\n> This started out as an attempt to make the backward compatibility notes\n> more parsable, but then I just kept going...\n\nThanks.\n\n>   * \"git bundle\" did not like a bundle created using a commit without\n> -   any message as its one of the prerequistes.\n> +   any message, as it is one of the prerequistes.\n\nThis is actually saying a different thing.  \n\nWhen you create a bundle, you can say \"you can only unbundle this in\na repository that has commit X\", with \"git bundle create $name ^X Y\nZ\".  Such a commit X is called the bundle's prerequisite.  You can\nhave more than one prerequisite, e.g. \"... ^X ^W Y Z\".\n\nBut if you create a bundle by using a commit that does not have any\nmessage as X (i.e. the bundle's prerequisite), the \"git bundle\" did\nnot like to read the resulting bundle output.\n\nSo <a commit <without any message> as its (bundle's) one of the\nprerequisites> is what the original wanted to say.  The rewrite\nmakes it read like \"For a commit, having a message is a requirement\nto be used in a bundle\", at least to me.\n\n>   * \"index-pack --fix-thin\" used an uninitialized value to compute\n> -   delta depths of objects it appends to the resulting pack.\n> +   the delta depths of objects it appends to the resulting pack.\n>  \n> - * \"index-pack --verify-stat\" used a few counters outside protection\n> -   of mutex, possibly showing incorrect numbers.\n> + * \"index-pack --verify-stat\" used a few counters outside the protection\n> +   of a mutex, possibly showing incorrect numbers.\n>  \n>   * The code to keep track of what directory names are known to Git on\n> -   platforms with case insensitive filesystems can get confused upon a\n> -   hash collision between these pathnames and looped forever.\n> +   platforms with case insensitive filesystems could get confused upon a\n> +   hash collision between these pathnames and would loop forever.\n>  \n> - * Annotated tags outside refs/tags/ hierarchy were not advertised\n> -   correctly to the ls-remote and fetch with recent version of Git.\n> + * Annotated tags outside the refs/tags/ hierarchy were not advertised\n> +   correctly to ls-remote and fetch with recent versions of Git.\n>  \n> - * Recent optimization broke shallow clones.\n> + * Recent optimizations broke shallow clones.\n>  \n>   * \"git cmd -- ':(top'\" was not diagnosed as an invalid syntax, and\n>     instead the parser kept reading beyond the end of the string.\n>  \n>   * \"git tag -f <tag>\" always said \"Updated tag '<tag>'\" even when\n> -   creating a new tag (i.e. not overwriting nor updating).\n> +   creating a new tag (i.e. neither overwriting nor updating).\n>  \n>   * \"git p4\" did not behave well when the path to the root of the P4\n>     client was not its real path.\n>     (merge bbd8486 pw/p4-symlinked-root later to maint).\n>  \n> - * \"git archive\" reports a failure when asked to create an archive out\n> -   of an empty tree.  It would be more intuitive to give an empty\n> + * \"git archive\" reported a failure when asked to create an archive out\n> +   of an empty tree.  It is more intuitive to give an empty\n>     archive back in such a case.\n>  \n> - * When \"format-patch\" quoted a non-ascii strings on the header files,\n> + * When \"format-patch\" quoted a non-ascii string in header files,\n>     it incorrectly applied rfc2047 and chopped a single character in\n> -   the middle of it.\n> +   the middle of the string.\n>  \n>   * An aliased command spawned from a bare repository that does not say\n> -   it is bare with \"core.bare = yes\" is treated as non-bare by mistake.\n> +   it is bare with \"core.bare = yes\" was treated as non-bare by mistake.\n>  \n> - * In \"git reflog expire\", REACHABLE bit was not cleared from the\n> + * In \"git reflog expire\", the REACHABLE bit was not cleared from the\n>     correct objects.\n>  \n>   * The logic used by \"git diff -M --stat\" to shorten the names of\n> @@ -347,9 +346,9 @@ details).\n>     common prefix and suffix between the two filenames overlapped.\n>  \n>   * The \"--match=<pattern>\" option of \"git describe\", when used with\n> -   \"--all\" to allow refs that are not annotated tags to be used as a\n> +   \"--all\" to allow refs that are not annotated tags to be a\n>     base of description, did not restrict the output from the command\n> -   to those that match the given pattern.\n> +   to those refs that match the given pattern.\n>  \n>   * Clarify in the documentation \"what\" gets pushed to \"where\" when the\n>     command line to \"git push\" does not say these explicitly.\n> @@ -357,7 +356,7 @@ details).\n>   * The \"--color=<when>\" argument to the commands in the diff family\n>     was described poorly.\n>  \n> - * The arguments given to pre-rebase hook were not documented.\n> + * The arguments given to the pre-rebase hook were not documented.\n>  \n>   * The v4 index format was not documented.\n>  \n> @@ -375,7 +374,7 @@ details).\n>  \n>   * In the v1.8.0 era, we changed symbols that do not have to be global\n>     to file scope static, but a few functions in graph.c were used by\n> -   CGit from sideways bypassing the entry points of the API the\n> +   CGit sideways, bypassing the entry points of the API the\n>     in-tree users use.\n>  \n>   * \"git update-index -h\" did not do the usual \"-h(elp)\" thing.\n> @@ -388,30 +387,30 @@ details).\n>     $msg already ended with one.\n>  \n>   * The SSL peer verification done by \"git imap-send\" did not ask for\n> -   Server Name Indication (RFC 4366), failing to connect SSL/TLS\n> +   Server Name Indication (RFC 4366), failing to connect to SSL/TLS\n>     sites that serve multiple hostnames on a single IP.\n>  \n>   * perl/Git.pm::cat_blob slurped everything in core only to write it\n>     out to a file descriptor, which was not a very smart thing to do.\n>  \n>   * \"git branch\" did not bother to check nonsense command line\n> -   parameters and issue errors in many cases.\n> +   parameters.  It now issues errors in many cases.\n>  \n> - * Verification of signed tags were not done correctly when not in C\n> + * Verification of signed tags was not done correctly when not in C\n>     or en/US locale.\n>  \n>   * Some platforms and users spell UTF-8 differently; retry with the\n>     most official \"UTF-8\" when the system does not understand the\n> -   user-supplied encoding name that are the common alternative\n> -   spellings of UTF-8.\n> +   user-supplied encoding name that is a common alternative\n> +   spelling of UTF-8.\n>  \n> - * When export-subst is used, \"zip\" output recorded incorrect\n> + * When export-subst is used, \"zip\" output recorded an incorrect\n>     size of the file.\n>  \n>   * \"git am $maildir/\" applied messages in an unexpected order; sort\n>     filenames read from the maildir/ in a way that is more likely to\n> -   sort messages in the order the writing MUA meant to, by sorting\n> -   numeric segment in numeric order and non-numeric segment in\n> +   sort the messages in the order the writing MUA meant to, by sorting\n> +   numeric segments in numeric order and non-numeric segments in\n>     alphabetical order.\n>  \n>   * \"git submodule update\", when recursed into sub-submodules, did not\n"},{"id":"215921","messageId":"7vhaipghpy.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"7vli81ghsf.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-29T21:16:57Z","receivedAt":"2013-04-29T21:16:57Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Marc Branchaud <marcnarc@xiplink.com> writes:\n>\n>> This started out as an attempt to make the backward compatibility notes\n>> more parsable, but then I just kept going...\n>\n> Thanks.\n>\n>>   * \"git bundle\" did not like a bundle created using a commit without\n>> -   any message as its one of the prerequistes.\n>> +   any message, as it is one of the prerequistes.\n>\n> This is actually saying a different thing.  \n>\n> When you create a bundle, you can say \"you can only unbundle this in\n> a repository that has commit X\", with \"git bundle create $name ^X Y\n> Z\".  Such a commit X is called the bundle's prerequisite.  You can\n> have more than one prerequisite, e.g. \"... ^X ^W Y Z\".\n>\n> But if you create a bundle by using a commit that does not have any\n> message as X (i.e. the bundle's prerequisite), the \"git bundle\" did\n> not like to read the resulting bundle output.\n>\n> So <a commit <without any message> as its (bundle's) one of the\n> prerequisites> is what the original wanted to say.  The rewrite\n> makes it read like \"For a commit, having a message is a requirement\n> to be used in a bundle\", at least to me.\n>\n>>   * \"index-pack --fix-thin\" used an uninitialized value to compute\n\nPlease disregard everything below this line; the message was sent\nunfinished by accident.\n"},{"id":"215922","messageId":"7va9ohghjt.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"1367262904-9885-1-git-send-email-marcnarc@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-04-29T21:20:38Z","receivedAt":"2013-04-29T21:20:38Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marc Branchaud <marcnarc@xiplink.com> writes:\n\n> Signed-off-by: Marc Branchaud <marcnarc@xiplink.com>\n>\n> ---\n>\n> This started out as an attempt to make the backward compatibility notes\n> more parsable, but then I just kept going...\n\nThanks; everything other than the \"bundle\" thing looked sensible.\n"},{"id":"215948","messageId":"CAPig+cSPbzU=vEFpE8AUZzvyHR3DbbJv-AxSmZ=dTyKzRUOQ2A@mail.gmail.com","threadId":"33626","inReplyTo":"1367262904-9885-1-git-send-email-marcnarc@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2013-04-30T03:09:19Z","receivedAt":"2013-04-30T03:09:19Z","isPatch":true,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Mon, Apr 29, 2013 at 3:15 PM, Marc Branchaud <marcnarc@xiplink.com> wrote:\n> --- a/Documentation/RelNotes/1.8.3.txt\n> +++ b/Documentation/RelNotes/1.8.3.txt\n> @@ -8,23 +8,22 @@ When \"git push [$there]\" does not say what to push, we have used the\n>  traditional \"matching\" semantics so far (all your branches were sent\n>  to the remote as long as there already are branches of the same name\n>  over there).  In Git 2.0, the default will change to the \"simple\"\n> -semantics that pushes the current branch to the branch with the same\n> -name, only when the current branch is set to integrate with that\n> -remote branch.  There is a user preference configuration variable\n> +semantics that pushes only the current branch to the branch with the same\n> +name, and only when the current branch is set to integrate with that\n> +remote branch.  Use the user preference configuration variable\n>  \"push.default\" to change this.  If you are an old-timer who is used\n> -to the \"matching\" semantics, you can set it to \"matching\" to keep the\n> +to the \"matching\" semantics, you can set the varaible to \"matching\" to keep the\n\ns/varaible/variable/\n\n>  traditional behaviour.  If you want to live in the future early,\n>  you can set it to \"simple\" today without waiting for Git 2.0.\n"},{"id":"215991","messageId":"517FD505.9000706@xiplink.com","threadId":"33626","inReplyTo":"7vli81ghsf.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2013-04-30T14:28:21Z","receivedAt":"2013-04-30T14:28:21Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"On 13-04-29 05:15 PM, Junio C Hamano wrote:\n> Marc Branchaud <marcnarc@xiplink.com> writes:\n> \n>> This started out as an attempt to make the backward compatibility notes\n>> more parsable, but then I just kept going...\n> \n> Thanks.\n> \n>>   * \"git bundle\" did not like a bundle created using a commit without\n>> -   any message as its one of the prerequistes.\n>> +   any message, as it is one of the prerequistes.\n> \n> This is actually saying a different thing.  \n> \n> When you create a bundle, you can say \"you can only unbundle this in\n> a repository that has commit X\", with \"git bundle create $name ^X Y\n> Z\".  Such a commit X is called the bundle's prerequisite.  You can\n> have more than one prerequisite, e.g. \"... ^X ^W Y Z\".\n> \n> But if you create a bundle by using a commit that does not have any\n> message as X (i.e. the bundle's prerequisite), the \"git bundle\" did\n> not like to read the resulting bundle output.\n> \n> So <a commit <without any message> as its (bundle's) one of the\n> prerequisites> is what the original wanted to say.  The rewrite\n> makes it read like \"For a commit, having a message is a requirement\n> to be used in a bundle\", at least to me.\n\nThanks, I did get that wrong.\n\nCC'ing Lukas, who wrote the relevant commit (5446e33f35).\n\nHow about:\n\n* \"git bundle\" can create a bundle that has a commit without a message as\n  a prerequisite, but it could not work with such a bundle.\n\n\t\tM.\n"},{"id":"215993","messageId":"CABURp0p+y_O4HVUfLpzTivmJrk61-bKdd+aEKm0uOtFCpTxBRQ@mail.gmail.com","threadId":"33626","inReplyTo":"1367262904-9885-1-git-send-email-marcnarc@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Phil Hord","fromEmail":"phil.hord@gmail.com","sentAt":"2013-04-30T15:12:35Z","receivedAt":"2013-04-30T15:12:35Z","isPatch":true,"sender":{"key":"phil.hord@gmail.com","avatar":"https://avatars.githubusercontent.com/u/123908?v=4"},"body":"On Mon, Apr 29, 2013 at 3:15 PM, Marc Branchaud <marcnarc@xiplink.com> wrote:\n> Signed-off-by: Marc Branchaud <marcnarc@xiplink.com>\n>\n> ---\n>\n> This started out as an attempt to make the backward compatibility notes\n> more parsable, but then I just kept going...\n\ns/parsable/parseable/\njk :-)\n\n>  Documentation/RelNotes/1.8.3.txt | 145 +++++++++++++++++++--------------------\n>  1 file changed, 72 insertions(+), 73 deletions(-)\n>\n> diff --git a/Documentation/RelNotes/1.8.3.txt b/Documentation/RelNotes/1.8.3.txt\n> index 6d25165..06bc831 100644\n> --- a/Documentation/RelNotes/1.8.3.txt\n> +++ b/Documentation/RelNotes/1.8.3.txt\n> @@ -8,23 +8,22 @@ When \"git push [$there]\" does not say what to push, we have used the\n>  traditional \"matching\" semantics so far (all your branches were sent\n>  to the remote as long as there already are branches of the same name\n...\n>\n> - * \"rev-list --stdin\" and friends kept bogus pointers into input\n> + * \"rev-list --stdin\" and friends kept bogus pointers into the input\n>     buffer around as human readble object names.  This was not a huge\n\nSo long as you're at it...\ns/readble/readable/\n\n> @@ -268,17 +267,17 @@ details).\n>   * \"git diff --diff-algorithm algo\" is also understood as \"git diff\n>     --diff-algorithm=algo\".\n>\n> - * The new core.commentchar configuration was not applied to a few\n> + * The new core.commentchar configuration was not applied in a few\n>     places.\n>\n>   * \"git bundle\" did not like a bundle created using a commit without\n> -   any message as its one of the prerequistes.\n> +   any message, as it is one of the prerequistes.\n\ns/prerequistes/prerequisites\n\n\nP\n"},{"id":"216126","messageId":"20130501082434.GA19771@blizzard","threadId":"33626","inReplyTo":"517FD505.9000706@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Lukas Fleischer","fromEmail":"git@cryptocrack.de","sentAt":"2013-05-01T08:24:34Z","receivedAt":"2013-05-01T08:24:34Z","isPatch":true,"sender":{"key":"git@cryptocrack.de","avatar":null},"body":"On Tue, Apr 30, 2013 at 10:28:21AM -0400, Marc Branchaud wrote:\n> On 13-04-29 05:15 PM, Junio C Hamano wrote:\n> > Marc Branchaud <marcnarc@xiplink.com> writes:\n> > \n> >> This started out as an attempt to make the backward compatibility notes\n> >> more parsable, but then I just kept going...\n> > \n> > Thanks.\n> > \n> >>   * \"git bundle\" did not like a bundle created using a commit without\n> >> -   any message as its one of the prerequistes.\n> >> +   any message, as it is one of the prerequistes.\n> > \n> > This is actually saying a different thing.  \n> > \n> > When you create a bundle, you can say \"you can only unbundle this in\n> > a repository that has commit X\", with \"git bundle create $name ^X Y\n> > Z\".  Such a commit X is called the bundle's prerequisite.  You can\n> > have more than one prerequisite, e.g. \"... ^X ^W Y Z\".\n> > \n> > But if you create a bundle by using a commit that does not have any\n> > message as X (i.e. the bundle's prerequisite), the \"git bundle\" did\n> > not like to read the resulting bundle output.\n> > \n> > So <a commit <without any message> as its (bundle's) one of the\n> > prerequisites> is what the original wanted to say.  The rewrite\n> > makes it read like \"For a commit, having a message is a requirement\n> > to be used in a bundle\", at least to me.\n> \n> Thanks, I did get that wrong.\n> \n> CC'ing Lukas, who wrote the relevant commit (5446e33f35).\n> \n> How about:\n> \n> * \"git bundle\" can create a bundle that has a commit without a message as\n>   a prerequisite, but it could not work with such a bundle.\n\nLooks fine to me.\n\nJunio's version with the last part changed to \"as one of its\nprerequisites\" sounds also good to me (in both cases, however, note the\nmissing \"i\" in \"prerequisites\").\n\nA third suggestion:\n\n    \"git bundle\" erroneously bailed out when parsing a valid bundle\n    containing a prerequisite commit without a commit message.\n\n> \n> \t\tM.\n"},{"id":"216144","messageId":"51812162.3060801@xiplink.com","threadId":"33626","inReplyTo":"20130501082434.GA19771@blizzard","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2013-05-01T14:06:26Z","receivedAt":"2013-05-01T14:06:26Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"On 13-05-01 04:24 AM, Lukas Fleischer wrote:\n> On Tue, Apr 30, 2013 at 10:28:21AM -0400, Marc Branchaud wrote:\n>> On 13-04-29 05:15 PM, Junio C Hamano wrote:\n>>> Marc Branchaud <marcnarc@xiplink.com> writes:\n>>>\n>>>> This started out as an attempt to make the backward compatibility notes\n>>>> more parsable, but then I just kept going...\n>>>\n>>> Thanks.\n>>>\n>>>>   * \"git bundle\" did not like a bundle created using a commit without\n>>>> -   any message as its one of the prerequistes.\n>>>> +   any message, as it is one of the prerequistes.\n>>>\n>>> This is actually saying a different thing.  \n>>>\n>>> When you create a bundle, you can say \"you can only unbundle this in\n>>> a repository that has commit X\", with \"git bundle create $name ^X Y\n>>> Z\".  Such a commit X is called the bundle's prerequisite.  You can\n>>> have more than one prerequisite, e.g. \"... ^X ^W Y Z\".\n>>>\n>>> But if you create a bundle by using a commit that does not have any\n>>> message as X (i.e. the bundle's prerequisite), the \"git bundle\" did\n>>> not like to read the resulting bundle output.\n>>>\n>>> So <a commit <without any message> as its (bundle's) one of the\n>>> prerequisites> is what the original wanted to say.  The rewrite\n>>> makes it read like \"For a commit, having a message is a requirement\n>>> to be used in a bundle\", at least to me.\n>>\n>> Thanks, I did get that wrong.\n>>\n>> CC'ing Lukas, who wrote the relevant commit (5446e33f35).\n>>\n>> How about:\n>>\n>> * \"git bundle\" can create a bundle that has a commit without a message as\n>>   a prerequisite, but it could not work with such a bundle.\n> \n> Looks fine to me.\n> \n> Junio's version with the last part changed to \"as one of its\n> prerequisites\" sounds also good to me (in both cases, however, note the\n> missing \"i\" in \"prerequisites\").\n> \n> A third suggestion:\n> \n>     \"git bundle\" erroneously bailed out when parsing a valid bundle\n>     containing a prerequisite commit without a commit message.\n\nI like that best.\n\n\t\tM.\n"},{"id":"216159","messageId":"7vmwse7ff9.fsf@alter.siamese.dyndns.org","threadId":"33626","inReplyTo":"51812162.3060801@xiplink.com","subject":"Re: [PATCH] Fix grammar in the 1.8.3 release notes.","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-01T17:55:54Z","receivedAt":"2013-05-01T17:55:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marc Branchaud <marcnarc@xiplink.com> writes:\n\n>> A third suggestion:\n>> \n>>     \"git bundle\" erroneously bailed out when parsing a valid bundle\n>>     containing a prerequisite commit without a commit message.\n>\n> I like that best.\n\nConcurred.  Thanks for your help, all.\n"}]}