gitweb.git
Merge git://git.kernel.org/pub/scm/gitk/gitkShawn O. Pearce Sun, 21 Oct 2007 03:41:20 +0000 (23:41 -0400)

Merge git://git.kernel.org/pub/scm/gitk/gitk

* git://git.kernel.org/pub/scm/gitk/gitk:
gitk: Fix "can't unset prevlines(...)" Tcl error
gitk: Avoid an error when cherry-picking if HEAD has moved on
gitk: Check that we are running on at least Tcl/Tk 8.4
gitk: Do not pick up file names of "copy from" lines
gitk: Add support for OS X mouse wheel
gitk: disable colours when calling git log

gitk: Fix "can't unset prevlines(...)" Tcl errorPaul Mackerras Sun, 21 Oct 2007 02:58:42 +0000 (12:58 +1000)

gitk: Fix "can't unset prevlines(...)" Tcl error

This fixes the error reported by Michele Ballabio, where gitk will
throw a Tcl error "can't unset prevlines(...)" when displaying a
commit that has a parent commit listed more than once, and the commit
is the first child of that parent.

The problem was basically that we had two variables, prevlines and
lineends, and were relying on the invariant that prevlines($id) was
set iff $id was in the lineends($r) list for some $r. But having
a duplicate parent breaks that invariant since we end up with the
parent listed twice in lineends.

This fixes it by simplifying the logic to use only a single variable,
lineend. It also rearranges things a little so that we don't try to
draw the line for the duplicated parent twice.

Signed-off-by: Paul Mackerras <paulus@samba.org>

Deduce exec_path also from calls to git with a relative... Johannes Schindelin Sat, 20 Oct 2007 07:21:34 +0000 (08:21 +0100)

Deduce exec_path also from calls to git with a relative path

There is already logic in the git wrapper to deduce the exec_path from
argv[0], when the git wrapper was called with an absolute path. Extend
that logic to handle relative paths as well.

For example, when you call "../../hello/world/git", it will not turn
"../../hello/world" into an absolute path, and use that.

Initial implementation by Scott R Parish.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-p4 support for perforce renames.Chris Pettitt Tue, 16 Oct 2007 05:15:06 +0000 (22:15 -0700)

git-p4 support for perforce renames.

The current git-p4 implementation does support file renames. However, because
it does not use the "p4 integrate" command, the history for the renamed file is
not linked to the new file.

This changeset adds support for perforce renames with the integrate command.
Currently this feature is only enabled when calling git-p4 submit with the -M
option. This is intended to look and behave similar to the "detect renames"
feature of other git commands.

The following sequence is used for renamed files:

p4 integrate -Dt x x'
p4 edit x'
rm x'
git apply
p4 delete x

By default, perforce will not allow an integration with a target file that has
been deleted. That is, if x' in the example above is the name of a previously
deleted file then perforce will fail the integrate. The -Dt option tells
perforce to allow the target of integrate to be a previously deleted file.

Signed-off-by: Chris Pettitt <cpettitt@gmail.com>
Signed-off-by: Simon Hausmann <simon@lst.de>

git-p4: When skipping a patch as part of "git-p4 submit... Simon Hausmann Thu, 13 Sep 2007 20:10:18 +0000 (22:10 +0200)

git-p4: When skipping a patch as part of "git-p4 submit" make sure we correctly revert to the previous state of the files using "p4 revert".

Signed-off-by: Simon Hausmann <simon@lst.de>

gitk: Avoid an error when cherry-picking if HEAD has... Paul Mackerras Sat, 20 Oct 2007 12:10:52 +0000 (22:10 +1000)

gitk: Avoid an error when cherry-picking if HEAD has moved on

This fixes an error reported by Adam Piątyszek: if the current HEAD
is not in the graph that gitk knows about when we do a cherry-pick
using gitk, then gitk hits an error when trying to update its
internal representation of the topology. This avoids the error by
not doing that update if the HEAD before the cherry-pick was a
commit that gitk doesn't know about.

Signed-off-by: Paul Mackerras <paulus@samba.org>

gitk: Check that we are running on at least Tcl/Tk 8.4Paul Mackerras Sat, 20 Oct 2007 11:21:03 +0000 (21:21 +1000)

gitk: Check that we are running on at least Tcl/Tk 8.4

This checks that we have Tcl/Tk 8.4 or later, and puts up an error
message in a window and quits if not.

This was prompted by a patch submitted by Steffen Prohaska, but is
done a bit differently (this uses package require rather than
looking at [info tclversion], and uses show_error to display the
error rather than printing it to stderr).

Signed-off-by: Paul Mackerras <paulus@samba.org>

gitk: Do not pick up file names of "copy from" linesJohannes Sixt Tue, 2 Oct 2007 14:16:54 +0000 (16:16 +0200)

gitk: Do not pick up file names of "copy from" lines

A file copy would be detected only if the original file was modified in the
same commit. This implies that there will be a patch listed under the
original file name, and we would expect that clicking the original file
name in the file list warps the patch window to that file's patch. (If the
original file was not modified, the copy would not be detected in the first
place, the copied file would be listed as "new file", and this whole matter
would not apply.)

However, if the name of the copy is sorted after the original file's patch,
then the logic introduced by commit d1cb298b0b (which picks up the link
information from the "copy from" line) would overwrite the link
information that is already present for the original file name, which was
parsed earlier. Hence, this patch reverts part of said commit.

Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

gitk: Add support for OS X mouse wheelJonathan del Strother Mon, 15 Oct 2007 09:33:07 +0000 (10:33 +0100)

gitk: Add support for OS X mouse wheel

(Väinö Järvelä supplied this patch a while ago for 1.5.2. It no longer
applied cleanly, so I'm reposting it.)

MacBook doesn't seem to recognize MouseRelease-4 and -5 events, at all.
So i added a support for the MouseWheel event, which i limited to Tcl/tk
aqua, as i couldn't test it neither on Linux or Windows. Tcl/tk needs to
be updated from the version that is shipped with OS X 10.4 Tiger, for
this patch to work.

Signed-off-by: Jonathan del Strother <jon.delStrother@bestbefore.tv>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'maint'Shawn O. Pearce Fri, 19 Oct 2007 06:18:21 +0000 (02:18 -0400)

Merge branch 'maint'

* maint:
Paper bag fix diff invocation in 'git stash show'

Paper bag fix diff invocation in 'git stash show'Shawn O. Pearce Fri, 19 Oct 2007 06:18:14 +0000 (02:18 -0400)

Paper bag fix diff invocation in 'git stash show'

In 89d750bf6fa025edeb31ad258cdd09a27a5c02fa I got a little too
aggressive with changing "git diff" to "git diff-tree". This is
shown to the user, who expects to see a full diff on their console,
and will want to see the output of their custom diff drivers (if
any) as the whole point of this call site is to show the diff to
the end-user.

Noticed by Johannes Sixt <j.sixt@viscovery.net>.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'maint'Shawn O. Pearce Fri, 19 Oct 2007 05:18:55 +0000 (01:18 -0400)

Merge branch 'maint'

* maint:
Further 1.5.3.5 fixes described in release notes
Avoid invoking diff drivers during git-stash
attr: fix segfault in gitattributes parsing code
Define NI_MAXSERV if not defined by operating system
Ensure we add directories in the correct order
Avoid scary errors about tagged trees/blobs during git-fetch

Further 1.5.3.5 fixes described in release notesShawn O. Pearce Fri, 19 Oct 2007 05:18:29 +0000 (01:18 -0400)

Further 1.5.3.5 fixes described in release notes

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Documentation/git-gc: improve description of --autoJeff King Fri, 19 Oct 2007 02:05:10 +0000 (22:05 -0400)

Documentation/git-gc: improve description of --auto

This patch tries to make the description of --auto a little
more clear for new users, especially those referred by the
"git-gc --auto" notification message.

It also cleans up some grammatical errors and typos in the
original description, as well as rewording for clarity.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Documentation/git-gc: explain --auto in descriptionJeff King Fri, 19 Oct 2007 02:01:11 +0000 (22:01 -0400)

Documentation/git-gc: explain --auto in description

Now that git-gc --auto tells the user to look at the man
page, it makes sense to mention the auto behavior near the
top (since this is likely to be most users' first exposure
to git-gc).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-gc: improve wording of --auto notificationJeff King Fri, 19 Oct 2007 01:12:11 +0000 (21:12 -0400)

git-gc: improve wording of --auto notification

The previous message had too much of a "boy, you should
really turn off this annoying gc" flair to it. Instead,
let's make sure the user understands what is happening, that
they can run it themselves, and where to find more info.

Suggested by Brian Gernhardt.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Avoid invoking diff drivers during git-stashShawn O. Pearce Fri, 19 Oct 2007 01:28:43 +0000 (21:28 -0400)

Avoid invoking diff drivers during git-stash

git-stash needs to restrict itself to plumbing when running automated
diffs as part of its operation as the user may have configured a
custom diff driver that opens an interactive UI for certain/all
files. Doing that during scripted actions is very unfriendly to
the end-user and may cause git-stash to fail to work.

Reported by Johannes Sixt

Acked-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

attr: fix segfault in gitattributes parsing codeSteffen Prohaska Thu, 18 Oct 2007 20:02:35 +0000 (22:02 +0200)

attr: fix segfault in gitattributes parsing code

git may segfault if gitattributes contains an invalid
entry. A test is added to t0020 that triggers the segfault.
The parsing code is fixed to avoid the crash.

Signed-off-by: Steffen Prohaska <prohaska@zib.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Define NI_MAXSERV if not defined by operating systemPatrick Welche Thu, 18 Oct 2007 17:17:39 +0000 (18:17 +0100)

Define NI_MAXSERV if not defined by operating system

I found I needed NI_MAXSERV as it is defined in netdb.h, which is
not included by daemon.c. Rather than including the whole header
we can define a reasonable fallback value.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Ensure we add directories in the correct orderAlex Bennee Thu, 18 Oct 2007 16:15:44 +0000 (17:15 +0100)

Ensure we add directories in the correct order

CVS gets understandably upset if you try and add a subdirectory
before it's parent directory. This patch fixes that.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Avoid scary errors about tagged trees/blobs during... Linus Torvalds Thu, 18 Oct 2007 23:24:47 +0000 (16:24 -0700)

Avoid scary errors about tagged trees/blobs during git-fetch

Ok, what is going on is:

- append_fetch_head() looks up the SHA1 for all heads (including tags):

if (get_sha1(head, sha1))
return error("Not a valid object name: %s", head);

- it then wants to check if it's a candidate for merging (because
fetching also does the whole "list which heads to merge" in case
it is going to be part of a "pull"):

commit = lookup_commit_reference(sha1);
if (!commit)
not_for_merge = 1;

- and that "lookup_commit_reference()" is just very vocal about the
case where it fails. It really shouldn't be, and it shouldn't
affect the actual end result, but that basically explains why
you get that scary warning.

In short, the warning is just bogus, and should be harmless, but
I agree that it's ugly. I think the appended patch should fix it.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'jc/am-quiet'Shawn O. Pearce Thu, 18 Oct 2007 07:45:05 +0000 (03:45 -0400)

Merge branch 'jc/am-quiet'

* jc/am-quiet:
git-am: fix typo in the previous one.
git-am: make the output quieter.

Merge branch 'maint'Shawn O. Pearce Thu, 18 Oct 2007 07:11:17 +0000 (03:11 -0400)

Merge branch 'maint'

* maint:
Yet more 1.5.3.5 fixes mentioned in release notes
cvsserver: Use exit 1 instead of die when req_Root fails.
git-blame shouldn't crash if run in an unmerged tree
git-config: print error message if the config file cannot be read
fixing output of non-fast-forward output of post-receive-email

Yet more 1.5.3.5 fixes mentioned in release notesShawn O. Pearce Thu, 18 Oct 2007 07:11:03 +0000 (03:11 -0400)

Yet more 1.5.3.5 fixes mentioned in release notes

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

cvsserver: Use exit 1 instead of die when req_Root... Brian Gernhardt Wed, 17 Oct 2007 14:05:47 +0000 (10:05 -0400)

cvsserver: Use exit 1 instead of die when req_Root fails.

This was causing test failures because die was exiting 255.

Signed-off-by: Brian Gernhardt <benji@silverinsanity.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-blame shouldn't crash if run in an unmerged treeLinus Torvalds Thu, 18 Oct 2007 06:31:30 +0000 (02:31 -0400)

git-blame shouldn't crash if run in an unmerged tree

If we are in the middle of resolving a merge conflict there may be
one or more files whose entries in the index represent an unmerged
state (index entries in the higher-order stages).

Attempting to run git-blame on any file in such a working directory
resulted in "fatal: internal error: ce_mode is 0" as we use the magic
marker for an unmerged entry is 0 (set up by things like diff-lib.c's
do_diff_cache() and builtin-read-tree.c's read_tree_unmerged())
and the ce_match_stat_basic() function gets upset about this.

I'm not entirely sure that the whole "ce_mode = 0" case is a good
idea to begin with, and maybe the right thing to do is to remove
that horrid freakish special case, but removing the internal error
seems to be the simplest fix for now.

Linus

[sp: Thanks to Björn Steinbrink for the test case]

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Teach core.autocrlf to 'git blame'Marius Storm-Olsen Wed, 17 Oct 2007 10:57:47 +0000 (12:57 +0200)

Teach core.autocrlf to 'git blame'

Pass the fake commit through convert_to_git, so that the
file is adjusted for local line-ending convention.

Signed-off-by: Marius Storm-Olsen <marius@trolltech.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-config: print error message if the config file... Gerrit Pape Fri, 12 Oct 2007 11:40:57 +0000 (11:40 +0000)

git-config: print error message if the config file cannot be read

Instead of simply exiting with 255, print an error message including
the reason why a config file specified through --file cannot be opened
or read.

The problem was noticed by Joey Hess, reported through
http://bugs.debian.org/445208

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'lt/diff-rename'Shawn O. Pearce Thu, 18 Oct 2007 05:30:19 +0000 (01:30 -0400)

Merge branch 'lt/diff-rename'

* lt/diff-rename:
optimize diffcore-delta by sorting hash entries.

Add a message explaining that automatic GC is about... koreth@midwinter.com Thu, 18 Oct 2007 04:41:43 +0000 (21:41 -0700)

Add a message explaining that automatic GC is about to start

Signed-off-by: Steven Grimm <koreth@midwinter.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

fixing output of non-fast-forward output of post-receiv... Robert Schiele Wed, 17 Oct 2007 22:27:51 +0000 (00:27 +0200)

fixing output of non-fast-forward output of post-receive-email

post-receive-email has one place where the variable fast_forward is not
spelled correctly. At the same place the logic was reversed. The
combination of both bugs made the script work correctly for fast-forward
commits but not for non-fast-forward ones. This change fixes this to
be correct in both cases.

Signed-off-by: Robert Schiele <rschiele@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

gitk: disable colours when calling git logSam Vilain Tue, 16 Oct 2007 22:33:04 +0000 (11:33 +1300)

gitk: disable colours when calling git log

If the user specifies 'diff.color = 1' in their configuration file,
then gitk will not start. Disable colours when calling git log.

Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: simplify the handling of fatal errorsBenoit Sigoure Tue, 16 Oct 2007 14:36:52 +0000 (16:36 +0200)

git-svn: simplify the handling of fatal errors

* git-svn.perl (&fatal): Append the newline at the end of the error
message.
Adjust all callers.

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: add git svn proplistBenoit Sigoure Tue, 16 Oct 2007 14:36:51 +0000 (16:36 +0200)

git-svn: add git svn proplist

This allows one to easily retrieve a list of svn properties from within
git-svn without requiring svn or knowing the URL of a repository.

* git-svn.perl (%cmd): Add the command `proplist'.
(&cmd_proplist): New.
* t/t9101-git-svn-props.sh: Test git svn proplist.

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: add git svn propgetBenoit Sigoure Tue, 16 Oct 2007 14:36:50 +0000 (16:36 +0200)

git-svn: add git svn propget

This allows one to easily retrieve a single SVN property from within
git-svn without requiring svn or remembering the URL of a repository

* git-svn.perl (%cmd): Add the new command `propget'.
($cmd_dir_prefix): New global.
(&get_svnprops): New helper.
(&cmd_propget): New. Use &get_svnprops.
* t/t9101-git-svn-props.sh: Add a test case for propget.

[ew: make sure the rev-parse --show-prefix call doesn't break
the `git-svn clone' command]

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: implement git svn create-ignoreBenoit Sigoure Tue, 16 Oct 2007 14:36:49 +0000 (16:36 +0200)

git-svn: implement git svn create-ignore

git svn create-ignore (to create one .gitignore per directory
from the svn:ignore properties. This has the disadvantage of
committing the .gitignore during the next dcommit, but when you
import a repo with tons of ignores (>1000), using git svn show-ignore
to build .git/info/exclude is *not* a good idea, because things like
git-status will end up doing >1000 fnmatch *per file* in the repo,
which leads to git-status taking more than 4s on my Core2Duo 2Ghz 2G
RAM)

* git-svn.perl (%cmd): Add the new command `create-ignore'.
(&cmd_create_ignore): New.
* t/t9101-git-svn-props.sh: Adjust the test-case for show-ignore and
add a test case for create-ignore.

[ew: added commit message from
<05CAB148-56ED-4FF1-8AAB-4BA2A0B70C2C@lrde.epita.fr> ]

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: add a generic tree traversal to fetch SVN... Benoit Sigoure Tue, 16 Oct 2007 14:36:48 +0000 (16:36 +0200)

git-svn: add a generic tree traversal to fetch SVN properties

* git-svn.perl (&traverse_ignore): Remove.
(&prop_walk): New.
(&cmd_show_ignore): Use prop_walk.

[ew: This will ease the implementation of the `create-ignore',
`propget', and `proplist' commands]

Signed-off-by: Benoit Sigoure <tsuna@lrde.epita.fr>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

gitweb: speed up project listing on large work trees... Luke Lu Wed, 17 Oct 2007 03:45:25 +0000 (20:45 -0700)

gitweb: speed up project listing on large work trees by limiting find depth

Signed-off-by: Luke Lu <git@vicaya.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'maint'Shawn O. Pearce Wed, 17 Oct 2007 03:32:03 +0000 (23:32 -0400)

Merge branch 'maint'

* maint:
Document additional 1.5.3.5 fixes in release notes
Avoid 'expr index' on Mac OS X as it isn't supported
filter-branch: update current branch when rewritten
fix filter-branch documentation
helpful error message when send-pack finds no refs in common.
Fix setup_git_directory_gently() with relative GIT_DIR & GIT_WORK_TREE
Correct typos in release notes for 1.5.3.5

Document additional 1.5.3.5 fixes in release notesShawn O. Pearce Wed, 17 Oct 2007 03:31:58 +0000 (23:31 -0400)

Document additional 1.5.3.5 fixes in release notes

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-cvsexportcommit.perl: git-apply no longer needs... Michael Witten Tue, 16 Oct 2007 08:08:14 +0000 (04:08 -0400)

git-cvsexportcommit.perl: git-apply no longer needs --binary

Signed-off-by: Michael Witten <mfwitten@mit.edu>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Avoid 'expr index' on Mac OS X as it isn't supportedShawn O. Pearce Wed, 17 Oct 2007 01:33:11 +0000 (21:33 -0400)

Avoid 'expr index' on Mac OS X as it isn't supported

This fixes git-instaweb so it can start an httpd without warning
about an invalid test command. Yes its ugly, but its also quite
portable.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

filter-branch: update current branch when rewrittenJohannes Schindelin Wed, 17 Oct 2007 02:23:10 +0000 (03:23 +0100)

filter-branch: update current branch when rewritten

Earlier, "git filter-branch --<options> HEAD" would not update the
working tree after rewriting the branch. This commit fixes it.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

fix filter-branch documentationJohannes Schindelin Wed, 17 Oct 2007 02:22:25 +0000 (03:22 +0100)

fix filter-branch documentation

The man page for filter-branch still talked about writing the result
to the branch "newbranch". This is hopefully the last place where the
old behaviour was described.

Noticed by Bill Lear.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

helpful error message when send-pack finds no refs... Andrew Clausen Tue, 16 Oct 2007 21:16:05 +0000 (17:16 -0400)

helpful error message when send-pack finds no refs in common.

Signed-off-by: Andrew Clausen <clausen@econ.upenn.edu>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: use "no warnings 'once'" to disable false... Eygene Ryabinkin Mon, 15 Oct 2007 07:19:12 +0000 (11:19 +0400)

git-svn: use "no warnings 'once'" to disable false-positives

Some variables coming from the Subversion's Perl bindings are used
in our code only once, so the interpreter warns us about it. These
warnings are false-positives, because the variables themselves are
initialized in the binding's guts, that are made by SWIG.

Credits to Sam Vilain for his note about "no warnings 'once'".

[ew: minor formatting change]

Signed-off-by: Eygene Ryabinkin <rea-git@codelabs.ru>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Fix setup_git_directory_gently() with relative GIT_DIR... Johannes Schindelin Tue, 16 Oct 2007 23:37:36 +0000 (00:37 +0100)

Fix setup_git_directory_gently() with relative GIT_DIR & GIT_WORK_TREE

There are a few programs, such as config and diff, which allow running
without a git repository. Therefore, they have to call
setup_git_directory_gently().

However, when GIT_DIR and GIT_WORK_TREE were set, and the current
directory was a subdirectory of the work tree,
setup_git_directory_gently() would return a bogus NULL prefix.

This patch fixes that.

Noticed by REPLeffect on IRC.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Correct typos in release notes for 1.5.3.5Shawn O. Pearce Wed, 17 Oct 2007 00:09:21 +0000 (20:09 -0400)

Correct typos in release notes for 1.5.3.5

Noticed by Michele Ballabio.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

cvsexportcommit: get rid of temporary directoryJohannes Schindelin Mon, 15 Oct 2007 22:32:45 +0000 (23:32 +0100)

cvsexportcommit: get rid of temporary directory

Since commit e86ad71fe5f53ae4434566bd09ea4256090e5a3a we do not use
a temporary directory in cvsexportcommit. So there is no need to set
one up.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-rebase: document suppression of duplicate commitsJeff King Mon, 15 Oct 2007 04:47:30 +0000 (00:47 -0400)

git-rebase: document suppression of duplicate commits

git-rebase uses format-patch's --ignore-if-in-upstream
option, but we never document the user-visible behavior. The
example is placed near the top of the example list rather
than at the bottom because it is:
a. a simple example
b. a reasonably common scenario for many projects (mail
some patches which get accepted upstream, then rebase)

[sp: Corrected direction of 'HEAD..<upstream>' set comparsion]

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Documentation/git-tag.txt: Document how to backdate... Michael Olson Mon, 15 Oct 2007 17:51:30 +0000 (13:51 -0400)

Documentation/git-tag.txt: Document how to backdate tags

Added a new section beneath "On Automatic following" called "On
Backdating Tags". This includes an explanation of when to use this
method, a brief explanation of the kind of date that can be used in
GIT_AUTHOR_DATE, and an example invocation of git-tag using a custom
setting of GIT_AUTHOR_DATE.

[sp: Corrected s/you/your/, noticed by Jeff King]

Signed-off-by: Michael W. Olson <mwolson@gnu.org>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-rebase--interactive.sh: Quote arguments to testMichael W. Olson Mon, 15 Oct 2007 17:48:27 +0000 (13:48 -0400)

git-rebase--interactive.sh: Quote arguments to test

If /bin/sh is /bin/dash, then the script will display an error if
$parent_sha1 is undefined. This patch works fixes the issue by
quoting both arguments to `test'. Arguments composed solely of
variable expansions should always be quoted, unless we know for
certain that the contents are defined.

Signed-off-by: Michael W. Olson <mwolson@gnu.org>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Merge branch 'maint'Shawn O. Pearce Tue, 16 Oct 2007 02:31:47 +0000 (22:31 -0400)

Merge branch 'maint'

* maint:
Whip post 1.5.3.4 maintenance series into shape.
rebase -i: use diff plumbing instead of porcelain
Do not remove distributed configure script
git-archive: document --exec
git-reflog: document --verbose
git-config: handle --file option with relative pathname properly
clear_commit_marks(): avoid deep recursion
git add -i: Remove unused variables
git add -i: Fix parsing of abbreviated hunk headers
git-config: don't silently ignore options after --list
Clean up "git log" format with DIFF_FORMAT_NO_OUTPUT
Fix embarrassing "git log --follow" bug

Conflicts:

RelNotes
git-rebase--interactive.sh

Whip post 1.5.3.4 maintenance series into shape.Shawn O. Pearce Tue, 16 Oct 2007 02:29:50 +0000 (22:29 -0400)

Whip post 1.5.3.4 maintenance series into shape.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Minor usage update in setgitperms.perlJosh England Tue, 9 Oct 2007 16:04:42 +0000 (10:04 -0600)

Minor usage update in setgitperms.perl

Signed-off-by: Josh England <jjengla@sandia.gov>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

manual: use 'URL' instead of 'url'.Ralf Wildenhues Tue, 9 Oct 2007 21:03:55 +0000 (23:03 +0200)

manual: use 'URL' instead of 'url'.

Just for consistency, use the spelling URL everywhere.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

manual: add some markup.Ralf Wildenhues Tue, 9 Oct 2007 21:03:11 +0000 (23:03 +0200)

manual: add some markup.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

manual: Fix example finding commits referencing given... Ralf Wildenhues Tue, 9 Oct 2007 21:02:19 +0000 (23:02 +0200)

manual: Fix example finding commits referencing given content.

If I'm handed a file, then it typically lives outside the
working directory. git-log only operates on in-tree files,
so the first 'filename' should be an in-tree one, or it should
look at all files. This patch does the latter, so it would
also find renamed files. However, it is also slower.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Fix wording in push definition.Ralf Wildenhues Tue, 9 Oct 2007 21:01:05 +0000 (23:01 +0200)

Fix wording in push definition.

Make the definition of push in the glossary readable.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Fix some typos, punctuation, missing words, minor markup.Ralf Wildenhues Tue, 9 Oct 2007 21:00:03 +0000 (23:00 +0200)

Fix some typos, punctuation, missing words, minor markup.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

manual: Fix or remove em dashes.Ralf Wildenhues Tue, 9 Oct 2007 21:05:30 +0000 (23:05 +0200)

manual: Fix or remove em dashes.

em dashes were used inconsistently in the manual.
This changes them to the way they are used in US English.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Add a --dry-run option to git-push.Brian Ewins Thu, 11 Oct 2007 19:32:27 +0000 (20:32 +0100)

Add a --dry-run option to git-push.

The default behaviour of git-push is potentially confusing
for new users, since it will push changes that are not on
the current branch. Publishing patches that were still
cooking on a development branch is hard to undo.

It would also be nice to be able to verify the expansion
of refspecs if you've edited them, so that you know
what branches matched on the server.

Adding a --dry-run flag allows the user to experiment
safely and learn how to use git-push properly. Originally
suggested by Steffen Prohaska.

Signed-off-by: Brian Ewins <brian.ewins@gmail.com>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Add a --dry-run option to git-send-pack.Brian Ewins Thu, 11 Oct 2007 19:32:26 +0000 (20:32 +0100)

Add a --dry-run option to git-send-pack.

Implement support for --dry-run, so that it can be used
in calls from git-push. With this flag set, git-send-pack
will not send any updates to the server.

Signed-off-by: Brian Ewins <brian.ewins@gmail.com>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Fix in-place editing functions in convert.cPierre Habouzit Fri, 5 Oct 2007 08:11:59 +0000 (10:11 +0200)

Fix in-place editing functions in convert.c

* crlf_to_git and ident_to_git:

Don't grow the buffer if there is enough space in the first place.
As a side effect, when the editing is done "in place", we don't grow, so
the buffer pointer doesn't changes, and `src' isn't invalidated anymore.

Thanks to Bernt Hansen for the bug report.

* apply_filter:

Fix memory leak due to fake in-place editing that didn't collected the
old buffer when the filter succeeds. Also a cosmetic fix.

Signed-off-by: Pierre Habouzit <madcoder@debian.org>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

instaweb: support for Ruby's WEBrick serverMike Dalessio Sat, 6 Oct 2007 17:29:49 +0000 (13:29 -0400)

instaweb: support for Ruby's WEBrick server

running the webrick server with git requires Ruby and Ruby's YAML and
Webrick libraries (both of which come standard with Ruby). nice for
single-user standalone invocations.

the --httpd=webrick option generates a ruby script on the fly to read
httpd.conf options and invoke the web server via library call. this
script is placed in the .git/gitweb directory. it also generates a
shell script in a feeble attempt to invoke ruby in a portable manner,
which assumes that 'ruby' is in the user's $PATH.

Signed-off-by: Mike Dalessio <mike@csa.net>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

instaweb: allow for use of auto-generated scriptsMike Dalessio Sat, 6 Oct 2007 17:29:48 +0000 (13:29 -0400)

instaweb: allow for use of auto-generated scripts

this patch allows scripts that reside in $fqgitdir/gitweb to be used
for firing up an instaweb server. this lays the groundwork for
extending instaweb support to non-standard web servers, which may
require a script for proper invocation.

Signed-off-by: Mike Dalessio <mike@csa.net>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Add 'git-p4 commit' as an alias for 'git-p4 submit'Marius Storm-Olsen Tue, 9 Oct 2007 14:16:09 +0000 (16:16 +0200)

Add 'git-p4 commit' as an alias for 'git-p4 submit'

Given that git uses 'commit', git-p4's 'sumbit' was a bit confusing at times;
often making me do 'git submit' and 'git-p4 commit' instead.

Signed-off-by: Marius Storm-Olsen <marius@trolltech.com>
Acked-By: Simon Hausmann <simon@lst.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

hg-to-git speedup through selectable repack intervalsMichael Gebetsroither Sat, 6 Oct 2007 21:16:51 +0000 (23:16 +0200)

hg-to-git speedup through selectable repack intervals

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-svn: respect Subversion's [auth] section configurat... Eygene Ryabinkin Sat, 6 Oct 2007 18:57:19 +0000 (22:57 +0400)

git-svn: respect Subversion's [auth] section configuration values

Parameters 'store-passwords' and 'store-auth-creds' from Subversion's
configuration (~/.subversion/config) were not respected. This was
fixed: the default values for these parameters are set to 'yes' to
follow Subversion behaviour.

Signed-off-by: Eygene Ryabinkin <rea-git@codelabs.ru>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

gtksourceview2 support for gitviewFrederick Akalin Fri, 5 Oct 2007 07:20:49 +0000 (00:20 -0700)

gtksourceview2 support for gitview

Added support for gtksourceview2 module (pygtksourceview 1.90.x) in
gitview. Also refactored code that creates the source buffer and view.

Signed-off-by: Frederick Akalin <akalin@akalin.cx>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

fix contrib/hooks/post-receive-email hooks.recipients... Jeff Muizelaar Thu, 11 Oct 2007 21:49:21 +0000 (17:49 -0400)

fix contrib/hooks/post-receive-email hooks.recipients error message

Have the error message for missing recipients actually report the
missing config variable and not a fictional one.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Support cvs via git-shellJohannes Schindelin Tue, 9 Oct 2007 14:33:25 +0000 (15:33 +0100)

Support cvs via git-shell

This adds cvs support to the git-shell; You can now give new users
a restricted git-shell and they still can commit via git's cvs
emulator.

Note that either the gecos information must be accurate, or you must
provide a $HOME/.gitconfig with the appropriate user credentials.
Since the git-shell is too restricted to allow the user to do it
(on purpose!), it is up to the administrator to take care of that.

Based on an idea by Jan Wielemaker.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

rebase -i: use diff plumbing instead of porcelainJohannes Schindelin Tue, 9 Oct 2007 12:59:43 +0000 (13:59 +0100)

rebase -i: use diff plumbing instead of porcelain

When diff drivers are installed, calling "git diff <tree1>..<tree2>"
calls those drivers. This borks the patch generation of rebase -i.
So use "git diff-tree -p" instead, which does not call diff drivers.

Noticed by Johannes Sixt.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Do not remove distributed configure scriptMathias Megyei Thu, 4 Oct 2007 21:49:19 +0000 (23:49 +0200)

Do not remove distributed configure script

Before this patch the clean target has removed the
configure script that comes with Git tar file.
That made compiling Git for different architectures
inconvenient.
This patch excludes configure from the files to be
deleted by 'make clean' and adds new target 'distclean'
to preserve old functionality.

Signed-off-by: Mathias Megyei <mathias@mnet-mail.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-archive: document --execMichele Ballabio Thu, 4 Oct 2007 22:41:11 +0000 (00:41 +0200)

git-archive: document --exec

Signed-off-by: Michele Ballabio <barra_cuda@katamail.com>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-reflog: document --verboseMichele Ballabio Thu, 4 Oct 2007 12:26:54 +0000 (14:26 +0200)

git-reflog: document --verbose

Signed-off-by: Michele Ballabio <barra_cuda@katamail.com>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

makefile: Add a cscope targetKristof Provost Sat, 6 Oct 2007 14:24:42 +0000 (16:24 +0200)

makefile: Add a cscope target

The current makefile supports ctags but not cscope. Some people prefer
cscope (I do), so this patch adds a cscope target.

I've also added cscope* to the .gitignore file. For some reason tags
and TAGS weren't in there either so I've added them too.

Signed-off-by: Kristof Provost <Kristof@provost-engineering.be>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-config: handle --file option with relative pathname... Gerrit Pape Fri, 12 Oct 2007 11:32:51 +0000 (11:32 +0000)

git-config: handle --file option with relative pathname properly

When calling git-config not from the top level directory of a repository,
it changes directory before trying to open the config file specified
through the --file option, which then fails if the config file was
specified by a relative pathname. This patch adjusts the pathname to
the config file if applicable.

The problem was noticed by Joey Hess, reported through
http://bugs.debian.org/445208

Signed-off-by: Gerrit Pape <pape@smarden.org>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

clear_commit_marks(): avoid deep recursionJohannes Schindelin Wed, 10 Oct 2007 22:14:35 +0000 (23:14 +0100)

clear_commit_marks(): avoid deep recursion

Before this patch, clear_commit_marks() recursed for each parent. This
could be potentially very expensive in terms of stack space. Probably
the only reason that this did not lead to problems is the fact that we
typically call clear_commit_marks() after marking a relatively small set
of commits.

Use (sort of) a tail recursion instead: first recurse on the parents
other than the first one, and then continue the loop with the first
parent.

Noticed by Shawn Pearce.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git add -i: Remove unused variablesJean-Luc Herren Tue, 9 Oct 2007 19:34:17 +0000 (21:34 +0200)

git add -i: Remove unused variables

Signed-off-by: Jean-Luc Herren <jlh@gmx.ch>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git add -i: Fix parsing of abbreviated hunk headersJean-Luc Herren Tue, 9 Oct 2007 19:29:26 +0000 (21:29 +0200)

git add -i: Fix parsing of abbreviated hunk headers

The unified diff format allows one-line ranges to be abbreviated
by omiting the size. The hunk header "@@ -10,1 +10,1 @@" can be
expressed as "@@ -10 +10 @@", but this wasn't properly parsed in
all cases.

Such abbreviated hunk headers are generated when a one-line change
(add, remove or modify) appears without context; for example
because the file is a one-liner itself or because GIT_DIFF_OPTS
was set to '-u0'. If the user then runs 'git add -i' and enters
the 'patch' command for that file, perl complains about undefined
variables.

Signed-off-by: Jean-Luc Herren <jlh@gmx.ch>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-config: don't silently ignore options after --listFrank Lichtenheld Fri, 5 Oct 2007 20:16:44 +0000 (22:16 +0200)

git-config: don't silently ignore options after --list

Error out if someone gives options after --list since that is
not a valid syntax.

Signed-off-by: Frank Lichtenheld <frank@lichtenheld.de>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-gc --auto: simplify "repack" command line buildingBrandon Casey Thu, 11 Oct 2007 01:00:27 +0000 (20:00 -0500)

git-gc --auto: simplify "repack" command line building

Since "-a" is removed from the base repack command line,
we can simplify how we add additional options to this
command line when using --auto.

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

git-gc: by default use safer "-A" option to repack... Brandon Casey Thu, 11 Oct 2007 00:59:19 +0000 (19:59 -0500)

git-gc: by default use safer "-A" option to repack when not --prune'ing

This makes use of repack's new "-A" option which does not drop packed
unreachable objects. This makes git-gc safe to call at any time,
particularly when a repository is referenced as an alternate by
another repository.

git-gc --prune will use the "-a" option to repack instead of "-A", so
that packed unreachable objects will be removed.

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

fix t5403-post-checkout-hook.sh: built-in test in dash... Alex Riesen Thu, 4 Oct 2007 13:32:53 +0000 (15:32 +0200)

fix t5403-post-checkout-hook.sh: built-in test in dash does not have "=="

Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Clean up "git log" format with DIFF_FORMAT_NO_OUTPUTLinus Torvalds Tue, 9 Oct 2007 16:35:22 +0000 (09:35 -0700)

Clean up "git log" format with DIFF_FORMAT_NO_OUTPUT

This fixes an unnecessary empty line that we add to the log message when
we generate diffs, but don't actually end up printing any due to having
DIFF_FORMAT_NO_OUTPUT set.

This can happen with pickaxe or with rename following. The reason is that
we normally add an empty line between the commit and the diff, but we do
that even for the case where we've then suppressed the actual printing of
the diff.

This also updates a couple of tests that assumed the extraneous empty
line would exist at the end of output.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Fix embarrassing "git log --follow" bugLinus Torvalds Mon, 8 Oct 2007 20:42:41 +0000 (13:42 -0700)

Fix embarrassing "git log --follow" bug

It turns out that I completely broke "git log --follow" with my recent
patch to revision.c ("Fix revision log diff setup, avoid unnecessary diff
generation", commit b7bb760d5ed4881422673d32f869d140221d3564).

Why? Because --follow obviously requires the diff machinery to function,
exactly the same way pickaxe does.

So everybody is away right now, but considering that nobody even noticed
this bug, I don't think it matters. But for the record, here's the trivial
one-liner fix (well, two, since I also fixed the comment).

Because of the nature of the bug, if you ask for patches when following
(which is one of the things I normally do), the bug is hidden, because
then the request for diff output will automatically also enable the diffs
themselves.

So while "git log --follow <filename>" didn't work, adding a "-p"
magically made it work again even without this fix.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Shawn O. Pearce <spearce@spearce.org>

Update state documentation link for 1.5.3.4Junio C Hamano Thu, 4 Oct 2007 08:35:01 +0000 (01:35 -0700)

Update state documentation link for 1.5.3.4

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

optimize diffcore-delta by sorting hash entries.Linus Torvalds Wed, 3 Oct 2007 02:28:19 +0000 (19:28 -0700)

optimize diffcore-delta by sorting hash entries.

Here's a test-patch. I don't guarantee anything, except that when I did
the timings I also did a "wc" on the result, and they matched..

Before:
[torvalds@woody linux]$ time git diff -l0 --stat -C v2.6.22.. | wc
7104 28574 438020

real 0m10.526s
user 0m10.401s
sys 0m0.136s

After:
[torvalds@woody linux]$ time ~/git/git diff -l0 --stat -C v2.6.22.. | wc
7104 28574 438020

real 0m8.876s
user 0m8.761s
sys 0m0.128s

but the diff is fairly simple, so if somebody will go over it and say
whether it's likely to be *correct* too, that 15% may well be worth it.

[ Side note, without rename detection, that diff takes just under three
seconds for me, so in that sense the improvement to the rename detection
itself is larger than the overall 15% - it brings the cost of just
rename detection from 7.5s to 5.9s, which would be on the order of just
over a 20% performance improvement. ]

Hmm. The patch depends on half-way subtle issues like the fact that the
hashtables are guaranteed to not be full => we're guaranteed to have zero
counts at the end => we don't need to do any steenking iterator count in
the loop. A few comments might in order.

Linus

Don't checkout the full tree if avoidableSteven Walter Fri, 28 Sep 2007 17:24:19 +0000 (13:24 -0400)

Don't checkout the full tree if avoidable

In most cases of branching, the tree is copied unmodified from the trunk
to the branch. When that is done, we can simply start with the parent's
index and apply the changes on the branch as usual.

[ew: rewritten from Steven's original to use SVN::Client instead
of the command-line svn client.

Since SVN::Client connects separately, we'll share our
authentication providers array between our usages of
SVN::Client and SVN::Ra, too. Bypassing the high-level
SVN::Client library can avoid this, but the code will be
much more complex. Regardless, any implementation of this
seems to require restarting a connection to the remote
server.

Also of note is that SVN 1.4 and later allows a more
efficient diff_summary to be done instead of a full diff,
but since this code is only to support SVN < 1.4.4, we'll
ignore it for now.]

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

builtin-apply: fix conversion error in strbuf seriesJunio C Hamano Thu, 4 Oct 2007 00:42:52 +0000 (17:42 -0700)

builtin-apply: fix conversion error in strbuf series

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

Merge branch 'mv/unknown'Junio C Hamano Wed, 3 Oct 2007 11:28:24 +0000 (04:28 -0700)

Merge branch 'mv/unknown'

* mv/unknown:
Don't use "<unknown>" for placeholders and suppress printing of empty user formats.

Merge branch 'ph/strbuf'Junio C Hamano Wed, 3 Oct 2007 10:06:02 +0000 (03:06 -0700)

Merge branch 'ph/strbuf'

* ph/strbuf: (44 commits)
Make read_patch_file work on a strbuf.
strbuf_read_file enhancement, and use it.
strbuf change: be sure ->buf is never ever NULL.
double free in builtin-update-index.c
Clean up stripspace a bit, use strbuf even more.
Add strbuf_read_file().
rerere: Fix use of an empty strbuf.buf
Small cache_tree_write refactor.
Make builtin-rerere use of strbuf nicer and more efficient.
Add strbuf_cmp.
strbuf_setlen(): do not barf on setting length of an empty buffer to 0
sq_quote_argv and add_to_string rework with strbuf's.
Full rework of quote_c_style and write_name_quoted.
Rework unquote_c_style to work on a strbuf.
strbuf API additions and enhancements.
nfv?asprintf are broken without va_copy, workaround them.
Fix the expansion pattern of the pseudo-static path buffer.
builtin-for-each-ref.c::copy_name() - do not overstep the buffer.
builtin-apply.c: fix a tiny leak introduced during xmemdupz() conversion.
Use xmemdupz() in many places.
...

Merge branch 'lh/merge'Junio C Hamano Wed, 3 Oct 2007 10:05:58 +0000 (03:05 -0700)

Merge branch 'lh/merge'

* lh/merge:
git-merge: add --ff and --no-ff options
git-merge: add support for --commit and --no-squash
git-merge: add support for branch.<name>.mergeoptions
git-merge: refactor option parsing
git-merge: fix faulty SQUASH_MSG
Add test-script for git-merge porcelain

Merge branch 'js/rebase-i'Junio C Hamano Wed, 3 Oct 2007 10:05:39 +0000 (03:05 -0700)

Merge branch 'js/rebase-i'

* js/rebase-i:
rebase -i: work on a detached HEAD

Merge branch 'jc/autogc'Junio C Hamano Wed, 3 Oct 2007 10:05:32 +0000 (03:05 -0700)

Merge branch 'jc/autogc'

* jc/autogc:
git-gc --auto: run "repack -A -d -l" as necessary.
git-gc --auto: restructure the way "repack" command line is built.
git-gc --auto: protect ourselves from accumulated cruft
git-gc --auto: add documentation.
git-gc --auto: move threshold check to need_to_gc() function.
repack -A -d: use --keep-unreachable when repacking
pack-objects --keep-unreachable
Export matches_pack_name() and fix its return value
Invoke "git gc --auto" from commit, merge, am and rebase.
Implement git gc --auto

Merge branch 'je/hooks'Junio C Hamano Wed, 3 Oct 2007 10:03:40 +0000 (03:03 -0700)

Merge branch 'je/hooks'

* je/hooks:
post-checkout hook, tests, and docs

Merge branch 'ap/dateformat'Junio C Hamano Wed, 3 Oct 2007 10:03:22 +0000 (03:03 -0700)

Merge branch 'ap/dateformat'

* ap/dateformat:
Add a test script for for-each-ref, including test of date formatting
dateformat: parse %(xxdate) %(yydate:format) correctly
Make for-each-ref's grab_date() support per-atom formatting
Make for-each-ref allow atom names like "<name>:<something>"
parse_date_format(): convert a format name to an enum date_mode

Sync with GIT 1.5.3.4Junio C Hamano Wed, 3 Oct 2007 10:03:15 +0000 (03:03 -0700)

Sync with GIT 1.5.3.4

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

GIT 1.5.3.4 v1.5.3.4Junio C Hamano Wed, 3 Oct 2007 09:33:11 +0000 (02:33 -0700)

GIT 1.5.3.4

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