git.git
17 years agoDocumentation: begin discussion of git-remote in user manual
J. Bruce Fields [Tue, 9 Jan 2007 05:18:09 +0000 (00:18 -0500)]
Documentation: begin discussion of git-remote in user manual

Start discussion of git-remote.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agoDocumentation: reorder development section, todo's
J. Bruce Fields [Tue, 9 Jan 2007 04:42:36 +0000 (23:42 -0500)]
Documentation: reorder development section, todo's

Update todo's.  Split out "sharing development" section into a separate
chapter, reorder.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agoDocumentation: more user-manual todo's
J. Bruce Fields [Tue, 9 Jan 2007 02:59:42 +0000 (21:59 -0500)]
Documentation: more user-manual todo's

Add some more todo's for the user manual.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agoDocumentation: git-rebase discussion, miscellaneous user-manual updates
J. Bruce Fields [Mon, 8 Jan 2007 04:57:41 +0000 (23:57 -0500)]
Documentation: git-rebase discussion, miscellaneous user-manual updates

Add discussion of git-rebase, patch series, history rewriting.

Mention "pull ." as a synonym for "merge".

Remind myself of another case I want to cover in the other-vcs's chapter.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agoDocumentation: expand preface and todo's
J. Bruce Fields [Mon, 8 Jan 2007 03:58:14 +0000 (22:58 -0500)]
Documentation: expand preface and todo's

Add a brief description of the organization to the preface, expand the
final notes/todo's section, in hopes maybe some others will want to
contribute.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agoDocumentation: add git user's manual
J. Bruce Fields [Mon, 8 Jan 2007 00:23:49 +0000 (19:23 -0500)]
Documentation: add git user's manual

The goals are:

- Readable from beginning to end in order without having read
  any other git documentation beforehand.
- Helpful section names and cross-references, so it's not too
  hard to skip around some if you need to.
- Organized to allow it to grow much larger (unlike the
  tutorials)

It's more liesurely than tutorial.txt, but tries to stay focused on
practical how-to stuff.  It adds a discussion of how to resolve merge
conflicts, and partial instructions on setting up and dealing with a
public repository.

I've lifted a little bit from "branching and merging" (e.g., some of the
discussion of history diagrams), and could probably steal more if that's
OK.  (Similarly anyone should of course feel free to reuse bits of this
if any parts seem more useful than the whole.)

There's a lot of detail on managing branches and using git-fetch, just
because those are essential even to people needing read-only access
(e.g., kernel testers).  I think those sections will be much shorter
once the new "git remote" command and the disconnected checkouts are
taken into account.

I do feel bad about adding yet another piece of documentation, but I we
need something that goes through all the basics in a logical order, and
I wasn't seeing how to grow the tutorials into that.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
17 years agogitweb: Fix split patches output (e.g. file to symlink)
Jakub Narebski [Sun, 7 Jan 2007 01:52:27 +0000 (02:52 +0100)]
gitweb: Fix split patches output (e.g. file to symlink)

Do not replace /dev/null in two-line from-file/to-file diff header for
split patches ("split" patch mean more than one patch per one
diff-tree raw line) by a/file or b/file link.

Split patches differ from pair of deletion/creation patch in git diff
header: both a/file and b/file are hyperlinks, in all patches in a
split.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRevert "gitweb: There can be empty patches (in git_patchset_body)"
Jakub Narebski [Sun, 7 Jan 2007 01:52:26 +0000 (02:52 +0100)]
Revert "gitweb: There can be empty patches (in git_patchset_body)"

This reverts commit 1ebb948f656c03a5bdaab4de1a113b9ffcb98bea,
as that patch quieted warning but was not proper solution.
The previous commit was.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix errors in git_patchset_body for empty patches
Jakub Narebski [Sun, 7 Jan 2007 01:52:25 +0000 (02:52 +0100)]
gitweb: Fix errors in git_patchset_body for empty patches

We now do not skip over empty patches in git_patchset_body (where
empty means that they consist only of git diff header, and of extended
diff header, for example "pure rename" patch).  This means that after
extended diff header there can be next patch (i.e. /^diff /) or end of
patchset, and not necessary patch body (i.e. /^--- /).

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix error in "rename to"/"copy to" git diff header output
Jakub Narebski [Sun, 7 Jan 2007 01:52:24 +0000 (02:52 +0100)]
gitweb: Fix error in "rename to"/"copy to" git diff header output

Fix error in git_patchset_body subroutine, which caused "rename to"/"copy
to" line in extended diff header to be displayed incorrectly.

While at it, fix align.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix error in git_patchest_body for file creation/deletion patch
Jakub Narebski [Sun, 7 Jan 2007 01:52:23 +0000 (02:52 +0100)]
gitweb: Fix error in git_patchest_body for file creation/deletion patch

$from_id, $to_id variables should be local per PATCH.

Fix error in git_patchset_body for file creation (deletion) patches,
where instead of /dev/null as from-file (to-file) diff header line, it
had link to previous file with current file name.  This error occured
only if there was another patch before file creation (deletion) patch.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: fix show-ignore
Eric Wong [Sun, 7 Jan 2007 06:25:55 +0000 (22:25 -0800)]
git-svn: fix show-ignore

Looks like I broke it in 747fa12cef73b6ca04fffaddaad7326cf546cdea
but never noticed.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: tutorial editing
J. Bruce Fields [Sun, 7 Jan 2007 03:38:38 +0000 (22:38 -0500)]
Documentation: tutorial editing

Edit for conciseness.

Add a "Making changes" section header.

When possible, make sure that stuff in text boxes could be entered literally.
(Don't use "..." unless we want a user to type that.)

Move 'commit -a' example into a literal code section, clarify that it finds
modified files automatically.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation/git-svn: clarify dcommit, rebase vs pull/merge
Eric Wong [Sun, 7 Jan 2007 01:53:40 +0000 (17:53 -0800)]
Documentation/git-svn: clarify dcommit, rebase vs pull/merge

Clarify that dcommit creates a revision in SVN for every commit
in git.  Also, add 'merge' to the rebase vs pull section because
git-merge is now a first-class UI.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svnimport: clean svn path when accessing SVN repo
Sasha Khapyorsky [Sun, 7 Jan 2007 00:22:10 +0000 (02:22 +0200)]
git-svnimport: clean svn path when accessing SVN repo

Clean svn path from leading '/' when accessing SVN repo.

Signed-off-by: Sasha Khapyorsky <sashak@voltaire.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svnimport: support for incremental import
Sasha Khapyorsky [Sun, 7 Jan 2007 00:17:19 +0000 (02:17 +0200)]
git-svnimport: support for incremental import

This adds ability to do import "in chunks" (default 1000 revisions),
after each chunk git repo will be repacked. The option -R is used to
change default value of chunk size (or how often repository will
repacked).

Signed-off-by: Sasha Khapyorsky <sashak@voltaire.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit.el: Avoid setting font lock keywords before entering log-edit mode.
Alexandre Julliard [Sat, 6 Jan 2007 13:46:47 +0000 (14:46 +0100)]
git.el: Avoid setting font lock keywords before entering log-edit mode.

Instead, reinitialize the keywords after the fact. This avoids
conflicts with other users of log-edit mode, like pcl-cvs.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoinstaweb: Nicer error message when the http daemon isn't found
Fredrik Kuivinen [Sat, 6 Jan 2007 10:40:06 +0000 (11:40 +0100)]
instaweb: Nicer error message when the http daemon isn't found

Signed-off-by: Fredrik Kuivinen <frekui@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit.el: Don't use --info-only when resolving a file.
Alexandre Julliard [Sat, 6 Jan 2007 10:20:57 +0000 (11:20 +0100)]
git.el: Don't use --info-only when resolving a file.

It doesn't make a difference for git.el, but it helps when interacting
with git-rebase and friends.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-clean: Fix the -q option.
Alexandre Julliard [Sat, 6 Jan 2007 10:19:44 +0000 (11:19 +0100)]
git-clean: Fix the -q option.

The 'quiet' flag is set by -q, but it's not used anywhere.
Remove it and set the 'echo1' variable instead.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoPrint a more accurate error message when we fail to create a lock file.
Steven Grimm [Sat, 6 Jan 2007 03:14:04 +0000 (19:14 -0800)]
Print a more accurate error message when we fail to create a lock file.

Signed-off-by: Steven Grimm <koreth@midwinter.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDescribe git-clone's actual behavior in the summary
Steven Grimm [Sat, 6 Jan 2007 01:52:20 +0000 (17:52 -0800)]
Describe git-clone's actual behavior in the summary

If a branch other than "master" is checked out in the origin repository,
git-clone makes a local copy of that branch rather than the origin's
"master"
branch. This patch describes the actual behavior.

Signed-off-by: Steven Grimm <koreth@midwinter.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoSet default "tar" umask to 002 and owner.group to root.root
René Scharfe [Fri, 5 Jan 2007 22:30:22 +0000 (23:30 +0100)]
Set default "tar" umask to 002 and owner.group to root.root

In order to make the generated tar files more friendly to users who
extract them as root using GNU tar and its implied -p option, change
the default umask to 002 and change the owner name and group name to
root.  This ensures that a) the extracted files and directories are
not world-writable and b) that they belong to user and group root.

Before they would have been assigned to a user and/or group named
git if it existed.  This also answers the question in the removed
comment: uid=0, gid=0, uname=root, gname=root is exactly what we
want.

Normal users who let tar apply their umask while extracting are
only affected if their umask allowed the world to change their
files (e.g. a umask of zero).  This case is so unlikely and strange
that we don't need to support it.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoFix timestamp for test-tick
Junio C Hamano [Sat, 6 Jan 2007 10:16:07 +0000 (02:16 -0800)]
Fix timestamp for test-tick

The earlier test timestamp was too old; I forgot that the bare
unixtime integer had to be after Jan 1, 2000.  This changes
test_tick to use the git-epoch timestamp.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agobuiltin-prune: memory diet.
Junio C Hamano [Fri, 5 Jan 2007 21:31:43 +0000 (13:31 -0800)]
builtin-prune: memory diet.

Somehow we forgot to turn save_commit_buffer off while walking
the reachable objects.  Releasing the memory for commit object
data that we do not use matters for large projects (for example,
about 90MB is saved while traversing linux-2.6 history).

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoBlame "linenr" link jumps to previous state at "orig_lineno"
Luben Tuikov [Fri, 5 Jan 2007 02:37:45 +0000 (18:37 -0800)]
Blame "linenr" link jumps to previous state at "orig_lineno"

Blame currently displays the commit id which introduced a
block of one or more lines, the line numbers wrt the current
listing of the file and the file's line contents.

The commit id displayed is hyperlinked to the commit.

Currently the linenr links are hyperlinked to the same
commit id displayed to the left, which is _no_ different
than the block of lines displayed, since it is the _same
commit_ that is hyperlinked.  And thus clicking on it leads
to the same state of the file for that chunk of
lines. I.e. data mining is not currently possible with
gitweb given a chunk of lines introduced by a commit.

This patch makes such data mining possible.

The line numbers are now hyperlinked to the parent of the
commit id of the block of lines.  Furthermore they are
linked to the line where that block was introduced.

Thus clicking on a linenr link will show you the file's
line(s) state prior to the commit id you were viewing.

So clicking continually on a linenr link shows you how this
line and its line number changed over time, leading to the
initial commit where it was first introduced.

Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix "Use of uninitialized value" warning in git_tags_body
Jakub Narebski [Wed, 3 Jan 2007 21:54:29 +0000 (22:54 +0100)]
gitweb: Fix "Use of uninitialized value" warning in git_tags_body

Fix "Use of uninitialized value" warning in git_tags_body generated
for lightweight tags of tree and blob object; those don't have age
($tag{'age'}) defined.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: make --repack work consistently between fetch and multi-fetch
Eric Wong [Fri, 5 Jan 2007 02:09:56 +0000 (18:09 -0800)]
git-svn: make --repack work consistently between fetch and multi-fetch

Since fetch reforks itself at most every 1000 revisions, we
need to update the counter in the parent process to have a
working count if we set our repack interval to be > ~1000
revisions.  multi-fetch has always done this correctly
because of an extra process; now fetch uses the extra process;
as well.

While we're at it, only compile the $sha1 regex that checks for
repacking once.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: update documentation for multi-{init|fetch}
Eric Wong [Fri, 5 Jan 2007 02:04:06 +0000 (18:04 -0800)]
git-svn: update documentation for multi-{init|fetch}

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: make multi-init less confusing
Eric Wong [Fri, 5 Jan 2007 02:02:00 +0000 (18:02 -0800)]
git-svn: make multi-init less confusing

It now requires at least one of the (trunk|branch|tags) arguments
(either from the command-line or in .git/config).  Also we make
sure that anything that is passed as a URL ('help') in David's
case is actually a URL.

Thanks to David Kågedal for reporting this issue.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRemove shadowing variable from traverse_trees()
René Scharfe [Thu, 4 Jan 2007 18:33:36 +0000 (19:33 +0100)]
Remove shadowing variable from traverse_trees()

The variable named entry is allocated using malloc() and then
forgotten, it being shadowed by an automatic variable of the
same name.  Fixing the array size at 3 worked so far because
the only caller of traverse_trees() needed only as much
entries.  Simply remove the shadowing varaible and we're able
to traverse more than three trees and save stack space at the
same time!

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMake check target depend on common-cmds.h
René Scharfe [Thu, 4 Jan 2007 18:33:48 +0000 (19:33 +0100)]
Make check target depend on common-cmds.h

This fixes sparse complaining about a missing include file
if 'make check' is run on clean sources.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agorerere: Fix removal of already resolved path.
Junio C Hamano [Thu, 4 Jan 2007 08:06:50 +0000 (00:06 -0800)]
rerere: Fix removal of already resolved path.

There was an obvious thinko in memmove() to remove an entry that
was resolved from the in-core data structure.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMerge branch 'maint'
Junio C Hamano [Fri, 5 Jan 2007 06:28:21 +0000 (22:28 -0800)]
Merge branch 'maint'

* maint:
  pack-check.c::verify_packfile(): don't run SHA-1 update on huge data
  Fix infinite loop when deleting multiple packed refs.

17 years agopack-check.c::verify_packfile(): don't run SHA-1 update on huge data v1.4.4.4
Junio C Hamano [Thu, 4 Jan 2007 07:09:08 +0000 (23:09 -0800)]
pack-check.c::verify_packfile(): don't run SHA-1 update on huge data

Running the SHA1_Update() on the whole packfile in a single call
revealed an overflow problem we had in the SHA-1 implementation
on POWER architecture some time ago, which was fixed with commit
b47f509b (June 19, 2006).  Other SHA-1 implementations may have
a similar problem.

The sliding mmap() series already makes chunked calls to
SHA1_Update(), so this patch itself will become moot when it
graduates to "master", but in the meantime, run the hash
function in smaller chunks to prevent possible future problems.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix shortlog only showing HEAD revision.
Robert Fitzsimons [Wed, 3 Jan 2007 20:25:55 +0000 (20:25 +0000)]
gitweb: Fix shortlog only showing HEAD revision.

My change in 190d7fdcf325bb444fa806f09ebbb403a4ae4ee6 had a small bug
found by Michael Krufky which caused the passed in hash value to be
ignored, so shortlog would only show the HEAD revision.

Signed-off-by: Robert Fitzsimons <robfitz@273k.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-verify-tag: make sure we remove temporary file.
Junio C Hamano [Wed, 3 Jan 2007 20:34:20 +0000 (12:34 -0800)]
git-verify-tag: make sure we remove temporary file.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-tag: add flag to verify a tag
Santi Béjar [Wed, 3 Jan 2007 12:59:00 +0000 (13:59 +0100)]
git-tag: add flag to verify a tag

This way "git tag -v $tag" is the UI for git-verify-tag.

Signed-off-by: Santi Béjar <sbejar@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRefactor print-functions in builtin-branch
Lars Hjemli [Wed, 3 Jan 2007 20:10:09 +0000 (21:10 +0100)]
Refactor print-functions in builtin-branch

This moves the guts of print_ref_list() into a revamped print_ref_info(),
which at the same time gets renamed to print_ref_item().

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: There can be empty patches (in git_patchset_body)
Jakub Narebski [Wed, 3 Jan 2007 19:47:25 +0000 (20:47 +0100)]
gitweb: There can be empty patches (in git_patchset_body)

We now do not skip over empty patches in git_patchset_body
(where empty means that they consist only of git diff header,
and of extended diff header), so uncomment branch of code dealing
with empty patches (patches which do not have even two-line
from/to header)

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix bug in git_difftree_body (was '!=' instead of 'ne')
Jakub Narebski [Wed, 3 Jan 2007 19:47:24 +0000 (20:47 +0100)]
gitweb: Fix bug in git_difftree_body (was '!=' instead of 'ne')

Fix bug in git_difftree_body subroutine; it was used '!=' comparison
operator for strings (file type) instead of correct 'ne'.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation/tutorial: misc updates
Santi Béjar [Wed, 3 Jan 2007 12:53:27 +0000 (13:53 +0100)]
Documentation/tutorial: misc updates

 - Teach how to delete a branch with "git branch -d name".
 - Usually a commit has one parent; merge has more.
 - Teach "git show" instead of "git cat-file -p".

Signed-off-by: Santi Béjar <sbejar@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agotutorial: misc updates.
Junio C Hamano [Wed, 3 Jan 2007 16:38:01 +0000 (08:38 -0800)]
tutorial: misc updates.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoSkip excessive blank lines before commit body
Lars Hjemli [Wed, 3 Jan 2007 13:34:13 +0000 (14:34 +0100)]
Skip excessive blank lines before commit body

This modifies pretty_print_commit() to make the output of git-rev-list and
friends a bit more predictable.

A commit body starting with blank lines might be unheard-of, but still possible
to create using git-commit-tree (so is bound to appear somewhere, sometime).

Signed-off-by: Lars Hjemli <hjemli@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoAdd documentation for git-branch's color configuration.
Brian Gernhardt [Wed, 3 Jan 2007 15:36:29 +0000 (10:36 -0500)]
Add documentation for git-branch's color configuration.

Added color.branch and color.branch.<slot> to configuration list.
Style copied from color.status and meanings derived from the code.

Moved the color meanings from color.diff.<slot> to color.branch.<slot>
since the latter comes first alphabetically.

Added --color and --no-color to git-branch's usage and documentation.

Signed-off-by: Brian Gernhardt <benji@silverinsanity.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogitweb: Fix error in git_project_index subroutine
Jakub Narebski [Wed, 3 Jan 2007 15:03:01 +0000 (16:03 +0100)]
gitweb: Fix error in git_project_index subroutine

Instead of "$projectroot/$pr->{'path'}" to get the path to project
GIT_DIR, it was used "$projectroot/$project" which is valid only
for actions where project parameter is set, and 'project_index' is not
one of them.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoFix infinite loop when deleting multiple packed refs.
Junio C Hamano [Tue, 2 Jan 2007 19:19:05 +0000 (11:19 -0800)]
Fix infinite loop when deleting multiple packed refs.

It was stupid to link the same element twice to lock_file_list
and end up in a loop, so we certainly need a fix.

But it is not like we are taking a lock on multiple files in
this case.  It is just that we leave the linked element on the
list even after commit_lock_file() successfully removes the
cruft.

We cannot remove the list element in commit_lock_file(); if we
are interrupted in the middle of list manipulation, the call to
remove_lock_file_on_signal() will happen with a broken list
structure pointed by lock_file_list, which would cause the cruft
to remain, so not removing the list element is the right thing
to do.  Instead we should be reusing the element already on the
list.

There is already a code for that in lock_file() function in
lockfile.c.  The code checks lk->next and the element is linked
only when it is not already on the list -- which is incorrect
for the last element on the list (which has NULL in its next
field), but if you read the check as "is this element already on
the list?" it actually makes sense.  We do not want to link it
on the list again, nor we would want to set up signal/atexit
over and over.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agosend pack check for failure to send revisions list
Andy Whitcroft [Tue, 2 Jan 2007 14:12:09 +0000 (14:12 +0000)]
send pack check for failure to send revisions list

When passing the revisions list to pack-objects we do not check for
errors nor short writes.  Introduce a new write_in_full which will
handle short writes and report errors to the caller.  Use this to
short cut the send on failure, allowing us to wait for and report
the child in case the failure is its fault.

Signed-off-by: Andy Whitcroft <apw@shadowen.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoinstaweb: load Apache mime and dir modules if they are needed
Eric Wong [Tue, 2 Jan 2007 08:57:11 +0000 (00:57 -0800)]
instaweb: load Apache mime and dir modules if they are needed

I've noticed that Apache 2.2 on a Debian etch machine has
these compiled as modules.

Also set ServerName to avoid a warning at startup.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agofetch-pack: do not use lockfile structure on stack.
Junio C Hamano [Tue, 2 Jan 2007 19:22:08 +0000 (11:22 -0800)]
fetch-pack: do not use lockfile structure on stack.

They are used in atexit() for clean-up, and you will be
accessing unallocated memory at that point.

See 31f584c2 for the fix for a similar problem.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRemove unused variable (git-commit.sh)
Junio C Hamano [Tue, 2 Jan 2007 07:42:53 +0000 (23:42 -0800)]
Remove unused variable (git-commit.sh)

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUpdate clone/fetch documentation with --depth (shallow clone) option
Junio C Hamano [Mon, 1 Jan 2007 23:08:06 +0000 (15:08 -0800)]
Update clone/fetch documentation with --depth (shallow clone) option

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoStrongly discourage --update-head-ok in fetch-options documentation.
Junio C Hamano [Mon, 1 Jan 2007 23:07:35 +0000 (15:07 -0800)]
Strongly discourage --update-head-ok in fetch-options documentation.

"Use it with care" is a wrong wording to say "this is purely internal
and you are supposed to know what you are doing if you use this".

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMerge branch 'sp/merge' (early part)
Junio C Hamano [Mon, 1 Jan 2007 22:40:37 +0000 (14:40 -0800)]
Merge branch 'sp/merge' (early part)

* 'sp/merge' (early part):
  Use merge-recursive in git-am -3.
  Allow merging bare trees in merge-recursive.
  Move better_branch_name above get_ref in merge-recursive.

17 years agoDocumentation: remove master:origin example from pull-fetch-param.txt
J. Bruce Fields [Mon, 1 Jan 2007 21:40:23 +0000 (16:40 -0500)]
Documentation: remove master:origin example from pull-fetch-param.txt

This is no longer a useful example.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: update git-pull.txt for new clone behavior
J. Bruce Fields [Mon, 1 Jan 2007 21:39:06 +0000 (16:39 -0500)]
Documentation: update git-pull.txt for new clone behavior

Update examples, stop using branch named "origin" as an example.
Remove large example of use of remotes; that particular case is
nicely automated by default, so it's not so pressing to explain, and
we can refer to git-repo-config for the details.

Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-fetch: remove .keep file at the end.
Junio C Hamano [Mon, 1 Jan 2007 20:24:15 +0000 (12:24 -0800)]
git-fetch: remove .keep file at the end.

Removal of them is needed regardless of errors.  The original
code had the removal outside of the process which sets the flag
to tell the later step what to remove, but it runs as a
downstream of a pipeline and its effect was lost.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agofail pull/merge early in the middle of conflicted merge
Junio C Hamano [Mon, 1 Jan 2007 07:21:50 +0000 (23:21 -0800)]
fail pull/merge early in the middle of conflicted merge

After a pull that results in a conflicted merge, a new user
often tries another "git pull" in desperation.  When the index
is unmerged, merge backends correctly bail out without touching
either index nor the working tree, so this does not make the
wound any worse.

The user will however see several lines of messsages during this
process, such as "filename: needs merge", "you need to resolve
your current index first", "Merging...", and "Entry ... would be
overwritten by merge. Cannot merge.".  They are unnecessarily
alarming, and cause useful conflict messages from the first pull
scroll off the top of the terminal.

This changes pull and merge to run "git-ls-files -u" upfront and
stop them much earlier than we currently do.  Old timers may
know better and would not to try pulling again before cleaning
things up; this change adds extra overhead that is unnecessary
for them.  But this would be worth paying for to save new people
from needless confusion.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUpdate send-pack pipeline documentation.
Junio C Hamano [Mon, 1 Jan 2007 20:07:26 +0000 (12:07 -0800)]
Update send-pack pipeline documentation.

The pipeline was much more complex and needed documentation, but
now it is trivial and straightforward.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: t/t91??-*: optimize the tests a bit
Eric Wong [Mon, 1 Jan 2007 05:49:47 +0000 (21:49 -0800)]
git-svn: t/t91??-*: optimize the tests a bit

This removes some unnecessary 'svn up' calls throughout

t9103-git-svn-graft-branches.sh:
  * removed an 'svn log' call that was leftover from debugging
  * removed multiple git-svn calls with a multi-init / multi-fetch
    combination (which weren't tested before, either)
  * replaced `rev-list ... | head -n1` with `rev-parse ...`
    (not sure what I was thinking when I wrote that)

All this saves about 9 seconds from a test run
(53s -> 44s for 'make t91*') on my 1.3GHz Athlon

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: t/t9100-git-svn-basic: remove old check for NO_SYMLINK
Eric Wong [Mon, 1 Jan 2007 05:49:46 +0000 (21:49 -0800)]
git-svn: t/t9100-git-svn-basic: remove old check for NO_SYMLINK

We don't support the svn command-line client anymore; nor
do we support anything before SVN 1.1.0, so we can be certain
symlinks will be supported in the SVN repository.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agogit-svn: remove svnadmin dependency from the tests
Eric Wong [Mon, 1 Jan 2007 05:49:45 +0000 (21:49 -0800)]
git-svn: remove svnadmin dependency from the tests

We require the libraries now, so we can create repositories
using them (and save some executable load time while we're at
it).

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoi18n: do not leak 'encoding' header even when we cheat the conversion.
Junio C Hamano [Mon, 1 Jan 2007 02:18:23 +0000 (18:18 -0800)]
i18n: do not leak 'encoding' header even when we cheat the conversion.

We special case the case where encoding recorded in the commit
and the output encoding are the same and do not call iconv().
But we should drop 'encoding' header for this case as well for
consistency.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDo not merge random set of refs out of wildcarded refs
Junio C Hamano [Mon, 1 Jan 2007 01:44:37 +0000 (17:44 -0800)]
Do not merge random set of refs out of wildcarded refs

When your fetch configuration has only the wildcards, we would
pick the lexicographically first ref from the remote side for
merging, which was complete nonsense.  Make sure nothing except
the one that is specified with branch.*.merge is merged in this
case.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoFix formatting for urls section of fetch, pull, and push manpages
Theodore Tso [Sun, 31 Dec 2006 04:11:52 +0000 (23:11 -0500)]
Fix formatting for urls section of fetch, pull, and push manpages

Updated to make the nroff'ed man pages look nicer.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: update tutorial's discussion of origin
J. Bruce Fields [Sun, 31 Dec 2006 23:47:38 +0000 (18:47 -0500)]
Documentation: update tutorial's discussion of origin

Update tutorial's discussion of origin branch to reflect new defaults,
and include a brief mention of git-repo-config.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: update glossary entry for "origin"
J. Bruce Fields [Sun, 31 Dec 2006 23:47:36 +0000 (18:47 -0500)]
Documentation: update glossary entry for "origin"

Update glossary entry for "origin" to reflect fact that it normally now refers
to a remote repository, not a branch.

Also, warning not to work on remote-tracking branches is no longer necessary
since git doesn't allow that.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: update git-clone.txt for clone's new default behavior
J. Bruce Fields [Sun, 31 Dec 2006 23:47:34 +0000 (18:47 -0500)]
Documentation: update git-clone.txt for clone's new default behavior

Fix a couple remaining references to the origin branch.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocs: update cvs-migration.txt to reflect clone's new default behavior
J. Bruce Fields [Sun, 31 Dec 2006 23:47:33 +0000 (18:47 -0500)]
Docs: update cvs-migration.txt to reflect clone's new default behavior

I couldn't think of a really quick way to give all the details, so just refer
readers to the git-repo-config man page instead.

I haven't tested recent cvs import behavior--some time presumably it should be
updated to do something more similar to clone.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agosend-pack: tell pack-objects to use its internal rev-list.
Junio C Hamano [Sun, 31 Dec 2006 09:26:53 +0000 (01:26 -0800)]
send-pack: tell pack-objects to use its internal rev-list.

This means one less process in the pipeline to worry about, and
removes about 1/8 of the code.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agosend-pack.c: use is_null_sha1()
Junio C Hamano [Sun, 31 Dec 2006 08:59:53 +0000 (00:59 -0800)]
send-pack.c: use is_null_sha1()

Everybody else uses is_null_sha1() -- there is no point to have its
own is_zero_sha1() anymore.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUpdate documentation for update hook.
Junio C Hamano [Sun, 31 Dec 2006 08:52:25 +0000 (00:52 -0800)]
Update documentation for update hook.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMerge branch 'jc/send-pack-pipeline'
Junio C Hamano [Sun, 31 Dec 2006 08:31:26 +0000 (00:31 -0800)]
Merge branch 'jc/send-pack-pipeline'

* jc/send-pack-pipeline:
  Documentation: illustrate send-pack pipeline.
  send-pack: fix pipeline.

17 years agoAdd test case for update hooks in receive-pack.
Shawn O. Pearce [Sun, 31 Dec 2006 07:20:23 +0000 (02:20 -0500)]
Add test case for update hooks in receive-pack.

Verify that the update hooks work as documented/advertised.  This is
a simple set of tests to check that the update hooks run with the
parameters expected, have their STDOUT and STDERR redirected to
the client side of the connection, and that their STDIN does not
contain any data (as its actually /dev/null).

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation/config.txt (and repo-config manpage): mark-up fix.
Junio C Hamano [Sun, 31 Dec 2006 06:39:24 +0000 (22:39 -0800)]
Documentation/config.txt (and repo-config manpage): mark-up fix.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoTeach Git how to parse standard power of 2 suffixes.
Shawn O. Pearce [Sun, 31 Dec 2006 03:13:05 +0000 (22:13 -0500)]
Teach Git how to parse standard power of 2 suffixes.

Sometimes its necessary to supply a value as a power of two in a
configuration parameter.  In this case the user may want to use the
standard suffixes such as K, M, or G to indicate that the numerical
value should be multiplied by a constant base before being used.

Shell scripts/etc. can also benefit from this automatic option
parsing with `git repo-config --int`.

[jc: with a couple of test and a slight input tightening]

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUse /dev/null for update hook stdin.
Shawn O. Pearce [Sun, 31 Dec 2006 02:55:22 +0000 (21:55 -0500)]
Use /dev/null for update hook stdin.

Currently the update hook invoked by receive-pack has its stdin
connected to the pushing client.  The hook shouldn't attempt to
read from this stream, and doing so may consume data that was
meant for receive-pack.  Instead we should give the update hook
/dev/null as its stdin, ensuring that it always receives EOF and
doesn't disrupt the protocol if it attempts to read any data.

The post-update hook is similar, as it gets invoked with /dev/null
on stdin to prevent the hook from reading data from the client.
Previously we had invoked it with stdout also connected to /dev/null,
throwing away anything on stdout, to prevent client protocol errors.
Instead we should redirect stdout to stderr, like we do with the
update hook.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRedirect update hook stdout to stderr.
Shawn O. Pearce [Sun, 31 Dec 2006 02:55:19 +0000 (21:55 -0500)]
Redirect update hook stdout to stderr.

If an update hook outputs to stdout then that output will be sent
back over the wire to the push client as though it were part of
the git protocol.  This tends to cause protocol errors on the
client end of the connection, as the hook output is not expected
in that context.  Most hook developers work around this by making
sure their hook outputs everything to stderr.

But hooks shouldn't need to perform such special behavior.  Instead
we can just dup stderr to stdout prior to invoking the update hook.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRemove unnecessary argc parameter from run_command_v.
Shawn O. Pearce [Sun, 31 Dec 2006 02:55:15 +0000 (21:55 -0500)]
Remove unnecessary argc parameter from run_command_v.

The argc parameter is never used by the run_command_v family of
functions.  Instead they require that the passed argv[] be NULL
terminated so they can rely on the operating system's execvp
function to correctly pass the arguments to the new process.

Making the caller pass the argc is just confusing, as the caller
could be mislead into believing that the argc might take precendece
over the argv, or that the argv does not need to be NULL terminated.
So goodbye argc.  Don't come back.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoAutomatically detect a bare git repository.
Shawn O. Pearce [Sun, 31 Dec 2006 04:30:19 +0000 (23:30 -0500)]
Automatically detect a bare git repository.

Many users find it unfriendly that they can create a bare git
repository easily with `git clone --bare` but are then unable to
run simple commands like `git log` once they cd into that newly
created bare repository.  This occurs because we do not check to
see if the current working directory is a git repository.

Instead of failing out with "fatal: Not a git repository" we should
try to automatically detect if the current working directory is
a bare repository and use that for GIT_DIR, and fail out only if
that doesn't appear to be true.

We test the current working directory only after we have tried
searching up the directory tree.  This is to retain backwards
compatibility with our previous behavior on the off chance that
a user has a 'refs' and 'objects' subdirectories and a 'HEAD'
file that looks like a symref, all stored within a repository's
associated working directory.

This change also consolidates the validation logic between the case
of GIT_DIR being supplied and GIT_DIR not being supplied, cleaning
up the code.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoReplace "GIT_DIR" with GIT_DIR_ENVIRONMENT.
Shawn O. Pearce [Sun, 31 Dec 2006 04:29:11 +0000 (23:29 -0500)]
Replace "GIT_DIR" with GIT_DIR_ENVIRONMENT.

We tend to use the nice constant GIT_DIR_ENVIRONMENT when we
are referring to the "GIT_DIR" constant, but git.c didn't do
so.  Now it does.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUse PATH_MAX constant for --bare.
Shawn O. Pearce [Sun, 31 Dec 2006 04:28:53 +0000 (23:28 -0500)]
Use PATH_MAX constant for --bare.

For easier portability we prefer PATH_MAX over seemingly random
constants like 1024.  Make it so.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoForce core.filemode to false on Cygwin.
Shawn O. Pearce [Sun, 31 Dec 2006 04:53:55 +0000 (23:53 -0500)]
Force core.filemode to false on Cygwin.

Many users have noticed that core.filemode doesn't appear to be
automatically set right on Cygwin when using a repository stored
on NTFS.  The issue is that Cygwin and NTFS correctly supports
the executable mode bit, and Git properly detected that, but most
native Windows applications tend to create files such that Cygwin
sees the executable bit set when it probably shouldn't be.

This is especially bad if the user's favorite editor deletes the
file then recreates it whenever they save (vs. just overwriting)
as now a file that was created with mode 0644 by checkout-index
appears to have mode 0755.

So we introduce NO_TRUSTABLE_FILEMODE, settable at compile time.
Setting this option forces core.filemode to false, even if the
detection code would have returned true.  This option should be
enabled by default on Cygwin.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoFix formatting for urls section of fetch, pull, and push manpages
Theodore Ts'o [Sun, 31 Dec 2006 01:03:51 +0000 (20:03 -0500)]
Fix formatting for urls section of fetch, pull, and push manpages

The line:

[remote "<remote>"]

was getting swallowed up by asciidoc, causing a critical line in the
explanation for how to store the .git/remotes information in .git/config
to go missing from the git-fetch, git-pull, and git-push manpages.

Put all of the examples into delimited blocks to fix this problem and to
make them look nicer.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoFix yet another subtle xdl_merge() bug
Johannes Schindelin [Sun, 31 Dec 2006 01:07:41 +0000 (02:07 +0100)]
Fix yet another subtle xdl_merge() bug

In very obscure cases, a merge can hit an unexpected code path (where the
original code went as far as saying that this was a bug). This failing
merge was noticed by Alexandre Juillard.

The problem is that the original file contains something like this:

-- snip --
two non-empty lines
before two empty lines

after two empty lines
-- snap --

and this snippet is reduced to _one_ empty line in _both_ new files.
However, it is ambiguous as to which hunk takes the empty line: the first
or the second one?

Indeed in Alexandre's example files, the xdiff algorithm attributes the
empty line to the first hunk in one case, and to the second hunk in the
other case.

(Trimming down the example files _changes_ that behaviour!)

Thus, the call to xdl_merge_cmp_lines() has no chance to realize that the
change is actually identical in both new files. Therefore,
xdl_refine_conflicts() finds an empty diff script, which was not expected
there, because (the original author of xdl_merge() thought)
xdl_merge_cmp_lines() would catch that case earlier.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoi18n: drop "encoding" header in the output after re-coding.
Junio C Hamano [Sat, 30 Dec 2006 23:49:32 +0000 (15:49 -0800)]
i18n: drop "encoding" header in the output after re-coding.

After re-coding the commit message into the encoding the user
specified (either with core.logoutputencidng or --encoding
option), this drops the "encoding" header altogether.  The
output is after re-coding as the user asked (either with the
config or --encoding=<encoding> option), and the extra header
becomes redundant information.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agocommit-tree: cope with different ways "utf-8" can be spelled.
Junio C Hamano [Sat, 30 Dec 2006 20:20:43 +0000 (12:20 -0800)]
commit-tree: cope with different ways "utf-8" can be spelled.

People can spell config.commitencoding differently from what we
internally have ("utf-8") to mean UTF-8.  Try to accept them and
treat them equally.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMove commit reencoding parameter parsing to revision.c
Junio C Hamano [Sat, 30 Dec 2006 19:59:08 +0000 (11:59 -0800)]
Move commit reencoding parameter parsing to revision.c

This way, git-rev-list and git-diff-tree with --pretty can use
it.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: minor rewording for git-log and git-show pages.
Junio C Hamano [Sat, 30 Dec 2006 10:21:48 +0000 (02:21 -0800)]
Documentation: minor rewording for git-log and git-show pages.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: i18n commit log message notes.
Junio C Hamano [Sat, 30 Dec 2006 10:22:38 +0000 (02:22 -0800)]
Documentation: i18n commit log message notes.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agot3900: test log --encoding=none
Junio C Hamano [Sat, 30 Dec 2006 10:35:14 +0000 (02:35 -0800)]
t3900: test log --encoding=none

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agocommit re-encoding: fix confusion between no and default conversion.
Junio C Hamano [Sat, 30 Dec 2006 10:18:24 +0000 (02:18 -0800)]
commit re-encoding: fix confusion between no and default conversion.

Telling the git-log family not to do any character conversion is
done with --encoding=none, which sets log_output_encoding to an
empty string.  However, logmsg_reencode() confused this with
log_output_encoding and commit_encoding set to NULL.  The latter
means we should use the default encoding (i.e. utf-8).

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoDocumentation: illustrate send-pack pipeline.
Junio C Hamano [Fri, 29 Dec 2006 20:14:30 +0000 (12:14 -0800)]
Documentation: illustrate send-pack pipeline.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agosend-pack: fix pipeline.
Junio C Hamano [Fri, 29 Dec 2006 10:35:40 +0000 (02:35 -0800)]
send-pack: fix pipeline.

send-pack builds a pipeline that runs "rev-list | pack-objects"
and sends the output from pack-objects to the other side, while
feeding the input side of that pipe from itself.  However, the
file descriptor that is given to this pipeline (so that it can
be dup2(2)'ed into file descriptor 1 of pack-objects) is closed
by the caller before the complex fork+exec dance!  Worse yet,
the caller already dup2's it to 1, so the child process did not
even have to.

I do not understand how this code could possibly have been
working, but it somehow was working by accident.

Merging the sliding mmap() code reveals this problem, presumably
because it keeps one extra file descriptor open for a packfile
and changes the way file descriptors are allocated.  I am too
tired to diagnose the problem now, but this seems to be a
sensible fix.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoMerge branch 'jc/curl'
Junio C Hamano [Fri, 29 Dec 2006 19:36:21 +0000 (11:36 -0800)]
Merge branch 'jc/curl'

* jc/curl:
  Work around http-fetch built with cURL 7.16.0

17 years agoFix 'git add' with .gitignore
Junio C Hamano [Fri, 29 Dec 2006 19:01:31 +0000 (11:01 -0800)]
Fix 'git add' with .gitignore

When '*.ig' is ignored, and you have two files f.ig and d.ig/foo
in the working tree,

$ git add .

correctly ignored f.ig but failed to ignore d.ig/foo.  This was
caused by a thinko in an earlier commit 4888c534, when we tried
to allow adding otherwise ignored files.

After reverting that commit, this takes a much simpler approach.
When we have an unmatched pathspec that talks about an existing
pathname, we know it is an ignored path the user tried to add,
so we include it in the set of paths directory walker returned.

This does not let you say "git add -f D" on an ignored directory
D and add everything under D.  People can submit a patch to
further allow it if they want to, but I think it is a saner
behaviour to require explicit paths to be spelled out in such a
case.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoRevert "read_directory: show_both option."
Junio C Hamano [Fri, 29 Dec 2006 18:08:19 +0000 (10:08 -0800)]
Revert "read_directory: show_both option."

This reverts commit 4888c534099012d71d24051deb5b14319747bd1a.

17 years agoAdd info about new test families (8 and 9) to t/README
Jakub Narebski [Fri, 29 Dec 2006 13:39:09 +0000 (14:39 +0100)]
Add info about new test families (8 and 9) to t/README

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agot5400 send-pack test: try a bit more nontrivial transfer.
Junio C Hamano [Fri, 29 Dec 2006 10:25:04 +0000 (02:25 -0800)]
t5400 send-pack test: try a bit more nontrivial transfer.

Not that this reveals anything new, but I did test_tick shell
function in test-lib and found it rather cute and nice.

Signed-off-by: Junio C Hamano <junkio@cox.net>
17 years agoUse merge-recursive in git-am -3.
Shawn O. Pearce [Thu, 28 Dec 2006 07:35:27 +0000 (02:35 -0500)]
Use merge-recursive in git-am -3.

By switching from merge-resolve to merge-recursive in the 3-way
fallback behavior of git-am we gain a few benefits:

 * renames are automatically handled, like in rebase -m;
 * conflict hunks can reference the patch name;
 * its faster on Cygwin (less forks).

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>