gitweb.git
git-am: Add am.keepcr and --no-keep-cr to override itStefan-W. Hahn Sat, 27 Feb 2010 14:20:27 +0000 (15:20 +0100)

git-am: Add am.keepcr and --no-keep-cr to override it

This patch adds the configuration `am.keepcr` for git-am. It also adds
`--no-keep-cr` parameter for git-am to give the possibility to
override configuration from command line.

Signed-off-by: Stefan-W. Hahn <stefan.hahn@s-hahn.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-am: Add command line parameter `--keep-cr` passing... Stefan-W. Hahn Sat, 27 Feb 2010 14:20:26 +0000 (15:20 +0100)

git-am: Add command line parameter `--keep-cr` passing it to git-mailsplit

c2ca1d7 (Allow mailsplit (and hence git-am) to handle mails with CRLF
line-endings, 2009-08-04) fixed "git mailsplit" to help people with
MUA whose output from save-as command uses CRLF as line terminators by
stripping CR at the end of lines.

However, when you know you are feeding output from "git format-patch"
directly to "git am", and especially when your contents have CR at the
end of line, such stripping is undesirable. To help such a use case,
teach --keep-cr option to "git am" and pass that to "git mailinfo".

Signed-off-by: Stefan-W. Hahn <stefan.hahn@s-hahn.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

documentation: 'git-mailsplit --keep-cr' is not hidden... Stefan-W. Hahn Sat, 27 Feb 2010 14:20:25 +0000 (15:20 +0100)

documentation: 'git-mailsplit --keep-cr' is not hidden anymore

So far this was an internal mechanism for rebase, but we will be exposing
it to the end users.

Signed-off-by: Stefan-W. Hahn <stefan.hahn@s-hahn.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Fri, 26 Feb 2010 07:21:50 +0000 (23:21 -0800)

Merge branch 'maint'

* maint:
t3301-notes: insert a shbang line in ./fake_editor.sh

Merge branch 'maint-1.6.6' into maintJunio C Hamano Fri, 26 Feb 2010 07:21:42 +0000 (23:21 -0800)

Merge branch 'maint-1.6.6' into maint

* maint-1.6.6:
t3301-notes: insert a shbang line in ./fake_editor.sh

Windows: redirect f[re]open("/dev/null") to f[re]open... Johannes Sixt Thu, 25 Feb 2010 20:03:44 +0000 (21:03 +0100)

Windows: redirect f[re]open("/dev/null") to f[re]open("nul")

On Windows, the equivalent of "/dev/null" is "nul". This implements
compatibility wrappers around fopen() and freopen() that check for this
particular file name.

The new tests exercise code paths where this is relevant.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

t3301-notes: insert a shbang line in ./fake_editor.shJohannes Sixt Thu, 25 Feb 2010 10:39:50 +0000 (11:39 +0100)

t3301-notes: insert a shbang line in ./fake_editor.sh

This is required on Windows because git-notes is now a built-in
rather than a shell script.

Signed-off-by: Johannes Sixt <j6t@kdbg.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Wed, 24 Feb 2010 23:34:07 +0000 (15:34 -0800)

Merge branch 'maint'

* maint:
commit: quote the user name in the example

commit: quote the user name in the exampleMatt Kraai Wed, 24 Feb 2010 14:18:25 +0000 (06:18 -0800)

commit: quote the user name in the example

If the user runs

git config --global user.name Your Name

as suggested, user.name will be set to "Your". With this patch, the
suggested command will be

git config --global user.name "Your Name"

which will set user.name to "Your Name" and hopefully help users avoid
the former mistake.

Signed-off-by: Matt Kraai <kraai@ftbfs.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'ml/maint-grep-doc' into maintJunio C Hamano Wed, 24 Feb 2010 23:33:23 +0000 (15:33 -0800)

Merge branch 'ml/maint-grep-doc' into maint

* ml/maint-grep-doc:
grep documentation: clarify what files match

Merge branch 'maint'Junio C Hamano Tue, 23 Feb 2010 22:27:55 +0000 (14:27 -0800)

Merge branch 'maint'

* maint:
am: remove rebase-apply directory before gc
rerere: fix memory leak if rerere images can't be read
Documentation: mention conflict marker size argument (%L) for merge driver

am: remove rebase-apply directory before gcJonathan Nieder Mon, 22 Feb 2010 14:35:46 +0000 (08:35 -0600)

am: remove rebase-apply directory before gc

When git am does an automatic gc it doesn't clean up the rebase-apply
directory until after this has finished. This means that if the user
aborts the gc then future am or rebase operations will report that an
existing operation is in progress, which is undesirable and confusing.

Reported by Mark Brown <broonie@debian.org> through
http://bugs.debian.org/570966

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

rerere: fix memory leak if rerere images can't be readBert Wesarg Tue, 23 Feb 2010 20:11:53 +0000 (21:11 +0100)

rerere: fix memory leak if rerere images can't be read

Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: mention conflict marker size argument... Bert Wesarg Tue, 23 Feb 2010 20:11:12 +0000 (21:11 +0100)

Documentation: mention conflict marker size argument (%L) for merge driver

23a64c9e (conflict-marker-size: new attribute, 2010-01-16) introduced the
new attribute and also pass the conflict marker size as %L to merge driver
commands. This documents the substitution.

Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'ml/maint-grep-doc'Junio C Hamano Sun, 21 Feb 2010 20:01:06 +0000 (12:01 -0800)

Merge branch 'ml/maint-grep-doc'

* ml/maint-grep-doc:
grep documentation: clarify what files match

Merge branch 'tc/maint-transport-ls-remote-with-void'Junio C Hamano Sun, 21 Feb 2010 20:01:03 +0000 (12:01 -0800)

Merge branch 'tc/maint-transport-ls-remote-with-void'

* tc/maint-transport-ls-remote-with-void:
transport: add got_remote_refs flag

Merge branch 'hm/maint-imap-send-crlf'Junio C Hamano Sun, 21 Feb 2010 20:00:21 +0000 (12:00 -0800)

Merge branch 'hm/maint-imap-send-crlf'

* hm/maint-imap-send-crlf:
git-imap-send: Convert LF to CRLF before storing patch to draft box

Merge branch 'sp/push-sideband'Junio C Hamano Sun, 21 Feb 2010 20:00:07 +0000 (12:00 -0800)

Merge branch 'sp/push-sideband'

* sp/push-sideband:
receive-pack: Send internal errors over side-band #2
t5401: Use a bare repository for the remote peer
receive-pack: Send hook output over side band #2
receive-pack: Wrap status reports inside side-band-64k
receive-pack: Refactor how capabilities are shown to the client
send-pack: demultiplex a sideband stream with status data
run-command: support custom fd-set in async
run-command: Allow stderr to be a caller supplied pipe

Merge branch 'jc/checkout-detached'Junio C Hamano Sun, 21 Feb 2010 19:59:42 +0000 (11:59 -0800)

Merge branch 'jc/checkout-detached'

* jc/checkout-detached:
Reword "detached HEAD" notification

Merge branch 'jc/maint-fix-test-perm'Junio C Hamano Sun, 21 Feb 2010 19:59:35 +0000 (11:59 -0800)

Merge branch 'jc/maint-fix-test-perm'

* jc/maint-fix-test-perm:
lib-patch-mode.sh: Fix permission
t6000lib: Fix permission

Merge branch 'jn/makefile-script-lib'Junio C Hamano Sun, 21 Feb 2010 19:59:22 +0000 (11:59 -0800)

Merge branch 'jn/makefile-script-lib'

* jn/makefile-script-lib:
Do not install shell libraries executable

Merge branch 'mv/request-pull-modernize'Junio C Hamano Sun, 21 Feb 2010 19:59:17 +0000 (11:59 -0800)

Merge branch 'mv/request-pull-modernize'

* mv/request-pull-modernize:
request-pull: avoid mentioning that the start point is a single commit

Merge branch 'maint'Junio C Hamano Sat, 20 Feb 2010 18:38:42 +0000 (10:38 -0800)

Merge branch 'maint'

* maint:
git-p4: fix bug in symlink handling
t1450: fix testcases that were wrongly expecting failure
Documentation: Fix indentation problem in git-commit(1)

git-p4: fix bug in symlink handlingEvan Powers Tue, 16 Feb 2010 08:44:08 +0000 (00:44 -0800)

git-p4: fix bug in symlink handling

Fix inadvertent breakage from b932705 (git-p4: stream from perforce to
speed up clones, 2009-07-30) in the code that strips the trailing '\n'
from p4 print on a symlink. (In practice, contents is of the form
['target\n', ''].)

Signed-off-by: Evan Powers <evan.powers@gmail.com>
Acked-by: Pete Wyckoff <pw@padd.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

t1450: fix testcases that were wrongly expecting failureThomas Rast Sat, 20 Feb 2010 00:18:44 +0000 (01:18 +0100)

t1450: fix testcases that were wrongly expecting failure

Almost exactly a year ago in 02a6552 (Test fsck a bit harder), I
introduced two testcases that were expecting failure.

However, the only bug was that the testcases wrote *blobs* because I
forgot to pass -t tag to hash-object. Fix this, and then adjust the
rest of the test to properly check the result.

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: Fix indentation problem in git-commit(1)Jacob Helwig Mon, 15 Feb 2010 12:33:06 +0000 (04:33 -0800)

Documentation: Fix indentation problem in git-commit(1)

Ever since the "See linkgit:git-config[1]..." paragraph was added to the
description for --untracked-files (d6293d1), the paragraphs for the
following options were indented at the same level as the "See
linkgit:git-config[1]" paragraph. This problem showed up in the
manpages, but not in the HTML documentation.

While this does fix the alignment of the options following
--untracked-files in the manpage, the "See linkgit..." portion of the
description does not retain its previous indentation level in the
manpages, or HTML documentation.

Signed-off-by: Jacob Helwig <jacob.helwig@gmail.com>
Acked-by: Thomas Rast <trast@student.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Fri, 19 Feb 2010 09:31:42 +0000 (01:31 -0800)

Merge branch 'maint'

* maint:
rm: fix bug in recursive subdirectory removal
Documentation: describe --thin more accurately

Merge branch 'jk/maint-rmdir-fix' into maintJunio C Hamano Fri, 19 Feb 2010 09:31:37 +0000 (01:31 -0800)

Merge branch 'jk/maint-rmdir-fix' into maint

* jk/maint-rmdir-fix:
rm: fix bug in recursive subdirectory removal

rm: fix bug in recursive subdirectory removalJeff King Fri, 19 Feb 2010 05:57:21 +0000 (00:57 -0500)

rm: fix bug in recursive subdirectory removal

If we remove a path in a/deep/subdirectory, we should try to
remove as many trailing components as possible (i.e.,
subdirectory, then deep, then a). However, the test for the
return value of rmdir was reversed, so we only ever deleted
at most one level.

The fix is in remove_path, so "apply" and "merge-recursive"
also are fixed.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: describe --thin more accuratelyStephen Boyd Thu, 18 Feb 2010 09:10:28 +0000 (01:10 -0800)

Documentation: describe --thin more accurately

The description for --thin was misleading and downright wrong. Correct
it with some inspiration from the description of index-pack's --fix-thin
and some background information from Nicolas Pitre <nico@fluxnic.net>.

Signed-off-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Thu, 18 Feb 2010 07:03:46 +0000 (23:03 -0800)

Merge branch 'maint'

* maint:
docs: don't talk about $GIT_DIR/refs/ everywhere

docs: don't talk about $GIT_DIR/refs/ everywhereJeff King Thu, 18 Feb 2010 01:16:20 +0000 (20:16 -0500)

docs: don't talk about $GIT_DIR/refs/ everywhere

It is misleading to say that we pull refs from $GIT_DIR/refs/*, because we
may also consult the packed refs mechanism. These days we tend to treat
the "refs hierarchy" as more of an abstract namespace that happens to be
represented as $GIT_DIR/refs. At best, this is a minor inaccuracy, but at
worst it can confuse users who then look in $GIT_DIR/refs and find that it
is missing some of the refs they expected to see.

This patch drops most uses of "$GIT_DIR/refs/*", changing them into just
"refs/*", under the assumption that users can handle the concept of an
abstract refs namespace. There are a few things to note:

- most cases just dropped the $GIT_DIR/ portion. But for cases where
that left _just_ the word "refs", I changed it to "refs/" to help
indicate that it was a hierarchy. I didn't do the same for longer
paths (e.g., "refs/heads" remained, instead of becoming
"refs/heads/").

- in some cases, no change was made, as the text was explicitly about
unpacked refs (e.g., the discussion in git-pack-refs).

- In some cases it made sense instead to note the existence of packed
refs (e.g., in check-ref-format and rev-parse).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'np/fast-import-idx-v2'Junio C Hamano Wed, 17 Feb 2010 23:28:25 +0000 (15:28 -0800)

Merge branch 'np/fast-import-idx-v2'

* np/fast-import-idx-v2:
fast-import: use the diff_delta() max_delta_size argument
fast-import: honor pack.indexversion and pack.packsizelimit config vars
fast-import: make default pack size unlimited
fast-import: use write_idx_file() instead of custom code
fast-import: use sha1write() for pack data
fast-import: start using struct pack_idx_entry

Acked-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Update draft release notes to 1.7.1Junio C Hamano Wed, 17 Feb 2010 23:01:11 +0000 (15:01 -0800)

Update draft release notes to 1.7.1

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Wed, 17 Feb 2010 23:00:10 +0000 (15:00 -0800)

Merge branch 'maint'

* maint:
Update 1.7.0.1 release notes

Update 1.7.0.1 release notesJunio C Hamano Wed, 17 Feb 2010 23:00:00 +0000 (15:00 -0800)

Update 1.7.0.1 release notes

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'jk/cherry-pick-reword' into maintJunio C Hamano Wed, 17 Feb 2010 22:55:24 +0000 (14:55 -0800)

Merge branch 'jk/cherry-pick-reword' into maint

* jk/cherry-pick-reword:
cherry-pick: prettify the advice message
cherry-pick: show commit name instead of sha1
cherry-pick: format help message as strbuf
cherry-pick: refactor commit parsing code
cherry-pick: rewrap advice message

Merge branch 'jk/grep-double-dash' into maintJunio C Hamano Wed, 17 Feb 2010 22:55:15 +0000 (14:55 -0800)

Merge branch 'jk/grep-double-dash' into maint

* jk/grep-double-dash:
accept "git grep -- pattern"

Merge branch 'jc/typo' into maintJunio C Hamano Wed, 17 Feb 2010 22:55:09 +0000 (14:55 -0800)

Merge branch 'jc/typo' into maint

* jc/typo:
Typofixes outside documentation area

gitweb: Protect escaping functions against calling... Jakub Narebski Sun, 7 Feb 2010 20:52:25 +0000 (21:52 +0100)

gitweb: Protect escaping functions against calling on undef

This is a bit of future-proofing esc_html and friends: when called
with undefined value they would now would return undef... which would
probably mean that error would still occur, but closer to the source
of problem.

This means that we can safely use
esc_html(shift) || "Internal Server Error"
in die_error() instead of
esc_html(shift || "Internal Server Error")

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

gitweb: esc_html (short) error message in die_errorJakub Narebski Sun, 7 Feb 2010 20:51:18 +0000 (21:51 +0100)

gitweb: esc_html (short) error message in die_error

The error message (second argument to die_error) is meant to be short,
one-line text description of given error. A few callers call
die_error with error message containing unescaped user supplied data
($hash, $file_name). Instead of forcing callers to escape data,
simply call esc_html on the parameter.

Note that optional third parameter, which contains detailed error
description, is meant to be HTML formatted, and therefore should be
not escaped.

While at it update esc_html synopsis/usage, and bring default error
description to read 'Internal Server Error' (titlecased).

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: use the diff_delta() max_delta_size argumentNicolas Pitre Wed, 17 Feb 2010 19:05:56 +0000 (14:05 -0500)

fast-import: use the diff_delta() max_delta_size argument

This let diff_delta() abort early if it is going to bust the given
size limit. Also, only objects larger than 20 bytes are considered
as objects smaller than that are most certainly going to produce
larger deltas than the original object due to the additional headers.

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: honor pack.indexversion and pack.packsizel... Nicolas Pitre Wed, 17 Feb 2010 19:05:55 +0000 (14:05 -0500)

fast-import: honor pack.indexversion and pack.packsizelimit config vars

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: make default pack size unlimitedNicolas Pitre Wed, 17 Feb 2010 19:05:54 +0000 (14:05 -0500)

fast-import: make default pack size unlimited

Now that fast-import is creating packs with index version 2, there is
no point limiting the pack size by default. A pack split will still
happen if off_t is not sufficiently large to hold large offsets.

While updating the doc, let's remove the "packfiles fit on CDs"
suggestion. Pack files created by fast-import are still suboptimal and
a 'git repack -a -f -d' or even 'git gc --aggressive' would be a pretty
good idea before considering storage on CDs.

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: use write_idx_file() instead of custom... Nicolas Pitre Wed, 17 Feb 2010 19:05:53 +0000 (14:05 -0500)

fast-import: use write_idx_file() instead of custom code

This allows for the creation of pack index version 2 with its object
CRC and the possibility for a pack to be larger than 4 GB.

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: use sha1write() for pack dataNicolas Pitre Wed, 17 Feb 2010 19:05:52 +0000 (14:05 -0500)

fast-import: use sha1write() for pack data

This is in preparation for using write_idx_file(). Also, by using
sha1write() we get some buffering to reduces the number of write
syscalls, and the written data is SHA1 summed which allows for the extra
data integrity validation check performed in fixup_pack_header_footer()
(details on this in commit abeb40e5aa).

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

fast-import: start using struct pack_idx_entryNicolas Pitre Wed, 17 Feb 2010 19:05:51 +0000 (14:05 -0500)

fast-import: start using struct pack_idx_entry

This is in preparation for using write_idx_file().

Signed-off-by: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

grep documentation: clarify what files matchMark Lodato Tue, 16 Feb 2010 00:25:40 +0000 (19:25 -0500)

grep documentation: clarify what files match

Clarify that git-grep(1) searches only tracked files, and that each
<pathspec> is a pathspec, as in any other ordinary git commands.

Add an example to show a simple use case for searching all .c and .h
files in the current directory and below.

Signed-off-by: Mark Lodato <lodatom@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'jc/typo'Junio C Hamano Wed, 17 Feb 2010 06:45:14 +0000 (22:45 -0800)

Merge branch 'jc/typo'

* jc/typo:
Typofixes outside documentation area

Merge branch 'maint'Junio C Hamano Wed, 17 Feb 2010 06:40:45 +0000 (22:40 -0800)

Merge branch 'maint'

* maint:
Prepare 1.7.0.1 release notes
Fix use of mutex in threaded grep
dwim_ref: fix dangling symref warning
stash pop: remove 'apply' options during 'drop' invocation
diff: make sure --output=/bad/path is caught
Remove hyphen from "git-command" in two error messages

Prepare 1.7.0.1 release notesJunio C Hamano Wed, 17 Feb 2010 06:25:03 +0000 (22:25 -0800)

Prepare 1.7.0.1 release notes

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'jc/maint-grep-one-thread-mutex-fix' into... Junio C Hamano Wed, 17 Feb 2010 06:23:25 +0000 (22:23 -0800)

Merge branch 'jc/maint-grep-one-thread-mutex-fix' into maint

* jc/maint-grep-one-thread-mutex-fix:
Fix use of mutex in threaded grep

Fix use of mutex in threaded grepJunio C Hamano Tue, 16 Feb 2010 02:34:28 +0000 (18:34 -0800)

Fix use of mutex in threaded grep

The program can decide at runtime not to use threading even if the support
is compiled in. In such a case, mutexes are not necessary and left
uninitialized. But the code incorrectly tried to take and release the
read_sha1_mutex unconditionally.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
Acked-by: Fredrik Kuivinen <frekui@gmail.com>

Merge branch 'maint-1.6.6' into maintJunio C Hamano Tue, 16 Feb 2010 23:05:02 +0000 (15:05 -0800)

Merge branch 'maint-1.6.6' into maint

* maint-1.6.6:
dwim_ref: fix dangling symref warning
stash pop: remove 'apply' options during 'drop' invocation
diff: make sure --output=/bad/path is caught
Remove hyphen from "git-command" in two error messages

Merge branch 'maint-1.6.5' into maint-1.6.6Junio C Hamano Tue, 16 Feb 2010 23:04:55 +0000 (15:04 -0800)

Merge branch 'maint-1.6.5' into maint-1.6.6

* maint-1.6.5:
dwim_ref: fix dangling symref warning
stash pop: remove 'apply' options during 'drop' invocation
diff: make sure --output=/bad/path is caught

transport: add got_remote_refs flagTay Ray Chuan Tue, 16 Feb 2010 07:18:21 +0000 (15:18 +0800)

transport: add got_remote_refs flag

transport_get_remote_refs() in tranport.c checks transport->remote_refs
to determine whether transport->get_refs_list() should be invoked. The
logic is "if it is NULL, we haven't run ls-remote to find out yet".

However, transport->remote_refs could still be NULL while cloning from
an empty repository. This causes get_refs_list() to be run unnecessarily.

Introduce a flag, transport->got_remote_refs, to more explicitly record
if we have run transport->get_refs_list() already.

Signed-off-by: Tay Ray Chuan <rctay89@gmail.com>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

dwim_ref: fix dangling symref warningJeff King Tue, 16 Feb 2010 07:03:16 +0000 (02:03 -0500)

dwim_ref: fix dangling symref warning

If we encounter a symref that is dangling, in most cases we will warn
about it. The one exception is a dangling HEAD, as that indicates a
branch yet to be born.

However, the check in dwim_ref was not quite right. If we were fed
something like "HEAD^0" we would try to resolve "HEAD", see that it is
dangling, and then check whether the _original_ string we got was
"HEAD" (which it wasn't in this case). And that makes no sense; the
dangling thing we found was not "HEAD^0" but rather "HEAD".

Fixing this squelches a scary warning from "submodule summary HEAD" (and
consequently "git status" with status.submodulesummary set) in an empty
repo, as the submodule script calls "git rev-parse -q --verify HEAD^0".

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

stash pop: remove 'apply' options during 'drop' invocationThomas Rast Mon, 15 Feb 2010 16:05:46 +0000 (17:05 +0100)

stash pop: remove 'apply' options during 'drop' invocation

The 'git stash pop' option parsing used to remove the first argument
in --index mode. At the time this was implemented, this first
argument was always --index. However, since the invention of the -q
option in fcdd0e9 (stash: teach quiet option, 2009-06-17) you can
cause an internal invocation of

git stash drop --index

by running

git stash pop -q --index

which then of course fails because drop doesn't know --index.

To handle this, instead let 'git stash apply' decide what the future
argument to 'drop' should be.

Warning: this means that 'git stash apply' must parse all options that
'drop' can take, and deal with them in the same way. This is
currently true for its only option -q.

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
Acked-by: Stephen Boyd <bebarino@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff: make sure --output=/bad/path is caughtLarry D'Anna Tue, 16 Feb 2010 04:10:45 +0000 (23:10 -0500)

diff: make sure --output=/bad/path is caught

The return value from fopen wasn't being checked.

Signed-off-by: Larry D'Anna <larry@elder-gods.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Remove hyphen from "git-command" in two error messagesPete Harlan Mon, 15 Feb 2010 23:33:18 +0000 (15:33 -0800)

Remove hyphen from "git-command" in two error messages

Signed-off-by: Pete Harlan <pgit@pcharlan.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Mon, 15 Feb 2010 02:59:20 +0000 (18:59 -0800)

Merge branch 'maint'

* maint:
fix minor memory leak in get_tree_entry()

Merge branch 'maint-1.6.6' into maintJunio C Hamano Mon, 15 Feb 2010 02:59:14 +0000 (18:59 -0800)

Merge branch 'maint-1.6.6' into maint

* maint-1.6.6:
fix minor memory leak in get_tree_entry()

fix minor memory leak in get_tree_entry()René Scharfe Sun, 14 Feb 2010 09:56:46 +0000 (10:56 +0100)

fix minor memory leak in get_tree_entry()

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Sat, 13 Feb 2010 23:14:04 +0000 (15:14 -0800)

Merge branch 'maint'

* maint:
Start 1.7.0 maintenance track

Conflicts:
RelNotes

Merge branch 'rs/git-dir-cleanup'Junio C Hamano Sat, 13 Feb 2010 23:09:33 +0000 (15:09 -0800)

Merge branch 'rs/git-dir-cleanup'

* rs/git-dir-cleanup:
Resurrect "git grep --no-index"
setenv(GIT_DIR) clean-up

Conflicts:
builtin-grep.c
t/t7002-grep.sh

Merge branch 'jk/cherry-pick-reword'Junio C Hamano Sat, 13 Feb 2010 23:09:33 +0000 (15:09 -0800)

Merge branch 'jk/cherry-pick-reword'

* jk/cherry-pick-reword:
cherry-pick: prettify the advice message
cherry-pick: show commit name instead of sha1
cherry-pick: format help message as strbuf
cherry-pick: refactor commit parsing code
cherry-pick: rewrap advice message

Merge branch 'jk/grep-double-dash'Junio C Hamano Sat, 13 Feb 2010 23:09:33 +0000 (15:09 -0800)

Merge branch 'jk/grep-double-dash'

* jk/grep-double-dash:
accept "git grep -- pattern"

Resurrect "git grep --no-index"Junio C Hamano Sat, 6 Feb 2010 18:40:08 +0000 (10:40 -0800)

Resurrect "git grep --no-index"

This reverts commit 3c8f6c8 (Revert 30816237 and 7e62265, 2010-02-05) as
the issue has been sorted out.

Start 1.7.0 maintenance trackJunio C Hamano Sat, 13 Feb 2010 23:04:00 +0000 (15:04 -0800)

Start 1.7.0 maintenance track

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Start 1.7.1 cycleJunio C Hamano Sat, 13 Feb 2010 23:01:37 +0000 (15:01 -0800)

Start 1.7.1 cycle

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Git 1.7.0 v1.7.0Junio C Hamano Fri, 12 Feb 2010 23:45:05 +0000 (15:45 -0800)

Git 1.7.0

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Fri, 12 Feb 2010 23:40:59 +0000 (15:40 -0800)

Merge branch 'maint'

* maint:
Fix typo in 1.6.6.2 release notes
Re-fix check-ref-format documentation mark-up

Fix typo in 1.6.6.2 release notesJunio C Hamano Fri, 12 Feb 2010 23:40:01 +0000 (15:40 -0800)

Fix typo in 1.6.6.2 release notes

Of course, these are changes since 1.6.6.1; changes since 1.6.6.2
would have been nil.

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Re-fix check-ref-format documentation mark-upJunio C Hamano Fri, 12 Feb 2010 23:39:03 +0000 (15:39 -0800)

Re-fix check-ref-format documentation mark-up

It is not double-backslash we forbid; backslashes are forbidden since
a4c2e699 (Disallow '\' in ref names, 2009-05-08)

Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-imap-send: Convert LF to CRLF before storing patch... Hitoshi Mitake Fri, 12 Feb 2010 11:36:12 +0000 (20:36 +0900)

git-imap-send: Convert LF to CRLF before storing patch to draft box

When storing a message over IMAP (RFC 3501 6.3.11), the message should be
in the format of an RFC 2822 message; most notably, CRLF must be used as
a line terminator.

Convert "\n" line endings in the payload to CRLF before feeding it to
IMAP APPEND command.

Signed-off-by: Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

archive documentation: attributes are taken from the... Junio C Hamano Wed, 10 Feb 2010 20:33:58 +0000 (12:33 -0800)

archive documentation: attributes are taken from the tree by default

By default, git-archive takes attributes from the tree being archived.
People however often wonder why their attempts to affect the way how the
command archives their tree by changing .gitattributes in their work tree
fail.

Add a bit of explanatory note to tell them how to achieve what they want
to do.

Noticed-by: Francois Marier
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: minor fixes to RelNotes-1.7.0Michael J Gruber Fri, 12 Feb 2010 09:47:53 +0000 (10:47 +0100)

Documentation: minor fixes to RelNotes-1.7.0

Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

bash: support 'git am's new '--continue' optionSZEDER Gábor Fri, 12 Feb 2010 14:11:47 +0000 (15:11 +0100)

bash: support 'git am's new '--continue' option

Signed-off-by: SZEDER Gábor <szeder@ira.uka.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint'Junio C Hamano Fri, 12 Feb 2010 07:06:32 +0000 (23:06 -0800)

Merge branch 'maint'

* maint:
filter-branch: Fix error message for --prune-empty --commit-filter

filter-branch: Fix error message for --prune-empty... Jacob Helwig Fri, 12 Feb 2010 02:46:22 +0000 (18:46 -0800)

filter-branch: Fix error message for --prune-empty --commit-filter

Running filter-branch with --prune-empty and --commit-filter reports:

"Cannot set --prune-empty and --filter-commit at the same time".

Change it to use the correct option name: --commit-filter

Signed-off-by: Jacob Helwig <jacob.helwig@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

cherry-pick: prettify the advice messageJeff King Thu, 11 Feb 2010 21:19:37 +0000 (16:19 -0500)

cherry-pick: prettify the advice message

It's hard to see the "how to commit" part of this message,
which users may want to cut and paste. On top of that,
having it in paragraph form means that a really long commit
name may cause ugly wrapping. Let's make it prettier, like:

Automatic cherry-pick failed. After resolving the conflicts,
mark the corrected paths with 'git add <paths>' or 'git rm <paths>'
and commit the result with:

git commit -c HEAD~23

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

cherry-pick: show commit name instead of sha1Jeff King Thu, 11 Feb 2010 21:08:15 +0000 (16:08 -0500)

cherry-pick: show commit name instead of sha1

When we have a conflict, we advise the user to do:

git commit -c $sha1

This works fine, but is unnecessarily confusing and annoying
for the user to type, when:

git commit -c $the_thing_you_called_cherry_pick_with

works just as well.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

cherry-pick: format help message as strbufJeff King Thu, 11 Feb 2010 21:07:06 +0000 (16:07 -0500)

cherry-pick: format help message as strbuf

This gets rid of the fixed-size buffer and an unchecked
sprintf. That sprintf is actually OK as the only
variable-sized thing put in it is an abbreviated sha1, which
is bounded at 40 characters. However, the next patch will
change that to something unbounded.

Note that this function now returns an allocated buffer
instead of a static one; however, it doesn't matter as the
only caller exits immediately.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

cherry-pick: refactor commit parsing codeJeff King Thu, 11 Feb 2010 21:06:43 +0000 (16:06 -0500)

cherry-pick: refactor commit parsing code

These lines are really just lookup_commit_reference
re-implemented.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

cherry-pick: rewrap advice messageJeff King Thu, 11 Feb 2010 21:06:01 +0000 (16:06 -0500)

cherry-pick: rewrap advice message

The current message overflows on an 80-character terminal.
While we're at it, fix the spelling of 'committing'.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

am: switch --resolved to --continueJeff King Thu, 11 Feb 2010 22:27:14 +0000 (17:27 -0500)

am: switch --resolved to --continue

Rebase calls this same function "--continue", which means
users may be trained to type it. There is no reason to
deprecate --resolved (or -r), so we will keep it as a
synonym.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Update draft release notes to 1.7.0 one more timeJunio C Hamano Wed, 10 Feb 2010 21:47:46 +0000 (13:47 -0800)

Update draft release notes to 1.7.0 one more time

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Sync with 1.6.6.2Junio C Hamano Wed, 10 Feb 2010 21:46:15 +0000 (13:46 -0800)

Sync with 1.6.6.2

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Git 1.6.6.2 v1.6.6.2Junio C Hamano Wed, 10 Feb 2010 21:44:11 +0000 (13:44 -0800)

Git 1.6.6.2

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'maint-1.6.5' into maintJunio C Hamano Wed, 10 Feb 2010 21:42:48 +0000 (13:42 -0800)

Merge branch 'maint-1.6.5' into maint

* maint-1.6.5:
t8003: check exit code of command and error message separately

t8003: check exit code of command and error message... Junio C Hamano Tue, 9 Feb 2010 18:06:33 +0000 (10:06 -0800)

t8003: check exit code of command and error message separately

Shell reports exit status only from the most downstream command
in a pipeline. In these tests, we want to make sure that the
command fails in a controlled way, and produces a correct error
message.

This issue was known by Jay who submitted the patch, and also was
pointed out by Hannes during the review process, but I forgot to
fix it up before applying. Sorry about that.

Signed-off-by: Junio C Hamano <gitster@pobox.com>

Merge branch 'sp/maint-fast-import-large-blob' into... Junio C Hamano Wed, 10 Feb 2010 21:32:20 +0000 (13:32 -0800)

Merge branch 'sp/maint-fast-import-large-blob' into maint

* sp/maint-fast-import-large-blob:
fast-import: Stream very large blobs directly to pack

Merge branch 'gp/maint-cvsserver' into maintJunio C Hamano Wed, 10 Feb 2010 21:02:52 +0000 (13:02 -0800)

Merge branch 'gp/maint-cvsserver' into maint

* gp/maint-cvsserver:
git-cvsserver: allow regex metacharacters in CVSROOT

Merge branch 'jc/maint-reflog-bad-timestamp' into maintJunio C Hamano Wed, 10 Feb 2010 21:02:43 +0000 (13:02 -0800)

Merge branch 'jc/maint-reflog-bad-timestamp' into maint

* jc/maint-reflog-bad-timestamp:
t0101: use a fixed timestamp when searching in the reflog
Update @{bogus.timestamp} fix not to die()
approxidate_careful() reports errorneous date string

Merge branch 'il/maint-xmallocz' into maintJunio C Hamano Wed, 10 Feb 2010 21:02:16 +0000 (13:02 -0800)

Merge branch 'il/maint-xmallocz' into maint

* il/maint-xmallocz:
Fix integer overflow in unpack_compressed_entry()
Fix integer overflow in unpack_sha1_rest()
Fix integer overflow in patch_delta()
Add xmallocz()

Merge branch 'jh/maint-config-file-prefix' into maintJunio C Hamano Wed, 10 Feb 2010 21:02:05 +0000 (13:02 -0800)

Merge branch 'jh/maint-config-file-prefix' into maint

* jh/maint-config-file-prefix:
builtin-config: Fix crash when using "-f <relative path>" from non-root dir

Merge branch 'nd/include-termios-for-osol' into maintJunio C Hamano Wed, 10 Feb 2010 21:01:55 +0000 (13:01 -0800)

Merge branch 'nd/include-termios-for-osol' into maint

* nd/include-termios-for-osol:
Add missing #include to support TIOCGWINSZ on Solaris

Merge branch 'jc/maint-1.6.1-checkout-m-custom-merge... Junio C Hamano Wed, 10 Feb 2010 20:54:15 +0000 (12:54 -0800)

Merge branch 'jc/maint-1.6.1-checkout-m-custom-merge' into maint

* jc/maint-1.6.1-checkout-m-custom-merge:
checkout -m path: fix recreating conflicts

Conflicts:
t/t7201-co.sh

Merge branch 'rs/maint-archive-match-pathspec' into... Junio C Hamano Wed, 10 Feb 2010 20:52:39 +0000 (12:52 -0800)

Merge branch 'rs/maint-archive-match-pathspec' into maint

* rs/maint-archive-match-pathspec:
archive: complain about path specs that don't match anything

check-ref-format documentation: fix enumeration mark-upJunio C Hamano Wed, 10 Feb 2010 18:18:55 +0000 (10:18 -0800)

check-ref-format documentation: fix enumeration mark-up

The last item in the enumerated refname rule was mistakenly made into
a sub-item of the 7th one. It should be the 8th one in the list on its
own.

Signed-off-by: Junio C Hamano <gitster@pobox.com>