| 7e20105 | Junio C Hamano | 17 October 2012, 17:36:01 UTC | Git 1.7.12.4 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 October 2012, 17:36:42 UTC |
| e98fa64 | Junio C Hamano | 17 October 2012, 17:29:24 UTC | Merge branch 'jk/maint-http-half-auth-push' into maint * jk/maint-http-half-auth-push: http: fix segfault in handle_curl_result | 17 October 2012, 17:29:24 UTC |
| 5a77ff7 | Junio C Hamano | 17 October 2012, 17:28:56 UTC | Merge branch 'rr/git-uri-doc' into maint * rr/git-uri-doc: Git url doc: mark ftp/ftps as read-only and deprecate them | 17 October 2012, 17:28:56 UTC |
| c15fada | Junio C Hamano | 17 October 2012, 17:28:48 UTC | Merge branch 'bw/cp-a-is-gnuism' into maint * bw/cp-a-is-gnuism: tests: "cp -a" is a GNUism | 17 October 2012, 17:28:48 UTC |
| 530d710 | Junio C Hamano | 17 October 2012, 17:28:42 UTC | Merge branch 'nd/doc-ignore' into maint * nd/doc-ignore: gitignore.txt: suggestions how to get literal # or ! at the beginning | 17 October 2012, 17:28:42 UTC |
| 352611f | Junio C Hamano | 17 October 2012, 17:28:34 UTC | Merge branch 'jc/doc-long-options' into maint * jc/doc-long-options: gitcli: parse-options lets you omit tail of long options | 17 October 2012, 17:28:34 UTC |
| 26c21f8 | Junio C Hamano | 17 October 2012, 17:28:19 UTC | Merge branch 'jc/maint-t1450-fsck-order-fix' into maint * jc/maint-t1450-fsck-order-fix: t1450: the order the objects are checked is undefined | 17 October 2012, 17:28:19 UTC |
| ff3ec8a | Junio C Hamano | 17 October 2012, 17:27:28 UTC | Merge branch 'rr/test-use-shell-path-not-shell' into maint * rr/test-use-shell-path-not-shell: test-lib: use $SHELL_PATH, not $SHELL | 17 October 2012, 17:27:28 UTC |
| 5178ee1 | Junio C Hamano | 17 October 2012, 17:27:21 UTC | Merge branch 'rr/test-make-sure-we-have-git' into maint * rr/test-make-sure-we-have-git: t/test-lib: make sure Git has already been built | 17 October 2012, 17:27:21 UTC |
| 756583f | Junio C Hamano | 17 October 2012, 17:27:10 UTC | Merge branch 'po/maint-docs' into maint * po/maint-docs: Doc branch: show -vv option and alternative Doc clean: add See Also link Doc add: link gitignore Doc: separate gitignore pattern sources Doc: shallow clone deepens _to_ new depth | 17 October 2012, 17:27:10 UTC |
| 8c4bcd3 | Junio C Hamano | 17 October 2012, 17:26:51 UTC | Merge branch 'jc/ll-merge-binary-ours' into maint * jc/ll-merge-binary-ours: ll-merge: warn about inability to merge binary files only when we can't attr: "binary" attribute should choose built-in "binary" merge driver merge: teach -Xours/-Xtheirs to binary ll-merge driver | 17 October 2012, 17:26:51 UTC |
| 19100d3 | Junio C Hamano | 17 October 2012, 17:25:37 UTC | Merge branch 'db/doc-custom-xmlto' into maint * db/doc-custom-xmlto: Documentation/Makefile: Allow custom XMLTO binary | 17 October 2012, 17:25:37 UTC |
| 7703477 | Richard Fearn | 13 October 2012, 22:32:50 UTC | Fix spelling error in post-receive-email hook Signed-off-by: Richard Fearn <richardfearn@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 14 October 2012, 04:50:14 UTC |
| 188923f | Jeff King | 12 October 2012, 06:22:49 UTC | http: fix segfault in handle_curl_result When we create an http active_request_slot, we can set its "results" pointer back to local storage. The http code will fill in the details of how the request went, and we can access those details even after the slot has been cleaned up. Commit 8809703 (http: factor out http error code handling) switched us from accessing our local results struct directly to accessing it via the "results" pointer of the slot. That means we're accessing the slot after it has been marked as finished, defeating the whole purpose of keeping the results storage separate. Most of the time this doesn't matter, as finishing the slot does not actually clean up the pointer. However, when using curl's multi interface with the dumb-http revision walker, we might actually start a new request before handing control back to the original caller. In that case, we may reuse the slot, zeroing its results pointer, and leading the original caller to segfault while looking for its results inside the slot. Instead, we need to pass a pointer to our local results storage to the handle_curl_result function, rather than relying on the pointer in the slot struct. This matches what the original code did before the refactoring (which did not use a separate function, and therefore just accessed the results struct directly). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 12 October 2012, 16:42:31 UTC |
| debf29d | Dylan Alex Simon | 11 October 2012, 20:40:35 UTC | gitweb.cgi: fix "comitter_tz" typo in feed gitweb's feeds sometimes contained committer timestamps in the wrong timezone due to a misspelling. Signed-off-by: Dylan Simon <dylan@dylex.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 12 October 2012, 15:43:12 UTC |
| 40701ad | Nguyen Thai Ngoc Duy | 10 October 2012, 13:55:52 UTC | attr: a note about the order of .gitattributes lookup This is the documentation part of 1a9d7e9 (attr.c: read .gitattributes from index as well. - 2007-08-14) 06f33c1 (Read attributes from the index that is being checked out - 2009-03-13) Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 10 October 2012, 18:09:20 UTC |
| 7bfffdc | Junio C Hamano | 09 October 2012, 18:48:53 UTC | Merge branch 'maint' of git://github.com/git-l10n/git-po into maint * 'maint' of git://github.com/git-l10n/git-po: l10n: de.po: fix a few minor typos | 09 October 2012, 18:48:53 UTC |
| d4a7ffa | Ben Walton | 08 October 2012, 08:08:01 UTC | tests: "cp -a" is a GNUism These tests just want a bit-for-bit identical copy; they do not need even -H (there is no symbolic link involved) nor -p (there is no funny permission or ownership issues involved). Just use "cp -R" instead. Signed-off-by: Ben Walton <bdwalton@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 October 2012, 21:37:43 UTC |
| 6347e71 | Ramkumar Ramachandra | 08 October 2012, 17:26:53 UTC | Git url doc: mark ftp/ftps as read-only and deprecate them It is not even worth mentioning their removal; just discourage people from using them. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 October 2012, 21:18:19 UTC |
| 234cd45 | Junio C Hamano | 08 October 2012, 18:40:43 UTC | Git 1.7.12.3 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 October 2012, 18:40:43 UTC |
| ff5702c | Junio C Hamano | 08 October 2012, 18:34:34 UTC | Merge branch 'os/commit-submodule-ignore' into maint "git status" honored the ignore=dirty settings in .gitmodules but "git commit" didn't. * os/commit-submodule-ignore: commit: pay attention to submodule.$name.ignore in .gitmodules | 08 October 2012, 18:34:34 UTC |
| 25c0890 | Junio C Hamano | 08 October 2012, 18:34:19 UTC | Merge branch 'jk/receive-pack-unpack-error-to-pusher' into maint "git receive-pack" (the counterpart to "git push") did not give progress output while processing objects it received to the puser when run over the smart-http protocol. * jk/receive-pack-unpack-error-to-pusher: receive-pack: drop "n/a" on unpacker errors receive-pack: send pack-processing stderr over sideband receive-pack: redirect unpack-objects stdout to /dev/null | 08 October 2012, 18:34:19 UTC |
| 9b4030c | Junio C Hamano | 08 October 2012, 18:34:02 UTC | Merge branch 'rt/maint-clone-single' into maint A repository created with "git clone --single" had its fetch refspecs set up just like a clone without "--single", leading the subsequent "git fetch" to slurp all the other branches, defeating the whole point of specifying "only this branch". * rt/maint-clone-single: clone --single: limit the fetch refspec to fetched branch | 08 October 2012, 18:34:02 UTC |
| 63c0c2c | Junio C Hamano | 08 October 2012, 18:33:35 UTC | Merge branch 'jc/blame-follows-renames' into maint It was unclear in the documentation for "git blame" that it is unnecessary for users to use the "--follow" option. * jc/blame-follows-renames: git blame: document that it always follows origin across whole-file renames | 08 October 2012, 18:33:35 UTC |
| 6e20357 | Junio C Hamano | 08 October 2012, 18:32:59 UTC | Merge branch 'lt/mailinfo-handle-attachment-more-sanely' into maint A patch attached as application/octet-stream (e.g. not text/*) were mishandled, not correctly honoring Content-Transfer-Encoding (e.g. base64). * lt/mailinfo-handle-attachment-more-sanely: mailinfo: don't require "text" mime type for attachments | 08 October 2012, 18:33:00 UTC |
| 866f5f8 | Nguyễn Thái Ngọc Duy | 07 October 2012, 03:13:54 UTC | gitignore.txt: suggestions how to get literal # or ! at the beginning We support backslash escape, but we hide the details behind the phrase "a shell glob suitable for consumption by fnmatch(3)". So it may not be obvious how one can get literal # or ! at the beginning of pattern. Add a few lines on how to work around the magic characters. Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 07 October 2012, 23:15:19 UTC |
| 30462a7 | Junio C Hamano | 04 October 2012, 17:13:49 UTC | gitcli: parse-options lets you omit tail of long options Describe the behaviour, but do warn people against taking it too literally and expect an abbreviation valid today will stay valid forever. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 05 October 2012, 03:30:27 UTC |
| 01cd63c | Simon Ruderich | 04 October 2012, 14:23:14 UTC | l10n: de.po: fix a few minor typos Signed-off-by: Simon Ruderich <simon@ruderich.org> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com> | 04 October 2012, 17:13:43 UTC |
| 9dad83b | Junio C Hamano | 02 October 2012, 22:08:16 UTC | t1450: the order the objects are checked is undefined When a tag T points at an object X that is of a type that is different from what the tag records as, fsck should report it as an error. However, depending on the order X and T are checked individually, the actual error message can be different. If X is checked first, fsck remembers X's type and then when it checks T, it notices that T records X as a wrong type (i.e. the complaint is about a broken tag T). If T is checked first, on the other hand, fsck remembers that we need to verify X is of the type tag records, and when it later checks X, it notices that X is of a wrong type (i.e. the complaint is about a broken object X). The important thing is that fsck notices such an error and diagnoses the issue on object X, but the test was expecting that we happen to check objects in the order to make us detect issues with tag T, not with object X. Remove this unwarranted assumption. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 02 October 2012, 22:08:16 UTC |
| 9376c86 | Junio C Hamano | 02 October 2012, 20:44:59 UTC | Start preparing for 1.7.12.3 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 02 October 2012, 20:44:59 UTC |
| e2c7a5b | Junio C Hamano | 02 October 2012, 20:42:32 UTC | Merge branch 'rr/maint-submodule-unknown-cmd' into maint "git submodule frotz" was not diagnosed as "frotz" being an unknown subcommand to "git submodule"; the user instead got a complaint that "git submodule status" was run with an unknown path "frotz". * rr/maint-submodule-unknown-cmd: submodule: if $command was not matched, don't parse other args | 02 October 2012, 20:42:32 UTC |
| 0a65df5 | Junio C Hamano | 02 October 2012, 20:42:13 UTC | Merge branch 'sp/maint-http-enable-gzip' into maint "git fetch" over http advertised that it supports "deflate", which is much less common, and did not advertise more common "gzip" on its Accept-Encoding header. * sp/maint-http-enable-gzip: Enable info/refs gzip decompression in HTTP client | 02 October 2012, 20:42:14 UTC |
| 8a477dd | Junio C Hamano | 02 October 2012, 20:41:38 UTC | Merge branch 'sp/maint-http-info-refs-no-retry' into maint "git fetch" over http had an old workaround for an unlikely server misconfiguration; it turns out that this hurts debuggability of the configuration in general, and has been reverted. * sp/maint-http-info-refs-no-retry: Revert "retry request without query when info/refs?query fails" | 02 October 2012, 20:41:38 UTC |
| a907309 | Peter Krefting | 02 October 2012, 07:25:32 UTC | l10n: Fix to Swedish translation Fix bad translation of "Receiving objects". Signed-off-by: Peter Krefting <peter@softwolves.pp.se> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 02 October 2012, 19:39:55 UTC |
| 9d55b2e | Linus Torvalds | 30 September 2012, 22:10:48 UTC | mailinfo: don't require "text" mime type for attachments Currently "git am" does insane things if the mbox it is given contains attachments with a MIME type that aren't "text/*". In particular, it will still decode them, and pass them "one line at a time" to the mail body filter, but because it has determined that they aren't text (without actually looking at the contents, just at the mime type) the "line" will be the encoding line (eg 'base64') rather than a line of *content*. Which then will cause the text filtering to fail, because we won't correctly notice when the attachment text switches from the commit message to the actual patch. Resulting in a patch failure, even if patch may be a perfectly well-formed attachment, it's just that the message type may be (for example) "application/octet-stream" instead of "text/plain". Just remove all the bogus games with the message_type. The only difference that code creates is how the data is passed to the filter function (chunked per-pred-code line or per post-decode line), and that difference is *wrong*, since chunking things per pre-decode line can never be a sensible operation, and cannot possibly matter for binary data anyway. This code goes all the way back to March of 2007, in commit 87ab79923463 ("builtin-mailinfo.c infrastrcture changes"), and apparently Don used to pass random mbox contents to git. However, the pre-decode vs post-decode logic really shouldn't matter even for that case, and more importantly, "I fed git am crap" is not a valid reason to break *real* patch attachments. If somebody really cares, and determines that some attachment is binary data (by looking at the data, not the MIME-type), the whole attachment should be dismissed, rather than fed in random-sized chunks to "handle_filter()". Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> Cc: Don Zickus <dzickus@redhat.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 01 October 2012, 00:29:27 UTC |
| d8cf053 | Junio C Hamano | 30 September 2012, 05:33:25 UTC | Git 1.7.12.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 30 September 2012, 06:20:47 UTC |
| d4018a5 | Junio C Hamano | 30 September 2012, 06:20:13 UTC | Merge branch 'maint' of git://github.com/git-l10n/git-po into maint Update German and Simplified Chinese translations. * 'maint' of git://github.com/git-l10n/git-po: l10n: de.po: correct translation of a 'rebase' message l10n: Improve many translation for zh_CN l10n: Unify the translation for '(un)expected' | 30 September 2012, 06:20:13 UTC |
| 31d69db | Junio C Hamano | 30 September 2012, 05:30:56 UTC | Merge branch 'jc/maint-log-grep-all-match-1' into maint * jc/maint-log-grep-all-match-1: grep.c: make two symbols really file-scope static this time t7810-grep: test --all-match with multiple --grep and --author options t7810-grep: test interaction of multiple --grep and --author options t7810-grep: test multiple --author with --all-match t7810-grep: test multiple --grep with and without --all-match t7810-grep: bring log --grep tests in common form grep.c: mark private file-scope symbols as static log: document use of multiple commit limiting options log --grep/--author: honor --all-match honored for multiple --grep patterns grep: show --debug output only once grep: teach --debug option to dump the parse tree | 30 September 2012, 05:30:56 UTC |
| 52938b1 | Junio C Hamano | 30 September 2012, 05:30:47 UTC | Merge branch 'jc/maint-mailinfo-mime-attr' into maint * jc/maint-mailinfo-mime-attr: mailinfo: do not concatenate charset= attribute values from mime headers | 30 September 2012, 05:30:48 UTC |
| 5b3ba71 | Jiang Xin | 27 September 2012, 22:49:08 UTC | Merge branch 'l10n-thynson' of git://github.com/thynson/git-po-zh_CN into maint * 'l10n-thynson' of git://github.com/thynson/git-po-zh_CN: l10n: Improve many translation for zh_CN l10n: Unify the translation for '(un)expected' | 27 September 2012, 22:49:08 UTC |
| 173c173 | Jiang Xin | 27 September 2012, 22:30:11 UTC | Merge branch 'maint' of https://github.com/ralfth/git-po-de into maint * 'maint' of https://github.com/ralfth/git-po-de: l10n: de.po: correct translation of a 'rebase' message | 27 September 2012, 22:30:11 UTC |
| 5f38e5e | Ralf Thielow | 24 September 2012, 17:16:21 UTC | l10n: de.po: correct translation of a 'rebase' message Noticed-by: Sascha Cunz <sascha-ml@babbelbox.org> Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com> | 27 September 2012, 18:58:51 UTC |
| af9c9f9 | Ramkumar Ramachandra | 22 September 2012, 11:27:59 UTC | submodule: if $command was not matched, don't parse other args "git submodule" command DWIMs the command line and assumes a unspecified action word for 'status' action. This is a UI mistake that leads to a confusing behaviour. A mistyped command name is instead treated as a request for 'status' of the submodule with that name, e.g. $ git submodule show error: pathspec 'show' did not match any file(s) known to git. Did you forget to 'git add'? Stop DWIMming an unknown or mistyped subcommand name as pathspec given to unspelled "status" subcommand. "git submodule" without any argument is still interpreted as "git submodule status", but its value is questionable. Adjust t7400 to match, and stop advertising the default subcommand being 'status' which does not help much in practice, other than promoting laziness and confusion. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 25 September 2012, 18:31:48 UTC |
| 4cde519 | Ramkumar Ramachandra | 22 September 2012, 04:55:10 UTC | test-lib: use $SHELL_PATH, not $SHELL The codepath for handling "--tee" ends up relaunching the test script under a shell, and that one has to be a Bourne. But we incorrectly used $SHELL, which could be a non-Bourne (e.g. zsh or csh); we have the Makefile variable $SHELL_PATH for exactly that, so use it instead. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 25 September 2012, 17:17:29 UTC |
| 666ca59 | Jeff King | 25 September 2012, 04:31:19 UTC | Revert "completion: fix shell expansion of items" This reverts commit 25ae7cfd19c8f21721363c64163cd5d9d1135b20. That patch does fix expansion of weird variables in some simple tests, but it also seems to break other things, like expansion of refs by "git checkout". While we're sorting out the correct solution, we are much better with the original bug (people with metacharacters in their completions occasionally see an error message) than the current bug (ref completion does not work at all). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 25 September 2012, 16:00:28 UTC |
| 1a002c7 | Junio C Hamano | 24 September 2012, 19:50:00 UTC | Start preparation for 1.7.12.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 September 2012, 19:50:00 UTC |
| 140011d | Junio C Hamano | 24 September 2012, 19:40:02 UTC | Merge branch 'jc/maint-blame-no-such-path' into maint Even during a conflicted merge, "git blame $path" always meant to blame uncommitted changes to the "working tree" version; make it more useful by showing cleanly merged parts as coming from the other branch that is being merged. This incidentally fixes an unrelated problem on a case insensitive filesystem, where "git blame MAKEFILE" run in a history that has "Makefile" but not "MAKEFILE" did not say "No such file MAKEFILE in HEAD" but pretended as if "MAKEFILE" was a newly added file. * jc/maint-blame-no-such-path: blame: allow "blame file" in the middle of a conflicted merge blame $path: avoid getting fooled by case insensitive filesystems | 24 September 2012, 19:40:02 UTC |
| 8144049 | Junio C Hamano | 24 September 2012, 19:39:21 UTC | Merge branch 'dj/fetch-all-tags' into maint "git fetch --all", when passed "--no-tags", did not honor the "--no-tags" option while fetching from individual remotes (the same issue existed with "--tags", but combination "--all --tags" makes much less sense than "--all --no-tags"). * dj/fetch-all-tags: fetch --all: pass --tags/--no-tags through to each remote submodule: use argv_array instead of hand-building arrays fetch: use argv_array instead of hand-building arrays argv-array: fix bogus cast when freeing array argv-array: add pop function | 24 September 2012, 19:39:21 UTC |
| f9db192 | Jonathan "Duke" Leto | 24 September 2012, 17:41:26 UTC | Improve the description of GIT_PS1_SHOWUPSTREAM Describe what '=' means in the output of __git_ps1 when using GIT_PS1_SHOWUPSTREAM, which was not previously described. Signed-off-by: Jonathan "Duke" Leto <jonathan@leto.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 September 2012, 19:38:41 UTC |
| 8f6811e | Orgad Shaneh | 23 September 2012, 07:37:47 UTC | commit: pay attention to submodule.$name.ignore in .gitmodules "git status" does not list a submodule with uncommitted working tree files as modified when "submodule.$name.ignore" is set to "dirty" in in-tree ".gitmodules" file. Both status and commit honor the setting in $GIT_DIR/config, but "commit" does not pick it up from .gitmodules, which is inconsistent. Teach "git commit" to pay attention to the setting in .gitmodules as well. Signed-off-by: Orgad Shaneh <orgads@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 September 2012, 16:28:36 UTC |
| e5dce96 | Junio C Hamano | 21 September 2012, 19:09:42 UTC | git blame: document that it always follows origin across whole-file renames Make it clear to people who (rightly or wrongly) think that the "--follow" option should follow origin across while-file renames that we already do so. That would explain the output that they see when they do give the "--follow" option to the command. We may or may not want to do a "--no-follow" patch as a follow-up, but that is a separate topic. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 21 September 2012, 19:14:19 UTC |
| 74eb32d | Jeff King | 21 September 2012, 05:38:46 UTC | receive-pack: drop "n/a" on unpacker errors The output from git push currently looks like this: $ git push dest HEAD fatal: [some message from index-pack] error: unpack failed: index-pack abnormal exit To dest ! [remote rejected] HEAD -> master (n/a (unpacker error)) That n/a is meant to be "the per-ref status is not available" but the nested parentheses just make it look ugly. Let's turn the final line into just: ! [remote rejected] HEAD -> master (unpacker error) Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 21 September 2012, 16:50:13 UTC |
| a22e6f8 | Jeff King | 21 September 2012, 05:34:55 UTC | receive-pack: send pack-processing stderr over sideband Receive-pack invokes either unpack-objects or index-pack to handle the incoming pack. However, we do not redirect the stderr of the sub-processes at all, so it is never seen by the client. From the initial thread adding sideband support, which is here: http://thread.gmane.org/gmane.comp.version-control.git/139471 it is clear that some messages are specifically kept off the sideband (with the assumption that they are of interest only to an administrator, not the client). The stderr of the subprocesses is mentioned in the thread, but it's unclear if they are included in that group, or were simply forgotten. However, there are a few good reasons to show them to the client: 1. In many cases, they are directly about the incoming packfile (e.g., fsck warnings with --strict, corruption in the packfile, etc). Without these messages, the client just gets "unpacker error" with no extra useful diagnosis. 2. No matter what the cause, we are probably better off showing the errors to the client. If the client and the server admin are not the same entity, it is probably much easier for the client to cut-and-paste the errors they see than for the admin to try to dig them out of a log and correlate them with a particular session. 3. Users of the ssh transport typically already see these stderr messages, as the remote's stderr is copied literally by ssh. This brings other transports (http, and push-over-git if you are crazy enough to enable it) more in line with ssh. As a bonus for ssh users, because the messages are now fed through the sideband and printed by the local git, they will have "remote:" prepended and be properly interleaved with any local output to stderr. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 21 September 2012, 16:49:47 UTC |
| 59bfdfb | Jeff King | 21 September 2012, 05:32:52 UTC | receive-pack: redirect unpack-objects stdout to /dev/null The unpack-objects command should not generally produce any output on stdout. However, if it's given extra input after the packfile, it will spew the remainder to stdout. When called by receive-pack, this means we will break protocol, since our stdout is connected to the remote send-pack. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 21 September 2012, 16:44:11 UTC |
| 8ef2794 | Junio C Hamano | 20 September 2012, 22:55:31 UTC | Merge branch 'nd/maint-diffstat-summary' into maint * nd/maint-diffstat-summary: Revert diffstat back to English | 20 September 2012, 22:55:31 UTC |
| 467ad25 | Junio C Hamano | 20 September 2012, 22:55:22 UTC | Merge branch 'jw/doc-commit-title' into maint * jw/doc-commit-title: Documentation: describe subject more precisely | 20 September 2012, 22:55:22 UTC |
| cc84144 | Junio C Hamano | 20 September 2012, 22:55:12 UTC | Merge branch 'dg/run-command-child-cleanup' into maint * dg/run-command-child-cleanup: run-command.c: fix broken list iteration in clear_child_for_cleanup | 20 September 2012, 22:55:12 UTC |
| 96c2abe | Junio C Hamano | 20 September 2012, 22:55:03 UTC | Merge branch 'jc/mailinfo-RE' into maint * jc/mailinfo-RE: mailinfo: strip "RE: " prefix | 20 September 2012, 22:55:03 UTC |
| ee70fb8 | Junio C Hamano | 20 September 2012, 22:54:57 UTC | Merge branch 'sn/ls-remote-get-url-doc' into maint * sn/ls-remote-get-url-doc: ls-remote: document the '--get-url' option | 20 September 2012, 22:54:57 UTC |
| 9fcacaa | Junio C Hamano | 20 September 2012, 22:54:43 UTC | Merge branch 'nd/log-n-doc' into maint * nd/log-n-doc: doc: move rev-list option -<n> from git-log.txt to rev-list-options.txt | 20 September 2012, 22:54:43 UTC |
| f9c2d2b | Junio C Hamano | 20 September 2012, 22:53:31 UTC | Merge branch 'nd/maint-remote-remove' into maint * nd/maint-remote-remove: remote: prefer subcommand name 'remove' to 'rm' | 20 September 2012, 22:53:31 UTC |
| 3083301 | Junio C Hamano | 20 September 2012, 21:20:09 UTC | grep.c: make two symbols really file-scope static this time Adding a declaration at the beginning is not sufficient for obvious reasons. The definition has to be made static. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 21:20:09 UTC |
| 31b808a | Ralf Thielow | 20 September 2012, 18:04:08 UTC | clone --single: limit the fetch refspec to fetched branch After running "git clone --single", the resulting repository has the usual default "+refs/heads/*:refs/remotes/origin/*" wildcard fetch refspec installed, which means that a subsequent "git fetch" will end up grabbing all the other branches. Update the fetch refspec to cover only the singly cloned ref instead to correct this. That means: If "--single" is used without "--branch" or "--mirror", the fetch refspec covers the branch on which remote's HEAD points to. If "--single" is used with "--branch", it'll cover only the branch specified in the "--branch" option. If "--single" is combined with "--mirror", then it'll cover all refs of the cloned repository. If "--single" is used with "--branch" that specifies a tag, then it'll cover only the ref for this tag. Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 21:15:35 UTC |
| f2fef7b | Stephen Boyd | 20 September 2012, 08:10:38 UTC | Documentation: Document signature showing options The pretty formats for GPG signatures were introduced but never documented. Use the documentation from the commit that introduced them. Do the same for the --show-signature option added to git log and friends. Signed-off-by: Stephen Boyd <bebarino@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 17:28:51 UTC |
| aa90b96 | Shawn O. Pearce | 19 September 2012, 23:12:02 UTC | Enable info/refs gzip decompression in HTTP client Some HTTP servers try to use gzip compression on the /info/refs request to save transfer bandwidth. Repositories with many tags may find the /info/refs request can be gzipped to be 50% of the original size due to the few but often repeated bytes used (hex SHA-1 and commonly digits in tag names). For most HTTP requests enable "Accept-Encoding: gzip" ensuring the /info/refs payload can use this encoding format. Only request gzip encoding from servers. Although deflate is supported by libcurl, most servers have standardized on gzip encoding for compression as that is what most browsers support. Asking for deflate increases request sizes by a few bytes, but is unlikely to ever be used by a server. Disable the Accept-Encoding header on probe RPCs as response bodies are supposed to be exactly 4 bytes long, "0000". The HTTP headers requesting and indicating compression use more space than the data transferred in the body. Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 17:26:50 UTC |
| 6ac964a | Shawn O. Pearce | 20 September 2012, 05:58:00 UTC | Revert "retry request without query when info/refs?query fails" This reverts commit 703e6e76a14825e5b0c960d525f34e607154b4f7. Retrying without the query parameter was added as a workaround for a single broken HTTP server at git.debian.org[1]. The server was misconfigured to route every request with a query parameter into gitweb.cgi. Admins fixed the server's configuration within 16 hours of the bug report to the Git mailing list, but we still patched Git with this fallback and have been paying for it since. Most Git hosting services configure the smart HTTP protocol and the retry logic confuses users when there is a transient HTTP error as Git dropped the real error from the smart HTTP request. Removing the retry makes root causes easier to identify. [1] http://thread.gmane.org/gmane.comp.version-control.git/137609 Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Acked-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 17:25:21 UTC |
| 25ae7cf | Felipe Contreras | 20 September 2012, 02:15:15 UTC | completion: fix shell expansion of items As reported by Jeroen Meijer[1]; the current code doesn't deal properly with items (tags, branches, etc.) that have ${} in them because they get expaned by bash while using compgen. A simple solution is to quote the items so they get expanded properly (\$\{\}). In order to achieve that I took bash-completion's quote() function, which is rather simple, and renamed it to __git_quote() as per Jeff King's suggestion. Solves the original problem for me. [1] http://article.gmane.org/gmane.comp.version-control.git/201596 Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> Acked-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 September 2012, 16:52:36 UTC |
| dd4f307 | Dave Borowitz | 19 September 2012, 17:06:37 UTC | Documentation/Makefile: Allow custom XMLTO binary Signed-off-by: Dave Borowitz <dborowitz@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 23:35:10 UTC |
| f0970fa | Philip Oakley | 18 September 2012, 23:25:34 UTC | Doc branch: show -vv option and alternative Indicate that the -v option can be given twice in the short options. Without it users pass over the option. Also indicate the alternate 'git remote show' method. Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 17:22:49 UTC |
| 068c674 | Philip Oakley | 18 September 2012, 23:25:33 UTC | Doc clean: add See Also link 'git clean' is controlled by gitignore. Provide See Also link for it. Use of core.excludesfile is implied. Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 04:49:50 UTC |
| a73d379 | Philip Oakley | 18 September 2012, 23:25:32 UTC | Doc add: link gitignore Use a gitignore link rather than the gitrepository- layout link. Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 04:49:46 UTC |
| 3f8c5a4 | Philip Oakley | 18 September 2012, 23:25:31 UTC | Doc: separate gitignore pattern sources Use separate bulleted paragraphs for the three different gitignore pattern sources. Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 04:47:45 UTC |
| bc40dea | Philip Oakley | 18 September 2012, 23:25:28 UTC | Doc: shallow clone deepens _to_ new depth Clarify that 'depth=' specifies the new depth from the remote's branch tip. It does not add the depth to the existing shallow clone. (details from pack-protocol.txt). Clarify that tags are not fetched. (details from shallow.txt) Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 September 2012, 04:47:12 UTC |
| 304b7d9 | Junio C Hamano | 18 September 2012, 21:34:31 UTC | Git 1.7.12.1 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 18 September 2012, 21:34:31 UTC |
| 39e2e02 | Junio C Hamano | 18 September 2012, 21:33:52 UTC | Merge branch 'er/doc-fast-import-done' into maint * er/doc-fast-import-done: fast-import: document the --done option | 18 September 2012, 21:33:52 UTC |
| 8ffc331 | Junio C Hamano | 18 September 2012, 21:24:06 UTC | Merge branch 'jk/config-warn-on-inaccessible-paths' into maint The attribute system may be asked for a path that itself or its leading directories no longer exists in the working tree, and it is fine if we cannot open .gitattribute file in such a case. Failure to open per-directory .gitattributes with error status other than ENOENT and ENOTDIR should be diagnosed. * jk/config-warn-on-inaccessible-paths: attr: failure to open a .gitattributes file is OK with ENOTDIR warn_on_inaccessible(): a helper to warn on inaccessible paths attr: warn on inaccessible attribute files gitignore: report access errors of exclude files config: warn on inaccessible files | 18 September 2012, 21:24:06 UTC |
| 2006f0a | Ramkumar Ramachandra | 17 September 2012, 17:06:19 UTC | t/test-lib: make sure Git has already been built When tests were run without building git, they stopped with: .: 54: Can't open /path/to/git/source/t/../GIT-BUILD-OPTIONS Move the check that makes sure that git has already been built from t0000 to test-lib, so that any test will do so before it runs. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 18 September 2012, 21:22:19 UTC |
| 01f7d7f | Philip Oakley | 16 September 2012, 10:57:58 UTC | Doc: Improve shallow depth wording Avoid confusion in compound sentence about the start of the commit set and the depth measure. Use two sentences. Signed-off-by: Philip Oakley <philipoakley@iee.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 18 September 2012, 20:35:56 UTC |
| 8093ae8 | Andreas Schwab | 18 September 2012, 15:55:08 UTC | Documentation/git-filter-branch: Move note about effect of removing commits The note that explains that changes introduced by removed commits are preserved should be placed directly after the paragraph that describes such commits removal. Otherwise the reference to "the commits" appears out of context. Also the big example that follows "Consider this history" is about rewriting part of the history DAG. Move the paragraph that describes the operation close to it. Signed-off-by: Andreas Schwab <schwab@linux-m68k.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 18 September 2012, 19:51:58 UTC |
| 176943b | Junio C Hamano | 17 September 2012, 21:17:58 UTC | mailinfo: do not concatenate charset= attribute values from mime headers "Content-type: text/plain; charset=UTF-8" header should not appear twice in the input, but it is always better to gracefully deal with such a case. The current code concatenates the value to the values we have seen previously, producing nonsense such as "utf8UTF-8". Instead of concatenating, forget the previous value and use the last value we see. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 22:24:52 UTC |
| 5805853 | Ramkumar Ramachandra | 17 September 2012, 17:06:18 UTC | t/perf: add "trash directory" to .gitignore Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 21:27:55 UTC |
| d9fcff2 | Adam Spiers | 17 September 2012, 11:38:51 UTC | Add missing -z to git check-attr usage text for consistency with man page Signed-off-by: Adam Spiers <git@adamspiers.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 20:45:32 UTC |
| 6108b04 | Mischa POSLAWSKY | 17 September 2012, 01:21:55 UTC | git-jump: ignore (custom) prefix in diff mode Matching the default file prefix b/ does not yield any results if config option diff.noprefix or diff.mnemonicprefix is enabled. Signed-off-by: Mischa POSLAWSKY <git@shiar.nl> Acked-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 19:31:57 UTC |
| 2ce4fee | Wesley J. Landaker | 17 September 2012, 14:22:15 UTC | Documentation: indent-with-non-tab uses "equivalent tabs" not 8 Update the documentation of the core.whitespace option "indent-with-non-tab" to correctly reflect that it catches the use of spaces instead of the equivalent tabs, rather than a fixed number. Signed-off-by: Wesley J. Landaker <wjl@icecavern.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 17:23:21 UTC |
| 7dd9ab0 | Yacine Belkadi | 16 September 2012, 16:06:11 UTC | completion: add --no-edit to git-commit Signed-off-by: Yacine Belkadi <yacine.belkadi.1@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 17 September 2012, 05:05:07 UTC |
| 39f2e01 | Michael J Gruber | 14 September 2012, 09:46:43 UTC | t7810-grep: test --all-match with multiple --grep and --author options The code used to have a bug that ignores "--all-match", that requires all "--grep" to have matched, when "--author" or "--committer" was used. Make sure the bug will not be reintroduced. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:40 UTC |
| 2cb03e7 | Michael J Gruber | 14 September 2012, 09:46:42 UTC | t7810-grep: test interaction of multiple --grep and --author options There are tests for this interaction already. Restructure slightly and avoid any claims about --all-match. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:40 UTC |
| 00f62a6 | Michael J Gruber | 14 September 2012, 09:46:41 UTC | t7810-grep: test multiple --author with --all-match The "--all-match" option is about "--grep", and does not affect how "--author" or "--committer" limitation is applied. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:40 UTC |
| dfe3642 | Michael J Gruber | 14 September 2012, 09:46:40 UTC | t7810-grep: test multiple --grep with and without --all-match Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:40 UTC |
| b327bf7 | Michael J Gruber | 14 September 2012, 09:46:39 UTC | t7810-grep: bring log --grep tests in common form The log --grep tests generate the expected out in different ways. Make them all use command blocks so that subshells are avoided and the expected output is easier to grasp visually. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:39 UTC |
| 07a7d65 | Junio C Hamano | 15 September 2012, 21:04:36 UTC | grep.c: mark private file-scope symbols as static Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 September 2012, 06:35:39 UTC |
| d4d39bc | Junio C Hamano | 15 September 2012, 04:37:51 UTC | Draft release notes to 1.7.12.1 We are almost there... Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 September 2012, 04:37:51 UTC |
| c13a5ac | Junio C Hamano | 15 September 2012, 04:32:07 UTC | Merge branch 'sb/send-email-reconfirm-fix' into maint * sb/send-email-reconfirm-fix: send-email: initial_to and initial_reply_to are both optional | 15 September 2012, 04:32:07 UTC |
| 448e370 | Junio C Hamano | 15 September 2012, 04:32:01 UTC | Merge branch 'jc/send-email-reconfirm' into maint * jc/send-email-reconfirm: send-email: validate & reconfirm interactive responses | 15 September 2012, 04:32:01 UTC |
| 78ed88d | Junio C Hamano | 15 September 2012, 04:24:18 UTC | Merge branch 'mz/cherry-pick-cmdline-order' into maint * mz/cherry-pick-cmdline-order: cherry-pick/revert: respect order of revisions to pick demonstrate broken 'git cherry-pick three one two' teach log --no-walk=unsorted, which avoids sorting | 15 September 2012, 04:24:18 UTC |
| c336bc1 | Junio C Hamano | 15 September 2012, 04:20:40 UTC | Sync with 1.7.11.7 | 15 September 2012, 04:20:40 UTC |
| bafc478 | Junio C Hamano | 15 September 2012, 03:57:23 UTC | Git 1.7.11.7 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 September 2012, 03:57:23 UTC |
| 19ece72 | Junio C Hamano | 15 September 2012, 03:48:31 UTC | Merge branch 'jk/maint-quiet-is-synonym-to-s-in-log' into maint-1.7.11 * jk/maint-quiet-is-synonym-to-s-in-log: log: fix --quiet synonym for -s | 15 September 2012, 03:48:31 UTC |
| ddbca33 | Junio C Hamano | 15 September 2012, 03:48:22 UTC | Merge branch 'jc/maint-ident-missing-human-name' into maint-1.7.11 * jc/maint-ident-missing-human-name: split_ident_line(): make best effort when parsing author/committer line | 15 September 2012, 03:48:22 UTC |
| dabdc01 | Junio C Hamano | 15 September 2012, 03:46:39 UTC | Merge branch 'rj/test-regex' into maint-1.7.11 * rj/test-regex: test-regex: Add a test to check for a bug in the regex routines | 15 September 2012, 03:46:39 UTC |
