gitweb.git
git.el: Make status refresh faster.Alexandre Julliard Tue, 8 Jan 2008 13:49:09 +0000 (14:49 +0100)

git.el: Make status refresh faster.

Don't set the needs-refresh flag when inserting a new file info, since
ewoc refreshes it upon insert already; this makes a full refresh twice
as fast.

Also make git-fileinfo-prettyprint a little faster by not retrieving
permission values twice.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git.el: Refresh files from their real state upon commit.Alexandre Julliard Tue, 8 Jan 2008 13:46:22 +0000 (14:46 +0100)

git.el: Refresh files from their real state upon commit.

Instead of just setting the state to up-to-date, retrieve the full
state again, so that the file type can be displayed properly.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git.el: Make sure we never insert the same file twice.Alexandre Julliard Tue, 8 Jan 2008 13:45:46 +0000 (14:45 +0100)

git.el: Make sure we never insert the same file twice.

Skip non-zero stage files during git-ls-files -c, they are handled
later. Also fix git-insert-info-list to merge duplicate file names.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

"git-apply --check" should not report "fixed"Junio C Hamano Tue, 8 Jan 2008 23:46:18 +0000 (15:46 -0800)

"git-apply --check" should not report "fixed"

When running "git apply --check" while --whitespace=fix is
enabled (either from the command line or via the configuration),
we reported that "N line(s) applied after _fixing_", but --check
by itself does not apply and this message was alarming.

We could even reword the message to say "N line(s) would have
been applied after fixing...", but this patch does not go that
far. Instead, we just make it use the "N lines add whitespace
errors" warning, which happens to be a good diagnostic message a
user would expect from the --check option.

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

Merge in GIT 1.5.3.8Junio C Hamano Tue, 8 Jan 2008 07:39:02 +0000 (23:39 -0800)

Merge in GIT 1.5.3.8

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

GIT 1.5.3.8 v1.5.3.8Junio C Hamano Tue, 8 Jan 2008 06:41:43 +0000 (22:41 -0800)

GIT 1.5.3.8

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

Documentation: config: add 'help.*' and 'instaweb.... Christian Couder Tue, 8 Jan 2008 03:55:14 +0000 (04:55 +0100)

Documentation: config: add 'help.*' and 'instaweb.*' variables.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Allow git-mergetool to handle paths with a leading... Rogan Dawes Mon, 7 Jan 2008 07:37:38 +0000 (09:37 +0200)

Allow git-mergetool to handle paths with a leading space

Signed-off-by: Rogan Dawes <rogan@dawes.za.net>

git-svn: clarify the "Ignoring error from SVN" pieceEric Wong Mon, 7 Jan 2008 10:40:40 +0000 (02:40 -0800)

git-svn: clarify the "Ignoring error from SVN" piece

I've heard of several users puzzled by this, and it sometimes it
appears as if git-svn is doing nothing on slower connections and
larger repositories.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-svn: support for funky branch and project names... Eric Wong Mon, 12 Nov 2007 07:37:42 +0000 (23:37 -0800)

git-svn: support for funky branch and project names over HTTP(S)

SVN requires that paths be URI-escaped for HTTP(S) repositories.
file:// and svn:// repositories do not need these rules.

Additionally, accessing individual paths inside repositories
(check_path() and get_log() do NOT require escapes to function
and in fact it breaks things).

Noticed-by: Michael J. Cohen <mjc@cruiseplanners.com>
Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

slightly better auto gc messageNicolas Pitre Mon, 7 Jan 2008 21:05:22 +0000 (16:05 -0500)

slightly better auto gc message

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

Documentation: typofixRalf Wildenhues Mon, 7 Jan 2008 21:43:27 +0000 (22:43 +0100)

Documentation: typofix

Signed-off-by: Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Update draft release notes for 1.5.4Junio C Hamano Mon, 7 Jan 2008 06:20:40 +0000 (22:20 -0800)

Update draft release notes for 1.5.4

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

diff: do not chomp hunk-header in the middle of a characterJunio C Hamano Wed, 2 Jan 2008 09:50:11 +0000 (01:50 -0800)

diff: do not chomp hunk-header in the middle of a character

We truncate hunk-header line at 80 bytes, but that 80th byte
could be in the middle of a character, which is bad. This uses
pick_one_utf8_char() function to make sure we do not cut a character
in the middle.

This assumes that the internal representation of the text is
UTF-8. This needs to be extended in the future but the optimal
direction has not been decided yet.

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

utf8_width(): allow non NUL-terminated inputJunio C Hamano Wed, 2 Jan 2008 09:49:58 +0000 (01:49 -0800)

utf8_width(): allow non NUL-terminated input

The original interface assumed that the input string is
always terminated with a NUL, but that wasn't too useful.

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

utf8: pick_one_utf8_char()Junio C Hamano Mon, 7 Jan 2008 03:02:22 +0000 (19:02 -0800)

utf8: pick_one_utf8_char()

utf8_width() function was doing two different things. To pick a
valid character from UTF-8 stream, and compute the display width of
that character. This splits the former to a separate function
pick_one_utf8_char().

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

git.el: Display file types and type changes.Alexandre Julliard Sun, 6 Jan 2008 11:13:36 +0000 (12:13 +0100)

git.el: Display file types and type changes.

Handle the T status from git-diff-index to display type changes
between file/symlink/subproject. Also always show the file type for
symlink and subprojects to indicate that they are not normal files.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git.el: Retrieve the permissions for up-to-date files.Alexandre Julliard Sun, 6 Jan 2008 11:13:01 +0000 (12:13 +0100)

git.el: Retrieve the permissions for up-to-date files.

This allows displaying correctly the executable flag for the initial
commit, and will make it possible to show the file type for up-to-date
symlinks and subprojects.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git.el: Support for getting diffs from inside the log... Alexandre Julliard Sun, 6 Jan 2008 11:12:24 +0000 (12:12 +0100)

git.el: Support for getting diffs from inside the log-edit buffer.

Take advantage of the new log-edit feature that allows to show a diff
with C-c C-d while editing the log message.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Make the git metapackage require the same version of... James Bowes Sun, 6 Jan 2008 17:35:02 +0000 (12:35 -0500)

Make the git metapackage require the same version of the subpackages.

Without explicit version deps in the rpm spec file, 'yum update git'
effectively does nothing. Require explicit versions of the subpackages,
so that they get pulled in on an update.

Signed-off-by: James Bowes <jbowes@dangerouslyinc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

tree-walk: don't parse incorrect entriesMartin Koegler Sun, 6 Jan 2008 17:21:10 +0000 (18:21 +0100)

tree-walk: don't parse incorrect entries

The current code can access memory outside of the tree buffer in the
case of malformed tree entries.

This patch prevents this by:

* The rest of the buffer must be at least 24 bytes (at least 1 byte
mode, 1 blank, at least one byte path name, 1 NUL, 20 bytes sha1).

* Check that the last NUL (21 bytes before the end) is present.
This ensures that strlen() and get_mode() calls stay within the
buffer.

* The mode may not be empty. We have only to reject a blank at the
begin, as the rest is handled by if (c < '0' || c > '7').

* The blank is ensured by get_mode().

* The path must contain at least one character.

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Document the color.interactive semanticsJeff King Sat, 5 Jan 2008 09:57:44 +0000 (04:57 -0500)

Document the color.interactive semantics

There are two possible confusions with the color.interactive
description:

1. the short name "interactive" implies that it covers all
interactive commands; let's explicitly make it so, even
though there are no other interactive commands which
currently use it

2. Not all parts of "git add --interactive" are controlled
by color.interactive (specifically, the diffs require
tweaking color.diff). So let's clarify that it applies
only to displays and prompts.

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

add--interactive: allow diff colors without interactive... Jeff King Fri, 4 Jan 2008 08:35:21 +0000 (03:35 -0500)

add--interactive: allow diff colors without interactive colors

Users with color.diff set to true/auto will not see color in
"git add -i" unless they also set color.interactive.

This changes the semantics of color.interactive to control only the
coloring of the interaction aspect of the command and let color.diff
to control the color of hunk picker, which would arguably be more
convenient.

Old $use_color variable is now renamed to $menu_use_color to make it
clear that it is about coloring the interaction.

The "colored" subroutine now checks if the passed color is defined,
instead of checking $use_color variable, to decide if the lines should
be colored. The various variables that define colors for different
parts of the output are set or unset depending on the setting of
color.interactive and color.diff configuration variables.

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

add--interactive: remove unused diff colorsJeff King Sun, 6 Jan 2008 04:24:09 +0000 (23:24 -0500)

add--interactive: remove unused diff colors

When color support was added, we colored the diffs ourselves.
However, 4af756f3 changed this to simply run "git diff-files"
twice, keeping the colored output separately.

This makes the internal diff color variables obsolete with
one exception: when splitting hunks, we have to manually
recreate the fragment for each part of the split. Thus we
keep $fraginfo_color around to do that correctly.

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

parse_tag_buffer: don't parse invalid tagsMartin Koegler Sun, 6 Jan 2008 19:03:10 +0000 (20:03 +0100)

parse_tag_buffer: don't parse invalid tags

The current tag parsing code can access memory outside the tag buffer,
if \n are missing. This patch prevent this behaviour.

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: rename gitlink macro to linkgitDan McGee Sat, 29 Dec 2007 06:20:38 +0000 (00:20 -0600)

Documentation: rename gitlink macro to linkgit

Between AsciiDoc 8.2.2 and 8.2.3, the following change was made to the stock
Asciidoc configuration:

@@ -149,7 +153,10 @@
# Inline macros.
# Backslash prefix required for escape processing.
# (?s) re flag for line spanning.
-(?su)[\\]?(?P<name>\w(\w|-)*?):(?P<target>\S*?)(\[(?P<attrlist>.*?)\])=
+
+# Explicit so they can be nested.
+(?su)[\\]?(?P<name>(http|https|ftp|file|mailto|callto|image|link)):(?P<target>\S*?)(\[(?P<attrlist>.*?)\])=
+
# Anchor: [[[id]]]. Bibliographic anchor.
(?su)[\\]?\[\[\[(?P<attrlist>[\w][\w-]*?)\]\]\]=anchor3
# Anchor: [[id,xreflabel]]

This default regex now matches explicit values, and unfortunately in this
case gitlink was being matched by just 'link', causing the wrong inline
macro template to be applied. By renaming the macro, we can avoid being
matched by the wrong regex.

Signed-off-by: Dan McGee <dpmcgee@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: fix "gitlink::foobar[s]"Junio C Hamano Sun, 6 Jan 2008 03:10:35 +0000 (19:10 -0800)

Documentation: fix "gitlink::foobar[s]"

They should be spelled with a single colon.

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

Documentation: remove gitman.info with "make clean"Junio C Hamano Sun, 6 Jan 2008 03:09:09 +0000 (19:09 -0800)

Documentation: remove gitman.info with "make clean"

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

custom pretty format: tolerate empty e-mail addressJunio C Hamano Sun, 6 Jan 2008 12:21:07 +0000 (04:21 -0800)

custom pretty format: tolerate empty e-mail address

When e-mail address is empty (e.g. "A U Thor <>"), --pretty=format
misparsed the commit header and did not pick up the date field correctly.

Noticed by Marco, fixed slightly differently with additional sanity
check and with a test.

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

Documentation/Makefile - honor $DESTDIR for quick-insta... Mark Levedahl Sat, 5 Jan 2008 22:32:21 +0000 (17:32 -0500)

Documentation/Makefile - honor $DESTDIR for quick-install target

Signed-off-by: Mark Levedahl <mdl123@verizon.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

filter-branch: work correctly with ambiguous refnamesJunio C Hamano Sat, 5 Jan 2008 20:18:43 +0000 (12:18 -0800)

filter-branch: work correctly with ambiguous refnames

'git-filter-branch branch' could fail producing the error:
"Which ref do you want to rewrite?" if existed another branch
or tag, which name was 'branch-something' or 'something/branch'.

[jc: original report and fix were done between Dmitry Potapov
and Dscho; I rewrote it using "rev-parse --symbolic-full-name"]

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

git-rev-parse --symbolic-full-nameJunio C Hamano Sat, 5 Jan 2008 20:09:55 +0000 (12:09 -0800)

git-rev-parse --symbolic-full-name

The plumbing level can understand that the user meant
"refs/heads/master" when the user says "master" or
"heads/master", but there is no easy way for the scripts to
figure it out without duplicating the dwim_ref() logic.

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

git-stash clear: refuse to work with extra parameter... Junio C Hamano Sat, 5 Jan 2008 09:35:54 +0000 (01:35 -0800)

git-stash clear: refuse to work with extra parameter for now

Because it is so tempting to expect "git stash clear stash@{4}"
to remove the fourth element in the stash while leaving other
elements intact, we should not blindly throw away everything
upon seeing such a command.

This may change when we start using "git reflog delete" to
selectively nuke a single (or multiple, for that matter) stash
entries with such a command line.

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

git-stash: use stdout instead of stderr for non error... Marco Costalba Sat, 5 Jan 2008 08:57:24 +0000 (09:57 +0100)

git-stash: use stdout instead of stderr for non error messages

Some scripts and libraries check stderr to detect a failing command,
instead of checking the exit code. Because the output from git-status
is not primarily for machine consumption, it would not hurt to send
these messages to stdout instead and it will make it easier to drive
the command for such callers.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Update draft release notes for 1.5.4Junio C Hamano Sat, 5 Jan 2008 07:13:47 +0000 (23:13 -0800)

Update draft release notes for 1.5.4

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

git-am: Run git gc only once and not for every patch.Michael Stefaniuc Fri, 4 Jan 2008 18:59:26 +0000 (19:59 +0100)

git-am: Run git gc only once and not for every patch.

With "too many unreachable loose objects" git gc --auto will always
trigger. This clutters the output of git am and thus git rebase.

Signed-off-by: Michael Stefaniuc <mstefani@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

t/t3800: do not use a temporary file to hold expected... Junio C Hamano Sat, 5 Jan 2008 06:45:08 +0000 (22:45 -0800)

t/t3800: do not use a temporary file to hold expected result.

It is a good practice to write program output to a temporary file
during the test, as it would allow easier postmortem when the tested
program does break. But there is no benefit in writing the expected
output out to the temporary.

This actually fixes a bug in check_verify_failure() routine.
The intention of the test seems to make sure the "git mktag" command
fails, and it spits out the expected error message. But if the
command did not fail as expected, the shell function as originally
written would not have detected the failure.

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

t/t{3600,3800,5401}: do not use egrep when grep would doJunio C Hamano Sat, 5 Jan 2008 06:28:13 +0000 (22:28 -0800)

t/t{3600,3800,5401}: do not use egrep when grep would do

There is nothing _wrong_ with egrep per se, but this way we
would have less dependency on external tools.

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

t/t7001: avoid unnecessary ERE when using grepMiklos Vajna Sat, 5 Jan 2008 04:06:48 +0000 (05:06 +0100)

t/t7001: avoid unnecessary ERE when using grep

As pointed out by Junio, it's unnecessary to use "grep -E" and ".+" when we can
just use "grep" and "..*".

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

t/t7600: avoid GNUism in grepJunio C Hamano Sat, 5 Jan 2008 06:22:55 +0000 (22:22 -0800)

t/t7600: avoid GNUism in grep

Using \+ to mean "one or more" in grep without -E is a GNU
extension outside POSIX. Avoid it.

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

Merge git://repo.or.cz/git-guiJunio C Hamano Sat, 5 Jan 2008 06:03:42 +0000 (22:03 -0800)

Merge git://repo.or.cz/git-gui

* git://repo.or.cz/git-gui:
git-gui: Make commit log messages end with a newline
Added Swedish translation.
git-gui: Unconditionally use absolute paths with Cygwin
git-gui: Handle file mode changes (644->755) in diff viewer
git-gui: Move frequently used commands to the top of the context menu.

Document git-reset defaults to HEAD if no commit is... Marco Costalba Fri, 4 Jan 2008 09:53:32 +0000 (10:53 +0100)

Document git-reset defaults to HEAD if no commit is given

Signed-off by: Marco Costalba <mcostalba@gmail.com>

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

diff: remove lazy config loadingJeff King Fri, 4 Jan 2008 09:45:31 +0000 (04:45 -0500)

diff: remove lazy config loading

There is no point to this. Either:

1. The program has already loaded git_diff_ui_config, in
which case this is a noop.
2. The program didn't, which means it is plumbing that
does not _want_ git_diff_ui_config to be loaded.

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

diff: load funcname patterns in "basic" configJeff King Fri, 4 Jan 2008 09:16:14 +0000 (04:16 -0500)

diff: load funcname patterns in "basic" config

The funcname patterns influence the "comment" on @@ lines of
the diff. They are safe to use with plumbing since they
don't fundamentally change the meaning of the diff in any
way.

Since all diff users call either diff_ui_config or
diff_basic_config, we can get rid of the lazy reading of the
config.

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

add a "basic" diff config callbackJeff King Fri, 4 Jan 2008 08:59:34 +0000 (03:59 -0500)

add a "basic" diff config callback

The diff porcelain uses git_diff_ui_config to set
porcelain-ish config options, like automatically turning on
color. The plumbing specifically avoids calling this
function, since it doesn't want things like automatic color
or rename detection.

However, some diff options should be set for both plumbing
and porcelain. For example, one can still turn on color in
git-diff-files using the --color command line option. This
means we want the color config from color.diff.* (so that
once color is on, we use the user's preferred scheme), but
_not_ the color.diff variable.

We split the diff config into "ui" and "basic", where
"basic" is suitable for use by plumbing (so _most_ things
affecting the output should still go into the "ui" part).

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

receive-pack: reject invalid refnamesMartin Koegler Fri, 4 Jan 2008 19:37:17 +0000 (20:37 +0100)

receive-pack: reject invalid refnames

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Don't access line[-1] for a zero-length "line" from... Jim Meyering Fri, 4 Jan 2008 17:37:41 +0000 (18:37 +0100)

Don't access line[-1] for a zero-length "line" from fgets.

A NUL byte at beginning of file, or just after a newline
would provoke an invalid buf[-1] access in a few places.

* builtin-grep.c (cmd_grep): Don't access buf[-1].
* builtin-pack-objects.c (get_object_list): Likewise.
* builtin-rev-list.c (read_revisions_from_stdin): Likewise.
* bundle.c (read_bundle_header): Likewise.
* server-info.c (read_pack_info_file): Likewise.
* transport.c (insert_packed_refs): Likewise.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-clean: make "Would remove ..." path relative to... Junio C Hamano Fri, 4 Jan 2008 03:04:37 +0000 (19:04 -0800)

git-clean: make "Would remove ..." path relative to cwd again

The rewrite changed the output to use the path relative to the
top of the work tree without a good reason. This fixes it.

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

Fix grammar nits in documentation and in code comments.Jim Meyering Thu, 3 Jan 2008 14:18:07 +0000 (15:18 +0100)

Fix grammar nits in documentation and in code comments.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Uninline prefixcmp()Junio C Hamano Thu, 3 Jan 2008 09:23:12 +0000 (01:23 -0800)

Uninline prefixcmp()

Now the routine is an open-coded loop that avoids an extra
strlen() in the previous implementation, it got a bit too big to
be inlined. Uninlining it makes code footprint smaller but the
result still retains the avoidance of strlen() cost.

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

Update callers of check_ref_format()Junio C Hamano Wed, 2 Jan 2008 19:14:40 +0000 (11:14 -0800)

Update callers of check_ref_format()

This updates send-pack and fast-import to use symbolic constants
for checking the return values from check_ref_format(), and also
futureproof the logic in lock_any_ref_for_update() to explicitly
name the case that is usually considered an error but is Ok for
this particular use.

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

git-svn: unlink index files that were globbed, tooEric Wong Wed, 2 Jan 2008 18:10:03 +0000 (10:10 -0800)

git-svn: unlink index files that were globbed, too

commit 3157dd9e89a71e80673d0bc21b5c0630f3b1fe68 (git-svn: unlink
internal index files after operations) introduced unlinking
index files after fetching. However, this missed indices for
refs that were created by globbing branches and tags. This will
track all refs we ever touch during a fetch and unlink them at
exit time.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-svn: allow dcommit --no-rebase to commit multiple... Eric Wong Wed, 2 Jan 2008 18:09:49 +0000 (10:09 -0800)

git-svn: allow dcommit --no-rebase to commit multiple, dependent changes

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

receive-pack: check object type of sha1 before using... Martin Koegler Wed, 2 Jan 2008 07:39:21 +0000 (08:39 +0100)

receive-pack: check object type of sha1 before using them as commits

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Optimize prefixcmp()Johannes Schindelin Sat, 29 Dec 2007 19:22:14 +0000 (20:22 +0100)

Optimize prefixcmp()

Certain codepaths (notably "git log --pretty=format...") use
prefixcmp() extensively, with very short prefixes. In those cases,
calling strlen() is a wasteful operation, so avoid it.

Initial patch by Marco Costalba.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

lock_any_ref_for_update(): reject wildcard return from... Junio C Hamano Wed, 2 Jan 2008 06:33:20 +0000 (22:33 -0800)

lock_any_ref_for_update(): reject wildcard return from check_ref_format

Recent check_ref_format() returns -3 as well as -1 (general
error) and -2 (less than two levels). The caller was explicitly
checking for -1, to allow "HEAD" but still needed to disallow
bogus refs.

This introduces symbolic constants for the return values from
check_ref_format() to make them read better and more
meaningful. Normal ref creation codepath can still treat
non-zero return values as errors.

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

git-reset: refuse to do hard reset in a bare repositoryJeff King Mon, 31 Dec 2007 07:13:52 +0000 (02:13 -0500)

git-reset: refuse to do hard reset in a bare repository

It makes no sense since there is no working tree. A soft
reset should be fine, though.

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

config: handle lack of newline at end of file betterJeff King Tue, 1 Jan 2008 06:17:34 +0000 (01:17 -0500)

config: handle lack of newline at end of file better

The config parsing routines use the static global
'config_file' to store the FILE* pointing to the current
config file being parsed. The function get_next_char()
automatically converts an EOF on this file to a newline for
the convenience of its callers, and it sets config_file to
NULL to indicate that EOF was reached.

This throws away useful information, though, since some
routines want to call ftell on 'config_file' to find out
exactly _where_ the routine ended. In the case of a key
ending at EOF boundary, we ended up segfaulting in some
cases (changing that key or adding another key in its
section), or failing to provide the necessary newline
(adding a new section).

This patch adds a new flag to indicate EOF and uses that
instead of setting config_file to NULL. It also makes sure
to add newlines where necessary for truncated input. All
three included tests fail without the patch.

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

Merge branch 'maint' of git://linux-nfs.org/~bfields/gitJunio C Hamano Mon, 31 Dec 2007 17:23:55 +0000 (09:23 -0800)

Merge branch 'maint' of git://linux-nfs.org/~bfields/git

* 'maint' of git://linux-nfs.org/~bfields/git:
Documentation/user-manual.txt: fix typo
Documentation: fix remote.<name>.skipDefaultUpdate description

Documentation/user-manual.txt: fix typoGustaf Hendeby Mon, 31 Dec 2007 13:31:35 +0000 (14:31 +0100)

Documentation/user-manual.txt: fix typo

Signed-off-by: Gustaf Hendeby <hendeby@isy.liu.se>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>

Documentation: fix remote.<name>.skipDefaultUpdate... J. Bruce Fields Sat, 22 Dec 2007 00:57:16 +0000 (19:57 -0500)

Documentation: fix remote.<name>.skipDefaultUpdate description

Fix the subcommand name.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>

git-rebase -i: clean-up error check codepath.Junio C Hamano Sun, 30 Dec 2007 20:51:42 +0000 (12:51 -0800)

git-rebase -i: clean-up error check codepath.

After replaying a single change, the code performed a number of checks,
but some of them were for sanity checking, failures from which should
make the command abort, and others were checks to see if it should make
a new commit object. Stringing them together with "&&" was wrong.

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

git-rebase -i behaves better on commits with incomplete... Junio C Hamano Sun, 30 Dec 2007 20:37:59 +0000 (12:37 -0800)

git-rebase -i behaves better on commits with incomplete messages

The commit message template when squashing multiple commits is
prepared by concatenating the messages of existing commits
together. If the messages from some of them end with incomplete
lines, this would result in a suboptimal message template. Make
sure that we add a terminating LF after each commit message.

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

Fix double-free() in http-push.c:remote_exists()Grégoire Barbier Sun, 30 Dec 2007 17:15:51 +0000 (18:15 +0100)

Fix double-free() in http-push.c:remote_exists()

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

git-gui: Make commit log messages end with a newline gitgui-0.9.1Bernt Hansen Fri, 28 Dec 2007 02:15:56 +0000 (21:15 -0500)

git-gui: Make commit log messages end with a newline

Concatenating commit log messages from multiple commits works better
when all of the commits end with a clean line break.

Its good to be strict in what you create, and lenient in what you
accept, and since we're creating here, we should always try to
Do The Right Thing(tm).

Signed-off-by: Bernt Hansen <bernt@alumni.uwaterloo.ca>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Added Swedish translation.Peter Karlsson Thu, 27 Dec 2007 12:34:31 +0000 (13:34 +0100)

Added Swedish translation.

"git pull --tags": error out with a better message.Junio C Hamano Fri, 28 Dec 2007 21:58:43 +0000 (13:58 -0800)

"git pull --tags": error out with a better message.

When "git pull --tags" is run without any other arguments, the
standard error message "You told me to fetch and merge stuff but
there is nothing to merge! You might want to fix your config"
is given.

While the error may be technically correct, fixing the config
would not help, as "git pull --tags" itself tells "git fetch"
not to use the configured refspecs.

This commit makes "git pull --tags" to issue a different error
message to avoid confusion. This is merely an interim solution.

In the longer term, it would be a better approach to change the
semantics of --tags option to make "git fetch" and "git pull"
to:

(1) behave as if no --tags was given (so an explicit refspec on
the command line overrides configured ones, or no explicit
refspecs on the command line takes configured ones); but

(2) no auto-following of tags is made even when using
configured refspecs; and

(3) fetch all tags as not-for-merge entries".

Then we would not need to have this separate error message, as
the ordinary merge will happen even with the --tags option.

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

git-sh-setup: document git_editor() and get_author_iden... Miklos Vajna Fri, 28 Dec 2007 22:25:39 +0000 (23:25 +0100)

git-sh-setup: document git_editor() and get_author_ident_from_commit()

These 2 functions were missing from the manpage.

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation/git-submodule.txt: typofixJunio C Hamano Fri, 28 Dec 2007 07:29:53 +0000 (23:29 -0800)

Documentation/git-submodule.txt: typofix

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

GIT 1.5.4-rc2 v1.5.4-rc2Junio C Hamano Thu, 27 Dec 2007 02:16:52 +0000 (18:16 -0800)

GIT 1.5.4-rc2

Although everybody was quiet during the Christmas holiday, it's been
a week since -rc1, so here is -rc2.

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

Merge branch 'rs/pretty-safety'Junio C Hamano Thu, 27 Dec 2007 01:52:40 +0000 (17:52 -0800)

Merge branch 'rs/pretty-safety'

* rs/pretty-safety:
Make "--pretty=format" parser a bit more careful.

Make "--pretty=format" parser a bit more careful.René Scharfe Thu, 20 Dec 2007 12:20:15 +0000 (13:20 +0100)

Make "--pretty=format" parser a bit more careful.

When a commit message that does not have a terminating LF is
read in and the memory that was allocated to read it happens to
have a LF immediately after that, the code was not careful and
went past the terminating NUL.

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

contrib: resurrect scripted git-revert.Junio C Hamano Thu, 27 Dec 2007 01:38:00 +0000 (17:38 -0800)

contrib: resurrect scripted git-revert.

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

Merge branch 'ar/commit-cleanup'Junio C Hamano Thu, 27 Dec 2007 01:35:38 +0000 (17:35 -0800)

Merge branch 'ar/commit-cleanup'

* ar/commit-cleanup:
Allow selection of different cleanup modes for commit messages
builtin-commit: avoid double-negation in the code.
builtin-commit: fix amending of the initial commit
t7005: do not exit inside test.

Fix rewrite_diff() name quoting.Junio C Hamano Thu, 27 Dec 2007 01:13:36 +0000 (17:13 -0800)

Fix rewrite_diff() name quoting.

This moves the logic to quote two paths (prefix + path) in
C-style introduced in the previous commit from the
dump_quoted_path() in combine-diff.c to quote.c, and uses it to
fix rewrite_diff() that never C-quoted the pathnames correctly.

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

combine-diff: Fix path quotingJunio C Hamano Thu, 27 Dec 2007 00:51:19 +0000 (16:51 -0800)

combine-diff: Fix path quoting

Earlier when showing combined diff, the filenames on the ---/+++
header lines were quoted incorrectly. a/ (or b/) prefix was
output literally and then the path was output, with c-quoting.

This fixes the quoting logic, and while at it, adjusts the code
to use the customizable prefix (a_prefix and b_prefix)
introduced recently.

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

Fix documentation of --first-parent in git-log and... Junio C Hamano Thu, 27 Dec 2007 00:04:05 +0000 (16:04 -0800)

Fix documentation of --first-parent in git-log and copy it to git-rev-list

Credit goes to Avi Kivity for noticing the lack of description in
rev-list manual page.

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

Fix "git log --diff-filter" bugArjen Laarhoven Tue, 25 Dec 2007 11:06:47 +0000 (12:06 +0100)

Fix "git log --diff-filter" bug

In commit b7bb760d5ed4881422673d32f869d140221d3564 (Fix revision
log diff setup, avoid unnecessary diff generation) an optimization was
made to avoid unnecessary diff generation. This was partly fixed in
99516e35d096f41e7133cacde8fbed8ee9a3ecd0 (Fix embarrassing "git log
--follow" bug). The '--diff-filter' option also needs the diff machinery
in action.

Signed-off-by: Arjen Laarhoven <arjen@yaph.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Improve error messages when int/long cannot be parsed... Shawn O. Pearce Tue, 25 Dec 2007 07:18:05 +0000 (02:18 -0500)

Improve error messages when int/long cannot be parsed from config

If a config file has become mildly corrupted due to a missing LF
we may discover some other option joined up against the end of a
numeric value. For example:

[section]
number = 1auto

where the "auto" flag was meant to occur on the next line, below
"number", but the missing LF has caused it to no longer be its
own option. Instead the word "auto" is parsed as a 'unit factor'
for the value of "number".

Before this change we got the confusing error message:

fatal: unknown unit: 'auto'

which told us nothing about where the problem appeared. Now we get:

fatal: bad config value for 'aninvalid.unit'

which at least points the user in the right direction of where to
search for the incorrectly formatted configuration file.

Noticed by erikh on #git, which received the original error from
a simple `git checkout -b` due to a midly corrupted config.

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

cvsimport: die on cvsps errorsJeff King Mon, 24 Dec 2007 03:08:19 +0000 (22:08 -0500)

cvsimport: die on cvsps errors

We were not previously checking the exit status of cvsps at
all. If it exited before producing any useful output, we
ended up with an empty import, which caused a spew of
confusing error messages from other parts of git:

$ git-cvsimport foo
Initialized empty Git repository in ...
some error from cvsps
fatal: refs/heads/origin: not a valid SHA1
fatal: master: not a valid SHA1
warning: You appear to be on a branch yet to be born.
warning: Forcing checkout of HEAD.
fatal: just how do you expect me to merge 0 trees?
checkout failed: 256

Now we get:

$ git-cvsimport foo
Initialized empty Git repository in ...
some error from cvsps
git-cvsimport: fatal: cvsps reported error

Signed-off-by: Jeff King <peff@peff.net>
Acked-by: Martin Langhoff <martin@catalyst.net.nz>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

shortlog manpage documentation: work around asciidoc... Gustaf Hendeby Sun, 23 Dec 2007 18:37:01 +0000 (10:37 -0800)

shortlog manpage documentation: work around asciidoc markup issues

We wanted to have a list in which one (and the sole, as it happen to
be) item in it is ".mailmap", but do not seem to be able to convince
AsciiDoc to format it correctly for manpages. Reformat it into a
paragraph that describes the said file to work around the issue.

Signed-off-by: Gustaf Hendeby <hendeby@isy.liu.se>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: describe 'union' low-level merge driverJunio C Hamano Sun, 23 Dec 2007 07:14:59 +0000 (23:14 -0800)

Documentation: describe 'union' low-level merge driver

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

Allow selection of different cleanup modes for commit... Alex Riesen Sat, 22 Dec 2007 18:46:24 +0000 (19:46 +0100)

Allow selection of different cleanup modes for commit messages

Although we traditionally stripped away excess blank lines, trailing
whitespaces and lines that begin with "#" from the commit log message,
sometimes the message just has to be the way user wants it.

For instance, a commit message template can contain lines that begin with
"#", the message must be kept as close to its original source as possible
if you are converting from a foreign SCM, or maybe the message has a shell
script including its comments for future reference.

The cleanup modes are default, verbatim, whitespace and strip. The
default mode depends on if the message is being edited and will either
strip whitespace and comments (if editor active) or just strip the
whitespace (for where the message is given explicitely).

Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

builtin-commit: avoid double-negation in the code.Junio C Hamano Sun, 23 Dec 2007 03:25:37 +0000 (19:25 -0800)

builtin-commit: avoid double-negation in the code.

The flag no_edit meant "we have got final message from the user
and will not editing it any further", but there were quite a few
places that needed to check !no_edit. Rename the variable to
use_editor and reverse the logic everywhere.

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

builtin-commit: fix amending of the initial commitJunio C Hamano Sun, 23 Dec 2007 03:22:29 +0000 (19:22 -0800)

builtin-commit: fix amending of the initial commit

When amending initial commit without editor, the command
incorrectly barfed because the check to see if there is anything
to commit referenced the non-existent HEAD^1.

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

t7005: do not exit inside test.Junio C Hamano Sun, 23 Dec 2007 03:14:33 +0000 (19:14 -0800)

t7005: do not exit inside test.

The way to signal failure is to leave non-zero in $?, not abort
the entire test.

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

Don't dereference NULL upon lookup failure.Jim Meyering Fri, 21 Dec 2007 10:56:32 +0000 (11:56 +0100)

Don't dereference NULL upon lookup failure.

Instead, signal the error just like the case we do upon encountering
an object with an unknown type.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Emit helpful status for accidental "git stash" saveWincent Colaiuta Sat, 22 Dec 2007 17:31:25 +0000 (18:31 +0100)

Emit helpful status for accidental "git stash" save

If the user types "git stash" mistakenly thinking that this will list
their stashes he/she may be surprised to see that it actually saved
a new stash and reset their working tree and index.

In the worst case they might not know how to recover the state. So
help them by telling them exactly what was saved and also how to
restore it immediately.

Signed-off-by: Wincent Colaiuta <win@wincent.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

parse-options: Add a gitcli(5) man page.Pierre Habouzit Thu, 13 Dec 2007 10:20:01 +0000 (11:20 +0100)

parse-options: Add a gitcli(5) man page.

This page should hold every information about the git ways to parse command
lines, and best practices to be used for scripting.

Signed-off-by: Pierre Habouzit <madcoder@debian.org>

Force the sticked form for options with optional arguments.Pierre Habouzit Fri, 21 Dec 2007 10:41:41 +0000 (11:41 +0100)

Force the sticked form for options with optional arguments.

This forbids "git tag -n <number> -l" we allowed earlier, so
adjust t7004 while at it.

Signed-off-by: Pierre Habouzit <madcoder@debian.org>

Fix $EDITOR regression introduced by rewrite in C.Junio C Hamano Sat, 22 Dec 2007 17:45:10 +0000 (09:45 -0800)

Fix $EDITOR regression introduced by rewrite in C.

When git-tag and git-commit launches the editor, they used to
honor EDITOR="editor -options args..." but recent rewrite in C
insisted on $EDITOR to be the path to the editor executable.

This restores the older behaviour.

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

Small comment fix for git-cvsimport.Stefan Sperling Fri, 21 Dec 2007 15:57:26 +0000 (16:57 +0100)

Small comment fix for git-cvsimport.

Signed-off-by: Stefan Sperling <stsp@stsp.name>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Make git send-email accept $EDITOR with argumentsGustaf Hendeby Sat, 22 Dec 2007 00:40:52 +0000 (01:40 +0100)

Make git send-email accept $EDITOR with arguments

Currently git send-email does not accept $EDITOR with arguments, eg,
emacs -nw, when starting an editor to produce a cover letter. This
patch changes this by letting the shell handle the option parsing.

Signed-off-by: Gustaf Hendeby <hendeby@isy.liu.se>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

git-tag: fix -l switch handling regression.Pierre Habouzit Fri, 21 Dec 2007 10:50:58 +0000 (11:50 +0100)

git-tag: fix -l switch handling regression.

The command itself takes an optional <pattern> argument that
limits the shown tags to the ones that match when in listing
mode that is triggered with '-l' option. The <pattern> is not
an optional option-argument to '-l'.

With this fix, "git tag -l -n 4 v0.99" works as expected.

It also removes a few bogus tests in t7004.

Signed-off-by: Pierre Habouzit <madcoder@debian.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Documentation: ls-files -v is about "assume unchanged".Junio C Hamano Fri, 21 Dec 2007 21:50:58 +0000 (13:50 -0800)

Documentation: ls-files -v is about "assume unchanged".

And refer the reader to update-index documentation.

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

Re(-re)*fix trim_common_tail()Linus Torvalds Fri, 21 Dec 2007 04:22:46 +0000 (20:22 -0800)

Re(-re)*fix trim_common_tail()

The tar-ball and the git archive itself is fine, but yes, the diff from
2.6.23 to 2.6.24-rc6 is bad. It's the "trim_common_tail()" optimization
that has caused way too much pain.

Very interesting breakage. The patch was actually "correct" in a (rather
limited) technical sense, but the context at the end was missing because
while the trim_common_tail() code made sure to keep enough common context
to allow a valid diff to be generated, the diff machinery itself could
decide that it could generate the diff differently than the "obvious"
solution.

Thee sad fact is that the git optimization (which is very important for
"git blame", which needs no context), is only really valid for that one
case where we really don't need any context.

[jc: since this is shared with "git diff -U0" codepath, context recovery
to the end of line needs to be done even for zero context case.]

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

gitweb: fix whitespace in config_to_multi (indent with... Jakub Narebski Thu, 20 Dec 2007 09:48:09 +0000 (10:48 +0100)

gitweb: fix whitespace in config_to_multi (indent with tab)

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

everyday: replace 'prune' and 'repack' with 'gc'Miklos Vajna Thu, 20 Dec 2007 14:05:45 +0000 (15:05 +0100)

everyday: replace 'prune' and 'repack' with 'gc'

In everyday tasks, "repack -a -d -f" won't be used, so there
is not much point mentioning "repack". By showing the --prune
option to "gc", we can do without mentioning "git prune", too.

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

Remove old generated files from .gitignore.Charles Bailey Thu, 20 Dec 2007 13:00:40 +0000 (13:00 +0000)

Remove old generated files from .gitignore.

Some entries in .gitignore are obselete. These should be cleaned up
just for the sake of general tidiness and so that any developers who
have a working tree that was moved forward without a clean know that
they have old stuff in their work tree.

Signed-off-by: Charles Bailey <charles@hashpling.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

clean up 1.5.4 release notesJeff King Thu, 20 Dec 2007 12:11:25 +0000 (07:11 -0500)

clean up 1.5.4 release notes

Mostly typo and small grammatical fixes with one or two rewordings for
clarity. But note the important fix for status.relativepaths.

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