git.git
18 years agoCommon option parsing for "git log --diff" and friends
Linus Torvalds [Fri, 14 Apr 2006 23:52:13 +0000 (16:52 -0700)]
Common option parsing for "git log --diff" and friends

This basically does a few things that are sadly somewhat interdependent,
and nontrivial to split out

 - get rid of "struct log_tree_opt"

   The fields in "log_tree_opt" are moved into "struct rev_info", and all
   users of log_tree_opt are changed to use the rev_info struct instead.

 - add the parsing for the log_tree_opt arguments to "setup_revision()"

 - make setup_revision set a flag (revs->diff) if the diff-related
   arguments were used. This allows "git log" to decide whether it wants
   to show diffs or not.

 - make setup_revision() also initialize the diffopt part of rev_info
   (which we had from before, but we just didn't initialize it)

 - make setup_revision() do all the "finishing touches" on it all (it will
   do the proper flag combination logic, and call "diff_setup_done()")

Now, that was the easy and straightforward part.

The slightly more involved part is that some of the programs that want to
use the new-and-improved rev_info parsing don't actually want _commits_,
they may want tree'ish arguments instead. That meant that I had to change
setup_revision() to parse the arguments not into the "revs->commits" list,
but into the "revs->pending_objects" list.

Then, when we do "prepare_revision_walk()", we walk that list, and create
the sorted commit list from there.

This actually cleaned some stuff up, but it's the less obvious part of the
patch, and re-organized the "revision.c" logic somewhat. It actually paves
the way for splitting argument parsing _entirely_ out of "revision.c",
since now the argument parsing really is totally independent of the commit
walking: that didn't use to be true, since there was lots of overlap with
get_commit_reference() handling etc, now the _only_ overlap is the shared
(and trivial) "add_pending_object()" thing.

However, I didn't do that file split, just because I wanted the diff
itself to be smaller, and show the actual changes more clearly. If this
gets accepted, I'll do further cleanups then - that includes the file
split, but also using the new infrastructure to do a nicer "git diff" etc.

Even in this form, it actually ends up removing more lines than it adds.

It's nice to note how simple and straightforward this makes the built-in
"git log" command, even though it continues to support all the diff flags
too. It doesn't get much simpler that this.

I think this is worth merging soonish, because it does allow for future
cleanup and even more sharing of code. However, it obviously touches
"revision.c", which is subtle. I've tested that it passes all the tests we
have, and it passes my "looks sane" detector, but somebody else should
also give it a good look-over.

[jc: squashed the original and three "oops this too" updates, with
 another fix-up.]

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'js/diffstat'
Junio C Hamano [Sat, 15 Apr 2006 04:55:23 +0000 (21:55 -0700)]
Merge branch 'js/diffstat'

* js/diffstat:
  diff --stat: no need to ask funcnames nor context.
  diff-options: add --stat (take 2)
  diff-options: add --stat (take 2)

18 years agoMerge branch 'jc/fix5500'
Junio C Hamano [Sat, 15 Apr 2006 04:55:22 +0000 (21:55 -0700)]
Merge branch 'jc/fix5500'

* jc/fix5500:
  t5500: test fix

18 years agoClean up trailing whitespace when pretty-printing commits
Linus Torvalds [Sat, 15 Apr 2006 04:20:51 +0000 (21:20 -0700)]
Clean up trailing whitespace when pretty-printing commits

Partly because we've messed up and now have some commits with trailing
whitespace, but partly because this also just simplifies the code, let's
remove trailing whitespace from the end when pretty-printing commits.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years ago"git cmd -h" for shell scripts.
Junio C Hamano [Fri, 14 Apr 2006 22:54:51 +0000 (15:54 -0700)]
"git cmd -h" for shell scripts.

Wrappers that use sh-setup took --help but not -h.  Noticed by
Sébastien Pierre.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-log <diff-options> <paths> documentation
Junio C Hamano [Fri, 14 Apr 2006 19:59:09 +0000 (12:59 -0700)]
git-log <diff-options> <paths> documentation

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoRetire git-log.sh (take #4)
Junio C Hamano [Fri, 14 Apr 2006 19:47:10 +0000 (12:47 -0700)]
Retire git-log.sh (take #4)

Noticed by Johannes.  We do not install it anymore, but still have
been shipping the source, which was crazy.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agostripspace: incomplete line fix (take #2)
Junio C Hamano [Fri, 14 Apr 2006 19:41:51 +0000 (12:41 -0700)]
stripspace: incomplete line fix (take #2)

This fixes f4ee3eb68906f079dea45de4f1bbb03d68189eb3 breakage, which
added an extra trailing blank line after stripping trailing blank lines
by mistake.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agot5500: test fix
Junio C Hamano [Fri, 14 Apr 2006 05:28:06 +0000 (22:28 -0700)]
t5500: test fix

Relying on eye-candy progress bar was fragile to begin with.
Run fetch-pack with -k option, and count the objects that are in
the pack that were transferred from the other end.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff --stat: no need to ask funcnames nor context.
Junio C Hamano [Fri, 14 Apr 2006 04:35:54 +0000 (21:35 -0700)]
diff --stat: no need to ask funcnames nor context.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoFix-up previous expr changes.
Junio C Hamano [Fri, 14 Apr 2006 02:05:38 +0000 (19:05 -0700)]
Fix-up previous expr changes.

The regexp on the right hand side of expr : operator somehow was
broken.

expr 'z+pu:refs/tags/ko-pu' : 'z\+\(.*\)'

does not strip '+'; write 'z+\(.*\)' instead.

We probably should switch to shell based substring post 1.3.0;
that's not bashism but just POSIX anyway.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-options: add --stat (take 2)
Johannes Schindelin [Thu, 13 Apr 2006 23:09:48 +0000 (01:09 +0200)]
diff-options: add --stat (take 2)

... and a fix for an invalid free():

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-options: add --stat (take 2)
Johannes Schindelin [Thu, 13 Apr 2006 22:15:30 +0000 (00:15 +0200)]
diff-options: add --stat (take 2)

Now, you can say "git diff --stat" (to get an idea how many changes are
uncommitted), or "git log --stat".

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoShell utilities: Guard against expr' magic tokens.
Mark Wooding [Thu, 13 Apr 2006 22:01:24 +0000 (22:01 +0000)]
Shell utilities: Guard against expr' magic tokens.

Some words, e.g., `match', are special to expr(1), and cause strange
parsing effects.  Track down all uses of expr and mangle the arguments
so that this isn't a problem.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agot3600-rm: skip failed-remove test when we cannot make an unremovable file.
Junio C Hamano [Thu, 13 Apr 2006 18:57:57 +0000 (11:57 -0700)]
t3600-rm: skip failed-remove test when we cannot make an unremovable file.

When running t3600-rm test under fakeroot (or as root), we
cannot make a file unremovable with "chmod a-w .".  Detect this
case early and skip that test.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoUse less memory in "git log"
Linus Torvalds [Thu, 13 Apr 2006 17:01:02 +0000 (10:01 -0700)]
Use less memory in "git log"

This trivially avoids keeping the commit message data around after we
don't need it any more, avoiding a continually growing "git log" memory
footprint.

It's not a huge deal, but it's somewhat noticeable. For the current kernel
tree, doing a full "git log" I got

 - before: /usr/bin/time git log > /dev/null
0.81user 0.02system 0:00.84elapsed 100%CPU (0avgtext+0avgdata 0maxresident)k
0inputs+0outputs (0major+8851minor)pagefaults 0swaps

 - after: /usr/bin/time git log > /dev/null
0.79user 0.03system 0:00.83elapsed 100%CPU (0avgtext+0avgdata 0maxresident)k
0inputs+0outputs (0major+5039minor)pagefaults 0swaps

ie the touched pages dropped from 8851 to 5039. For the historic kernel
archive, the numbers are 18357->11037 minor page faults.

We could/should in theory free the commits themselves, but that's really a
lot harder, since during revision traversal we may hit the same commit
twice through different children having it as a parent, even after we've
shown it once (when that happens, we'll silently ignore it next time, but
we still need the "struct commit" to know).

And as the commit message data is clearly the biggest part of the commit,
this is the really easy 60% solution.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-log: do not output excess blank line between commits
Junio C Hamano [Thu, 13 Apr 2006 07:26:21 +0000 (00:26 -0700)]
git-log: do not output excess blank line between commits

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMakefile: $(MAKE) check-docs
Junio C Hamano [Thu, 13 Apr 2006 07:17:19 +0000 (00:17 -0700)]
Makefile: $(MAKE) check-docs

This target lists undocumented commands, and/or whose document
is not referenced from the main git documentation.

For now, there are some exceptions I added primarily because I
lack the energy to document them myself:

 - merge backends (we should really document them)
 - ssh-push/ssh-pull (does anybody still use them?)
 - annotate and blame (maybe after one of them eats the other ;-)

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocumentation: add a couple of missing docs.
Junio C Hamano [Thu, 13 Apr 2006 07:09:54 +0000 (00:09 -0700)]
Documentation: add a couple of missing docs.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/combine' into next
Junio C Hamano [Wed, 12 Apr 2006 20:24:48 +0000 (13:24 -0700)]
Merge branch 'jc/combine' into next

* jc/combine:
  stripspace: make sure not to leave an incomplete line.
  git-commit: do not muck with commit message when no_edit is set.
  When showing a commit message, do not lose an incomplete line.
  Retire t5501-old-fetch-and-upload test.
  combine-diff: type fix.

18 years agoMerge branch 'master' into jc/combine
Junio C Hamano [Wed, 12 Apr 2006 20:24:04 +0000 (13:24 -0700)]
Merge branch 'master' into jc/combine

* master:
  stripspace: make sure not to leave an incomplete line.
  git-commit: do not muck with commit message when no_edit is set.
  When showing a commit message, do not lose an incomplete line.
  Retire t5501-old-fetch-and-upload test.

18 years agocombine-diff: type fix.
Junio C Hamano [Wed, 12 Apr 2006 20:23:50 +0000 (13:23 -0700)]
combine-diff: type fix.

The variable hunk_end points at a line number, which is
represented as unsigned long by all the other variables.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agostripspace: make sure not to leave an incomplete line.
Junio C Hamano [Wed, 12 Apr 2006 20:10:27 +0000 (13:10 -0700)]
stripspace: make sure not to leave an incomplete line.

When dealing with a commit log message for human consumption, it
never makes sense to keep a log that ends with an incomplete
line, so make it a part of the clean-up process done by
git-stripspace.

Acked-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-commit: do not muck with commit message when no_edit is set.
Junio C Hamano [Wed, 12 Apr 2006 18:45:18 +0000 (11:45 -0700)]
git-commit: do not muck with commit message when no_edit is set.

Spotted by Linus and Darrin Thompson.  When we took a commit
message from -F <file> with an incomplete line, we appended "git
status" output, which ended up attaching a lone "#" at the end.

We still need the "do we have anything to commit?" check by
running "status" (which has to know what to do in different
cases with -i/-o/-a), but there is no point appending its output
to the proposed commit message given by the user.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoWhen showing a commit message, do not lose an incomplete line.
Linus Torvalds [Wed, 12 Apr 2006 18:31:23 +0000 (11:31 -0700)]
When showing a commit message, do not lose an incomplete line.

18 years agoRetire t5501-old-fetch-and-upload test.
Junio C Hamano [Tue, 11 Apr 2006 23:35:39 +0000 (16:35 -0700)]
Retire t5501-old-fetch-and-upload test.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/combine' into next
Junio C Hamano [Tue, 11 Apr 2006 21:34:59 +0000 (14:34 -0700)]
Merge branch 'jc/combine' into next

* jc/combine:
  combine-diff: fix hunks at the end (take #2).
  combine-diff: do not lose hunks with only deletion at end.

18 years agoMerge branch 'jc/diff' into next
Junio C Hamano [Tue, 11 Apr 2006 21:34:53 +0000 (14:34 -0700)]
Merge branch 'jc/diff' into next

* jc/diff:
  blame and friends: adjust to multiple pathspec change.
  git log --full-diff
  tree-diff: do not assume we use only one pathspec

18 years agocombine-diff: fix hunks at the end (take #2).
Junio C Hamano [Tue, 11 Apr 2006 21:31:31 +0000 (14:31 -0700)]
combine-diff: fix hunks at the end (take #2).

The previous round showed the delete-only hunks at the end, but
forgot to mark them interesting when they were.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombine-diff: do not lose hunks with only deletion at end.
Junio C Hamano [Tue, 11 Apr 2006 10:13:29 +0000 (03:13 -0700)]
combine-diff: do not lose hunks with only deletion at end.

We used to lose hunks that appear at the end and have only
deletion.  This makes sure that the record beyond the end of
file (which holds such deletions) is examined.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'ds/index' into next
Junio C Hamano [Tue, 11 Apr 2006 18:52:36 +0000 (11:52 -0700)]
Merge branch 'ds/index' into next

* ds/index:
  Replace index() with strchr().
  Solaris 9 also wants our own unsetenv/setenv.
  Retire git-log.sh (take #3)

18 years agoMerge branch 'jc/withraw' into next
Junio C Hamano [Tue, 11 Apr 2006 18:52:01 +0000 (11:52 -0700)]
Merge branch 'jc/withraw' into next

* jc/withraw:
  Separate the raw diff and patch with a newline
  Document --patch-with-raw

18 years agoReplace index() with strchr().
Dennis Stosberg [Tue, 11 Apr 2006 16:37:58 +0000 (18:37 +0200)]
Replace index() with strchr().

strchr() is more portable than index() and is used everywhere in
git already.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoSolaris 9 also wants our own unsetenv/setenv.
Dennis Stosberg [Tue, 11 Apr 2006 18:42:26 +0000 (11:42 -0700)]
Solaris 9 also wants our own unsetenv/setenv.

[jc: the original had "index() is evil" but that should be a
separate patch.]

18 years agoRetire git-log.sh (take #3)
Junio C Hamano [Tue, 11 Apr 2006 18:29:36 +0000 (11:29 -0700)]
Retire git-log.sh (take #3)

Do not install built-in commands as separate files -- use
hardlinks instead.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoSeparate the raw diff and patch with a newline
Petr Baudis [Tue, 11 Apr 2006 11:30:46 +0000 (13:30 +0200)]
Separate the raw diff and patch with a newline

More friendly for human reading I believe, and possibly friendlier to some
parsers (although only by an epsilon).

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocument --patch-with-raw
Petr Baudis [Tue, 11 Apr 2006 11:22:17 +0000 (13:22 +0200)]
Document --patch-with-raw

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'eb/apply' into next
Junio C Hamano [Tue, 11 Apr 2006 02:45:30 +0000 (19:45 -0700)]
Merge branch 'eb/apply' into next

* eb/apply:
  Implement limited context matching in git-apply.

18 years agoMerge branch 'jc/withraw' into next
Junio C Hamano [Tue, 11 Apr 2006 02:44:35 +0000 (19:44 -0700)]
Merge branch 'jc/withraw' into next

* jc/withraw:
  diff-* --patch-with-raw
  Retire git-log.sh (take#2)
  Retire git-log.sh
  Retire diffcore-pathspec.
  Improve the git-diff-tree -c/-cc documentation

18 years agodiff-* --patch-with-raw
Junio C Hamano [Tue, 11 Apr 2006 00:36:53 +0000 (17:36 -0700)]
diff-* --patch-with-raw

This new flag outputs the diff-raw output and diff-patch output
at the same time.  Requested by Cogito.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoImplement limited context matching in git-apply.
Eric W. Biederman [Mon, 10 Apr 2006 09:33:06 +0000 (03:33 -0600)]
Implement limited context matching in git-apply.

Ok this really should be the good version.  The option
handling has been reworked to be automation safe.

Currently to import the -mm tree I have to work around
git-apply by using patch.  Because some of Andrews
patches in quilt will only apply with fuzz.

I started out implementing a --fuzz option and then I realized
fuzz is not a very safe concept for an automated system.  What
you really want is a minimum number of context lines that must
match.  This allows policy to be set without knowing how many
lines of context a patch actually provides.   By default
the policy remains to match all provided lines of context.

Allowng git-apply to match a restricted set of context makes
it much easier to import the -mm tree into git.  I am still only
processing  1.5 to 1.6 patches a second for the 692 patches in
2.6.17-rc1-mm2 is still painful but it does help.

If I just loop through all of Andrews patches in order
and run git-apply --index -C1 I process the entire patchset
in 1m53s or about 6 patches per second.  So running
git-mailinfo, git-write-tree, git-commit-tree, and
git-update-ref everytime has a measurable impact,
and shows things can be speeded up even more.

All of these timings were taking on my poor 700Mhz Athlon
with 512MB of ram.  So people with fast machiens should
see much better performance.

When a match is found after the number of context are reduced a
warning is generated.  Since this is a rare event and possibly
dangerous this seems to make sense.  Unless you are patching
a single file the error message is a little bit terse at
the moment, but it should be easy to go back and fix.

I have also updated the documentation for git-apply to reflect
the new -C option that sets the minimum number of context
lines that must match.

Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoRetire git-log.sh (take#2)
Junio C Hamano [Tue, 11 Apr 2006 00:37:58 +0000 (17:37 -0700)]
Retire git-log.sh (take#2)

... and install a replacement properly.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoblame and friends: adjust to multiple pathspec change.
Junio C Hamano [Tue, 11 Apr 2006 01:14:54 +0000 (18:14 -0700)]
blame and friends: adjust to multiple pathspec change.

This makes things that include revision.h build again.

Blame is also built, but I am not sure how well it works (or how
well it worked to begin with) -- it was relying on tree-diff to
be using whatever pathspec was used the last time, which smells
a bit suspicious.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoRetire git-log.sh
Junio C Hamano [Tue, 11 Apr 2006 00:37:58 +0000 (17:37 -0700)]
Retire git-log.sh

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit log --full-diff
Junio C Hamano [Mon, 10 Apr 2006 23:40:40 +0000 (16:40 -0700)]
git log --full-diff

Without this flag, "git log -p paths..." shows commits that
touch the specified paths, and diffs about the same specified
paths.  With this, the full diff is shown for commits that touch
the specified paths.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agotree-diff: do not assume we use only one pathspec
Junio C Hamano [Mon, 10 Apr 2006 23:39:11 +0000 (16:39 -0700)]
tree-diff: do not assume we use only one pathspec

The way tree-diff was set up assumed we would use only one set
of pathspec during the entire life of the program.  Move the
pathspec related static variables out to diff_options structure
so that we can filter commits with one set of paths while show
the actual diffs using different set of paths.

I suspect this breaks blame.c, and makes "git log paths..." to
default to the --full-diff, the latter of which is dealt with
the next commit.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master' into jc/diff
Junio C Hamano [Mon, 10 Apr 2006 23:44:59 +0000 (16:44 -0700)]
Merge branch 'master' into jc/diff

* master:
  Make "--parents" logs also be incremental
  Retire diffcore-pathspec.
  Improve the git-diff-tree -c/-cc documentation

18 years agoMerge branch 'lt/rev'
Junio C Hamano [Mon, 10 Apr 2006 22:58:41 +0000 (15:58 -0700)]
Merge branch 'lt/rev'

* lt/rev:
  Make "--parents" logs also be incremental

18 years agoRetire diffcore-pathspec.
Junio C Hamano [Mon, 10 Apr 2006 22:57:24 +0000 (15:57 -0700)]
Retire diffcore-pathspec.

Nobody except diff-stages used it -- the callers instead filtered
the input to diffcore themselves.  Make diff-stages do that as
well and retire diffcore-pathspec.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoImprove the git-diff-tree -c/-cc documentation
Petr Baudis [Sun, 9 Apr 2006 13:43:17 +0000 (15:43 +0200)]
Improve the git-diff-tree -c/-cc documentation

This tries to clarify the -c/-cc documentation and clean up the style and
grammar.

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/diff' into next
Junio C Hamano [Sun, 9 Apr 2006 09:07:33 +0000 (02:07 -0700)]
Merge branch 'jc/diff' into next

* jc/diff:
  git log [diff-tree options]...
  log-tree: separate major part of diff-tree.

18 years agogit log [diff-tree options]...
Junio C Hamano [Sun, 9 Apr 2006 08:59:03 +0000 (01:59 -0700)]
git log [diff-tree options]...

And this makes "git log" to take common diff-tree options, so
that it can be used as "git whatchanged".

The recent revision walker updates by Linus to make path
limiting low-latency helps this quite a bit.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agolog-tree: separate major part of diff-tree.
Junio C Hamano [Sun, 9 Apr 2006 08:11:11 +0000 (01:11 -0700)]
log-tree: separate major part of diff-tree.

This separates out the part that deals with one-commit diff-tree
(and --stdin form) into a separate log-tree module.

There are two goals with this.  The more important one is to be
able to make this part available to "git log --diff", so that we
can have a native "git whatchanged" command.  Another is to
simplify the commit log generation part simpler.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMake "--parents" logs also be incremental
Linus Torvalds [Sun, 9 Apr 2006 00:05:58 +0000 (17:05 -0700)]
Make "--parents" logs also be incremental

The parent rewriting feature caused us to create the whole history in one
go, and then simplify it later, because of how rewrite_parents() had been
written. However, with a little tweaking, it's perfectly possible to do
even that one incrementally.

Right now, this doesn't really much matter, because every user of
"--parents" will probably generally _also_ use "--topo-order", which will
cause the old non-incremental behaviour anyway. However, I'm hopeful that
we could make even the topological sort incremental, or at least
_partially_ so (for example, make it incremental up to the first merge).

In the meantime, this at least moves things in the right direction, and
removes a strange special case.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoxdiff/xdiffi.c: fix warnings about possibly uninitialized variables
Marco Roeland [Sat, 8 Apr 2006 15:27:20 +0000 (17:27 +0200)]
xdiff/xdiffi.c: fix warnings about possibly uninitialized variables

Compiling this module gave the following warnings (some double dutch!):

xdiff/xdiffi.c: In functie 'xdl_recs_cmp':
xdiff/xdiffi.c:298: let op: 'spl.i1' may be used uninitialized in this function
xdiff/xdiffi.c:298: let op: 'spl.i2' may be used uninitialized in this function
xdiff/xdiffi.c:219: let op: 'fbest1' may be used uninitialized in this function
xdiff/xdiffi.c:219: let op: 'bbest1' may be used uninitialized in this function

A superficial tracking of their usage, without deeper knowledge about the
algorithm, indeed confirms that there are code paths on which these
variables will be used uninitialized. In practice these code paths might never
be reached, but then these fixes will not change the algorithm. If these
code paths are ever reached we now at least have a predictable outcome. And
should the very small performance impact of these initializations be
noticeable, then they should at least be replaced by comments why certain
code paths will never be reached.

Some extra initializations in this patch now fix the warnings.

18 years agodiffcore-rename: fix merging back a broken pair.
Junio C Hamano [Sun, 9 Apr 2006 03:17:46 +0000 (20:17 -0700)]
diffcore-rename: fix merging back a broken pair.

When a broken pair is matched up by rename detector to be merged
back, we do not want to say it is "dissimilar" with the
similarity index.  The output should just say they were changed,
taking the break score left by the earlier diffcore-break run if
any.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff: fix output of total-rewrite diff.
Junio C Hamano [Sun, 9 Apr 2006 02:45:39 +0000 (19:45 -0700)]
diff: fix output of total-rewrite diff.

We did not read in the file data before emitting the
total-rewrite diff.  Noticed by Pasky.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-log: match rev-list --abbrev and --abbrev-commit
Junio C Hamano [Sat, 8 Apr 2006 05:34:11 +0000 (22:34 -0700)]
git-log: match rev-list --abbrev and --abbrev-commit

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master' into next
Junio C Hamano [Sat, 8 Apr 2006 01:03:07 +0000 (18:03 -0700)]
Merge branch 'master' into next

* master:
  GIT 1.3.0-rc3

18 years agoGIT 1.3.0-rc3 v1.3.0-rc3
Junio C Hamano [Sat, 8 Apr 2006 01:02:40 +0000 (18:02 -0700)]
GIT 1.3.0-rc3

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'kh/svn'
Junio C Hamano [Sat, 8 Apr 2006 01:01:55 +0000 (18:01 -0700)]
Merge branch 'kh/svn'

* kh/svn:
  git-svnimport: Don't assume that copied files haven't changed

18 years agoMerge branch 'jc/thinpack'
Junio C Hamano [Sat, 8 Apr 2006 01:00:16 +0000 (18:00 -0700)]
Merge branch 'jc/thinpack'

* jc/thinpack:
  Thin pack generation: optimization.

18 years agoMerge branch 'jc/date'
Junio C Hamano [Sat, 8 Apr 2006 01:00:06 +0000 (18:00 -0700)]
Merge branch 'jc/date'

* jc/date:
  date parsing: be friendlier to our European friends.

18 years agoMerge branch 'nh/http'
Junio C Hamano [Sat, 8 Apr 2006 00:59:36 +0000 (17:59 -0700)]
Merge branch 'nh/http'

* nh/http:
  Fix compile with expat, but an old curl version
  http-fetch: add optional DAV-based pack list

18 years agoMerge branch 'ew/rev-abbrev'
Junio C Hamano [Sat, 8 Apr 2006 00:59:10 +0000 (17:59 -0700)]
Merge branch 'ew/rev-abbrev'

* ew/rev-abbrev:
  rev-list --abbrev-commit

18 years agoMerge branch 'jc/blame'
Junio C Hamano [Sat, 8 Apr 2006 00:57:46 +0000 (17:57 -0700)]
Merge branch 'jc/blame'

* jc/blame:
  blame -S <ancestry-file>
  Match ofs/cnt types in diff interface.
  blame: use built-in xdiff
  combine-diff: move the code to parse hunk-header into common library.
  combine-diff: refactor built-in xdiff interface.
  combine-diff: use built-in xdiff.

18 years agoMerge branch 'master' into next
Junio C Hamano [Fri, 7 Apr 2006 23:53:06 +0000 (16:53 -0700)]
Merge branch 'master' into next

* master:
  gitk: Fix incorrect invocation of getmergediffline
  [PATCH] gitk: Fix searching for filenames in gitk
  count-delta: match get_delta_hdr_size() changes.
  check patch_delta bounds more carefully

18 years agoMerge git://git.kernel.org/pub/scm/gitk/gitk
Junio C Hamano [Fri, 7 Apr 2006 23:52:59 +0000 (16:52 -0700)]
Merge git://git.kernel.org/pub/scm/gitk/gitk

* git://git.kernel.org/pub/scm/gitk/gitk:
  gitk: Fix incorrect invocation of getmergediffline
  [PATCH] gitk: Fix searching for filenames in gitk

18 years agoMerge branch 'maint'
Junio C Hamano [Fri, 7 Apr 2006 23:51:55 +0000 (16:51 -0700)]
Merge branch 'maint'

* maint:
  count-delta: match get_delta_hdr_size() changes.
  check patch_delta bounds more carefully

18 years agocount-delta: match get_delta_hdr_size() changes. v1.2.6
Junio C Hamano [Fri, 7 Apr 2006 23:48:09 +0000 (16:48 -0700)]
count-delta: match get_delta_hdr_size() changes.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocheck patch_delta bounds more carefully
Nicolas Pitre [Fri, 7 Apr 2006 19:26:10 +0000 (15:26 -0400)]
check patch_delta bounds more carefully

Let's avoid going south with invalid delta data.

Signed-off-by: Nicolas Pitre <nico@cam.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/thinpack' into next
Junio C Hamano [Fri, 7 Apr 2006 09:13:20 +0000 (02:13 -0700)]
Merge branch 'jc/thinpack' into next

* jc/thinpack:
  Thin pack generation: optimization.

18 years agoMerge branch 'ew/rev-abbrev' into next
Junio C Hamano [Fri, 7 Apr 2006 09:12:55 +0000 (02:12 -0700)]
Merge branch 'ew/rev-abbrev' into next

* ew/rev-abbrev:
  rev-list --abbrev-commit

18 years agoMerge branch 'jc/blame' into next
Junio C Hamano [Fri, 7 Apr 2006 09:12:48 +0000 (02:12 -0700)]
Merge branch 'jc/blame' into next

* jc/blame:
  blame -S <ancestry-file>
  Match ofs/cnt types in diff interface.

18 years agoMerge branch 'kh/svn' into next
Junio C Hamano [Fri, 7 Apr 2006 09:12:18 +0000 (02:12 -0700)]
Merge branch 'kh/svn' into next

* kh/svn:
  git-svnimport: Don't assume that copied files haven't changed
  Add Documentation/technical/pack-format.txt
  Added Packing Heursitics IRC writeup.
  Add documentation for git-imap-send.

18 years agogit-svnimport: Don't assume that copied files haven't changed
Karl Hasselström [Fri, 7 Apr 2006 06:06:09 +0000 (08:06 +0200)]
git-svnimport: Don't assume that copied files haven't changed

Don't assume that a file that SVN claims was copied from somewhere
else is bit-for-bit identical with its parent, since SVN allows
changes to copied files before they are committed.

Without this fix, such copy-modify-commit operations causes the
imported file to lack the "modify" part -- that is, we get subtle data
corruption.

Signed-off-by: Karl Hasselström <kha@treskal.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agorev-list --abbrev-commit
Junio C Hamano [Fri, 7 Apr 2006 04:32:36 +0000 (21:32 -0700)]
rev-list --abbrev-commit

This should make --pretty=oneline a whole lot more readable for
people using 80-column terminals.  Originally from Eric Wong.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoThin pack generation: optimization.
Junio C Hamano [Thu, 6 Apr 2006 06:24:57 +0000 (23:24 -0700)]
Thin pack generation: optimization.

Jens Axboe noticed that recent "git push" has become very slow
since we made --thin transfer the default.

Thin pack generation to push a handful revisions that touch
relatively small number of paths out of huge tree was stupid; it
registered _everything_ from the excluded revisions.  As a
result, "Counting objects" phase was unnecessarily expensive.

This changes the logic to register the blobs and trees from
excluded revisions only for paths we are actually going to send
to the other end.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAdd Documentation/technical/pack-format.txt
Junio C Hamano [Fri, 7 Apr 2006 09:07:40 +0000 (02:07 -0700)]
Add Documentation/technical/pack-format.txt

... along with the previous one, pack-heuristics, by popular
demand.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAdded Packing Heursitics IRC writeup.
Jon Loeliger [Fri, 3 Mar 2006 01:19:29 +0000 (19:19 -0600)]
Added Packing Heursitics IRC writeup.

Signed-off-by: Jon Loeliger <jdl@jdl.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoblame -S <ancestry-file>
Junio C Hamano [Fri, 7 Apr 2006 06:58:51 +0000 (23:58 -0700)]
blame -S <ancestry-file>

This adds the -S <ancestry-file> option to blame, which is
needed by the CVS server emulation.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMatch ofs/cnt types in diff interface.
Junio C Hamano [Fri, 7 Apr 2006 05:29:55 +0000 (22:29 -0700)]
Match ofs/cnt types in diff interface.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAdd documentation for git-imap-send.
Mike McCormack [Thu, 6 Apr 2006 03:32:19 +0000 (12:32 +0900)]
Add documentation for git-imap-send.

Signed-off-by: Mike McCormack <mike@codeweavers.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master' into next
Junio C Hamano [Thu, 6 Apr 2006 01:22:19 +0000 (18:22 -0700)]
Merge branch 'master' into next

* master:
  blame.c: fix completely broken ancestry traversal.

18 years agoblame.c: fix completely broken ancestry traversal.
Junio C Hamano [Thu, 6 Apr 2006 01:21:17 +0000 (18:21 -0700)]
blame.c: fix completely broken ancestry traversal.

Recent revision.c updates completely broken the assignment of
blames by not rewriting commit->parents field unless explicitly
asked to by the caller.  The caller needs to set revs.parents.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogitk: Fix incorrect invocation of getmergediffline
Paul Mackerras [Thu, 6 Apr 2006 00:20:03 +0000 (10:20 +1000)]
gitk: Fix incorrect invocation of getmergediffline

Signed-off-by: Paul Mackerras <paulus@samba.org>
18 years ago[PATCH] gitk: Fix searching for filenames in gitk
Pavel Roskin [Wed, 5 Apr 2006 23:02:50 +0000 (19:02 -0400)]
[PATCH] gitk: Fix searching for filenames in gitk

findcont should not accept any arguments.

Signed-off-by: Pavel Roskin <proski@gnu.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
18 years agoMerge branch 'jc/date' into next
Junio C Hamano [Wed, 5 Apr 2006 22:47:29 +0000 (15:47 -0700)]
Merge branch 'jc/date' into next

* jc/date:
  date parsing: be friendlier to our European friends.
  Tweaks to make asciidoc play nice.
  git-commit: document --amend
  Avoid a crash if realloc returns a different pointer.
  Avoid a divide by zero if there's no messages to send.
  [PATCH] Provide configurable UI font for gitk
  [PATCH] gitk: Use git wrapper to run git-ls-remote.
  [PATCH] gitk: add key bindings for selecting first and last commit
  gitk: Add a help menu item to display key bindings
  [PATCH] gitk: allow goto heads
  gitk: replace parent and children arrays with lists

18 years agodate parsing: be friendlier to our European friends.
Junio C Hamano [Wed, 5 Apr 2006 22:31:12 +0000 (15:31 -0700)]
date parsing: be friendlier to our European friends.

This does three things, only applies to cases where the user
manually tries to override the author/commit time by environment
variables, with non-ISO, non-2822 format date-string:

 - Refuses to use the interpretation to put the date in the
   future; recent kernel history has a commit made with
   10/03/2006 which is recorded as October 3rd.

 - Adds '.' as the possible year-month-date separator.  We
   learned from our European friends on the #git channel that
   dd.mm.yyyy is the norm there.

 - When the separator is '.', we prefer dd.mm.yyyy over
   mm.dd.yyyy; otherwise mm/dd/yy[yy] takes precedence over
   dd/mm/yy[yy].

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoTweaks to make asciidoc play nice.
Francis Daly [Wed, 5 Apr 2006 22:25:17 +0000 (23:25 +0100)]
Tweaks to make asciidoc play nice.

Once the content has been generated, the formatting elves can reorder
it to be pretty...

Signed-off-by: Francis Daly <francis@daoine.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/blame' into next
Junio C Hamano [Wed, 5 Apr 2006 21:26:00 +0000 (14:26 -0700)]
Merge branch 'jc/blame' into next

* jc/blame:
  blame: use built-in xdiff

18 years agoMerge branch 'nh/http' into next
Junio C Hamano [Wed, 5 Apr 2006 21:25:57 +0000 (14:25 -0700)]
Merge branch 'nh/http' into next

* nh/http:
  Fix compile with expat, but an old curl version

18 years agoblame: use built-in xdiff
Junio C Hamano [Wed, 5 Apr 2006 19:23:29 +0000 (12:23 -0700)]
blame: use built-in xdiff

This removes the last use of external diff from core git suite.
Also addresses the use of index() -- elsewhere we tend to use
strchr().

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-commit: document --amend
Marco Roeland [Wed, 5 Apr 2006 20:28:28 +0000 (22:28 +0200)]
git-commit: document --amend

The "--amend" option is used to amend the tip of the current branch. This
documentation text was copied straight from the commit that implemented it.

Some minor format tweaks for asciidoc were taken from work by Francis Daly
in commit b0d08a5.. It looks good now also in the html page.

[jc: amended further to follow the recommendation by Francis in
commit 3070b60].

Signed-off-by: Marco Roeland <marco.roeland@xs4all.nl>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/clone'
Junio C Hamano [Wed, 5 Apr 2006 21:06:50 +0000 (14:06 -0700)]
Merge branch 'jc/clone'

* jc/clone:
  git-clone: fix handling of upsteram whose HEAD does not point at master.

18 years agoMerge branch 'pb/regex'
Junio C Hamano [Wed, 5 Apr 2006 21:06:26 +0000 (14:06 -0700)]
Merge branch 'pb/regex'

* pb/regex:
  On some platforms, certain headers need to be included before regex.h
  Support for pickaxe matching regular expressions

18 years agoAvoid a crash if realloc returns a different pointer.
Mike McCormack [Wed, 5 Apr 2006 14:22:19 +0000 (23:22 +0900)]
Avoid a crash if realloc returns a different pointer.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAvoid a divide by zero if there's no messages to send.
Mike McCormack [Wed, 5 Apr 2006 14:22:52 +0000 (23:22 +0900)]
Avoid a divide by zero if there's no messages to send.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoFix compile with expat, but an old curl version
Johannes Schindelin [Wed, 5 Apr 2006 14:22:40 +0000 (16:22 +0200)]
Fix compile with expat, but an old curl version

With an old curl version, git-http-push is not compiled. But git-http-fetch
still needs to be linked with expat if NO_EXPAT is not defined.

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge git://git.kernel.org/pub/scm/gitk/gitk
Junio C Hamano [Wed, 5 Apr 2006 19:45:17 +0000 (12:45 -0700)]
Merge git://git.kernel.org/pub/scm/gitk/gitk

* git://git.kernel.org/pub/scm/gitk/gitk:
  [PATCH] Provide configurable UI font for gitk
  [PATCH] gitk: Use git wrapper to run git-ls-remote.
  [PATCH] gitk: add key bindings for selecting first and last commit
  gitk: Add a help menu item to display key bindings
  [PATCH] gitk: allow goto heads
  gitk: replace parent and children arrays with lists