3d5ae43 | Junio C Hamano | 20 May 2011, 03:30:44 UTC | Git 1.7.5.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 May 2011, 03:30:44 UTC |
36268b7 | Jonathan Nieder | 19 May 2011, 15:14:25 UTC | provide a copy of the LGPLv2.1 The LGPL seems to require providing a copy of the license when distributing xdiff, compat/fnmatch, and so on, or altering the license notices to refer to the GPL intead. Since we don't want to do the latter, let's do the former. It's nice to let people know their rights anyway. Inspired-by: Erik Faye-Lund <kusmabite@gmail.com> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 20 May 2011, 01:23:17 UTC |
ea1ab4b | Junio C Hamano | 16 May 2011, 23:44:50 UTC | Update draft release notes to 1.7.5.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 May 2011, 23:44:50 UTC |
4bbfd8e | Junio C Hamano | 16 May 2011, 23:39:09 UTC | Merge branch 'jn/maint-format-patch-doc' into maint * jn/maint-format-patch-doc: Documentation: describe the format of messages with inline patches | 16 May 2011, 23:39:09 UTC |
0f51322 | Junio C Hamano | 16 May 2011, 23:38:56 UTC | Merge branch 'ss/cherry-pick-x-doc' into maint * ss/cherry-pick-x-doc: doc: Clarify that "cherry-pick -x" does not use "git notes" | 16 May 2011, 23:38:56 UTC |
8de4338 | Junio C Hamano | 16 May 2011, 23:38:46 UTC | Merge branch 'vr/merge-base-doc' into maint * vr/merge-base-doc: Restructure documentation for git-merge-base. Documentation: update to git-merge-base --octopus | 16 May 2011, 23:38:46 UTC |
f704591 | Junio C Hamano | 16 May 2011, 23:38:16 UTC | Merge branch 'sr/maint-fast-import-tighten-option-parsing' into maint * sr/maint-fast-import-tighten-option-parsing: fast-import: fix option parser for no-arg options | 16 May 2011, 23:38:16 UTC |
a8793d2 | Junio C Hamano | 16 May 2011, 23:37:54 UTC | Merge branch 'jc/t1506-shell-param-expansion-gotcha' into maint * jc/t1506-shell-param-expansion-gotcha: t1507: avoid "${parameter<op>'word'}" inside double-quotes | 16 May 2011, 23:37:54 UTC |
90e8b9b | Junio C Hamano | 16 May 2011, 23:37:33 UTC | Merge branch 'jc/fix-add-u-unmerged' into maint * jc/fix-add-u-unmerged: Fix "add -u" that sometimes fails to resolve unmerged paths Conflicts: builtin/add.c | 16 May 2011, 23:37:33 UTC |
81f9068 | Junio C Hamano | 16 May 2011, 23:37:12 UTC | Merge branch 'jn/gitweb-dependency' into maint * jn/gitweb-dependency: Remove gitweb/gitweb.cgi and other legacy targets from main Makefile git-instaweb: Simplify build dependency on gitweb | 16 May 2011, 23:37:12 UTC |
2f0db1d | Junio C Hamano | 16 May 2011, 23:37:07 UTC | Merge branch 'jc/maint-branch-mergeoptions' into maint * jc/maint-branch-mergeoptions: merge: make branch.<name>.mergeoptions correctly override merge.<option> Conflicts: builtin/merge.c | 16 May 2011, 23:37:07 UTC |
c69e8b6 | Junio C Hamano | 16 May 2011, 23:36:46 UTC | Merge branch 'jc/maint-add-p-overlapping-hunks' into maint * jc/maint-add-p-overlapping-hunks: t3701: add-p-fix makes the last test to pass "add -p": work-around an old laziness that does not coalesce hunks add--interactive.perl: factor out repeated --recount option t3701: Editing a split hunk in an "add -p" session add -p: 'q' should really quit | 16 May 2011, 23:36:46 UTC |
43d532e | Jim Meyering | 16 May 2011, 18:23:35 UTC | Documentation/git-fsck.txt: fix typo: unreadable -> unreachable Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 16 May 2011, 23:35:08 UTC |
96dbe93 | Junio C Hamano | 15 May 2011, 23:11:55 UTC | Update draft release notes to 1.7.5.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 May 2011, 23:11:55 UTC |
06c0f42 | Junio C Hamano | 15 May 2011, 23:10:49 UTC | Merge branch 'cn/format-patch-quiet' into maint * cn/format-patch-quiet: format-patch: document --quiet option format-patch: don't pass on the --quiet flag | 15 May 2011, 23:10:49 UTC |
ab02095 | Junio C Hamano | 15 May 2011, 22:57:16 UTC | Merge branch 'jm/mergetool-submodules' into maint * jm/mergetool-submodules: mergetool: Teach about submodules | 15 May 2011, 22:57:16 UTC |
92b501f | Junio C Hamano | 15 May 2011, 22:56:44 UTC | Merge branch 'jk/format-patch-quote-special-in-from' into maint * jk/format-patch-quote-special-in-from: pretty: quote rfc822 specials in email addresses | 15 May 2011, 22:56:44 UTC |
e5c1650 | Junio C Hamano | 15 May 2011, 22:52:40 UTC | Merge branch 'vh/git-svn-doc' into maint * vh/git-svn-doc: git-svn.txt: small typeface improvements git-svn.txt: move option descriptions git-svn.txt: fix usage of --add-author-from | 15 May 2011, 22:52:40 UTC |
f4e5168 | Junio C Hamano | 15 May 2011, 19:16:29 UTC | git_open_noatime(): drop unused parameter Since commit c793430 (Limit file descriptors used by packs, 2011-02-28), the extra parameter added in f2e872aa (Work around EMFILE when there are too many pack files, 2010-11-01) is not used anymore. Remove it. Signed-off-by: Junio C Hamano <gitster@pobox.com> Acked-by: Shawn O. Pearce <spearce@spearce.org> | 15 May 2011, 22:24:52 UTC |
ccf5ace | Junio C Hamano | 15 May 2011, 19:16:03 UTC | sha1_file: typofix The number zero is spelled "zero", not "zer0". Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 May 2011, 22:24:36 UTC |
8c2be75 | Jonathan Nieder | 14 May 2011, 20:19:21 UTC | add, merge, diff: do not use strcasecmp to compare config variable names The config machinery already makes section and variable names lowercase when parsing them, so using strcasecmp for comparison just feels wasteful. No noticeable change intended. Noticed-by: Jay Soffian <jaysoffian@gmail.com> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 May 2011, 01:53:39 UTC |
375f8a0 | Junio C Hamano | 13 May 2011, 17:57:09 UTC | Prepare for 1.7.5.2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 May 2011, 17:57:09 UTC |
e31b018 | Junio C Hamano | 13 May 2011, 17:45:21 UTC | Merge branch 'aw/maint-rebase-i-p-no-ff' into maint * aw/maint-rebase-i-p-no-ff: git-rebase--interactive.sh: preserve-merges fails on merges created with no-ff | 13 May 2011, 17:45:21 UTC |
bc67ad8 | Junio C Hamano | 13 May 2011, 17:45:00 UTC | Merge branch 'js/blame-parsename' into maint * js/blame-parsename: t/annotate-tests: Use echo & cat instead of sed blame: tolerate bogus e-mail addresses a bit better | 13 May 2011, 17:45:00 UTC |
978471d | Junio C Hamano | 13 May 2011, 17:44:54 UTC | Merge branch 'gr/cvsimport-alternative-cvspass-location' into maint * gr/cvsimport-alternative-cvspass-location: Look for password in both CVS and CVSNT password files. | 13 May 2011, 17:44:54 UTC |
e82c746 | Junio C Hamano | 13 May 2011, 17:44:46 UTC | Merge branch 'cj/p4merge' into maint * cj/p4merge: Pass empty file to p4merge where no base is suitable. | 13 May 2011, 17:44:46 UTC |
8272bca | Junio C Hamano | 13 May 2011, 17:44:19 UTC | Merge branch 'jk/merge-one-file-working-tree' into maint * jk/merge-one-file-working-tree: merge-one-file: fix broken merges with alternate work trees add tests for merge-index / merge-one-file | 13 May 2011, 17:44:19 UTC |
a613b53 | Junio C Hamano | 13 May 2011, 17:41:54 UTC | Merge branch 'jc/fix-diff-files-unmerged' into maint * jc/fix-diff-files-unmerged: diff-files: show unmerged entries correctly diff: remove often unused parameters from diff_unmerge() diff.c: return filepair from diff_unmerge() test: use $_z40 from test-lib | 13 May 2011, 17:41:54 UTC |
4bdbfb1 | Junio C Hamano | 13 May 2011, 17:41:24 UTC | Merge branch 'mz/maint-rename-unmerged' into maint * mz/maint-rename-unmerged: diffcore-rename: don't consider unmerged path as source | 13 May 2011, 17:41:24 UTC |
ebe8621 | Johan Herland | 13 May 2011, 16:43:29 UTC | t5400: Fix a couple of typos Signed-off-by: Johan Herland <johan@herland.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 May 2011, 16:57:16 UTC |
365c2aa | Junio C Hamano | 09 May 2011, 04:43:20 UTC | t1507: avoid "${parameter<op>'word'}" inside double-quotes Kacper Kornet noticed that a $variable in "word" in the above construct is not substituted by his pdksh. Modern POSIX compliant shells (e.g. dash, ksh, bash) all seem to interpret POSIX "2.6.2 Parameter Expansion" that says "word shall be subjected to tilde expansion, parameter expansion, command substitution, and arithmetic expansion" in ${parameter<op>word}, to mean that the word is expanded as if it appeared in dq pairs, so if the word were "'$variable'" (sans dq) it would expand to a single quote, the value of the $variable and then a single quote. Johannes Sixt reports that the behavior of quoting at the right of :- when the ${...:-...} expansion appears in double-quotes was debated recently at length at the Austin group. We can avoid this issue and future-proof the test by a slight rewrite. Helped-by: Johannes Sixt <j.sixt@viscovery.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 09 May 2011, 16:19:42 UTC |
0bf9fc0 | Junio C Hamano | 08 May 2011, 20:43:04 UTC | t3701: add-p-fix makes the last test to pass Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 May 2011, 20:43:04 UTC |
f09f1d3 | Jakub Narebski | 07 May 2011, 12:45:21 UTC | Remove gitweb/gitweb.cgi and other legacy targets from main Makefile Now that there is gitweb/Makefile, let's leave only "gitweb" and "install-gitweb" targets in main Makefile. Those targets just delegate to gitweb's Makefile. Requested-by: Junio C Hamano <gitster@pobox.com> Signed-off-by: Jakub Narebski <jnareb@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 May 2011, 18:52:57 UTC |
ff2e2cd | Jakub Narebski | 07 May 2011, 12:45:20 UTC | git-instaweb: Simplify build dependency on gitweb Since c0cb4ed (git-instaweb: Configure it to work with new gitweb structure, 2010-05-28) git-instaweb does not re-create gitweb.cgi etc., but makes use of installed gitweb. Therefore simplify git-instaweb dependency on gitweb subsystem in main Makefile from 'gitweb/gitweb.cgi gitweb/static/gitweb.css gitweb/static/gitweb.js' to simply 'gitweb'. This is preparation for splitting gitweb.perl script, and for splitting gitweb.js (to be reassembled / combined on build). This way we don't have to duplicate parts of gitweb/Makefile in main Makefile... it is also more correct description of git-instaweb dependency. Reported-by: Junio C Hamano <gitster@pobox.com> Signed-off-by: Jakub Narebski <jnareb@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 08 May 2011, 18:52:05 UTC |
0d8fc3e | Junio C Hamano | 05 May 2011, 00:42:51 UTC | merge: make branch.<name>.mergeoptions correctly override merge.<option> The parsing of the additional command line parameters supplied to the branch.<name>.mergeoptions configuration variable was implemented at the wrong stage. If any merge-related variable came after we read branch.<name>.mergeoptions, the earlier value was overwritten. We should first read all the merge.* configuration, override them by reading from branch.<name>.mergeoptions and then finally read from the command line. This patch should fix it, even though I now strongly suspect that branch.<name>.mergeoptions that gives a single command line that needs to be parsed was likely to be an ill-conceived idea to begin with. Sigh... Helped-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 06 May 2011, 22:01:56 UTC |
4c007ae | João Britto | 06 May 2011, 04:16:17 UTC | Remove duplicated "is a" Signed-off-by: João Britto <jabcalves@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 06 May 2011, 05:14:15 UTC |
4cce4ef | Sverre Rabbelier | 05 May 2011, 18:56:00 UTC | fast-import: fix option parser for no-arg options While refactoring the options parser in bc3c79a (fast-import: add (non-)relative-marks feature, 2009-12-04), it was made too lenient for options that take no argument, fix that. Signed-off-by: Sverre Rabbelier <srabbelier@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 06 May 2011, 04:21:24 UTC |
8fe6177 | Junio C Hamano | 05 May 2011, 18:28:52 UTC | t3701: fix here document A broken here-document was not caught because end of file is taken by an implicit end of the here document (POSIX does not seem to say it is an error to lack the delimiter), and everything in the test just turned into a single "cat into a file". Noticed-by: Kacper Kornet <draenog@pld-linux.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 05 May 2011, 18:28:52 UTC |
9fee24c | Michael J Gruber | 05 May 2011, 09:13:38 UTC | git-fast-import.txt: --relative-marks takes no parameter Remove spurious "=" after --relative-marks. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 05 May 2011, 17:18:18 UTC |
2d9932c | Dima Sharov | 05 May 2011, 06:40:17 UTC | shell: add missing initialization of argv0_path According to c6dfb39 (remote-curl: add missing initialization of argv0_path, 2009-10-13), stand-alone programs (non-builtins) must call git_extract_argv0_path(argv[0]) in order to help builds that derive the installation prefix at runtime. Without this call, the program segfaults (or raises an assertion failure). Signed-off-by: Dima Sharov <git.avalakvista@gmail.com> Acked-by: Johannes Sixt <j6t@kdbg.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 05 May 2011, 16:32:28 UTC |
c01bd5a | Brian Gernhardt | 05 May 2011, 04:43:37 UTC | t/annotate-tests: Use echo & cat instead of sed The use of the sed command "1i No robots allowed" caused the version of sed in OS X to die with sed: 1: "1i "No robots allowed"\n": command i expects \ followed by text Since this command was just trying to add a single line to the beginning of the file, do the same with "echo" followed by "cat". Unbreaks t8001 and t8002 on OS X 10.6.7 Signed-off-by: Brian Gernhardt <brian@gernhardtsoftware.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 05 May 2011, 16:24:21 UTC |
2f312e8 | Junio C Hamano | 04 May 2011, 22:47:40 UTC | Git 1.7.5.1 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 04 May 2011, 22:47:40 UTC |
f5bf1b5 | Junio C Hamano | 04 May 2011, 21:59:07 UTC | Merge branch 'jh/dirstat' into maint * jh/dirstat: --dirstat: In case of renames, use target filename instead of source filename Teach --dirstat not to completely ignore rearranged lines within a file --dirstat-by-file: Make it faster and more correct --dirstat: Describe non-obvious differences relative to --stat or regular diff | 04 May 2011, 21:59:07 UTC |
419272d | Junio C Hamano | 04 May 2011, 21:58:56 UTC | Merge branch 'sp/maint-clear-postfields' into maint * sp/maint-clear-postfields: http: clear POSTFIELDS when initializing a slot | 04 May 2011, 21:58:56 UTC |
be44a20 | Junio C Hamano | 04 May 2011, 21:58:42 UTC | Merge branch 'jk/maint-stash-oob' into maint * jk/maint-stash-oob: stash: fix false positive in the invalid ref test. stash: fix accidental apply of non-existent stashes Conflicts: t/t3903-stash.sh | 04 May 2011, 21:58:42 UTC |
476e424 | Junio C Hamano | 04 May 2011, 21:58:13 UTC | Merge branch 'jk/maint-upload-pack-shallow' into maint * jk/maint-upload-pack-shallow: upload-pack: start pack-objects before async rev-list | 04 May 2011, 21:58:13 UTC |
e9f1878 | Junio C Hamano | 04 May 2011, 21:57:49 UTC | Merge branch 'dm/stash-k-i-p' into maint * dm/stash-k-i-p: stash: ensure --no-keep-index and --patch can be used in any order stash: add two more tests for --no-keep-index | 04 May 2011, 21:57:49 UTC |
f28d2e3 | Junio C Hamano | 04 May 2011, 21:57:38 UTC | Merge branch 'jc/pack-objects-bigfile' into maint * jc/pack-objects-bigfile: Teach core.bigfilethreashold to pack-objects | 04 May 2011, 21:57:38 UTC |
f6bfe76 | Junio C Hamano | 04 May 2011, 21:57:33 UTC | Merge branch 'mg/reflog-with-options' into maint * mg/reflog-with-options: reflog: fix overriding of command line options t/t1411: test reflog with formats builtin/log.c: separate default and setup of cmd_log_init() | 04 May 2011, 21:57:33 UTC |
0752000 | Junio C Hamano | 04 May 2011, 21:57:15 UTC | Merge branch 'jk/stash-loosen-safety' into maint * jk/stash-loosen-safety: stash: drop dirty worktree check on apply | 04 May 2011, 21:57:15 UTC |
391b493 | Junio C Hamano | 04 May 2011, 21:57:06 UTC | Merge branch 'ar/clean-rmdir-empty' into maint * ar/clean-rmdir-empty: clean: unreadable directory may still be rmdir-able if it is empty | 04 May 2011, 21:57:06 UTC |
a3402c1 | Junio C Hamano | 04 May 2011, 21:56:51 UTC | Merge branch 'mg/sha1-path-advise' into maint * mg/sha1-path-advise: sha1_name: Suggest commit:./file for path in subdir t1506: factor out test for "Did you mean..." | 04 May 2011, 21:56:51 UTC |
d022443 | Valentin Haenel | 04 May 2011, 19:58:17 UTC | git-add.txt: document 'add.ignoreErrors' Signed-off-by: Valentin Haenel <valentin.haenel@gmx.de> Noticed-by: Junio C Hamano <gitster@pobox.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 04 May 2011, 21:52:30 UTC |
0a0ec7b | Ciaran Jessup | 01 May 2011, 22:16:02 UTC | Pass empty file to p4merge where no base is suitable. Modify the p4merge client command to pass a reference to an empty file instead of the local file when no base revision available. In the situation where a merge tries to add a file from one branch into a branch that already contains that file (by name), p4merge currently seems to have successfully automatically resolved the 'conflict' when it is opened (correctly if the files differed by just whitespace for example) but leaves the save button disabled. This means the user of the p4merge client cannot commit the resolved changes back to disk and merely exits, leaving the original (merge-conflicted) file intact on the disk. Provide an empty base file to p4merge so that it leaves the save button enabled. This will allow saving of the auto-resolution to disk. Signed-off-by: Ciaran Jessup <ciaranj@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 01 May 2011, 22:56:05 UTC |
58fdef0 | Guy Rouillier | 01 May 2011, 05:33:52 UTC | Look for password in both CVS and CVSNT password files. In conn, if password is not passed on command line, look for a password entry in both the CVS password file and the CVSNT password file. If only one file is found and the requested repository is in that file, or if both files are found but the requested repository is found in only one file, use the password from the single file containing the repository entry. If both files are found and the requested repository is found in both files, then produce an error message. The CVS password file separates tokens with a space character, while the CVSNT password file separates tokens with an equal (=) character. Add a sub find_password_entry that accepts the password file name and a delimiter to eliminate code duplication. Signed-off-by: Guy Rouillier <guyr@burntmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 01 May 2011, 18:42:59 UTC |
6aaeca9 | Jeff King | 29 April 2011, 22:24:32 UTC | merge-one-file: fix broken merges with alternate work trees The merge-one-file tool predates the invention of GIT_WORK_TREE. By the time GIT_WORK_TREE was invented, most people were using the merge-recursive strategy, which handles resolving internally. Therefore these features have had very little testing together. For the most part, merge-one-file just works with GIT_WORK_TREE; most of its heavy lifting is done by plumbing commands which do respect GIT_WORK_TREE properly. The one exception is a shell redirection which touches the worktree directly, writing results to the wrong place in the presence of a GIT_WORK_TREE variable. This means that merges won't even fail; they will silently produce incorrect results, throwing out the entire "theirs" side of files which need content-level merging! This patch makes merge-one-file chdir to the toplevel of the working tree (and exit if we don't have one). This most closely matches the assumption made by the original script (before separate work trees were invented), and matches what happens when the script is called as part of a merge strategy. While we're at it, we'll also error-check the call to cat. Merging a file in a subdirectory could in fact fail, as the redirection relies on the "checkout-index" call just prior to create leading directories. But we never noticed, since we ignored the error return from running cat. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:37:43 UTC |
cf1af1b | Jeff King | 29 April 2011, 22:23:54 UTC | add tests for merge-index / merge-one-file There were no tests for either, except a brief use in t1200-tutorial. These tools are not used much these days, as most people use the merge-recursive strategy, which handles everything internally. However, they are used by the "octopus" and "resolve" strategies, as well as any custom strategies or merge scripts people have built around them. For example, together with read-tree, they are the simplest way to do a basic content-level merge without checking out the entire repository contents beforehand. This script adds a basic test of the tools to perform one content-level merge. It also shows a failure of the tools to work properly in the face of GIT_WORK_TREE or core.worktree. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:37:08 UTC |
933e44d | Junio C Hamano | 06 April 2011, 21:20:57 UTC | "add -p": work-around an old laziness that does not coalesce hunks Since 0beee4c (git-add--interactive: remove hunk coalescing, 2008-07-02), "git add--interactive" behaves lazily and passes overlapping hunks to the underlying "git apply" without coalescing. This was partially corrected by 7a26e65 (its partial revert, 2009-05-16), but overlapping hunks are still passed when the patch is edited. Teach "git apply" the --allow-overlap option that disables a safety feature that avoids misapplication of patches by not applying patches to overlapping hunks, and pass this option form "add -p" codepath. Do not even advertise the option, as this is merely a workaround, and the correct fix is to make "add -p" correctly coalesce adjacent patch hunks. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:27:59 UTC |
9dce832 | Junio C Hamano | 06 April 2011, 21:12:34 UTC | add--interactive.perl: factor out repeated --recount option Depending on the direction and the target of patch application, we would need to pass --cached and --reverse to underlying "git apply". Also we only pass --check when we are not applying but just checking. But we always pass --recount since 8cbd431 (git-add--interactive: replace hunk recounting with apply --recount, 2008-07-02). Instead of repeating the same --recount over and over again, move it to a single place that actually runs the command, namely, "run_git_apply" subroutine. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:27:59 UTC |
f3217e2 | Junio C Hamano | 29 April 2011, 19:41:16 UTC | t3701: Editing a split hunk in an "add -p" session Arnaud Lacombe reported that with the recent change to reject overlapping hunks fed to "git apply", the edit mode of an "add -p" session that lazily feeds overlapping hunks without coalescing adjacent ones claim that the patch does not apply. Expose the problem to be fixed. Cf. http://thread.gmane.org/gmane.comp.version-control.git/170685/focus=171000 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:26:41 UTC |
f5ea3f2 | Junio C Hamano | 29 April 2011, 22:12:32 UTC | add -p: 'q' should really quit The "quit" command was added in 9a7a1e0 (git add -p: new "quit" command at the prompt, 2009-04-10) to allow the user to say that hunks other than what have already been chosen are undesirable, and exit the interactive loop immediately. It forgot that there may be an undecided hunk before the current one. In such a case, the interactive loop still goes back to the beginning. Clear all the USE bit for undecided hunks and exit the loop. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 22:25:03 UTC |
65f13f2 | Junio C Hamano | 29 April 2011, 18:47:10 UTC | Start 1.7.5.1 maintenance track Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 18:47:35 UTC |
04a67dc | Junio C Hamano | 29 April 2011, 18:43:18 UTC | Merge branch 'mg/x-years-12-months' into maint * mg/x-years-12-months: date: avoid "X years, 12 months" in relative dates | 29 April 2011, 18:43:18 UTC |
a1dd7e1 | Sylvain Rabot | 29 April 2011, 18:23:24 UTC | git-send-email: fix missing space in error message When the command cannot make a connection to the SMTP server the error message to diagnose the broken configuration is issued. However, when an optional smtp-server-port is given and needs to be reported, the message lacked a space between "hello=<smtp-domain>" and "port=<smtp-server-port>". Signed-off-by: Sylvain Rabot <sylvain@abstraction.fr> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 18:34:32 UTC |
9b01f00 | Josh Stone | 21 April 2011, 22:07:36 UTC | blame: tolerate bogus e-mail addresses a bit better The names and e-mails are sanitized by fmt_ident() when creating commits, so that they do not contain "<" nor ">", and the "committer" and "author" lines in the commit object will always be in the form: ("author" | "committer") name SP "<" email ">" SP timestamp SP zone When parsing the email part out, the current code looks for SP starting from the end of the email part, but the author could obfuscate the address as "author at example dot com". We should instead look for SP followed by "<", to match the logic of the side that formats these lines. Signed-off-by: Josh Stone <jistone@redhat.com> Reviewed-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 29 April 2011, 18:09:42 UTC |
c192f9c | Andrew Wong | 28 April 2011, 04:35:55 UTC | git-rebase--interactive.sh: preserve-merges fails on merges created with no-ff 'git rebase' uses 'git merge' to preserve merges (-p). This preserves the original merge commit correctly, except when the original merge commit was created by 'git merge --no-ff'. In this case, 'git rebase' will fail to preserve the merge, because during 'git rebase', 'git merge' will simply fast-forward and skip the commit. For example: B / \ A---M / ---o---O---P---Q If we try to rebase M onto P, we lose the merge commit and this happens: A---B / ---o---O---P---Q To correct this, we simply do a "no fast-forward" on all merge commits when rebasing. Since by the time we decided to do a 'git merge' inside 'git rebase', it means there was a merge originally, so 'git merge' should always create a merge commit regardless of what the merge branches look like. This way, when rebase M onto P from the above example, we get: B / \ A---M / ---o---O---P---Q Signed-off-by: Andrew Wong <andrew.kw.w@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 28 April 2011, 16:56:55 UTC |
4d03c18 | Jeff King | 08 April 2011, 22:40:36 UTC | pretty: quote rfc822 specials in email addresses If somebody has a name that includes an rfc822 special, we will output it literally in the "From:" header. This is usually OK, but certain characters (like ".") are supposed to be enclosed in double-quotes in a mail header. In practice, whether this matters may depend on your MUA. Some MUAs will happily take in: From: Foo B. Bar <author@example.com> without quotes, and properly quote the "." when they send the actual mail. Others may not, or may screw up harder things like: From: Foo "The Baz" Bar <author@example.com> For example, mutt will strip the quotes, thinking they are actual syntactic rfc822 quotes. So let's quote properly, and then (if necessary) we still apply rfc2047 encoding on top of that, which should make all MUAs happy. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 April 2011, 22:07:02 UTC |
7733b27 | Valentin Haenel | 26 April 2011, 11:34:19 UTC | git-svn.txt: small typeface improvements Mark subcommand names as 'subcommand' to make them stand out. Signed-off-by: Valentin Haenel <valentin.haenel@gmx.de> Acked-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 April 2011, 18:48:48 UTC |
c88d18f | Valentin Haenel | 26 April 2011, 11:34:18 UTC | git-svn.txt: move option descriptions The options '---use-log-author' and '--add-author-from' are applicable to other subcommands except 'fetch' -- therefore move them from the 'fetch' section to the more general 'options' section. Signed-off-by: Valentin Haenel <valentin.haenel@gmx.de> Acked-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 April 2011, 18:48:43 UTC |
12d3065 | Valentin Haenel | 25 April 2011, 21:36:15 UTC | git-svn.txt: fix usage of --add-author-from The option '--add-author-from' is used in 'commit-diff', 'set-tree', and 'dcommit' subcommands. Signed-off-by: Valentin Haenel <valentin.haenel@gmx.de> Acked-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 26 April 2011, 18:48:34 UTC |
1e41827 | Junio C Hamano | 26 April 2011, 15:04:49 UTC | http: clear POSTFIELDS when initializing a slot After posting a short request using CURLOPT_POSTFIELDS, if the slot is reused for posting a large payload, the slot ends up having both POSTFIELDS (which now points at a random garbage) and READFUNCTION, in which case the curl library tries to use the stale POSTFIELDS. Clear it as part of the general slot initialization in get_active_slot(). Heavylifting-by: Shawn Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> Acked-by: Shawn Pearce <spearce@spearce.org> | 26 April 2011, 17:44:33 UTC |
ec014ea | Junio C Hamano | 24 April 2011, 06:36:32 UTC | Git 1.7.5 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 06:36:32 UTC |
75973b2 | Junio C Hamano | 21 April 2011, 01:11:19 UTC | Fix "add -u" that sometimes fails to resolve unmerged paths "git add -u" updates the index with the updated contents from the working tree by internally running "diff-files" to grab the set of paths that are different from the index. Then it updates the index entries for the paths that are modified in the working tree, and deletes the index entries for the paths that are deleted in the working tree. It ignored the output from the diff-files that indicated that a path is unmerged. For these paths, it instead relied on the fact that an unmerged path is followed by the result of comparison between stage #2 (ours) and the working tree, and used that to update or delete such a path when it is used to record the resolution of a conflict. As the result, when a path did not have stage #2 (e.g. "we deleted while the other side added"), these unmerged stages were left behind, instead of recording what the user resolved in the working tree. Since we recently fixed "diff-files" to indicate if the corresponding path exists on the working tree for an unmerged path, we do not have to rely on the comparison with stage #2 anymore. We can instead tell the diff-files not to compare with higher stages, and use the unmerged output to update the index to reflect the state of the working tree. The changes to the test vector in t2200 illustrates the nature of the bug and the fix. The test expected stage #1 and #3 entries be left behind, but it was codifying the buggy behaviour. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 06:13:28 UTC |
095ce95 | Junio C Hamano | 22 April 2011, 23:19:27 UTC | diff-files: show unmerged entries correctly Earlier, e9c8409 (diff-index --cached --raw: show tree entry on the LHS for unmerged entries., 2007-01-05) taught the command to show the object name and the mode from the entry coming from the tree side when comparing a tree with an unmerged index. This is a belated companion patch that teaches diff-files to show the mode from the entry coming from the working tree side, when comparing an unmerged index and the working tree. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 05:35:13 UTC |
fa7b290 | Junio C Hamano | 22 April 2011, 23:05:58 UTC | diff: remove often unused parameters from diff_unmerge() e9c8409 (diff-index --cached --raw: show tree entry on the LHS for unmerged entries., 2007-01-05) added a <mode, object name> pair as parameters to this function, to store them in the pre-image side of an unmerged file pair. Now the function is fixed to return the filepair it queued, we can make the caller on the special case codepath to do so. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 05:34:43 UTC |
76399c0 | Junio C Hamano | 22 April 2011, 22:55:55 UTC | diff.c: return filepair from diff_unmerge() The underlying diff_queue() returns diff_filepair so that the caller can further add information to it, and the helper function diff_unmerge() utilizes the feature itself, but does not expose it to its callers, which was kind of selfish. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 05:34:43 UTC |
3749fde | Junio C Hamano | 24 April 2011, 05:34:13 UTC | test: use $_z40 from test-lib There is no need to duplicate the definition of $_z40 and $_x40 that test-lib.sh supplies the test scripts. Signed-off-by: Junio C Hamano <gitster@pobox.com> | 24 April 2011, 05:34:43 UTC |
f1e9c54 | Michael J Gruber | 20 April 2011, 09:12:11 UTC | date: avoid "X years, 12 months" in relative dates When relative dates are more than about a year ago, we start writing them as "Y years, M months". At the point where we calculate Y and M, we have the time delta specified as a number of days. We calculate these integers as: Y = days / 365 M = (days % 365 + 15) / 30 This rounds days in the latter half of a month up to the nearest month, so that day 16 is "1 month" (or day 381 is "1 year, 1 month"). We don't round the year at all, though, meaning we can end up with "1 year, 12 months", which is silly; it should just be "2 years". Implement this differently with months of size onemonth = 365/12 so that totalmonths = (long)( (days + onemonth/2)/onemonth ) years = totalmonths / 12 months = totalmonths % 12 In order to do this without floats, we write the first formula as totalmonths = (days*12*2 + 365) / (365*2) Tests and inspiration by Jeff King. Helped-by: Jeff King <peff@peff.net> Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 21 April 2011, 02:23:16 UTC |
6ceb270 | Junio C Hamano | 19 April 2011, 18:51:00 UTC | Git 1.7.5-rc3 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 April 2011, 18:51:00 UTC |
0e73bb4 | Junio C Hamano | 19 April 2011, 18:49:13 UTC | Sync with 1.7.4.5 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 April 2011, 18:49:13 UTC |
4fec830 | Junio C Hamano | 19 April 2011, 18:45:38 UTC | Git 1.7.4.5 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 April 2011, 18:45:38 UTC |
ccc2fcf | Michael J Gruber | 19 April 2011, 12:24:27 UTC | git-svn.txt: Document --mergeinfo 6abd933 (git-svn: allow the mergeinfo property to be set, 2010-09-24) introduced the --mergeinfo option. Document it. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 19 April 2011, 18:29:45 UTC |
60e199c | Junio C Hamano | 18 April 2011, 21:14:53 UTC | Revert "run-command: prettify -D_FORTIFY_SOURCE workaround" This reverts commit ebec842773932e6f853acac70c80f84209b5f83e, which somehow mistakenly thought that any non-zero return from write(2) is an error. | 18 April 2011, 21:14:53 UTC |
bea7d16 | Sebastian Schuberth | 15 April 2011, 17:53:51 UTC | doc: Clarify that "cherry-pick -x" does not use "git notes" The documentation for "cherry-pick -x" could be misread in the way that a "git notes" object is attached to the new commit, which is not the case. Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 April 2011, 20:53:00 UTC |
e0d4827 | Jonathan Nieder | 15 April 2011, 02:22:02 UTC | Documentation: describe the format of messages with inline patches Add a DISCUSSION section to the "git format-patch" manual to encourage people to send patches in a form that can be applied by "git am" automatically. There are two such forms: 1. The default form in which most metadata goes in the mail header and the message body starts with the patch description; 2. The snipsnip form in which a message starts with pertinent discussion and ends with a patch after a "scissors" mark. The example requires QP encoding in the "Subject:" header intended for the mailer to give the reader a chance to reflect on that, rather than being startled by it later. By contrast, in-body "From:" and "Subject:" lines should be human-readable and not QP encoded. Inspired-by: Jim Meyering <jim@meyering.net> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com> Improved-by: Junio C Hamano <gitster@pobox.com> Improved-by: Drew Northup <drew.northup@maine.edu> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 April 2011, 20:24:53 UTC |
ded7e04 | Jonathan Nieder | 15 April 2011, 08:38:55 UTC | Restructure documentation for git-merge-base. Restructure the text of git-merge-base to better explain more clearly the different modes of operation. Signed-off-by: Vincent van Ravesteijn <vfr@lyx.org> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 April 2011, 17:16:24 UTC |
5729482 | Vincent van Ravesteijn | 15 April 2011, 08:34:03 UTC | Documentation: update to git-merge-base --octopus Unlike plain merge-base, merge-base --octopus only requires at least one commit argument; update the synopsis to reflect that. Add a sentence to the discussion that when --octopus is used, we do expect '2' (the common ansestor across all) as the result. Signed-off-by: Vincent van Ravesteijn <vfr@lyx.org> Reviewed-by: Jonathan Nieder <jrnieder@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 15 April 2011, 17:13:52 UTC |
4d8b32a | Junio C Hamano | 14 April 2011, 19:26:45 UTC | Merge branch 'maint' * maint: archive: document limitation of tar.umask config setting t3306,t5304: avoid clock skew issues git.txt: fix list continuation | 14 April 2011, 19:26:45 UTC |
810cae5 | René Scharfe | 14 April 2011, 18:04:57 UTC | archive: document limitation of tar.umask config setting The local value of the config variable tar.umask is not passed to the other side with --remote. We may want to change that, but for now just document this fact. Reported-by: Jacek Masiulaniec <jacek.masiulaniec@gmail.com> Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 14 April 2011, 19:24:02 UTC |
e3b02bc | Michael J Gruber | 14 April 2011, 17:38:13 UTC | t3306,t5304: avoid clock skew issues On systems where the local time and file modification time may be out of sync (e.g. test directory on NFS) t3306 and t5305 can fail because prune compares times such as "now" (client time) with file modification times (server times for remote file systems). I.e., these are spurious test failures. Avoid this by setting the relevant modification times to the local time. Noticed on a system with as little as 2s time skew. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 14 April 2011, 17:47:18 UTC |
fd5858b | Michael J Gruber | 14 April 2011, 07:17:26 UTC | git.txt: fix list continuation Remove a spurious empty line which prevented asciidoc from recognizing a list continuation mark ('+'), so that it does not get output literally any more. Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 14 April 2011, 16:19:51 UTC |
865d370 | Junio C Hamano | 13 April 2011, 21:01:03 UTC | Git 1.7.5-rc2 Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 April 2011, 21:02:00 UTC |
6c4c9bd | Junio C Hamano | 13 April 2011, 20:59:19 UTC | Merge branch 'maint' * maint: | 13 April 2011, 20:59:19 UTC |
06ff449 | Junio C Hamano | 13 April 2011, 20:56:52 UTC | Merge branch 'jc/rev-list-options-fix' into maint * jc/rev-list-options-fix: "log --cherry-pick" documentation regression fix | 13 April 2011, 20:56:52 UTC |
302de7a | Junio C Hamano | 13 April 2011, 20:55:53 UTC | Merge branch 'js/checkout-untracked-symlink' into maint * js/checkout-untracked-symlink: t2021: mark a test as fixed | 13 April 2011, 20:55:53 UTC |
ff7f089 | Jonathon Mah | 13 April 2011, 10:00:48 UTC | mergetool: Teach about submodules When the index has conflicted submodules, mergetool used to mildly clobber the module, renaming it to mymodule.BACKUP.nnnn, then failing to copy it non-recursively. Recognize submodules and offer a resolution instead: Submodule merge conflict for 'Shared': {local}: submodule commit ad9f12e3e6205381bf2163a793d1e596a9e211d0 {remote}: submodule commit f5893fb70ec5646efcd9aa643c5136753ac89253 Use (l)ocal or (r)emote, or (a)bort? Selecting a commit will stage it, but not update the submodule (as git does had there been no conflict). Type changes are also supported, should the path be a submodule on one side, and a file, symlink, directory, or deleted on the other. Signed-off-by: Jonathon Mah <me@JonathonMah.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 April 2011, 19:21:45 UTC |
0353a0c | Jim Meyering | 13 April 2011, 15:39:40 UTC | remove doubled words, e.g., s/to to/to/, and fix related typos I found that some doubled words had snuck back into projects from which I'd already removed them, so now there's a "syntax-check" makefile rule in gnulib to help prevent recurrence. Running the command below spotted a few in git, too: git ls-files | xargs perl -0777 -n \ -e 'while (/\b(then?|[iao]n|i[fst]|but|f?or|at|and|[dt])\s+\1\b/gims)' \ -e '{$n=($` =~ tr/\n/\n/ + 1); ($v=$&)=~s/\n/\\n/g;' \ -e 'print "$ARGV:$n:$v\n"}' Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 April 2011, 18:59:11 UTC |
5e31075 | Ramkumar Ramachandra | 10 April 2011, 15:39:14 UTC | revert: Hide '-r' option in default usage The '-r' command-line option is a no-op provided only for backward compatiblity since abd6970 (cherry-pick: make -r the default, 2006-10-05), and somehow ended up surviving across reimplementation in C at 9509af6 (Make git-revert & git-cherry-pick a builtin, 2007-03-01) and another rewrite of the command line parser at f810379 (Make builtin-revert.c use parse_options, 2007-10-07). We should have stopped advertising the option long time ago. Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 13 April 2011, 18:02:49 UTC |
b781cfa | Carlos Martín Nieto | 12 April 2011, 15:51:37 UTC | format-patch: document --quiet option Signed-off-by: Carlos Martín Nieto <cmn@elego.de> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 12 April 2011, 19:50:32 UTC |
250087f | Carlos Martín Nieto | 12 April 2011, 15:35:38 UTC | format-patch: don't pass on the --quiet flag The --quiet flag is not meant to be passed on to the diff, as the user always wants the patches to be produced so catch it and pass it to reopen_stdout which decides whether to print the filename or not. Noticed by Paul Gortmaker Signed-off-by: Carlos Martín Nieto <cmn@elego.de> Signed-off-by: Junio C Hamano <gitster@pobox.com> | 12 April 2011, 19:49:11 UTC |