| 689efb7 | Junio C Hamano | 08 September 2015, 22:38:43 UTC | Git 2.6-rc1 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 September 2015, 22:38:43 UTC |
| d6a2b05 | Junio C Hamano | 08 September 2015, 22:34:57 UTC | Merge branch 'jc/builtin-am-signoff-regression-fix' Recent "git am" had regression when adding a Signed-off-by line with its "-s" option by an unintended tightening of how an existing trailer block is detected. * jc/builtin-am-signoff-regression-fix: am: match --signoff to the original scripted version | 08 September 2015, 22:35:05 UTC |
| aab8454 | Junio C Hamano | 06 September 2015, 02:56:20 UTC | am: match --signoff to the original scripted version Linus noticed that the recently reimplemented "git am -s" defines the trailer block too rigidly, resulting in an unnecessary blank line between the existing sign-offs and his new sign-off. An e-mail submission sent to Linus in real life ends with mixture of sign-offs and commentaries, e.g. title here message here Signed-off-by: Original Author <original@auth.or> [rv: tweaked frotz and nitfol] Signed-off-by: Re Viewer <rv@ew.er> Signed-off-by: Other Reviewer <other@rev.ewer> --- patch here Because the reimplementation reused append_signoff() helper that is used by other codepaths, which is unaware that people intermix such comments with their sign-offs in the trailer block, such a message was judged to end with a non-trailer, resulting in an extra blank line before adding a new sign-off. The original scripted version of "git am" used a lot looser definition, i.e. "if and only if there is no line that begins with Signed-off-by:, add a blank line before adding a new sign-off". For the upcoming release, stop using the append_signoff() in "git am" and reimplement the looser definition used by the scripted version to use only in "git am" to fix this regression in "am" while avoiding new regressions to other users of append_signoff(). In the longer term, we should look into loosening append_signoff() so that other codepaths that add a new sign-off behave the same way as "git am -s", but that is a task for post-release. Reported-by: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 07 September 2015, 02:59:40 UTC |
| ec371ff | Junio C Hamano | 04 September 2015, 21:34:57 UTC | Sync with maint * maint: | 04 September 2015, 21:34:57 UTC |
| fb8880d | Junio C Hamano | 04 September 2015, 02:18:05 UTC | Merge branch 'ee/clean-test-fixes' into maint * ee/clean-test-fixes: t7300: fix broken && chains | 04 September 2015, 02:18:05 UTC |
| 5af77d1 | Junio C Hamano | 04 September 2015, 02:18:03 UTC | Merge branch 'jk/log-missing-default-HEAD' into maint "git init empty && git -C empty log" said "bad default revision 'HEAD'", which was found to be a bit confusing to new users. * jk/log-missing-default-HEAD: log: diagnose empty HEAD more clearly | 04 September 2015, 02:18:04 UTC |
| 9d93988 | Junio C Hamano | 04 September 2015, 02:18:03 UTC | Merge branch 'cc/trailers-corner-case-fix' into maint The "interpret-trailers" helper mistook a multi-paragraph title of a commit log message with a colon in it as the end of the trailer block. * cc/trailers-corner-case-fix: trailer: support multiline title trailer: retitle a test and correct an in-comment message trailer: ignore first line of message | 04 September 2015, 02:18:03 UTC |
| 311e5ce | Junio C Hamano | 04 September 2015, 02:18:02 UTC | Merge branch 'dt/commit-preserve-base-index-upon-opportunistic-cache-tree-update' into maint When re-priming the cache-tree opportunistically while committing the in-core index as-is, we mistakenly invalidated the in-core index too aggressively, causing the experimental split-index code to unnecessarily rewrite the on-disk index file(s). * dt/commit-preserve-base-index-upon-opportunistic-cache-tree-update: commit: don't rewrite shared index unnecessarily | 04 September 2015, 02:18:02 UTC |
| 1c82039 | Junio C Hamano | 04 September 2015, 02:18:01 UTC | Merge branch 'rs/archive-zip-many' into maint "git archive" did not use zip64 extension when creating an archive with more than 64k entries, which nobody should need, right ;-)? * rs/archive-zip-many: archive-zip: support more than 65535 entries archive-zip: use a local variable to store the creator version t5004: test ZIP archives with many entries | 04 September 2015, 02:18:01 UTC |
| ae6ac84 | Junio C Hamano | 04 September 2015, 02:18:00 UTC | Merge branch 'jc/calloc-pathspec' into maint Minor code cleanup. * jc/calloc-pathspec: ps_matched: xcalloc() takes nmemb and then element size | 04 September 2015, 02:18:00 UTC |
| 8136099 | Junio C Hamano | 04 September 2015, 02:17:59 UTC | Merge branch 'ss/fix-config-fd-leak' into maint * ss/fix-config-fd-leak: config: close config file handle in case of error | 04 September 2015, 02:17:59 UTC |
| dc4e7b0 | Junio C Hamano | 04 September 2015, 02:17:57 UTC | Merge branch 'sg/wt-status-header-inclusion' into maint * sg/wt-status-header-inclusion: wt-status: move #include "pathspec.h" to the header | 04 September 2015, 02:17:57 UTC |
| 659227b | Junio C Hamano | 04 September 2015, 02:17:56 UTC | Merge branch 'po/po-readme' into maint Doc updates for i18n. * po/po-readme: po/README: Update directions for l10n contributors | 04 September 2015, 02:17:56 UTC |
| 57a2bb1 | Junio C Hamano | 04 September 2015, 02:17:54 UTC | Merge branch 'sg/t3020-typofix' into maint * sg/t3020-typofix: t3020: fix typo in test description | 04 September 2015, 02:17:55 UTC |
| c1fa16b | Junio C Hamano | 04 September 2015, 02:17:53 UTC | Merge branch 'as/docfix-reflog-expire-unreachable' into maint Docfix. * as/docfix-reflog-expire-unreachable: Documentation/config: fix inconsistent label on gc.*.reflogExpireUnreachable | 04 September 2015, 02:17:53 UTC |
| d6c196a | Junio C Hamano | 04 September 2015, 02:17:53 UTC | Merge branch 'nd/fixup-linked-gitdir' into maint The code in "multiple-worktree" support that attempted to recover from an inconsistent state updated an incorrect file. * nd/fixup-linked-gitdir: setup: update the right file in multiple checkouts | 04 September 2015, 02:17:53 UTC |
| e654e3b | Junio C Hamano | 04 September 2015, 02:17:52 UTC | Merge branch 'jk/rev-list-has-no-notes' into maint "git rev-list" does not take "--notes" option, but did not complain when one is given. * jk/rev-list-has-no-notes: rev-list: make it obvious that we do not support notes | 04 September 2015, 02:17:53 UTC |
| fa6d374 | Junio C Hamano | 04 September 2015, 02:17:52 UTC | Merge branch 'jk/fix-alias-pager-config-key-warnings' into maint Because the configuration system does not allow "alias.0foo" and "pager.0foo" as the configuration key, the user cannot use '0foo' as a custom command name anyway, but "git 0foo" tried to look these keys up and emitted useless warnings before saying '0foo is not a git command'. These warning messages have been squelched. * jk/fix-alias-pager-config-key-warnings: config: silence warnings for command names with invalid keys | 04 September 2015, 02:17:52 UTC |
| 0b2cef2 | Junio C Hamano | 04 September 2015, 02:17:51 UTC | Merge branch 'nd/dwim-wildcards-as-pathspecs' into maint Test updates for Windows. * nd/dwim-wildcards-as-pathspecs: t2019: skip test requiring '*' in a file name non Windows | 04 September 2015, 02:17:51 UTC |
| 969560b | Junio C Hamano | 04 September 2015, 02:17:51 UTC | Merge branch 'sg/help-group' into maint We rewrote one of the build scripts in Perl but this reimplements in Bourne shell. * sg/help-group: generate-cmdlist: re-implement as shell script | 04 September 2015, 02:17:51 UTC |
| d11448f | Junio C Hamano | 04 September 2015, 02:17:50 UTC | Merge branch 'ps/t1509-chroot-test-fixup' into maint t1509 test that requires a dedicated VM environment had some bitrot, which has been corrected. * ps/t1509-chroot-test-fixup: tests: fix cleanup after tests in t1509-root-worktree tests: fix broken && chains in t1509-root-worktree | 04 September 2015, 02:17:50 UTC |
| 8b27071 | Junio C Hamano | 04 September 2015, 02:17:50 UTC | Merge branch 'jh/strbuf-read-use-read-in-full' into maint strbuf_read() used to have one extra iteration (and an unnecessary strbuf_grow() of 8kB), which was eliminated. * jh/strbuf-read-use-read-in-full: strbuf_read(): skip unnecessary strbuf_grow() at eof | 04 September 2015, 02:17:50 UTC |
| 6c0850f | Junio C Hamano | 04 September 2015, 02:17:49 UTC | Merge branch 'jk/long-error-messages' into maint The codepath to produce error messages had a hard-coded limit to the size of the message, primarily to avoid memory allocation while calling die(). * jk/long-error-messages: vreportf: avoid intermediate buffer vreportf: report to arbitrary filehandles | 04 September 2015, 02:17:49 UTC |
| cbcd3dc | Junio C Hamano | 04 September 2015, 02:17:49 UTC | Merge branch 'cb/open-noatime-clear-errno' into maint When trying to see that an object does not exist, a state errno leaked from our "first try to open a packfile with O_NOATIME and then if it fails retry without it" logic on a system that refuses O_NOATIME. This confused us and caused us to die, saying that the packfile is unreadable, when we should have just reported that the object does not exist in that packfile to the caller. * cb/open-noatime-clear-errno: git_open_noatime: return with errno=0 on success | 04 September 2015, 02:17:49 UTC |
| 03ea027 | Junio C Hamano | 04 September 2015, 02:17:47 UTC | Merge branch 'mh/get-remote-group-fix' into maint An off-by-one error made "git remote" to mishandle a remote with a single letter nickname. * mh/get-remote-group-fix: get_remote_group(): use skip_prefix() get_remote_group(): eliminate superfluous call to strcspn() get_remote_group(): rename local variable "space" to "wordlen" get_remote_group(): handle remotes with single-character names | 04 September 2015, 02:17:48 UTC |
| 7662973 | Junio C Hamano | 03 September 2015, 21:14:01 UTC | Merge branch 'jk/am-rerere-lock-fix' Recent "git am" introduced a double-locking failure when used with the "--3way" option that invokes rerere machinery. * jk/am-rerere-lock-fix: rerere: release lockfile in non-writing functions | 03 September 2015, 21:14:01 UTC |
| 16ffa64 | Junio C Hamano | 02 September 2015, 19:55:21 UTC | Git 2.6-rc0 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 02 September 2015, 19:55:28 UTC |
| cdd00df | Junio C Hamano | 02 September 2015, 19:50:21 UTC | Merge branch 'cc/trailers-corner-case-fix' The "interpret-trailers" helper mistook a multi-paragraph title of a commit log message with a colon in it as the end of the trailer block. * cc/trailers-corner-case-fix: trailer: support multiline title | 02 September 2015, 19:50:21 UTC |
| 8f8386e | Junio C Hamano | 02 September 2015, 19:50:18 UTC | Merge branch 'sb/read-cache-one-indent-style-fix' * sb/read-cache-one-indent-style-fix: read-cache: fix indentation in read_index_from | 02 September 2015, 19:50:18 UTC |
| 83d9092 | Junio C Hamano | 02 September 2015, 19:50:16 UTC | Merge branch 'ee/clean-test-fixes' * ee/clean-test-fixes: t7300: fix broken && chains | 02 September 2015, 19:50:16 UTC |
| 699a0f3 | Junio C Hamano | 02 September 2015, 19:50:08 UTC | Merge branch 'jk/log-missing-default-HEAD' "git init empty && git -C empty log" said "bad default revision 'HEAD'", which was found to be a bit confusing to new users. * jk/log-missing-default-HEAD: log: diagnose empty HEAD more clearly | 02 September 2015, 19:50:10 UTC |
| 7aa67f6 | Junio C Hamano | 01 September 2015, 23:31:58 UTC | Ninth batch for 2.6 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 01 September 2015, 23:31:58 UTC |
| 81d0e33 | Junio C Hamano | 01 September 2015, 23:31:29 UTC | Merge branch 'dt/commit-preserve-base-index-upon-opportunistic-cache-tree-update' When re-priming the cache-tree opportunistically while committing the in-core index as-is, we mistakenly invalidated the in-core index too aggressively, causing the experimental split-index code to unnecessarily rewrite the on-disk index file(s). * dt/commit-preserve-base-index-upon-opportunistic-cache-tree-update: commit: don't rewrite shared index unnecessarily | 01 September 2015, 23:31:29 UTC |
| 0fb8e24 | Junio C Hamano | 01 September 2015, 23:31:26 UTC | Merge branch 'rt/remove-hold-lockfile-for-append' * rt/remove-hold-lockfile-for-append: lockfile: remove function "hold_lock_file_for_append" | 01 September 2015, 23:31:27 UTC |
| 565f575 | Junio C Hamano | 01 September 2015, 23:31:24 UTC | Merge branch 'rs/archive-zip-many' "git archive" did not use zip64 extension when creating an archive with more than 64k entries, which nobody should need, right ;-)? * rs/archive-zip-many: archive-zip: support more than 65535 entries archive-zip: use a local variable to store the creator version t5004: test ZIP archives with many entries | 01 September 2015, 23:31:24 UTC |
| bb84dce | Junio C Hamano | 01 September 2015, 23:31:22 UTC | Merge branch 'ls/p4-fold-case-client-specs' On case insensitive systems, "git p4" did not work well with client specs. * ls/p4-fold-case-client-specs: git-p4: honor core.ignorecase when using P4 client specs | 01 September 2015, 23:31:22 UTC |
| 2953140 | Junio C Hamano | 01 September 2015, 23:31:21 UTC | Merge branch 'ah/submodule-typofix-in-error' Error string fix. * ah/submodule-typofix-in-error: git-submodule: remove extraneous space from error message | 01 September 2015, 23:31:21 UTC |
| bc1c600 | Junio C Hamano | 01 September 2015, 23:31:18 UTC | Merge branch 'ah/reflog-typofix-in-error' Error string fix. * ah/reflog-typofix-in-error: reflog: add missing single quote to error message | 01 September 2015, 23:31:18 UTC |
| 0b20a46 | Junio C Hamano | 01 September 2015, 23:31:15 UTC | Merge branch 'ah/read-tree-usage-string' Usage string fix. * ah/read-tree-usage-string: read-tree: replace bracket set with parentheses to clarify usage | 01 September 2015, 23:31:16 UTC |
| 8746e30 | Junio C Hamano | 01 September 2015, 23:31:12 UTC | Merge branch 'ah/pack-objects-usage-strings' Usage string fix. * ah/pack-objects-usage-strings: pack-objects: place angle brackets around placeholders in usage strings | 01 September 2015, 23:31:12 UTC |
| 49c15c0 | Junio C Hamano | 01 September 2015, 23:31:09 UTC | Merge branch 'br/svn-doc-include-paths-config' * br/svn-doc-include-paths-config: git-svn doc: mention "svn-remote.<name>.include-paths" | 01 September 2015, 23:31:10 UTC |
| 91d5469 | Junio C Hamano | 01 September 2015, 23:31:06 UTC | Merge branch 'nd/fixup-linked-gitdir' The code in "multiple-worktree" support that attempted to recover from an inconsistent state updated an incorrect file. * nd/fixup-linked-gitdir: setup: update the right file in multiple checkouts | 01 September 2015, 23:31:07 UTC |
| 9dd330e | Jeff King | 01 September 2015, 22:14:09 UTC | rerere: release lockfile in non-writing functions There's a bug in builtin/am.c in which we take a lock on MERGE_RR recursively. But rather than fix am.c, this patch fixes the confusing interface from rerere.c that caused the bug. Read on for the gory details. The setup_rerere() function both reads the existing MERGE_RR file, and takes MERGE_RR.lock. In the rerere() and rerere_forget() functions, we end up in write_rr(), which will then commit the lock file. But for functions like rerere_clear() that do not write to MERGE_RR, we expect the caller to have handled setup_rerere(). That caller would then need to release the lockfile, but it can't; the lock struct is local to rerere.c. For builtin/rerere.c, this is OK. We run a single rerere operation and then exit immediately, which has the side effect of rolling back the lockfile. But in builtin/am.c, this is actively wrong. If we run "git am -3 --skip", we call setup-rerere twice without releasing the lock: 1. The "--skip" causes us to call am_rerere_clear(), which calls setup_rerere(), but never drops the lock. 2. We then proceed to the next patch. 3. The "--3way" may cause us to call rerere() to handle conflicts in that patch, but we are already holding the lock. The lockfile code dies with: BUG: prepare_tempfile_object called for active object We could fix this by having rerere_clear() call rollback_lock_file(). But it feels a bit odd for it to roll back a lockfile that it did not itself take. So let's simplify the interface further, and handle setup_rerere in the function itself, taking away the question from the caller over whether they need to do so. We can give rerere_gc() the same treatment, as well (even though it doesn't have any callers besides builtin/rerere.c at this point). Note that these functions don't take flags from their callers to pass along to setup_rerere; that's OK, because the flags would not be meaningful for what they are doing. Both of those functions need to hold the lock because even though they do not write to MERGE_RR, they are still writing and should be protected from a simultaneous "rerere" run. But rerere_remaining(), "rerere diff", and "rerere status" are all read-only operations. They want to setup_rerere(), but do not care about taking the lock in the first place. Since our update of MERGE_RR is the usual atomic rename done by commit_lock_file, they can just do a lockless read. For that, we teach setup_rerere a READONLY flag to avoid the lock. As a bonus, this pushes builtin/rerere.c's setup_rerere call closer to the functions that use it. Which means that "git rerere totally-bogus-command" will no longer silently exit(0) in a repository without rerere enabled. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 01 September 2015, 22:52:54 UTC |
| 1616360 | Junio C Hamano | 31 August 2015, 22:40:24 UTC | Eighth batch for 2.6 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 22:40:24 UTC |
| e95c3fb | Junio C Hamano | 31 August 2015, 22:39:10 UTC | Merge branch 'sg/describe-contains' "git describe" without argument defaulted to describe the HEAD commit, but "git describe --contains" didn't. Arguably, in a repository used for active development, such defaulting would not be very useful as the tip of branch is typically not tagged, but it is better to be consistent. * sg/describe-contains: describe --contains: default to HEAD when no commit-ish is given | 31 August 2015, 22:39:10 UTC |
| b21089d | Junio C Hamano | 31 August 2015, 22:39:07 UTC | Merge branch 'db/push-sign-if-asked' The client side codepaths in "git push" have been cleaned up and the user can request to perform an optional "signed push", i.e. sign only when the other end accepts signed push. * db/push-sign-if-asked: push: add a config option push.gpgSign for default signed pushes push: support signing pushes iff the server supports it builtin/send-pack.c: use parse_options API config.c: rename git_config_maybe_bool_text and export it as git_parse_maybe_bool transport: remove git_transport_options.push_cert gitremote-helpers.txt: document pushcert option Documentation/git-send-pack.txt: document --signed Documentation/git-send-pack.txt: wrap long synopsis line Documentation/git-push.txt: document when --signed may fail | 31 August 2015, 22:39:08 UTC |
| 5b6211a | Junio C Hamano | 31 August 2015, 22:39:05 UTC | Merge branch 'jk/notes-merge-config' "git notes merge" can be told with "--strategy=<how>" option how to automatically handle conflicts; this can now be configured by setting notes.mergeStrategy configuration variable. * jk/notes-merge-config: notes: teach git-notes about notes.<name>.mergeStrategy option notes: add notes.mergeStrategy option to select default strategy notes: add tests for --commit/--abort/--strategy exclusivity notes: extract parse_notes_merge_strategy to notes-utils notes: extract enum notes_merge_strategy to notes-utils.h notes: document cat_sort_uniq rewriteMode | 31 August 2015, 22:39:05 UTC |
| d75bb73 | Junio C Hamano | 31 August 2015, 22:39:01 UTC | Merge branch 'jc/am-state-fix' Recent reimplementation of "git am" changed the format of state files kept in $GIT_DIR/rebase-apply/ without meaning to do so, primarily because write_file() API was cumbersome to use and it was easy to mistakenly make text files with incomplete lines. Update write_file() interface to make it harder to misuse. * jc/am-state-fix: write_file(): drop caller-supplied LF from calls to create a one-liner file write_file_v(): do not leave incomplete line at the end write_file(): drop "fatal" parameter builtin/am: make sure state files are text builtin/am: introduce write_state_*() helper functions | 31 August 2015, 22:39:03 UTC |
| 2ba6183 | Junio C Hamano | 31 August 2015, 22:38:59 UTC | Merge branch 'jc/log-p-cc' "git log --cc" did not show any patch, even though most of the time the user meant "git log --cc -p -m" to see patch output for commits with a single parent, and combined diff for merge commits. The command is taught to DWIM "--cc" (without "--raw" and other forms of output specification) to "--cc -p -m". * jc/log-p-cc: builtin/log.c: minor reformat log: show merge commit when --cc is given log: when --cc is given, default to -p unless told otherwise log: rename "tweak" helpers | 31 August 2015, 22:38:59 UTC |
| 7b7c10b | Junio C Hamano | 31 August 2015, 22:38:57 UTC | Merge branch 'jk/fix-alias-pager-config-key-warnings' Because the configuration system does not allow "alias.0foo" and "pager.0foo" as the configuration key, the user cannot use '0foo' as a custom command name anyway, but "git 0foo" tried to look these keys up and emitted useless warnings before saying '0foo is not a git command'. These warning messages have been squelched. * jk/fix-alias-pager-config-key-warnings: config: silence warnings for command names with invalid keys | 31 August 2015, 22:38:57 UTC |
| 0bb71fb | Junio C Hamano | 31 August 2015, 22:38:54 UTC | Merge branch 'jk/rev-list-has-no-notes' "git rev-list" does not take "--notes" option, but did not complain when one is given. * jk/rev-list-has-no-notes: rev-list: make it obvious that we do not support notes | 31 August 2015, 22:38:55 UTC |
| 5a4f07b | Junio C Hamano | 31 August 2015, 22:38:52 UTC | Merge branch 'hv/submodule-config' The gitmodules API accessed from the C code learned to cache stuff lazily. * hv/submodule-config: submodule: allow erroneous values for the fetchRecurseSubmodules option submodule: use new config API for worktree configurations submodule: extract functions for config set and lookup submodule: implement a config API for lookup of .gitmodules values | 31 August 2015, 22:38:52 UTC |
| fc9dfda | Junio C Hamano | 31 August 2015, 22:38:50 UTC | Merge branch 'sg/config-name-only' "git config --list" output was hard to parse when values consist of multiple lines. "--name-only" option is added to help this. * sg/config-name-only: get_urlmatch: avoid useless strbuf write format_config: simplify buffer handling format_config: don't init strbuf config: restructure format_config() for better control flow completion: list variable names reliably with 'git config --name-only' config: add '--name-only' option to list only variable names | 31 August 2015, 22:38:50 UTC |
| 6bea53c | Stefan Beller | 31 August 2015, 18:43:29 UTC | read-cache: fix indentation in read_index_from Signed-off-by: Stefan Beller <sbeller@google.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 19:31:00 UTC |
| 5c99995 | Christian Couder | 30 August 2015, 19:14:40 UTC | trailer: support multiline title We currently ignore the first line passed to `git interpret-trailers`, when looking for the beginning of the trailers. Unfortunately this does not work well when a commit is created with a line break in the title, using for example the following command: git commit -m 'place of code: change we made' That's why instead of ignoring only the first line, it is better to ignore the first paragraph. Signed-off-by: Christian Couder <christian.couder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 18:14:08 UTC |
| 1733ed3 | Erik Elfström | 30 August 2015, 09:18:09 UTC | t7300: fix broken && chains While we are here, remove some boilerplate by using test_commit. Signed-off-by: Erik Elfström <erik.elfstrom@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 16:46:36 UTC |
| ce11360 | Jeff King | 29 August 2015, 05:04:18 UTC | log: diagnose empty HEAD more clearly If you init or clone an empty repository, the initial message from running "git log" is not very friendly: $ git init Initialized empty Git repository in /home/peff/foo/.git/ $ git log fatal: bad default revision 'HEAD' Let's detect this situation and write a more friendly message: $ git log fatal: your current branch 'master' does not have any commits yet We also detect the case that 'HEAD' points to a broken ref; this should be even less common, but is easy to see. Note that we do not diagnose all possible cases. We rely on resolve_ref, which means we do not get information about complex cases. E.g., "--default master" would use dwim_ref to find "refs/heads/master", but we notice only that "master" does not exist. Similarly, a complex sha1 expression like "--default HEAD^2" will not resolve as a ref. But that's OK. We fall back to a generic error message in those cases, and they are unlikely to be used anyway. Catching an empty or broken "HEAD" improves the common case, and the other cases are not regressed. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 16:34:20 UTC |
| 475a344 | David Turner | 27 August 2015, 17:07:54 UTC | commit: don't rewrite shared index unnecessarily Remove a cache invalidation which would cause the shared index to be rewritten on as-is commits. When the cache-tree has changed, we need to update it. But we don't necessarily need to update the shared index. So setting active_cache_changed to SOMETHING_CHANGED is unnecessary. Instead, we let update_main_cache_tree just update the CACHE_TREE_CHANGED bit. In order to test this, make test-dump-split-index not segfault on missing replace_bitmap/delete_bitmap. This new codepath is not called now that the test passes, but is necessary to avoid a segfault when the new test is run with the old builtin/commit.c code. Signed-off-by: David Turner <dturner@twopensource.com> Acked-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 31 August 2015, 15:41:07 UTC |
| aecce6d | Junio C Hamano | 28 August 2015, 19:32:45 UTC | Sync with 2.5.1 | 28 August 2015, 19:32:45 UTC |
| e6837c8 | Junio C Hamano | 28 August 2015, 19:27:39 UTC | Seventh batch for 2.6 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 19:32:25 UTC |
| 483c9b8 | Junio C Hamano | 28 August 2015, 19:32:17 UTC | Merge branch 'cc/trailers-corner-case-fix' "interpret-trailers" helper mistook a single-liner log message that has a colon as the end of existing trailer. * cc/trailers-corner-case-fix: trailer: retitle a test and correct an in-comment message trailer: ignore first line of message | 28 August 2015, 19:32:17 UTC |
| 038226e | Junio C Hamano | 28 August 2015, 19:32:14 UTC | Merge branch 'dt/untracked-subdir' The experimental untracked-cache feature were buggy when paths with a few levels of subdirectories are involved. * dt/untracked-subdir: untracked cache: fix entry invalidation untracked-cache: fix subdirectory handling | 28 August 2015, 19:32:15 UTC |
| 9476c2c | Alex Henrie | 27 August 2015, 04:27:06 UTC | read-tree: replace bracket set with parentheses to clarify usage -u and -i can only be given if -m, --reset, or --prefix is given. Without parentheses, it looks like -u and -i can be used no matter what, and the second pair of brackets is confusing. Signed-off-by: Alex Henrie <alexhenrie24@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 19:01:37 UTC |
| b8c1d27 | Alex Henrie | 27 August 2015, 04:26:36 UTC | pack-objects: place angle brackets around placeholders in usage strings Signed-off-by: Alex Henrie <alexhenrie24@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 18:59:10 UTC |
| b80fa84 | Alex Henrie | 27 August 2015, 04:26:19 UTC | git-submodule: remove extraneous space from error message Signed-off-by: Alex Henrie <alexhenrie24@gmail.com> Acked-by: Chris Packham <judge.packham@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 18:57:24 UTC |
| aae42e4 | Ralf Thielow | 28 August 2015, 16:55:52 UTC | lockfile: remove function "hold_lock_file_for_append" With 77b9b1d (add_to_alternates_file: don't add duplicate entries, 2015-08-10) the last caller of function "hold_lock_file_for_append" has been removed, so we can remove the function as well. Signed-off-by: Ralf Thielow <ralf.thielow@gmail.com> Acked-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 18:32:01 UTC |
| c415fb7 | Junio C Hamano | 28 August 2015, 18:19:57 UTC | Git 2.5.1 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 18:19:57 UTC |
| c3cb7b6 | Jose F. Morales | 28 August 2015, 09:43:37 UTC | Mingw: verify both ends of the pipe () call The code to open and test the second end of the pipe clearly imitates the code for the first end. A little too closely, though... Let's fix the obvious copy-edit bug. Signed-off-by: Jose F. Morales <jfmcjf@gmail.com> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Acked-by: Johannes Sixt <j6t@kdbg.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 18:11:50 UTC |
| a0a50d8 | Lars Schneider | 28 August 2015, 12:00:34 UTC | git-p4: honor core.ignorecase when using P4 client specs Perforce depot may record paths in mixed cases, e.g. "p4 files" may show that there are these two paths: //depot/Path/to/file1 //depot/pATH/to/file2 and with "p4" or "p4v", these end up in the same directory, e.g. //depot/Path/to/file1 //depot/Path/to/file2 which is the desired outcome on case insensitive systems. If git-p4 is used with client spec "//depot/Path/...", however, then all files not matching the case in the client spec are ignored (in the example above "//depot/pATH/to/file2"). Fix this by using the path case that appears first in lexicographical order when core.ignorecase is set to true. This behavior is consistent with "p4" and "p4v". Signed-off-by: Lars Schneider <larsxschneider@gmail.com> Acked-by: Luke Diamand <luke@diamand.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 17:33:16 UTC |
| 99885bc | Alex Henrie | 27 August 2015, 04:26:02 UTC | reflog: add missing single quote to error message The error message can be seen by running `git config gc.reflogexpire foo` and then `git reflog expire`. Signed-off-by: Alex Henrie <alexhenrie24@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 16:42:13 UTC |
| 88329ca | René Scharfe | 22 August 2015, 19:06:45 UTC | archive-zip: support more than 65535 entries Support more than 65535 entries cleanly by writing a "zip64 end of central directory record" (with a 64-bit field for the number of entries) before the usual "end of central directory record" (which contains only a 16-bit field). InfoZIP's zip does the same. Archives with 65535 or less entries are not affected. Programs that extract all files like InfoZIP's zip and 7-Zip ignored the field and could extract all files already. Software that relies on the ZIP file directory to show a list of contained files quickly to simulate to normal directory like Windows' built-in ZIP functionality only saw a subset of the included files. Windows supports ZIP64 since Vista according to https://en.wikipedia.org/wiki/Zip_%28file_format%29#ZIP64. Suggested-by: Johannes Schauer <josch@debian.org> Signed-off-by: Rene Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 15:54:57 UTC |
| 0f747f9 | René Scharfe | 22 August 2015, 19:06:31 UTC | archive-zip: use a local variable to store the creator version Use a simpler conditional right next to the code which makes a higher creator version necessary -- namely symlink handling and support for executable files -- instead of a long line with a ternary operator. The resulting code has more lines but is simpler and allows reuse of the value easily. Signed-off-by: Rene Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 15:52:22 UTC |
| 19ee294 | René Scharfe | 22 August 2015, 19:06:12 UTC | t5004: test ZIP archives with many entries A ZIP file directory has a 16-bit field for the number of entries it contains. There are 64-bit extensions to deal with that. Demonstrate that git archive --format=zip currently doesn't use them and instead overflows the field. InfoZIP's unzip doesn't care about this field and extracts all files anyway. Software that uses the directory for presenting a filesystem like view quickly -- notably Windows -- depends on it, but doesn't lend itself to an automatic test case easily. Use InfoZIP's zipinfo, which probably isn't available everywhere but at least can provides *some* way to check this field. To speed things up a bit create and commit only a subset of the files and build a fake tree out of duplicates and pass that to git archive. Signed-off-by: Rene Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 August 2015, 15:52:10 UTC |
| 3a9835b | Junio C Hamano | 26 August 2015, 22:47:48 UTC | Sixth batch for 2.6 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 August 2015, 22:47:48 UTC |
| 678c5a4 | Junio C Hamano | 26 August 2015, 22:45:39 UTC | Merge branch 'sg/help-group' We rewrote one of the build scripts in Perl but this reimplements in Bourne shell. * sg/help-group: generate-cmdlist: re-implement as shell script | 26 August 2015, 22:45:40 UTC |
| 6525496 | Junio C Hamano | 26 August 2015, 22:45:38 UTC | Merge branch 'sg/wt-status-header-inclusion' * sg/wt-status-header-inclusion: wt-status: move #include "pathspec.h" to the header | 26 August 2015, 22:45:38 UTC |
| cfcd38f | Junio C Hamano | 26 August 2015, 22:45:37 UTC | Merge branch 'as/docfix-reflog-expire-unreachable' * as/docfix-reflog-expire-unreachable: Documentation/config: fix inconsistent label on gc.*.reflogExpireUnreachable | 26 August 2015, 22:45:37 UTC |
| 7205e22 | Junio C Hamano | 26 August 2015, 22:45:35 UTC | Merge branch 'sg/t3020-typofix' * sg/t3020-typofix: t3020: fix typo in test description | 26 August 2015, 22:45:36 UTC |
| 88bd19b | Junio C Hamano | 26 August 2015, 22:45:33 UTC | Merge branch 'jc/calloc-pathspec' * jc/calloc-pathspec: ps_matched: xcalloc() takes nmemb and then element size | 26 August 2015, 22:45:34 UTC |
| a5315dd | Junio C Hamano | 26 August 2015, 22:45:33 UTC | Merge branch 'dt/untracked-sparse' Test update. * dt/untracked-sparse: t7063: use --force-untracked-cache to speed up a bit | 26 August 2015, 22:45:33 UTC |
| 74702c3 | Junio C Hamano | 26 August 2015, 22:45:33 UTC | Merge branch 'pt/am-builtin' Rewrite "am" in "C". * pt/am-builtin: i18n: am: fix typo in description of -b option | 26 August 2015, 22:45:33 UTC |
| b7d2a15 | Junio C Hamano | 26 August 2015, 22:45:32 UTC | Merge branch 'pt/am-builtin-abort-fix' "git am" that was recently reimplemented in C had a performance regression in "git am --abort" that goes back to the version before an attempted (and failed) patch application. * pt/am-builtin-abort-fix: am --skip/--abort: merge HEAD/ORIG_HEAD tree into index | 26 August 2015, 22:45:32 UTC |
| 788f211 | Junio C Hamano | 26 August 2015, 22:45:32 UTC | Merge branch 'po/po-readme' * po/po-readme: po/README: Update directions for l10n contributors | 26 August 2015, 22:45:32 UTC |
| 629ac65 | Junio C Hamano | 26 August 2015, 22:45:31 UTC | Merge branch 'jv/send-email-selective-smtp-auth' "git send-email" learned a new option --smtp-auth to limit the SMTP AUTH mechanisms to be used to a subset of what the system library supports. * jv/send-email-selective-smtp-auth: send-email: provide whitelist of SMTP AUTH mechanisms | 26 August 2015, 22:45:31 UTC |
| ed070a4 | Junio C Hamano | 26 August 2015, 22:45:31 UTC | Merge branch 'ep/http-configure-ssl-version' A new configuration variable http.sslVersion can be used to specify what specific version of SSL/TLS to use to make a connection. * ep/http-configure-ssl-version: http: add support for specifying the SSL version | 26 August 2015, 22:45:31 UTC |
| 51e83a4 | Junio C Hamano | 26 August 2015, 22:45:30 UTC | Merge branch 'ss/fix-config-fd-leak' * ss/fix-config-fd-leak: config: close config file handle in case of error | 26 August 2015, 22:45:30 UTC |
| 1fb5a0e | Jiang Xin | 26 August 2015, 15:51:19 UTC | i18n: am: fix typo in description of -b option Signed-off-by: Jiang Xin <worldhello.net@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 August 2015, 19:51:43 UTC |
| 6262fe9 | Christian Couder | 26 August 2015, 02:51:00 UTC | trailer: retitle a test and correct an in-comment message Signed-off-by: Christian Couder <christian.couder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 August 2015, 19:16:56 UTC |
| 486e1e1 | Brett Randall | 24 August 2015, 00:23:45 UTC | git-svn doc: mention "svn-remote.<name>.include-paths" Mention the configuration variable in a way similar to how "svn-remote.<name>.ignore-paths" is mentioned. Signed-off-by: Brett Randall <javabrett@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 August 2015, 17:27:26 UTC |
| 33f2c4f | Junio C Hamano | 25 August 2015, 23:09:30 UTC | Sync with maint * maint: pull: pass upload_pack only when it was given | 25 August 2015, 23:09:30 UTC |
| 52f6893 | Junio C Hamano | 25 August 2015, 23:09:17 UTC | Merge branch 'jk/guess-repo-name-regression-fix' into maint "git clone $URL" in recent releases of Git contains a regression in the code that invents a new repository name incorrectly based on the $URL. This has been corrected. * jk/guess-repo-name-regression-fix: clone: use computed length in guess_dir_name clone: add tests for output directory | 25 August 2015, 23:09:17 UTC |
| 84deb3e | Junio C Hamano | 25 August 2015, 23:09:16 UTC | Merge branch 'jk/test-with-x' into maint Running tests with the "-x" option to make them verbose had some unpleasant interactions with other features of the test suite. * jk/test-with-x: test-lib: disable trace when test is not verbose test-lib: turn off "-x" tracing during chain-lint check | 25 August 2015, 23:09:16 UTC |
| 7a23807 | Junio C Hamano | 25 August 2015, 23:09:16 UTC | Merge branch 'sb/check-return-from-read-ref' into maint * sb/check-return-from-read-ref: transport-helper: die on errors reading refs. | 25 August 2015, 23:09:16 UTC |
| 425a4c7 | Junio C Hamano | 25 August 2015, 23:09:15 UTC | Merge branch 'mm/pull-upload-pack' into maint "git pull" in recent releases of Git has a regression in the code that allows custom path to the --upload-pack=<program>. This has been corrected. Note that this is irrelevant for 'master' with "git pull" rewritten in C. * mm/pull-upload-pack: pull: pass upload_pack only when it was given pull.sh: quote $upload_pack when passing it to git-fetch | 25 August 2015, 23:09:15 UTC |
| 13e0e28 | Junio C Hamano | 25 August 2015, 23:06:53 UTC | pull: pass upload_pack only when it was given The upload_pack shell variable is initialized to an empty string, so conditional expansion with ${upload_pack+"$upload_pack"} would not work very well. You need a colon there. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 25 August 2015, 23:08:58 UTC |
| 8da8889 | Junio C Hamano | 25 August 2015, 22:00:15 UTC | Fifth batch for 2.6 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 25 August 2015, 22:00:25 UTC |
| 9b8d731 | Junio C Hamano | 25 August 2015, 21:57:10 UTC | Merge branch 'cb/open-noatime-clear-errno' When trying to see that an object does not exist, a state errno leaked from our "first try to open a packfile with O_NOATIME and then if it fails retry without it" logic on a system that refuses O_NOATIME. This confused us and caused us to die, saying that the packfile is unreadable, when we should have just reported that the object does not exist in that packfile to the caller. * cb/open-noatime-clear-errno: git_open_noatime: return with errno=0 on success | 25 August 2015, 21:57:10 UTC |
| db86e61 | Junio C Hamano | 25 August 2015, 21:57:09 UTC | Merge branch 'mh/tempfile' The "lockfile" API has been rebuilt on top of a new "tempfile" API. * mh/tempfile: credential-cache--daemon: use tempfile module credential-cache--daemon: delete socket from main() gc: use tempfile module to handle gc.pid file lock_repo_for_gc(): compute the path to "gc.pid" only once diff: use tempfile module setup_temporary_shallow(): use tempfile module write_shared_index(): use tempfile module register_tempfile(): new function to handle an existing temporary file tempfile: add several functions for creating temporary files prepare_tempfile_object(): new function, extracted from create_tempfile() tempfile: a new module for handling temporary files commit_lock_file(): use get_locked_file_path() lockfile: add accessor get_lock_file_path() lockfile: add accessors get_lock_file_fd() and get_lock_file_fp() create_bundle(): duplicate file descriptor to avoid closing it twice lockfile: move documentation to lockfile.h and lockfile.c | 25 August 2015, 21:57:09 UTC |
| 424f89f | Junio C Hamano | 25 August 2015, 21:57:08 UTC | Merge branch 'pt/am-builtin-options' After "git am --opt1" stops, running "git am --opt2" pays attention to "--opt2" only for the patch that caused the original invocation to stop. * pt/am-builtin-options: am: let --signoff override --no-signoff am: let command-line options override saved options test_terminal: redirect child process' stdin to a pty | 25 August 2015, 21:57:08 UTC |
| 080cc64 | Junio C Hamano | 25 August 2015, 21:57:08 UTC | Merge branch 'dt/refs-pseudo' To prepare for allowing a different "ref" backend to be plugged in to the system, update_ref()/delete_ref() have been taught about ref-like things like MERGE_HEAD that are per-worktree (they will always be written to the filesystem inside $GIT_DIR). * dt/refs-pseudo: pseudoref: check return values from read_ref() sequencer: replace write_cherry_pick_head with update_ref bisect: use update_ref pseudorefs: create and use pseudoref update and delete functions refs: add ref_type function refs: introduce pseudoref and per-worktree ref concepts | 25 August 2015, 21:57:08 UTC |
