13 years agoMerge branch 'jc/clone'
Junio C Hamano [Sat, 11 Aug 2007 06:05:04 +0000 (23:05 -0700)]
Merge branch 'jc/clone'

* jc/clone:
  git-clone: aggressively optimize local clone behaviour.
  connect: accept file:// URL scheme

13 years agoOptimize the three-way merge of git-read-tree
Linus Torvalds [Fri, 10 Aug 2007 19:53:51 +0000 (12:53 -0700)]
Optimize the three-way merge of git-read-tree

As mentioned, the three-way case *should* be as trivial as the
following. It passes all the tests, and I verified that a conflicting
merge in the 100,000 file horror-case merged correctly (with the conflict
markers) in 0.687 seconds with this, so it works, but I'm lazy and
somebody else should double-check it [jc: followed all three-way merge
codepaths and verified it removes when it should].

Without this patch, the merge took 8.355 seconds, so this patch
really does make a huge difference for merge performance with lots and
lots of files, and we're not talking percentages, we're talking
orders-of-magnitude differences!

Now "unpack_trees()" is just fast enough that we don't need to avoid it
(although it's probably still a good idea to eventually convert it to use
the traverse_trees() infrastructure some day - just to avoid having
extraneous tree traversal functions).

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoFix filehandle leak in "git branch -D"
Alex Riesen [Fri, 10 Aug 2007 13:06:22 +0000 (15:06 +0200)]
Fix filehandle leak in "git branch -D"

On Windows (it can't touch open files in any way) the following fails:

    git branch -D branch1 branch2

if the both branches are in packed-refs.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
13 years agobuiltin-bundle - use buffered reads for bundle header
Mark Levedahl [Sat, 11 Aug 2007 00:39:24 +0000 (20:39 -0400)]
builtin-bundle - use buffered reads for bundle header

This eliminates all use of byte-at-a-time reading of data in this
function: as Junio noted, a bundle file is seekable so we can
reset the file position to the first part of the pack-file using lseek
after reading the header.

Signed-off-by: Mark Levedahl <>
Signed-off-by: Junio C Hamano <>
13 years agobuiltin-bundle.c - use stream buffered input for rev-list
Mark Levedahl [Fri, 10 Aug 2007 22:29:49 +0000 (18:29 -0400)]
builtin-bundle.c - use stream buffered input for rev-list

git-bundle create on cygwin was nearly unusable due to 1 character
at a time (unbuffered) reading from an exec'ed process. Fix by using
fdopen to get a buffered stream.

Results for "time git bundle create test.bdl v1.0.3..v1.5.2" are:

before this patch:
         cygwin         linux
real    1m38.828s      0m3.578s
user    0m12.122s      0m2.896s
sys     1m28.215s      0m0.692s

after this patch:
real    0m3.688s       0m2.835s
user    0m3.075s       0m2.731s
sys     0m1.075s       0m0.149s

Signed-off-by: Mark Levedahl <>
Signed-off-by: Junio C Hamano <>
13 years agoallow git-bundle to create bottomless bundle
Junio C Hamano [Thu, 9 Aug 2007 05:04:06 +0000 (22:04 -0700)]
allow git-bundle to create bottomless bundle

Mark Levedahl <> writes:

> Junio C Hamano wrote:
>> While "git bundle" was a useful way to sneakernet incremental
>> changes, we did not allow:
> Thanks - I've been thinking for months I could fix this bug, never
> figured it out and didn't want to nag Dscho one more time. I confirm
> that this allows creation of bundles with arbitrary refs, not just
> those under refs/heads. Yahoo!

Actually, there is another bug nearby.

If you do:

git bundle create v2.6-20-v2.6.22.bndl v2.6.20..v2.6.22

the bundle records that it requires v2.6.20^0 commit (correct)
and gives you tag v2.6.22 (incorrect); the bug is that the
object it lists in fact is the commit v2.6.22^0, not the tag.

This is because the revision range operation .. is always about
set of commits, but the code near where my patch touches does
not validate that the sha1 value obtained from dwim_ref()
against the commit object name e->item->sha1 before placing the
head information in the commit.

The attached patch attempts to fix this problem.

Signed-off-by: Junio C Hamano <>
13 years agoallow git-bundle to create bottomless bundle
Junio C Hamano [Thu, 9 Aug 2007 00:01:49 +0000 (17:01 -0700)]
allow git-bundle to create bottomless bundle

While "git bundle" was a useful way to sneakernet incremental
changes, we did not allow:

$ git bundle create v2.6.20.bndl v2.6.20

to create a bundle that contains the whole history to a
well-known good revision.  Such a bundle can be mirrored
everywhere, and people can prime their repository with it to
reduce the load on the repository that serves near the tip of
the development.

Signed-off-by: Junio C Hamano <>
13 years agoOptimize the two-way merge of git-read-tree too
Linus Torvalds [Fri, 10 Aug 2007 19:31:20 +0000 (12:31 -0700)]
Optimize the two-way merge of git-read-tree too

This trivially optimizes the two-way merge case of git-read-tree too,
which affects switching branches.

When you have tons and tons of files in your repository, but there are
only small differences in the branches (maybe just a couple of files
changed), the biggest cost of the branch switching was actually just the
index calculations.

This fixes it (timings for switching between the "testing" and "master"
branches in the 100,000 file testing-repo-from-hell, where the branches
only differ in one small file).

[torvalds@woody bummer]$ time git checkout master
real    0m9.919s
user    0m8.461s
sys     0m0.264s

[torvalds@woody bummer]$ time git checkout testing
real    0m0.576s
user    0m0.348s
sys     0m0.228s

so it's easily an order of magnitude different.

This concludes the series. I think we could/should do the three-way merge
too (to speed up merges), but I'm lazy. Somebody else can do it.

The rule is very simple: you need to remove the old entry if:
 - you want to remove the file entirely
 - you replace it with a "merge conflict" entry (ie a non-stage-0 entry)

and you can avoid removing it if you either

 - keep the old one
 - or resolve it to a new one.

and these rules should all be valid for the three-way case too.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoOptimize the common cases of git-read-tree
Linus Torvalds [Fri, 10 Aug 2007 19:21:20 +0000 (12:21 -0700)]
Optimize the common cases of git-read-tree

This optimizes bind_merge() and oneway_merge() to not unnecessarily
remove and re-add the old index entries when they can just get replaced
by updated ones.

This makes these operations much faster for large trees (where "large"
is in the 50,000+ file range), because we don't unnecessarily move index
entries around in the index array all the time.

Using the "bummer" tree (a test-tree with 100,000 files) we get:

[torvalds@woody bummer]$ time git commit -m"Change one file" 50/500
real    0m9.470s
user    0m8.729s
sys     0m0.476s

[torvalds@woody bummer]$ time git commit -m"Change one file" 50/500
real    0m1.173s
user    0m0.720s
sys     0m0.452s

so for large trees this is easily very noticeable indeed.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoMove old index entry removal from "unpack_trees()" into the individual functions
Linus Torvalds [Fri, 10 Aug 2007 19:15:54 +0000 (12:15 -0700)]
Move old index entry removal from "unpack_trees()" into the individual functions

This makes no changes to current code, but it allows the individual merge
functions to decide what to do about the old entry.  They might decide to
update it in place, rather than force them to always delete and re-add it.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'lt/readtree'
Junio C Hamano [Fri, 10 Aug 2007 20:58:45 +0000 (13:58 -0700)]
Merge branch 'lt/readtree'

* lt/readtree:
  Start moving unpack-trees to "struct tree_desc"

13 years agoFix "git commit directory/" performance anomaly
Linus Torvalds [Fri, 10 Aug 2007 16:51:58 +0000 (09:51 -0700)]
Fix "git commit directory/" performance anomaly

This trivial patch avoids re-hashing files that are already clean in the
index. This mirrors what commit 0781b8a9b2fe760fc4ed519a3a26e4b9bd6ccffe
did for "git add .", only for "git commit ." instead.

This improves the cold-cache case immensely, since we don't need to bring
in all the file contents, just the index and any files dirty in the index.


[torvalds@woody linux]$ time git commit .
real    1m49.537s
user    0m3.892s
sys     0m2.432s


[torvalds@woody linux]$ time git commit .
real    0m14.273s
user    0m1.312s
sys     0m0.516s

(both after doing a "echo 3 > /proc/sys/vm/drop_caches" to get cold-cache
behaviour - even with the index optimization git still has to "lstat()"
all the files, so with a truly cold cache, bringing all the inodes in
will take some time).

[jc: trivial "return 0;" fixed]

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoOptimize "diff --cached" performance.
Junio C Hamano [Thu, 9 Aug 2007 20:42:50 +0000 (13:42 -0700)]
Optimize "diff --cached" performance.

The read_tree() function is called only from the call chain to
run "git diff --cached" (this includes the internal call made by
git-runstatus to run_diff_index()).  The function vacates stage
without any funky "merge" magic.  The caller then goes and
compares stage #1 entries from the tree with stage #0 entries
from the original index.

When adding the cache entries this way, it used the general
purpose add_cache_entry().  This function looks for an existing
entry to replace or if there is none to find where to insert the
new entry, resolves D/F conflict and all the other things.

For the purpose of reading entries into an empty stage, none of
that processing is needed.  We can instead append everything and
then sort the result at the end.

This commit changes read_tree() to first make sure that there is
no existing cache entries at specified stage, and if that is the
case, it runs add_cache_entry() with ADD_CACHE_JUST_APPEND flag
(new), and then sort the resulting cache using qsort().

This new flag tells add_cache_entry() to omit all the checks
such as "Does this path already exist?  Does adding this path
remove other existing entries because it turns a directory to a
file?" and instead append the given cache entry straight at the
end of the active cache.  The caller of course is expected to
sort the resulting cache at the end before using the result.

Signed-off-by: Junio C Hamano <>
13 years agoRevert "tweak manpage formatting"
Junio C Hamano [Fri, 10 Aug 2007 18:32:42 +0000 (11:32 -0700)]
Revert "tweak manpage formatting"

This reverts commit 524e5ffcf41a67ec113a9c3730ddc8fb8d3317f5.
It is reported that this change breaks formatting with docbook

13 years agoStart moving unpack-trees to "struct tree_desc"
Linus Torvalds [Fri, 10 Aug 2007 05:21:29 +0000 (22:21 -0700)]
Start moving unpack-trees to "struct tree_desc"

This doesn't actually change any real code, but it changes the interface
to unpack_trees() to take an array of "struct tree_desc" entries, the same
way the tree-walk.c functions do.

The reason for this is that we would be much better off if we can do the
tree-unpacking using the generic "traverse_trees()" functionality instead
of having to the special "unpack" infrastructure.

This really is a pretty minimal diff, just to change the calling
convention. It passes all the tests, and looks sane. There were only two
users of "unpack_trees()": builtin-read-tree and merge-recursive, and I
tried to keep the changes minimal.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoReinstate the old behaviour when GIT_DIR is set and GIT_WORK_TREE is unset
Johannes Schindelin [Sun, 5 Aug 2007 13:12:53 +0000 (14:12 +0100)]
Reinstate the old behaviour when GIT_DIR is set and GIT_WORK_TREE is unset

The old behaviour was to unilaterally default to the cwd is the work tree
when GIT_DIR was set, but GIT_WORK_TREE wasn't, no matter if we are inside
the GIT_DIR, or if GIT_DIR is actually something like ../../../.git.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agotweak manpage formatting
Junio C Hamano [Fri, 10 Aug 2007 07:49:26 +0000 (00:49 -0700)]
tweak manpage formatting

This attempts to force fixed-font in manpages for literal
blocks.  I have tested this with docbook 1.71 and it seems to
work as expected.

Signed-off-by: Junio C Hamano <>
13 years agoFix an illustration in git-rev-parse.txt
Junio C Hamano [Fri, 10 Aug 2007 07:49:27 +0000 (00:49 -0700)]
Fix an illustration in git-rev-parse.txt

This hides the backslash at the end of line from AsciiDoc
toolchain by introducing a trailing whitespace on one line in an
illustration in git-rev-parse.txt.

Signed-off-by: Junio C Hamano <>
13 years agosend-email: get all the quoting of realnames right
Uwe Kleine-König [Thu, 9 Aug 2007 13:27:58 +0000 (15:27 +0200)]
send-email: get all the quoting of realnames right

- when sending several mails I got a slightly different behaviour for the first
  mail compared to the second to last one.  The reason is that $from was
  assigned in line 608 and was not reset when beginning to handle the next

- Email::Valid can only handle properly quoted real names, so quote arguments
  to extract_valid_address.

This patch cleans up variable naming to better differentiate between sender of
the mail and it's author.

Signed-off-by: Uwe Kleine-König <>
Signed-off-by: Junio C Hamano <>
13 years agosend-email: rfc822 forbids using <address@domain> without a non-empty "phrase"
Uwe Kleine-König [Thu, 9 Aug 2007 13:27:57 +0000 (15:27 +0200)]
send-email: rfc822 forbids using <address@domain> without a non-empty "phrase"

Email::Valid does respect this considering such a mailbox specification
invalid.  b06c6bc831cbb9e9eb82fd3ffd5a2b674cd940d0 addressed the issue, but
only if Email::Valid is available.

Signed-off-by: Uwe Kleine-König <>
Signed-off-by: Junio C Hamano <>
13 years agoUse the empty tree for base diff in paranoid-update on new branches
Shawn O. Pearce [Thu, 9 Aug 2007 06:38:16 +0000 (02:38 -0400)]
Use the empty tree for base diff in paranoid-update on new branches

We have to load a tree difference for the purpose of testing
file patterns.  But if our branch is being created and there is no
specific base to difference against in the rule our base will be
'0'x40.  This is (usually) not a valid tree-ish object in a Git
repository, so there's nothing to difference against.

Instead of creating the empty tree and running git-diff against
that we just take the output of `ls-tree -r --name-only` and mark
every returned pathname as an add.

Signed-off-by: Shawn O. Pearce <>
Signed-off-by: Junio C Hamano <>
13 years agoTeach the update-paranoid to look at file differences
Shawn O. Pearce [Thu, 9 Aug 2007 06:38:12 +0000 (02:38 -0400)]
Teach the update-paranoid to look at file differences

In some applications of the update hook a user may be allowed to
modify a branch, but only if the file level difference is also an
allowed change.  This is the commonly requested feature of allowing
users to modify only certain files.

A new repository.*.allow syntax permits granting the three basic
file level operations:

  A: file is added relative to the other tree
  M: file exists in both trees, but its SHA-1 or mode differs
  D: file is removed relative to the other tree

on a per-branch and path-name basis.  The user must also have a
branch level allow line already granting them access to create,
rewind or update (CRU) that branch before the hook will consult
any file level rules.

In order for a branch change to succeed _all_ files that differ
relative to some base (by default the old value of this branch,
but it can also be any valid tree-ish) must be allowed by file
level allow rules.  A push is rejected if any diff exists that
is not covered by at least one allow rule.

Signed-off-by: Shawn O. Pearce <>
Signed-off-by: Junio C Hamano <>
13 years agoTeach update-paranoid how to store ACLs organized by groups
Shawn O. Pearce [Thu, 9 Aug 2007 06:38:09 +0000 (02:38 -0400)]
Teach update-paranoid how to store ACLs organized by groups

In some applications of this paranoid update hook the set of ACL
rules that need to be applied to a user can be large, and the
number of users that those rules must also be applied to can be
more than a handful of individuals.  Rather than repeating the same
rules multiple times (once for each user) we now allow users to be
members of groups, where the group supplies the list of ACL rules.
For various reasons we don't depend on the underlying OS groups
and instead perform our own group handling.

Users can be made a member of one or more groups by setting the
user.memberOf property within the "users/$who.acl" file:

    memberOf = developer
memberOf = administrator

This will cause the hook to also parse the "groups/$groupname.acl"
file for each value of user.memberOf, and merge any allow rules
that match the current repository with the user's own private rules
(if they had any).

Since some rules are basically the same but may have a component
differ based on the individual user, any user.* key may be inserted
into a rule using the "${}" syntax.  The allow rule does
not match if the user does not define one (and exactly one) value
for the key "foo".

Signed-off-by: Shawn O. Pearce <>
Signed-off-by: Junio C Hamano <>
13 years agoFix formatting of git-blame documentation.
Junio C Hamano [Fri, 10 Aug 2007 07:47:28 +0000 (00:47 -0700)]
Fix formatting of git-blame documentation.

blame-options.txt did not format multi-paragraph option description

Signed-off-by: Junio C Hamano <>
13 years agocvsserver: Fix for work trees
Brian Downing [Thu, 9 Aug 2007 04:26:10 +0000 (23:26 -0500)]
cvsserver: Fix for work trees

git-cvsserver used checkout-index internally for commit and annotate.
Since a work tree is required for this to function now, this was
breaking.  Work around this by defining GIT_WORK_TREE=. in the
appropriate places.

Signed-off-by: Brian Downing <>
Signed-off-by: Junio C Hamano <>
13 years agogit-p4: Fix git-p4 submit to include only changed files in the perforce submit template.
Simon Hausmann [Wed, 8 Aug 2007 15:06:55 +0000 (17:06 +0200)]
git-p4: Fix git-p4 submit to include only changed files in the perforce submit template.

Parse the files section in the "p4 change -o" output and remove lines with file changes in unrelated depot paths.

Signed-off-by: Simon Hausmann <>
Signed-off-by: Marius Storm-Olsen <>
Signed-off-by: Junio C Hamano <>
13 years agoReorder the list of commands in the manual.
Junio C Hamano [Wed, 8 Aug 2007 20:41:46 +0000 (13:41 -0700)]
Reorder the list of commands in the manual.

The basic idea was proposed by Steve Hoelzer; in order to make
the list easier to search, we keep the command list in the
script that generates it with "sort -d".

Signed-off-by: Junio C Hamano <>
13 years agogit-p4: Fix support for symlinks.
Simon Hausmann [Tue, 7 Aug 2007 10:28:00 +0000 (12:28 +0200)]
git-p4: Fix support for symlinks.

Detect symlinks as file type, set the git file mode accordingly and strip off the trailing newline in the p4 print output.
Make the mode handling a bit more readable at the same time.

Signed-off-by: Simon Hausmann <>
Acked-by: Brian Swetland <>
Signed-off-by: Junio C Hamano <>
13 years agogit-stash documentation: add missing backtick
Steve Hoelzer [Tue, 7 Aug 2007 17:38:29 +0000 (12:38 -0500)]
git-stash documentation: add missing backtick

Signed-off-by: Steve Hoelzer <>
Signed-off-by: Junio C Hamano <>
13 years agogit-stash documentation: stash numbering starts at zero, not one
Steve Hoelzer [Tue, 7 Aug 2007 17:39:03 +0000 (12:39 -0500)]
git-stash documentation: stash numbering starts at zero, not one

Signed-off-by: Steve Hoelzer <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd a note about the index being updated by git-status in some cases
Steven Grimm [Tue, 7 Aug 2007 05:57:47 +0000 (22:57 -0700)]
Add a note about the index being updated by git-status in some cases

Signed-off-by: Steven Grimm <>
Signed-off-by: Junio C Hamano <>
13 years agoDocumentation/git-commit.txt: correct bad list formatting.
David Kastrup [Mon, 6 Aug 2007 12:56:32 +0000 (14:56 +0200)]
Documentation/git-commit.txt: correct bad list formatting.

Signed-off-by: David Kastrup <>
Signed-off-by: Junio C Hamano <>
13 years agosend-email: teach sanitize_address to do rfc2047 quoting
Uwe Kleine-K\e,Av\e(Bnig [Mon, 6 Aug 2007 20:34:50 +0000 (22:34 +0200)]
send-email: teach sanitize_address to do rfc2047 quoting

Without this patch I'm not able to properly send emails as I have a
non-ascii character in my name.

I removed the _rfc822 suffix from the function name as it now does more
than rfc822 quoting.

I dug through rfc822 to do the double quoting right.  Only if that is not
possible rfc2047 quoting is applied.

Signed-off-by: Uwe Kleine-K\e,Av\e(Bnig <>
Cc: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
13 years agoFix "make GZ=1 quick-install-doc"
Junio C Hamano [Tue, 7 Aug 2007 04:15:15 +0000 (21:15 -0700)]
Fix "make GZ=1 quick-install-doc"

The basic idea is from Mark Levedahl.  I do not use GZ=1 nor
quick-install-doc myself (there obviously is a chicken-and-egg
issue with quick-install-doc for me).

Signed-off-by: Junio C Hamano <>
13 years agopager: find out pager setting from configuration
Junio C Hamano [Tue, 7 Aug 2007 04:08:43 +0000 (21:08 -0700)]
pager: find out pager setting from configuration

It was very unfortunate that we added core.pager setting to the
configuration file; even when the underlying command does not care
if there is no git repository is involved (think "git diff --no-index"),
the user would now rightfully want the configuration setting to be
honored, which means we would need to read the configuration file before
we launch the pager.

This is a minimum change in the sense that it restores the old
behaviour of not even reading config in setup_git_directory(),
but have the core.pager honored when we know it matters.

Note that this does not cover "git -p --git-dir where command";
the -p option immediately trigger the pager settings before we
even see --git-dir to learn where the configuration file is, so
we will end up reading the configuration from the place where
we would _normally_ find the git repository.

Signed-off-by: Junio C Hamano <>
13 years agogit-am: initialize variable $resume on startup
Gerrit Pape [Mon, 6 Aug 2007 14:15:30 +0000 (14:15 +0000)]
git-am: initialize variable $resume on startup

git-am expects the variable $resume to be empty or unset, which might not
be the case if $resume is set in the user's environment.  So initialize
it to an empty value on startup.

The problem was noticed by Pierre Habouzit and reported through

Signed-off-by: Gerrit Pape <>
Signed-off-by: Junio C Hamano <>
13 years agoDocumentation/git-svn: how to clone a git-svn-created repository
Adam Roben [Mon, 6 Aug 2007 08:16:43 +0000 (01:16 -0700)]
Documentation/git-svn: how to clone a git-svn-created repository

These instructions tell you how to create a clone of a repository
created with git-svn, that can in turn be used with git-svn.

Signed-off-by: Adam Roben <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'maint'
Junio C Hamano [Mon, 6 Aug 2007 08:37:10 +0000 (01:37 -0700)]
Merge branch 'maint'

* maint:
  apply: remove directory that becomes empty by renaming the last file away
  setup.c:verify_non_filename(): don't die unnecessarily while disambiguating

13 years agoapply: remove directory that becomes empty by renaming the last file away
Linus Torvalds [Sun, 5 Aug 2007 04:48:08 +0000 (21:48 -0700)]
apply: remove directory that becomes empty by renaming the last file away

We attempt to remove directory that becomes empty after removal
of a file.  We should do the same when we rename an existing
file away.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agosetup.c:verify_non_filename(): don't die unnecessarily while disambiguating
Junio C Hamano [Mon, 6 Aug 2007 07:20:06 +0000 (00:20 -0700)]
setup.c:verify_non_filename(): don't die unnecessarily while disambiguating

If you have a working tree _file_ "foo", attempt to refer to a
branch "foo/bar" without -- to disambiguate, like this:

$ git log foo/bar

tried to make sure that foo/bar cannot be naming a working tree
file "foo/bar" (in which case we would say "which one do you
want?  A rev or a working tree file?  clarify with -- please").
We run lstat("foo/bar") to check that.  If it does not succeed,
there is no ambiguity.

That is good.  But we also checked the error status for the
lstat() and expected it to fail with ENOENT.  In this particular
case, however, it fails with ENOTDIR.  That should be treated as
"expected error" as well.

Signed-off-by: Junio C Hamano <>
13 years agodocumentation: use the word "index" in the git-commit man page
J. Bruce Fields [Mon, 6 Aug 2007 04:34:02 +0000 (00:34 -0400)]
documentation: use the word "index" in the git-commit man page

As with git-add, I think previous updates to the git-commit man page did
indeed help make it more user-friendly.  But I think the banishment of
the word "index" from the description goes too far; reinstate its use,
to simplify some of the language slightly and smooth the transition to
other documentation.

Signed-off-by: J. Bruce Fields <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'master' of git://
Junio C Hamano [Mon, 6 Aug 2007 00:55:52 +0000 (17:55 -0700)]
Merge branch 'master' of git://

* 'master' of git://
  documentation: use the word "index" in the git-add manual page
  user-manual: mention git-gui
  user-manual: mention git stash
  user-manual: update for new default --track behavior

13 years agoMerge branch 'maint'
J. Bruce Fields [Sun, 5 Aug 2007 23:18:39 +0000 (19:18 -0400)]
Merge branch 'maint'

13 years agodocumentation: use the word "index" in the git-add manual page
J. Bruce Fields [Sun, 5 Aug 2007 23:16:09 +0000 (19:16 -0400)]
documentation: use the word "index" in the git-add manual page

It was a neat trick to show that you could introduce the git-add manual
page without using the word "index", and it was certainly an improvement
over the previous man page (which started out "A simple wrapper for
git-update-index to add files to the index...").

But it's possible to use the standard terminology without sacrificing
user-friendliness.  So, rewrite to use the word "index" when

Signed-off-by: J. Bruce Fields <>
13 years agouser-manual: mention git-gui
J. Bruce Fields [Sun, 5 Aug 2007 22:12:37 +0000 (18:12 -0400)]
user-manual: mention git-gui

The git gui project seems to be still in early stages, but at a point
where it's worth mentioning as an alternative way of creating commits.

One feature of interest is the ability to manipulate individual diff
hunks.  However, people have found that feature not to be easily
discoverable from the user-interface.  Pending some ui improvements, a
parenthetical hint here may help.

(Thanks to Steffen Prohask and Junio Hamano for suggesting the

Signed-off-by: J. Bruce Fields <>
13 years agouser-manual: mention git stash
Junio C Hamano [Fri, 3 Aug 2007 22:22:59 +0000 (15:22 -0700)]
user-manual: mention git stash

Mention the git-stash command as a way to temporarily set aside work in

Signed-off-by: J. Bruce Fields <>
13 years agouser-manual: update for new default --track behavior
J. Bruce Fields [Sun, 5 Aug 2007 21:18:52 +0000 (17:18 -0400)]
user-manual: update for new default --track behavior

Update documentation to reflect the --track default.

That change seems to have happened in the 1.5.3 -rc's, so bump the "for
version x.y.z or newer" warning as well.

Signed-off-by: J. Bruce Fields <>
13 years agoFix install-doc-quick target
Junio C Hamano [Sun, 5 Aug 2007 06:48:27 +0000 (23:48 -0700)]
Fix install-doc-quick target

The script starts in a subdirectory of the source directory to
muck with a branch whose structure does not have anything to
do with the actual work tree.  Go up to the top to make it clear
that we operate on the whole tree.

It also exported GIT_DIR without any good reason.  Remove it.

Signed-off-by: Junio C Hamano <>
13 years agounpack-trees.c: assume submodules are clean during check-out
Junio C Hamano [Sat, 4 Aug 2007 05:13:09 +0000 (22:13 -0700)]
unpack-trees.c: assume submodules are clean during check-out

Sven originally raised this issue:

    If you have a submodule checked out and you go back (or
    forward) to a revision of the supermodule that contains a
    different revision of the submodule and then switch to
    another revision, it will complain that the submodule is not
    uptodate, because git simply didn't update the submodule in
    the first move.

The current policy is to consider it is perfectly normal that
checked-out submodule is out-of-sync wrt the supermodule index.
At least until we introduce a superproject repository
configuration option that says "in this repository, I do care
about this submodule and at any time I move around in the
superproject, recursively check out the submodule to match", it
is a reasonable policy, as we currently do not recursively
checkout the submodules at all.  The most extreme case of this
policy is that the superproject index knows about the submodule
but the subdirectory does not even have to be checked out.

The function verify_uptodate(), called during the two-way merge
aka branch switching, is about "make sure the filesystem entity
that corresponds to this cache entry is up to date, lest we lose
the local modifications".  As we explicitly allow submodule
checkout to drift from the supermodule index entry, the check
should say "Ok, for submodules, not matching is the norm" for

Later when we have the ability to mark "I care about this
submodule to be always in sync with the superproject" (thereby
implementing automatic recursive checkout and perhaps diff,
among other things), we should check if the submodule in
question is marked as such and perform the current test.

Acked-by: Lars Hjemli <>
Acked-by: Sven Verdoolaege <>
Signed-off-by: Junio C Hamano <>
13 years agoFixed git-push manpage
Jyotirmoy Bhattacharya [Sun, 5 Aug 2007 05:22:15 +0000 (10:52 +0530)]
Fixed git-push manpage

In git-push it is the remote repository and not the
local repository which is fast forwarded. The description
of the -f option in the git-push manpage gets it the other
way round.

Signed-off-by: Jyotirmoy Bhattacharya <>
Signed-off-by: Junio C Hamano <>
13 years agocheckout-index needs a working tree
Johannes Schindelin [Sat, 4 Aug 2007 22:20:07 +0000 (23:20 +0100)]
checkout-index needs a working tree

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoadd "test-absolute-path" to .gitignore
Randal L. Schwartz [Sat, 4 Aug 2007 08:57:29 +0000 (01:57 -0700)]
add "test-absolute-path" to .gitignore

New file requires new ignore.

Signed-off-by: Junio C Hamano <>
13 years agoDocument GIT_SSH environment variable alongside other variables
Shawn O. Pearce [Sat, 4 Aug 2007 06:06:52 +0000 (02:06 -0400)]
Document GIT_SSH environment variable alongside other variables

The GIT_SSH environment variable has survived for quite a while
without being documented, but has been mentioned on list and on
my day-job repositories can only be accessed via magic supplied
through the wonderous hack that is GIT_SSH.

Advertising it alongside other "low level magic" such as GIT_PAGER
and GIT_MERGE_VERBOSITY will certainly help others who need to
spread their own pixie dust to make things work.

Signed-off-by: Shawn O. Pearce <>
Signed-off-by: Junio C Hamano <>
13 years agoINSTALL: add warning on docbook-xsl 1.72 and 1.73
Junio C Hamano [Sat, 4 Aug 2007 03:19:09 +0000 (20:19 -0700)]
INSTALL: add warning on docbook-xsl 1.72 and 1.73

Signed-off-by: Junio C Hamano <>
13 years agogitweb: Fix handling of $file_name in feed generation
Jakub Narebski [Fri, 3 Aug 2007 17:50:42 +0000 (19:50 +0200)]
gitweb: Fix handling of $file_name in feed generation

The commit b6093a5c, by Robert Fitzsimons:
  "gitweb: Change atom, rss actions to use parse_commits."
forgot to pass $file_name parameter to parse_commits subroutine.

If git_feed is provided a file name, it ought to show only the history
affecting that file or a directory.  The title was being set
correctly, but all commits from history were being shown.

Signed-off-by: Steven Walter <>
Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
13 years agogit-completion: add "git stash"
Junio C Hamano [Fri, 3 Aug 2007 09:04:37 +0000 (02:04 -0700)]
git-completion: add "git stash"

This is a new addition to 1.5.3; let's teach it to the
completion before the final release.

[sp: Added missing git-stash completion configuration]

Signed-off-by: Junio C Hamano <>
Signed-off-by: Shawn O. Pearce <>
13 years agoGIT 1.5.3-rc4 v1.5.3-rc4
Junio C Hamano [Fri, 3 Aug 2007 22:33:57 +0000 (15:33 -0700)]
GIT 1.5.3-rc4

Signed-off-by: Junio C Hamano <>
13 years agoFix bogus use of printf in t3700 test
Junio C Hamano [Fri, 3 Aug 2007 21:31:47 +0000 (14:31 -0700)]
Fix bogus use of printf in t3700 test

The hashed contents did not matter in the end result, but it passed
an uninitialized variable to printf, which caused it to emit empty
while giving an error/usage message.

Signed-off-by: Junio C Hamano <>
13 years agogitweb: do not choke on recursive symlink
Junio C Hamano [Fri, 27 Jul 2007 08:23:03 +0000 (01:23 -0700)]
gitweb: do not choke on recursive symlink

If somebody used to advertise his repository that physically
resides at /pub/lic.git/ as:


but now wants to use --base-path to allow:


she can start git-daemon with --base-path option, like this:

git-daemon --base-path=/pub --export-all

During the transition, however, she would also want to allow
older URL as well.  One natural way to achieve that is to create
a symlink:

ln -s /pub /pub/pub

so that a request to git:// is first
translated by --base-path to a request to /pub/pub/lic.git/
which goes to /pub/lic.git, thanks to the symlink.

So far so good.

However, gitweb chokes if there is such a symlink (File::Find
barfs with "/pub/pub is a recursive symbolic link").  Make the
code ignore such a symlink.

Signed-off-by: Junio C Hamano <>
13 years agoFix work-tree related breakages
Junio C Hamano [Thu, 2 Aug 2007 22:10:56 +0000 (15:10 -0700)]
Fix work-tree related breakages

In set_work_tree(), variable rel needs to be reinitialized to
NULL on every call (it should not be static).

Make sure the incoming dir variable is not too long before
copying to the temporary buffer, and make sure chdir to the
resulting directory succeeds.

This was spotted and fixed by Alex and Johannes in a handful
patch exchanges.  Here is the final version.

Signed-off-by: Junio C Hamano <>
Acked-by: Johannes Schindelin <>
13 years agoFix documentation for core.gitproxy to reflect code
David Symonds [Thu, 2 Aug 2007 22:45:56 +0000 (08:45 +1000)]
Fix documentation for core.gitproxy to reflect code

The current implementation of core.gitproxy only operates on
git:// URLs, so the ssh:// examples and custom protocol examples
have been removed or edited.

Signed-off-by: Junio C Hamano <>
13 years agoread-tree: remove unnecessary call to setup_git_directory()
Johannes Schindelin [Thu, 2 Aug 2007 21:36:37 +0000 (22:36 +0100)]
read-tree: remove unnecessary call to setup_git_directory()

read-tree is already marked with RUN_SETUP in git.c, so there is
no need to call setup_git_directory() a second time.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoSupport building on GNU/Hurd
Thomas Schwinge [Thu, 2 Aug 2007 08:56:42 +0000 (10:56 +0200)]
Support building on GNU/Hurd

GNU/Hurd systems don't have strlcpy.

Signed-off-by: Thomas Schwinge <>
Signed-off-by: Junio C Hamano <>
13 years agoFix style nit in Python slicing.
Han-Wen Nienhuys [Mon, 23 Jul 2007 22:56:37 +0000 (15:56 -0700)]
Fix style nit in Python slicing.

Python slices start at 0 by default.

Signed-off-by: Han-Wen Nienhuys <>
Signed-off-by: Simon Hausmann <>
13 years agoSort output of "p4 change" in incremental import before further
Han-Wen Nienhuys [Mon, 23 Jul 2007 22:51:49 +0000 (15:51 -0700)]
Sort output of "p4 change" in incremental import before further


P4 change outputs the changes sorted for each directory separately. We
want the global ordering on the changes, hence we sort.

Signed-off-by: Han-Wen Nienhuys <>
Signed-off-by: Simon Hausmann <>
13 years agoRelNotes 1.5.3 updates before -rc4
Junio C Hamano [Thu, 2 Aug 2007 07:13:03 +0000 (00:13 -0700)]
RelNotes 1.5.3 updates before -rc4

Signed-off-by: Junio C Hamano <>
13 years agogit-clone: aggressively optimize local clone behaviour.
Junio C Hamano [Thu, 2 Aug 2007 06:42:36 +0000 (23:42 -0700)]
git-clone: aggressively optimize local clone behaviour.

This changes the behaviour of cloning from a repository on the
local machine, by defaulting to "-l" (use hardlinks to share
files under .git/objects) and making "-l" a no-op.  A new
option, --no-hardlinks, is also added to cause file-level copy
of files under .git/objects while still avoiding the normal
"pack to pipe, then receive and index pack" network transfer
overhead.  The old behaviour of local cloning without -l nor -s
is availble by specifying the source repository with the newly
introduced file:///path/to/repo.git/ syntax (i.e. "same as
network" cloning).

 * With --no-hardlinks (i.e. have all .git/objects/ copied via
   cpio) would not catch the source repository corruption, and
   also risks corrupted recipient repository if an
   alpha-particle hits memory cell while indexing and resolving
   deltas.  As long as the recipient is created uncorrupted, you
   have a good back-up.

 * same-as-network is expensive, but it would catch the breakage
   of the source repository.  It still risks corrupted recipient
   repository due to hardware failure.  As long as the recipient
   is created uncorrupted, you have a good back-up.

 * The new default on the same filesystem, as long as the source
   repository is healthy, it is very likely that the recipient
   would be, too.  Also it is very cheap.  You do not get any
   back-up benefit, though.

None of the method is resilient against the source repository
corruption, so let's discount that from the comparison.  Then
the difference with and without --no-hardlinks matters primarily
if you value the back-up benefit or not.  If you want to use the
cloned repository as a back-up, then it is cheaper to do a clone
with --no-hardlinks and two git-fsck (source before clone,
recipient after clone) than same-as-network clone, especially as
you are likely to do a git-fsck on the recipient if you are so
paranoid anyway.

Which leads me to believe that being able to use file:/// is
probably a good idea, if only for testability, but probably of
little practical value.  We default to hardlinked clone for
everyday use, and paranoids can use --no-hardlinks as a way to
make a back-up.

Signed-off-by: Junio C Hamano <>
13 years agoconnect: accept file:// URL scheme
Linus Torvalds [Wed, 1 Aug 2007 17:03:37 +0000 (10:03 -0700)]
connect: accept file:// URL scheme

We might make it something like: "if you use an url, we don't default to
local", so the difference would be that

git clone file:///directory/to/repo

would work the way it does now, but

git clone /directory/to/repo

would default to "-l" behaviour. That kind of would make sense (and should
be easy to implement.

This adds support for "file://" URL to underlying connect
codepath to make it happen.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoWhen generating manpages, delete outdated targets first.
David Kastrup [Wed, 1 Aug 2007 23:48:44 +0000 (01:48 +0200)]
When generating manpages, delete outdated targets first.

This makes "make doc" work even if you made "sudo make doc" previously
by mistake.  Apparently an oversight: the other targets did this already.

Signed-off-by: David Kastrup <>
Signed-off-by: Junio C Hamano <>
13 years agorebase -i: fix for optional [branch] parameter
Johannes Schindelin [Wed, 1 Aug 2007 22:31:03 +0000 (23:31 +0100)]
rebase -i: fix for optional [branch] parameter

When calling "git rebase -i <upstream> <branch>", git should switch
to <branch> first.  This worked before, but I broke it by my
"Shut git rebase -i up" patch.

Fix that, and add a test to make sure that it does not break again.

Signed-off-by: Johannes Schindelin <>
Acked-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
13 years agoTry to be consistent with capitalization in the documentation
Steve Hoelzer [Wed, 1 Aug 2007 15:43:06 +0000 (10:43 -0500)]
Try to be consistent with capitalization in the documentation

Signed-off-by: Steve Hoelzer <>
Signed-off-by: Junio C Hamano <>
13 years Permit the --amend message to be given with -m/-c/-C/-F.
Junio C Hamano [Thu, 2 Aug 2007 01:14:41 +0000 (18:14 -0700)] Permit the --amend message to be given with -m/-c/-C/-F.

[jc: adjusted t/t7501 as this makes -F and --amend compatible]

Signed-off-by: David Kastrup <>
Signed-off-by: Junio C Hamano <>
13 years make GIT_EDITOR/core.editor/VISUAL/EDITOR accept commands
David Kastrup [Wed, 1 Aug 2007 21:47:20 +0000 (23:47 +0200)] make GIT_EDITOR/core.editor/VISUAL/EDITOR accept commands

The previous code only allowed specifying a single executable rather
than a complete command like "emacsclient --alternate-editor vi" in
those variables.  Since VISUAL/EDITOR appear to be traditionally
passed to a shell for interpretation (as corroborated with "less",
"mail" and "mailx", while the really ancient "more" indeed allows only
an executable name), the shell function git_editor has been amended

"eval" is employed to have quotes and similar interpreted _after_
expansion, so that specifying
EDITOR='"/home/dak/My Commands/notepad.exe"'
can be used for actually using commands with blanks.

Instead of passing just the first argument of git_editor on, we pass
all of them (so that +lineno might be employed at a later point of
time, or so that multiple files may be edited when appropriate).

Strictly speaking, there is a change in behavior: when
git config core.editor
returns a valid but empty string, the fallbacks are still searched.
This is more consistent, and the old code was problematic with regard
to multiple blanks.  Putting in additional quotes might have worked,
but quotes inside of command substitution inside of quotes is nasty
enough to not reliably work the same across "Bourne shells".

Signed-off-by: David Kastrup <>
Signed-off-by: Junio C Hamano <>
13 years agoHack git-add--interactive to make it work with ActiveState Perl
Alex Riesen [Wed, 1 Aug 2007 12:57:43 +0000 (14:57 +0200)]
Hack git-add--interactive to make it work with ActiveState Perl

It wont work for arguments with special characters (like ", : or *).
It is generally not possible on Windows, so I didn't even try.

Signed-off-by: Junio C Hamano <>
13 years agorebase -i: ignore patches that are already in the upstream
Johannes Schindelin [Wed, 1 Aug 2007 14:59:35 +0000 (15:59 +0100)]
rebase -i: ignore patches that are already in the upstream

Non-interactive rebase had this from the beginning -- match it by
using --cherry-pick option to rev-list.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoget_relative_cwd(): clarify why it handles dir == NULL
Johannes Schindelin [Wed, 1 Aug 2007 18:26:59 +0000 (19:26 +0100)]
get_relative_cwd(): clarify why it handles dir == NULL

The comment did not make a good case why it makes sense.
Clarify, and remove stale comment about the caller being lazy.
The behaviour on NULL input is pretty much intentional.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoClean up work-tree handling
Johannes Schindelin [Wed, 1 Aug 2007 00:30:14 +0000 (01:30 +0100)]
Clean up work-tree handling

The old version of work-tree support was an unholy mess, barely readable,
and not to the point.

For example, why do you have to provide a worktree, when it is not used?
As in "git status".  Now it works.

Another riddle was: if you can have work trees inside the git dir, why
are some programs complaining that they need a work tree?

IOW it is allowed to call

$ git --git-dir=../ --work-tree=. bla

when you really want to.  In this case, you are both in the git directory
and in the working tree.  So, programs have to actually test for the right
thing, namely if they are inside a working tree, and not if they are
inside a git directory.

Also, GIT_DIR=../.git should behave the same as if no GIT_DIR was
specified, unless there is a repository in the current working directory.
It does now.

The logic to determine if a repository is bare, or has a work tree
(tertium non datur), is this:

--work-tree=bla overrides GIT_WORK_TREE, which overrides core.bare = true,
which overrides core.worktree, which overrides GIT_DIR/.. when GIT_DIR
ends in /.git, which overrides the directory in which .git/ was found.

In related news, a long standing bug was fixed: when in .git/bla/x.git/,
which is a bare repository, git formerly assumed ../.. to be the
appropriate git dir.  This problem was reported by Shawn Pearce to have
caused much pain, where a colleague mistakenly ran "git init" in "/" a
long time ago, and bare repositories just would not work.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd set_git_dir() function
Johannes Schindelin [Wed, 1 Aug 2007 00:29:38 +0000 (01:29 +0100)]
Add set_git_dir() function

With the function set_git_dir() you can reset the path that will
be used for git_path(), git_dir() and friends.

The responsibility to close files and throw away information from the
old git_dir lies with the caller.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd functions get_relative_cwd() and is_inside_dir()
Johannes Schindelin [Wed, 1 Aug 2007 00:29:17 +0000 (01:29 +0100)]
Add functions get_relative_cwd() and is_inside_dir()

The function get_relative_cwd() works just as getcwd(), only that it
takes an absolute path as additional parameter, returning the prefix
of the current working directory relative to the given path.  If the
cwd is no subdirectory of the given path, it returns NULL.

is_inside_dir() is just a trivial wrapper over get_relative_cwd().

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd is_absolute_path() and make_absolute_path()
Johannes Schindelin [Wed, 1 Aug 2007 00:28:59 +0000 (01:28 +0100)]
Add is_absolute_path() and make_absolute_path()

This patch adds convenience functions to work with absolute paths.
The function is_absolute_path() should help the efforts to integrate
the MinGW fork.

Note that make_absolute_path() returns a pointer to a static buffer.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years ago--base-path-relaxed option
Jens Axboe [Fri, 27 Jul 2007 21:00:29 +0000 (14:00 -0700)]
--base-path-relaxed option

I switched to --base-path a few minutes ago, to get rid of
a /data/git postfix in the posted urls. But transitioning is tricky,
since now all old paths will fail miserably.

So I added this --base-path-relaxed option, that will make git-daemon
try the absolute path without prefixing --base-path before giving up.
With this in place and --base-path-relaxed added, both my new url of


and the old


work fine.

Signed-off-by: Jens Axboe <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd test case for basic commit functionality.
Kristian Høgsberg [Tue, 31 Jul 2007 19:37:30 +0000 (15:37 -0400)]
Add test case for basic commit functionality.

Signed-off-by: Kristian Høgsberg <>
Signed-off-by: Junio C Hamano <>
13 years agogit.el: Take into account the core.excludesfile config option.
Alexandre Julliard [Tue, 31 Jul 2007 10:36:32 +0000 (12:36 +0200)]
git.el: Take into account the core.excludesfile config option.

Also don't require .git/info/exclude to exist in order to list unknown

Signed-off-by: Alexandre Julliard <>
Acked-by: Karl Hasselström <>
Signed-off-by: Junio C Hamano <>
13 years agogit.el: Avoid using ewoc-set-data for compatibility with Emacs 21.
Alexandre Julliard [Wed, 1 Aug 2007 06:03:41 +0000 (23:03 -0700)]
git.el: Avoid using ewoc-set-data for compatibility with Emacs 21.

Signed-off-by: Alexandre Julliard <>
Acked-by: Karl Hasselström <>
Signed-off-by: Junio C Hamano <>
13 years agoMake verse of git-config manpage more readable
Alex Riesen [Tue, 31 Jul 2007 22:01:17 +0000 (00:01 +0200)]
Make verse of git-config manpage more readable

Also mention '--file' in FILES.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd a test for git-config --file
Alex Riesen [Tue, 31 Jul 2007 10:30:52 +0000 (12:30 +0200)]
Add a test for git-config --file

Check for non-0 exit code if the confiog file does not exist and
if it works exactly like when setting GIT_CONFIG.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
13 years agoAdd an option to specify a file to config builtin
Alex Riesen [Tue, 31 Jul 2007 09:58:43 +0000 (11:58 +0200)]
Add an option to specify a file to config builtin

There are (really!) systems where using environment variables is very
cumbersome (yes, Windows, it has problems unsetting them). Besides this
form is shorter.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
13 years agomake the name of the library directory a config option
Robert Schiele [Wed, 1 Aug 2007 04:30:35 +0000 (06:30 +0200)]
make the name of the library directory a config option

Introduce new makefile variable lib to hold the name of the lib
directory ("lib" by default).  Also introduce a switch for configure
to specify this name with --with-lib=ARG.  This is useful for systems
that use a different name than "lib" (like "lib64" on some 64 bit
Linux architectures).

Signed-off-by: Robert Schiele <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'maint'
Junio C Hamano [Wed, 1 Aug 2007 04:12:32 +0000 (21:12 -0700)]
Merge branch 'maint'

* maint:
  rev-list --bisect: fix allocation of "int*" instead of "int".

13 years agorev-list --bisect: fix allocation of "int*" instead of "int".
Christian Couder [Tue, 31 Jul 2007 12:48:29 +0000 (14:48 +0200)]
rev-list --bisect: fix allocation of "int*" instead of "int".

Signed-off-by: Christian Couder <>
Signed-off-by: Junio C Hamano <>
13 years agogit.c: execution path
Junio C Hamano [Tue, 31 Jul 2007 09:06:14 +0000 (02:06 -0700)]
git.c: execution path

The comment before executing git subcommands were stale and confusing.
Noticed by Jeff King.

Signed-off-by: Junio C Hamano <>
13 years agoadd option to find zlib in custom path
Robert Schiele [Sun, 29 Jul 2007 18:35:45 +0000 (20:35 +0200)]
add option to find zlib in custom path

Some systems do not provide zlib development headers and libraries in
default search path of the compiler.  For these systems we should allow
specifying the location by --with-zlib=PATH or by setting ZLIB_PATH in
the makefile.

Signed-off-by: Robert Schiele <>
Signed-off-by: Junio C Hamano <>
13 years agoUnset GIT_EDITOR while running tests.
Junio C Hamano [Tue, 31 Jul 2007 05:16:40 +0000 (22:16 -0700)]
Unset GIT_EDITOR while running tests.

Signed-off-by: Junio C Hamano <>
13 years agoadd_file_to_index: skip rehashing if the cached stat already matches
Junio C Hamano [Tue, 31 Jul 2007 00:12:58 +0000 (17:12 -0700)]
add_file_to_index: skip rehashing if the cached stat already matches

An earlier commit 366bfcb6 broke git-add by moving read_cache()
call down, because it wanted the directory walking code to grab
paths that are already in the index.  The change serves its
purpose, but introduces a regression because the responsibility
of avoiding unnecessary reindexing by matching the cached stat
is shifted nowhere.

This makes it the job of add_file_to_index() function.

Signed-off-by: Junio C Hamano <>
13 years agogit-svn: Translate invalid characters in refname
Robert Ewald [Mon, 30 Jul 2007 09:08:21 +0000 (11:08 +0200)]
git-svn: Translate invalid characters in refname

In git some characters are invalid as documented
in git-check-ref-format. In subversion these characters might
be valid, so a translation is required.

This patch does this translation by url escaping characters, that
are not allowed.

Credit goes to Eric Wong, martin f. krafft and Jan Hudec

Signed-off-by: Robert Ewald <>
Acked-by: Eric Wong <>
Signed-off-by: Junio C Hamano <>
13 years agowhite space fixes in setup.c
Johannes Schindelin [Sun, 29 Jul 2007 23:24:38 +0000 (00:24 +0100)]
white space fixes in setup.c

Some lines were not indented by tabs but by spaces.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoDocumentation/gitattributes.txt: typofix
David Soria Parra [Mon, 30 Jul 2007 01:50:24 +0000 (22:50 -0300)]
Documentation/gitattributes.txt: typofix

The file used for per-repository attribute setting is not
$GIT_DIR/info/gitattributes, but $GIT_DIR/info/attributes.

Signed-off-by: Junio C Hamano <>
13 years agorebase: try not to munge commit log message
Junio C Hamano [Sun, 29 Jul 2007 00:57:25 +0000 (17:57 -0700)]
rebase: try not to munge commit log message

This makes rebase/am keep the original commit log message
better, even when it does not conform to "single line paragraph
to say what it does, then explain and defend why it is a good
change in later paragraphs" convention.

This change is a two-edged sword.  While the earlier behaviour
would make such commit log messages more friendly to readers who
expect to get the birds-eye view with oneline summary formats,
users who primarily use git as a way to interact with foreign
SCM systems would not care much about the convenience of oneline
git log tools, but care more about preserving their own
convention.  This changes their commits less useful to readers
who read them with git tools while keeping them more consistent
with the foreign SCM systems they interact with.

Signed-off-by: Junio C Hamano <>
13 years agosymbolic-ref, update-ref: do not refuse reflog message with LF
Junio C Hamano [Sun, 29 Jul 2007 00:20:52 +0000 (17:20 -0700)]
symbolic-ref, update-ref: do not refuse reflog message with LF

Earlier these tools refused to create a reflog entry when the
message given by the calling Porcelain had a LF in it, partially
to keep the file format integrity of reflog file, which is
one-entry-per-line.  These tools should not be dictating such a

Instead, let the codepath to write out the reflog entry worry
about the format integrity and allow messages with LF in them.

Signed-off-by: Junio C Hamano <>
13 years agolog_ref_write() -- do not chomp reflog message at the first LF
Junio C Hamano [Sun, 29 Jul 2007 00:17:17 +0000 (17:17 -0700)]
log_ref_write() -- do not chomp reflog message at the first LF

A reflog file is organized as one-line-per-entry records, and we
enforced the file format integrity by chomping the given message
at the first LF.  This changes it to convert them to SP, which
is more in line with the --pretty=oneline format.

Signed-off-by: Junio C Hamano <>