Code

git.git
18 years agoMerge branch 'master'
Junio C Hamano [Sun, 12 Feb 2006 13:03:40 +0000 (05:03 -0800)]
Merge branch 'master'

* master:
  Add howto about separating topics.
  Merge branch 'pb/repo'
  Add support for explicit type specifiers when calling git-repo-config
  Merge branch 'jc/fixdiff'
  diff-tree: do not default to -c
  Avoid using "git-var -l" until it gets fixed.
  t5500: adjust to change in pack-object reporting behaviour.
  Only call git-rerere if $GIT_DIR/rr-cache exists.
  Use a relative path for SVN importing
  fetch-clone progress: finishing touches.
  Fix fetch-clone in the presense of signals
  Make "git clone" pack-fetching download statistics better
  Make "git clone" less of a deathly quiet experience

18 years agoAdd howto about separating topics.
kent@lysator.liu.se [Sun, 12 Feb 2006 12:00:52 +0000 (13:00 +0100)]
Add howto about separating topics.

This howto consists of a footnote from an email by JC to the git
mailing list (<7vfyms0x4p.fsf@assigned-by-dhcp.cox.net>).

Signed-off-by: Kent Engstrom <kent@lysator.liu.se>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'pb/repo'
Junio C Hamano [Sun, 12 Feb 2006 13:02:30 +0000 (05:02 -0800)]
Merge branch 'pb/repo'

* pb/repo:
  Add support for explicit type specifiers when calling git-repo-config

18 years agoMerge branch 'jc/fixdiff'
Junio C Hamano [Sun, 12 Feb 2006 13:02:25 +0000 (05:02 -0800)]
Merge branch 'jc/fixdiff'

* jc/fixdiff:
  diff-tree: do not default to -c

18 years agoAvoid using "git-var -l" until it gets fixed.
Junio C Hamano [Sat, 11 Feb 2006 20:39:11 +0000 (12:39 -0800)]
Avoid using "git-var -l" until it gets fixed.

This is to be nicer to people with unusable GECOS field.

"git-var -l" is currently broken in that when used by a user who
does not have a usable GECOS field and has not corrected it by
exporting GIT_COMMITTER_NAME environment variable it dies when
it tries to output GIT_COMMITTER_IDENT (same thing for AUTHOR).

"git-pull" used "git-var -l" only because it needed to get a
configuration variable before "git-repo-config --get" was
introduced.  Use the latter tool designed exactly for this
purpose.

"git-sh-setup" used "git-var GIT_AUTHOR_IDENT" without actually
wanting to use its value.  The only purpose was to cause the
command to check and barf if the repository format version
recorded in the $GIT_DIR/config file is too new for us to deal
with correctly.  Instead, use "repo-config --get" on a random
property and see if it die()s, and check if the exit status is
128 (comes from die -- missing variable is reported with exit
status 1, so we can tell that case apart).

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/nostat'
Junio C Hamano [Sun, 12 Feb 2006 12:15:50 +0000 (04:15 -0800)]
Merge branch 'jc/nostat'

* jc/nostat:
  "assume unchanged" git: documentation.
  ls-files: split "show-valid-bit" into a different option.

18 years ago"assume unchanged" git: documentation.
Junio C Hamano [Sun, 12 Feb 2006 09:48:47 +0000 (01:48 -0800)]
"assume unchanged" git: documentation.

This updates documentation to describe the "assume unchanged"
behaviour.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agols-files: split "show-valid-bit" into a different option.
Junio C Hamano [Sun, 12 Feb 2006 09:47:57 +0000 (01:47 -0800)]
ls-files: split "show-valid-bit" into a different option.

To preserve compatibility with scripts that expect uppercase
letters to be shown, do not make '-t' to unconditionally show
the valid bit.  Introduce '-v' option for that.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAdd support for explicit type specifiers when calling git-repo-config
Petr Baudis [Sun, 12 Feb 2006 03:14:48 +0000 (04:14 +0100)]
Add support for explicit type specifiers when calling git-repo-config

Currently, git-repo-config will just return the raw value of option
as specified in the config file; this makes things difficult for scripts
calling it, especially if the value is supposed to be boolean.

This patch makes it possible to ask git-repo-config to check if the option
is of the given type (int or bool) and write out the value in its
canonical form. If you do not pass --int or --bool, the behaviour stays
unchanged and the raw value is emitted.

This also incidentally fixes the segfault when option with no value is
encountered.

[jc: tweaked the option parsing a bit to make it easier to see
 that the patch does not change anything but the type stuff in
 the diff output.  Also changed to avoid "foo ? : bar" construct. ]

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-tree: do not default to -c
Junio C Hamano [Sun, 12 Feb 2006 00:43:30 +0000 (16:43 -0800)]
diff-tree: do not default to -c

Marco says it breaks qgit.  This makes the flags a bit more
orthogonal.

  $ git-diff-tree -r --abbrev ca18

    No output from this command because you asked to skip merge by
    not having -m there.

  $ git-diff-tree -r -m --abbrev ca18
  ca182053c7710a286d72102f4576cf32e0dafcfb
  :100644 100644 538d21d... 59042d1... M Makefile
  :100644 100644 410b758... 6c47c3a... M entry.c
  ca182053c7710a286d72102f4576cf32e0dafcfb
  :100644 100644 30479b4... 59042d1... M Makefile

    The same "independent sets of diff" as before without -c.

  $ git-diff-tree -r -m -c --abbrev ca18
  ca182053c7710a286d72102f4576cf32e0dafcfb
  ::100644 100644 100644 538d21d... 30479b4... 59042d1... MM Makefile

    Combined.

  $ git-diff-tree -r -c --abbrev ca18
  ca182053c7710a286d72102f4576cf32e0dafcfb
  ::100644 100644 100644 538d21d... 30479b4... 59042d1... MM Makefile

    Asking for combined without -m does not make sense, so -c
    implies -m.

We need to supply -c as default to whatchanged, which is a
one-liner.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agot5500: adjust to change in pack-object reporting behaviour.
Junio C Hamano [Sun, 12 Feb 2006 07:08:23 +0000 (23:08 -0800)]
t5500: adjust to change in pack-object reporting behaviour.

Now pack-object is not as chatty when its stderr is not connected
to a terminal, so the test needs to be adjusted for that.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoOnly call git-rerere if $GIT_DIR/rr-cache exists.
Junio C Hamano [Sun, 12 Feb 2006 02:55:43 +0000 (18:55 -0800)]
Only call git-rerere if $GIT_DIR/rr-cache exists.

Johannes noticed that git-rerere depends on Digest.pm, and if
one does not use the command, one can live without it.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoUse a relative path for SVN importing
Christian Biesinger [Sat, 11 Feb 2006 15:44:11 +0000 (16:44 +0100)]
Use a relative path for SVN importing

The absolute path (with the leading slash) breaks SVN importing,
because it then looks for /trunk/... instead of /svn/trunk/...
(in my case, the repository URL was https://servername/svn/)

Signed-off-by: Christian Biesinger <cbiesinger@web.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agofetch-clone progress: finishing touches.
Junio C Hamano [Sun, 12 Feb 2006 01:54:18 +0000 (17:54 -0800)]
fetch-clone progress: finishing touches.

This makes fetch-pack also report the progress of packing part.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoFix fetch-clone in the presense of signals
Linus Torvalds [Sat, 11 Feb 2006 18:41:22 +0000 (10:41 -0800)]
Fix fetch-clone in the presense of signals

We shouldn't fail a fetch just because a signal might have interrupted
the read.

Normally, we don't install any signal handlers, so EINTR really shouldn't
happen. That said, really old versions of Linux will interrupt an
interruptible system call even for signals that turn out to be ignored
(SIGWINCH is the classic example - resizing your xterm would cause it).
The same might well be true elsewhere too.

Also, since receive_keep_pack() doesn't control the caller, it can't know
that no signal handlers exist.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMake "git clone" pack-fetching download statistics better
Linus Torvalds [Sat, 11 Feb 2006 18:43:56 +0000 (10:43 -0800)]
Make "git clone" pack-fetching download statistics better

Average it out over a few events to make the numbers stable, and fix the
silly usec->binary-ms conversion.

Yeah, yeah, it's arguably eye-candy to keep the user calm, but let's do
that right.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMake "git clone" less of a deathly quiet experience
Linus Torvalds [Sat, 11 Feb 2006 04:31:09 +0000 (20:31 -0800)]
Make "git clone" less of a deathly quiet experience

It used to be that "git-unpack-objects" would give nice percentages, but
now that we don't unpack the initial clone pack any more, it doesn't. And
I'd love to do that nice percentage view in the pack objects downloader
too, but the thing doesn't even read the pack header, much less know how
much it's going to get, so I was lazy and didn't.

Instead, it at least prints out how much data it's gotten, and what the
packing speed is. Which makes the user realize that it's actually doing
something useful instead of sitting there silently (and if the recipient
knows how large the final result is, he can at least make a guess about
when it migt be done).

So with this patch, I get something like this on my DSL line:

[torvalds@g5 ~]$ time git clone master.kernel.org:/pub/scm/linux/kernel/git/torvalds/linux-2.6 clone-test
Packing 188543 objects
  48.398MB  (154 kB/s)

where even the speed approximation seems to be roughtly correct (even
though my algorithm is a truly stupid one, and only really gives "speed in
the last half second or so").

Anyway, _something_ like this is definitely needed. It could certainly be
better (if it showed the same kind of thing that git-unpack-objects did,
that would be much nicer, but would require parsing the object stream as
it comes in). But this is  big step forward, I think.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master'
Junio C Hamano [Sat, 11 Feb 2006 03:12:57 +0000 (19:12 -0800)]
Merge branch 'master'

* master:
  Define GIT_(AUTHOR|COMMITTER)_(NAME|EMAIL) to known values.
  Merge branch 'lt/diff-tree'
  git-commit -v: have patch at the end.

18 years agoDefine GIT_(AUTHOR|COMMITTER)_(NAME|EMAIL) to known values.
Junio C Hamano [Sat, 11 Feb 2006 03:11:23 +0000 (19:11 -0800)]
Define GIT_(AUTHOR|COMMITTER)_(NAME|EMAIL) to known values.

Without these, running tests with an account with empty gecos
field would fail.

We might want to loosen error from "git-var -l" (but not
"git-var GIT_AUTHOR_NAME") later, but that is more or less an
independent issue.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'lt/diff-tree'
Junio C Hamano [Sat, 11 Feb 2006 02:47:41 +0000 (18:47 -0800)]
Merge branch 'lt/diff-tree'

* lt/diff-tree:
  combine-diff: Record diff status a bit more faithfully
  find_unique_abbrev() simplification.
  combine-diff: move formatting logic to show_combined_diff()
  combined-diff: use diffcore before intersecting paths.
  diff-tree -c raw output

18 years agogit-commit -v: have patch at the end.
Junio C Hamano [Sat, 11 Feb 2006 02:38:24 +0000 (18:38 -0800)]
git-commit -v: have patch at the end.

It was pointed out that otherwise more important summary
information prefixed with '#' would become prone to be missed.

Also instead of chopping at the first '^---$' line, stop at the
first 'diff --git a/' line.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master'
Junio C Hamano [Fri, 10 Feb 2006 19:57:08 +0000 (11:57 -0800)]
Merge branch 'master'

* master:
  rev-list: default to abbreviate merge parent names under --pretty.
  delta micro optimization
  count-delta.c: comment fixes
  Merge branch 'jc/empty-commit'

18 years agorev-list: default to abbreviate merge parent names under --pretty.
Junio C Hamano [Fri, 10 Feb 2006 19:56:42 +0000 (11:56 -0800)]
rev-list: default to abbreviate merge parent names under --pretty.

When we prettyprint commit log messages, merge parent names were
often very long and there was no way to abbreviate it.

This changes them to be abbreviated by default, and non-default
abbreviations can be specified with --no-abbrev or --abbrev=<n>
options.

Note that this affects only the prettyprinted parent names.  The
output from --show-parents is meant for machine consumption and
is not affected by this flag.

18 years agodelta micro optimization
Nicolas Pitre [Fri, 10 Feb 2006 18:42:05 +0000 (13:42 -0500)]
delta micro optimization

My kernel work habit made me look at the generated assembly for the
delta code, and one obvious albeit small improvement is this patch.

Signed-off-by: Nicolas Pitre <nico@cam.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocount-delta.c: comment fixes
Nicolas Pitre [Fri, 10 Feb 2006 15:20:40 +0000 (10:20 -0500)]
count-delta.c: comment fixes

There was a stale comment that explains why the old code could
undercount when delta data copied things around inside detination
buffer.  We do not use that kind of delta, so the comment does
not apply.

18 years agoMerge branch 'jc/empty-commit'
Junio C Hamano [Fri, 10 Feb 2006 15:14:55 +0000 (07:14 -0800)]
Merge branch 'jc/empty-commit'

* jc/empty-commit:
  t6000: fix a careless test library add-on.
  Do not allow empty name or email.

18 years agoMerge branch 'lt/diff-tree'
Junio C Hamano [Fri, 10 Feb 2006 14:51:28 +0000 (06:51 -0800)]
Merge branch 'lt/diff-tree'

* lt/diff-tree:
  combine-diff: Record diff status a bit more faithfully
  find_unique_abbrev() simplification.

18 years agocombine-diff: Record diff status a bit more faithfully
Junio C Hamano [Fri, 10 Feb 2006 10:30:52 +0000 (02:30 -0800)]
combine-diff: Record diff status a bit more faithfully

This shows "new file mode XXXX" and "deleted file mode XXXX"
lines like two-way diff-patch output does, by checking the
status from each parent.

The diff-raw output for combined diff is made a bit uglier by
showing diff status letters with each parent.  While most of the
case you would see "MM" in the output, an Evil Merge that
touches a path that was added by inheriting from one parent is
possible and it would be shown like these:

    $ git-diff-tree --abbrev -c HEAD
    2d7ca89675eb8888b0b88a91102f096d4471f09f
    ::000000 000000 100644 0000000... 0000000... 31dd686... AA b
    ::000000 100644 100644 0000000... 6c884ae... c6d4fa8... AM d
    ::100644 100644 100644 4f7cbe7... f8c295c... 19d5d80... RR e

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agofind_unique_abbrev() simplification.
Junio C Hamano [Fri, 10 Feb 2006 09:51:12 +0000 (01:51 -0800)]
find_unique_abbrev() simplification.

Earlier it did not grok the 0{40} SHA1 very well, but what it
needed to do was to find the shortest 0{N} that is not used as a
valid object name to be consistent with the way names of valid
objects are abbreviated.  This makes some users simpler.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/status'
Junio C Hamano [Fri, 10 Feb 2006 08:55:34 +0000 (00:55 -0800)]
Merge branch 'jc/status'

* jc/status:
  git-status -v

18 years agogit-status -v
Junio C Hamano [Fri, 10 Feb 2006 08:45:59 +0000 (00:45 -0800)]
git-status -v

This revamps the git-status command to take the same set of
parameters as git commit.  It gives a preview of what is being
committed with that command.  With -v flag, it shows the diff
output between the HEAD commit and the index that would be
committed if these flags were given to git-commit command.

git-commit also acquires -v flag (it used to mean "verify" but
that is the default anyway and there is --no-verify to turn it
off, so not much is lost), which uses the updated git-status -v
to seed the commit log buffer.  This is handy for writing a log
message while reviewing the changes one last time.

Now, git-commit and git-status are internally share the same
implementation.

Unlike previous git-commit change, this uses a temporary index
to prepare the index file that would become the real index file
after a successful commit, and moves it to the real index file
once the commit is actually made.  This makes it safer than the
previous scheme, which stashed away the original index file and
restored it after an aborted commit.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'master'
Junio C Hamano [Fri, 10 Feb 2006 06:19:21 +0000 (22:19 -0800)]
Merge branch 'master'

* master:
  Merge branch 'jc/ls-files-o'
  count-delta.c: Match the delta data semantics change in version 3.
  remove delta-against-self bit
  stat() for existence in safe_create_leading_directories()
  call git_config() after setup_git_directory()
  Add --diff-filter= documentation paragraph

18 years agoMerge branch 'jc/ls-files-o'
Junio C Hamano [Fri, 10 Feb 2006 06:19:07 +0000 (22:19 -0800)]
Merge branch 'jc/ls-files-o'

* jc/ls-files-o:
  ls-files: honour per-directory ignore file from higher directories.

18 years agoMerge branch 'lt/diff-tree'
Junio C Hamano [Fri, 10 Feb 2006 05:10:52 +0000 (21:10 -0800)]
Merge branch 'lt/diff-tree'

* lt/diff-tree:
  combine-diff: move formatting logic to show_combined_diff()
  combined-diff: use diffcore before intersecting paths.
  diff-tree -c raw output

18 years agocount-delta.c: Match the delta data semantics change in version 3.
Junio C Hamano [Fri, 10 Feb 2006 01:15:59 +0000 (17:15 -0800)]
count-delta.c: Match the delta data semantics change in version 3.

This matches the count_delta() logic to the change previous
commit introduces to patch_delta().

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoremove delta-against-self bit
Nicolas Pitre [Thu, 9 Feb 2006 22:50:04 +0000 (17:50 -0500)]
remove delta-against-self bit

After experimenting with code to add the ability to encode a delta
against part of the deltified file, it turns out that resulting packs
are _bigger_ than when this ability is not used.  The raw delta output
might be smaller, but it doesn't compress as well using gzip with a
negative net saving on average.

Said bit would in fact be more useful to allow for encoding the copying
of chunks larger than 64KB providing more savings with large files.
This will correspond to packs version 3.

While the current code still produces packs version 2, it is made future
proof so pack versions 2 and 3 are accepted.  Any pack version 2 are
compatible with version 3 since the redefined bit was never used before.
When enough time has passed, code to use that bit to produce version 3
packs could be added.

Signed-off-by: Nicolas Pitre <nico@cam.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agostat() for existence in safe_create_leading_directories()
Jason Riedy [Fri, 10 Feb 2006 01:56:13 +0000 (17:56 -0800)]
stat() for existence in safe_create_leading_directories()

Use stat() to explicitly check for existence rather than
relying on the non-portable EEXIST error in sha1_file.c's
safe_create_leading_directories().  There certainly are
optimizations possible, but then the code becomes almost
the same as that in coreutil's lib/mkdir-p.c.

Other uses of EEXIST seem ok.  Tested on Solaris 8, AIX 5.2L,
and a few Linux versions.  AIX has some unrelated (I think)
failures right now; I haven't tried many recent gits there.
Anyone have an old Ultrix box to break everything?  ;)

Also remove extraneous #includes.  Everything's already in
git-compat-util.h, included through cache.h.

Signed-off-by: Jason Riedy <ejr@cs.berkeley.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombine-diff: move formatting logic to show_combined_diff()
Junio C Hamano [Thu, 9 Feb 2006 23:23:06 +0000 (15:23 -0800)]
combine-diff: move formatting logic to show_combined_diff()

This way, diff-files can make use of it.  Also implement the
full suite of what diff_flush_raw() supports just for
consistency.  With this, 'diff-tree -c -r --name-status' would
show what is expected.

There is no way to get the historical output (useful for
debugging and low-level Plumbing work) anymore, so tentatively
it makes '-m' to mean "do not combine and show individual diffs
with parents".

diff-files matches diff-tree to produce raw output for -c.  For
textual combined diff, use -p -c.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocall git_config() after setup_git_directory()
Junio C Hamano [Thu, 9 Feb 2006 22:41:39 +0000 (14:41 -0800)]
call git_config() after setup_git_directory()

If you call setup_git_directory() to work from a subdirectory,
that should be run first before running git_config().  Otherwise
you would not read the configuration file from the correct place.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombined-diff: use diffcore before intersecting paths.
Junio C Hamano [Thu, 9 Feb 2006 22:35:19 +0000 (14:35 -0800)]
combined-diff: use diffcore before intersecting paths.

This is needed to make "diff-tree -c -M" to work semi-sensibly.
Otherwise rename detection, pickaxe and friends would never be
invoked.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoAdd --diff-filter= documentation paragraph
Jon Loeliger [Thu, 9 Feb 2006 15:12:11 +0000 (09:12 -0600)]
Add --diff-filter= documentation paragraph

Signed-off-by: Jon Loeliger <jdl@jdl.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-tree -c raw output
Linus Torvalds [Thu, 9 Feb 2006 18:30:28 +0000 (10:30 -0800)]
diff-tree -c raw output

NOTE! This makes "-c" be the default, which effectively means that merges
are never ignored any more, and "-m" is a no-op. So it changes semantics.

I would also like to make "--cc" the default if you do patches, but didn't
actually do that.

The raw output format is not wonderfully pretty, but it's distinguishable
from a "normal patch" in that a normal patch with just one parent has just
one colon at the beginning, while a multi-parent raw diff has <n> colons
for <n> parents.

So now, in the kernel, when you do

git-diff-tree cce0cac125623f9b68f25dd1350f6d616220a8dd

(to see the manual ARM merge that had a conflict in arch/arm/Kconfig), you
get

cce0cac125623f9b68f25dd1350f6d616220a8dd
::100644 100644 100644 4a63a8e2e45247a11c068c6ed66c6e7aba29ddd9 77eee38762d69d3de95ae45dd9278df9b8225e2c 2f61726d2f4b636f6e66696700dbf71a59dad287       arch/arm/Kconfig

ie you see two colons (two parents), then three modes (parent modes
followed by result mode), then three sha1s (parent sha1s followed by
result sha1).

Which is pretty close to the normal raw diff output.

Cool/stupid exercise:

$ git-whatchanged | grep '^::' | cut -f2- | sort |
  uniq -c | sort -n | less -S

will show which files have needed the most file-level merge conflict
resolution. Useful? Probably not. But kind of interesting.

For the kernel, it's

     ....
     10 arch/ia64/Kconfig
     11 drivers/scsi/Kconfig
     12 drivers/net/Makefile
     17 include/linux/libata.h
     18 include/linux/pci_ids.h
     23 drivers/net/Kconfig
     24 drivers/scsi/libata-scsi.c
     28 drivers/scsi/libata-core.c
     43 MAINTAINERS

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/nostat'
Junio C Hamano [Thu, 9 Feb 2006 08:55:45 +0000 (00:55 -0800)]
Merge branch 'jc/nostat'

* jc/nostat:
  "Assume unchanged" git: --really-refresh fix.

18 years ago"Assume unchanged" git: --really-refresh fix.
Junio C Hamano [Thu, 9 Feb 2006 08:55:17 +0000 (00:55 -0800)]
"Assume unchanged" git: --really-refresh fix.

The earlier round failed to make --really-refresh to mark
up-to-date index entry to valid again due to a trivial thinko.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/ls-files-o'
Junio C Hamano [Thu, 9 Feb 2006 08:21:27 +0000 (00:21 -0800)]
Merge branch 'jc/ls-files-o'

* jc/ls-files-o:
  ls-files: honour per-directory ignore file from higher directories.

18 years agols-files: honour per-directory ignore file from higher directories.
Junio C Hamano [Thu, 9 Feb 2006 08:08:31 +0000 (00:08 -0800)]
ls-files: honour per-directory ignore file from higher directories.

When git-ls-files -o --exclude-per-directory=.gitignore is run
from a subdirectory, it did not read from .gitignore from its
parent directory.  Reading from them makes output from these two
commands consistent:

    $ git ls-files -o --exclude-per-directory=.gitignore Documentation
    $ cd Documentation &&
      git ls-files -o --exclude-per-directory=.gitignore

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branches 'jc/nostat' and 'jc/empty-commit'
Junio C Hamano [Thu, 9 Feb 2006 05:56:05 +0000 (21:56 -0800)]
Merge branches 'jc/nostat' and 'jc/empty-commit'

* jc/nostat:
  ls-files: debugging aid for CE_VALID changes.
  "Assume unchanged" git: do not set CE_VALID with --refresh
  "Assume unchanged" git

* jc/empty-commit:
  t6000: fix a careless test library add-on.
  Do not allow empty name or email.

18 years agot6000: fix a careless test library add-on.
Junio C Hamano [Tue, 7 Feb 2006 23:35:46 +0000 (15:35 -0800)]
t6000: fix a careless test library add-on.

It tried to "restore" GIT_AUTHOR_EMAIL environment variable but
the variable started out as unset, so ended up setting it to an
empty string.  This is now caught as an error.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDo not allow empty name or email.
Junio C Hamano [Tue, 7 Feb 2006 21:19:10 +0000 (13:19 -0800)]
Do not allow empty name or email.

Instead of silently allowing to create a bogus commit that lacks
information by mistake, complain loudly and die.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agols-files: debugging aid for CE_VALID changes.
Junio C Hamano [Thu, 9 Feb 2006 05:50:18 +0000 (21:50 -0800)]
ls-files: debugging aid for CE_VALID changes.

This is not really part of the proposed updates for CE_VALID,
but with this change, ls-files -t shows CE_VALID paths with
lowercase tag letters instead of the usual uppercase.  Useful
for checking out what is going on.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years ago"Assume unchanged" git: do not set CE_VALID with --refresh
Junio C Hamano [Thu, 9 Feb 2006 05:49:47 +0000 (21:49 -0800)]
"Assume unchanged" git: do not set CE_VALID with --refresh

When working with automatic assume-unchanged mode using
core.ignorestat, setting CE_VALID after --refresh makes things
more cumbersome to use.  Consider this scenario:

 (1) the working tree is on a filesystem with slow lstat(2).
     The user sets core.ignorestat = true.

 (2) "git checkout" to switch to a different branch (or initial
     checkout) updates all paths and the index starts out with
     "all clean".

 (3) The user knows she wants to edit certain paths.  She uses
     update-index --no-assume-unchanged (we could call it --edit;
     the name is inmaterial) to mark these paths and starts
     editing.

 (4) After editing half of the paths marked to be edited, she
     runs "git status".  This runs "update-index --refresh" to
     reduce the false hits from diff-files.

 (5) Now the other half of the paths, since she has not changed
     them, are found to match the index, and CE_VALID is set on
     them again.

For this reason, this commit makes update-index --refresh not to
set CE_VALID even after the path without CE_VALID are verified
to be up to date.  The user still can run --really-refresh to
force lstat() to match the index entries to the reality.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years ago"Assume unchanged" git
Junio C Hamano [Thu, 9 Feb 2006 05:15:24 +0000 (21:15 -0800)]
"Assume unchanged" git

This adds "assume unchanged" logic, started by this message in the list
discussion recently:

<Pine.LNX.4.64.0601311807470.7301@g5.osdl.org>

This is a workaround for filesystems that do not have lstat()
that is quick enough for the index mechanism to take advantage
of.  On the paths marked as "assumed to be unchanged", the user
needs to explicitly use update-index to register the object name
to be in the next commit.

You can use two new options to update-index to set and reset the
CE_VALID bit:

git-update-index --assume-unchanged path...
git-update-index --no-assume-unchanged path...

These forms manipulate only the CE_VALID bit; it does not change
the object name recorded in the index file.  Nor they add a new
entry to the index.

When the configuration variable "core.ignorestat = true" is set,
the index entries are marked with CE_VALID bit automatically
after:

 - update-index to explicitly register the current object name to the
   index file.

 - when update-index --refresh finds the path to be up-to-date.

 - when tools like read-tree -u and apply --index update the working
   tree file and register the current object name to the index file.

The flag is dropped upon read-tree that does not check out the index
entry.  This happens regardless of the core.ignorestat settings.

Index entries marked with CE_VALID bit are assumed to be
unchanged most of the time.  However, there are cases that
CE_VALID bit is ignored for the sake of safety and usability:

 - while "git-read-tree -m" or git-apply need to make sure
   that the paths involved in the merge do not have local
   modifications.  This sacrifices performance for safety.

 - when git-checkout-index -f -q -u -a tries to see if it needs
   to checkout the paths.  Otherwise you can never check
   anything out ;-).

 - when git-update-index --really-refresh (a new flag) tries to
   see if the index entry is up to date.  You can start with
   everything marked as CE_VALID and run this once to drop
   CE_VALID bit for paths that are modified.

Most notably, "update-index --refresh" honours CE_VALID and does
not actively stat, so after you modified a file in the working
tree, update-index --refresh would not notice until you tell the
index about it with "git-update-index path" or "git-update-index
--no-assume-unchanged path".

This version is not expected to be perfect.  I think diff
between index and/or tree and working files may need some
adjustment, and there probably needs other cases we should
automatically unmark paths that are marked to be CE_VALID.

But the basics seem to work, and ready to be tested by people
who asked for this feature.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years ago.gitignore git-rerere and config.mak
Andreas Ericsson [Tue, 7 Feb 2006 17:21:02 +0000 (18:21 +0100)]
.gitignore git-rerere and config.mak

Signed-off-by: Andreas Ericsson <ae@op5.se>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoFix "git diff a..b" breakage
Linus Torvalds [Tue, 7 Feb 2006 18:26:41 +0000 (10:26 -0800)]
Fix "git diff a..b" breakage

The "--cc" implies "-p", but without the recursive part.

Linus

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoBasic documentation for git-show
Petr Baudis [Tue, 7 Feb 2006 11:47:47 +0000 (12:47 +0100)]
Basic documentation for git-show

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocument git-diff-tree --always
Petr Baudis [Tue, 7 Feb 2006 11:47:44 +0000 (12:47 +0100)]
Document git-diff-tree --always

Signed-off-by: Petr Baudis <pasky@suse.cz>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agohttp-fetch: Abort requests for objects which arrived in packs
Mark Wooding [Tue, 7 Feb 2006 10:07:39 +0000 (10:07 +0000)]
http-fetch: Abort requests for objects which arrived in packs

In fetch_object, there's a call to release an object request if the
object mysteriously arrived, say in a pack.  Unfortunately, the fetch
attempt for this object might already be in progress, and we'll leak the
descriptor.  Instead, try to tidy away the request.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoformat-patch: Remove last vestiges of --mbox option
Andreas Ericsson [Tue, 7 Feb 2006 09:37:54 +0000 (09:37 +0000)]
format-patch: Remove last vestiges of --mbox option

Don't mention it in docs or --help output.
Remove mbox, date and author variables from git-format-patch.sh.

Use DESCRIPTION text from man-page to update LONG_USAGE output. It's
a bit silly to have two texts saying the same thing in different words,
and I'm too lazy to update both.

Signed-off-by: Andreas Ericsson <ae@op5.se>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge git://git.kernel.org/pub/scm/gitk/gitk
Junio C Hamano [Tue, 7 Feb 2006 09:19:49 +0000 (01:19 -0800)]
Merge git://git.kernel.org/pub/scm/gitk/gitk

* git://git.kernel.org/pub/scm/gitk/gitk:
  gitk: Use git-diff-tree --cc for showing the diffs for merges
  gitk: Add braces around if expressions

18 years agogit-commit: finishing touches.
Junio C Hamano [Mon, 6 Feb 2006 00:08:01 +0000 (16:08 -0800)]
git-commit: finishing touches.

Introduce --only flag to allow the new "partial commit"
semantics when paths are specified.  The default is still the
traditional --include semantics.  Once peoples' fingers and
scripts that want the traditional behaviour are updated to
explicitly say --include, we could change it to either default
to --only, or refuse to operate without either --only/--include
when paths are specified.

This also fixes a couple of bugs in the previous round.  Namely:

 - forgot to save/restore index in some cases.

 - forgot to use the temporary index to show status when '--only
   paths...' semantics was used.

 - --author did not take precedence when reusing an existing
   commit.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-commit: revamp the git-commit semantics.
Junio C Hamano [Sun, 5 Feb 2006 08:07:44 +0000 (00:07 -0800)]
git-commit: revamp the git-commit semantics.

 - "git commit" without _any_ parameter keeps the traditional
   behaviour.  It commits the current index.

   We commit the whole index even when this form is run from a
   subdirectory.

 - "git commit --include paths..." (or "git commit -i paths...")
   is equivalent to:

    git update-index --remove paths...
        git commit

 - "git commit paths..." acquires a new semantics.  This is an
   incompatible change that needs user training, which I am
   still a bit reluctant to swallow, but enough people seem to
   have complained that it is confusing to them.  It

   1. refuses to run if $GIT_DIR/MERGE_HEAD exists, and reminds
      trained git users that the traditional semantics now needs
      -i flag.

   2. refuses to run if named paths... are different in HEAD and
      the index (ditto about reminding).  Added paths are OK.

   3. reads HEAD commit into a temporary index file.

   4. updates named paths... from the working tree in this
      temporary index.

   5. does the same updates of the paths... from the working
      tree to the real index.

   6. makes a commit using the temporary index that has the
      current HEAD as the parent, and updates the HEAD with this
      new commit.

 - "git commit --all" can run from a subdirectory, but it updates
   the index with all the modified files and does a whole tree
   commit.

 - In all cases, when the command decides not to create a new
   commit, the index is left as it was before the command is
   run.  This means that the two "git diff" in the following
   sequence:

       $ git diff
       $ git commit -a
       $ git diff

   would show the same diff if you abort the commit process by
   making the commit log message empty.

This commit also introduces much requested --author option.

$ git commit --author 'A U Thor <author@example.com>'

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-rerere: reuse recorded resolve.
Junio C Hamano [Sun, 29 Jan 2006 07:15:24 +0000 (23:15 -0800)]
git-rerere: reuse recorded resolve.

In a workflow that employs relatively long lived topic branches,
the developer sometimes needs to resolve the same conflict over
and over again until the topic branches are done (either merged
to the "release" branch, or sent out and accepted upstream).

This commit introduces a new command, "git rerere", to help this
process by recording the conflicted automerge results and
corresponding hand-resolve results on the initial manual merge,
and later by noticing the same conflicted automerge and applying
the previously recorded hand resolution using three-way merge.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agofmt-merge-msg: show summary of what is merged.
Junio C Hamano [Sat, 4 Feb 2006 03:20:16 +0000 (19:20 -0800)]
fmt-merge-msg: show summary of what is merged.

In addition to the branch names, populate the log message with
one-line description from actual commits that are being merged.

This was prompted by Len's 12-way octopus.  You need to have
'merge.summary' in the configuration file to enable it:

$ git repo-config merge.summary yes

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoread-tree --aggressive
Junio C Hamano [Sat, 4 Feb 2006 06:04:14 +0000 (22:04 -0800)]
read-tree --aggressive

A new flag --aggressive resolves what we traditionally resolved
with external git-merge-one-file inside index while read-tree
3-way merge works.

git-merge-octopus and git-merge-resolve use this flag before
running git-merge-index with git-merge-one-file.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years ago[PATCH] mailinfo: reset CTE after each multipart
Junio C Hamano [Tue, 7 Feb 2006 05:35:25 +0000 (21:35 -0800)]
[PATCH] mailinfo: reset CTE after each multipart

If the first part uses quoted-printable to protect iso8859-1
name in the commit log, and the second part was plain ascii text
patchfile without even Content-Transfer-Encoding subheader, we
incorrectly tried to decode the patch as quoted printable.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocs: minor git-push copyediting
J. Bruce Fields [Mon, 6 Feb 2006 00:42:27 +0000 (19:42 -0500)]
Docs: minor git-push copyediting

Minor git-push copyediting

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocs: move git url and remotes text to separate sections
J. Bruce Fields [Sun, 5 Feb 2006 23:29:49 +0000 (18:29 -0500)]
Docs: move git url and remotes text to separate sections

The sections on git urls and remotes files in the git-fetch,
git-pull, and git-push manpages seem long enough to be worth a
manpage section of their own.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDocs: split up pull-fetch-param.txt
J. Bruce Fields [Sun, 5 Feb 2006 22:43:47 +0000 (17:43 -0500)]
Docs: split up pull-fetch-param.txt

The push and pull man pages include a bunch of shared text from
pull-fetch-param.txt.  This simplifies maintenance somewhat, but
there's actually quite a bit of text that applies only to one or the
other.

So, separate out the push- and pull/fetch-specific text into
pull-fetch-param.txt and git-push.txt, then include the largest chunk
of common stuff (the description of protocols and url's) from
urls.txt.  That cuts some irrelevant stuff from the man pages without
making us duplicate too much.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombine-diff: do not punt on removed or added files.
Junio C Hamano [Tue, 7 Feb 2006 02:54:08 +0000 (18:54 -0800)]
combine-diff: do not punt on removed or added files.

When we remove a file, the parents' contents are all removed so
it is not that interesting to show all of them, but the fact it
was removed when all parents had it *is* unusual.  When we add a
file, similarly the fact it was added when no parent wanted it
*is* unusual, and in addition the result matters, so show it.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogitk: Use git-diff-tree --cc for showing the diffs for merges
Paul Mackerras [Mon, 6 Feb 2006 22:13:52 +0000 (09:13 +1100)]
gitk: Use git-diff-tree --cc for showing the diffs for merges

This replaces a lot of code that used the result from several 2-way
diffs to generate a combined diff for a merge.  Now we just use
git-diff-tree --cc and colorize the output a bit, which is a lot
simpler, and has the enormous advantage that if the diff doesn't
show quite what someone thinks it should show, I can deflect the
blame to someone else. :)

Signed-off-by: Paul Mackerras <paulus@samba.org>
18 years agogitk: Add braces around if expressions
Paul Mackerras [Mon, 6 Feb 2006 22:10:18 +0000 (09:10 +1100)]
gitk: Add braces around if expressions

Apparently this simplifies things for the parser/compiler and makes
it go slightly faster (since without the braces, it potentially has
to do two levels of substitutions rather than one).

Signed-off-by: Paul Mackerras <paulus@samba.org>
18 years agocombine-diff: show mode changes as well.
Junio C Hamano [Mon, 6 Feb 2006 20:53:07 +0000 (12:53 -0800)]
combine-diff: show mode changes as well.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombine-diff: do not send NULL to printf
Junio C Hamano [Mon, 6 Feb 2006 20:30:00 +0000 (12:30 -0800)]
combine-diff: do not send NULL to printf

When we run combined diff from working tree (diff-files --cc),
we sent NULL to printf that is returned by find_unique_abbrev().

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocore-tutorial: adjust to recent reality.
Junio C Hamano [Mon, 6 Feb 2006 20:27:33 +0000 (12:27 -0800)]
core-tutorial: adjust to recent reality.

We still talked about HEAD symlinks but these days we use
symrefs by default.

Also 'failed/prevented' message is now gone from the merge
output.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-diff: do not fall back on --cc when -[123], --ours etc. are given.
Junio C Hamano [Mon, 6 Feb 2006 19:17:56 +0000 (11:17 -0800)]
git-diff: do not fall back on --cc when -[123], --ours etc. are given.

These flags ask diff with a specific unmerged stage, so it
should fall back on -p instead.  Also when -c is given, we
should not do --cc.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branch 'jc/diff'
Junio C Hamano [Mon, 6 Feb 2006 07:55:45 +0000 (23:55 -0800)]
Merge branch 'jc/diff'

* jc/diff:
  git-diff-tree --stdin: show all parents.
  combine-diff: remove misguided --show-empty hack.

18 years agoMerge branches 'lt/show' and 'lt/revlist'
Junio C Hamano [Mon, 6 Feb 2006 07:55:09 +0000 (23:55 -0800)]
Merge branches 'lt/show' and 'lt/revlist'

* lt/show:
  git-show

* lt/revlist:
  rev-parse lstat() workaround cleanup.

18 years agoMerge branches 'jc/daemon' and 'mw/http'
Junio C Hamano [Mon, 6 Feb 2006 07:54:14 +0000 (23:54 -0800)]
Merge branches 'jc/daemon' and 'mw/http'

* jc/daemon:
  daemon: extend user-relative path notation.
  daemon: Set SO_REUSEADDR on listening sockets.
  daemon: do not forbid user relative paths unconditionally under --base-path

* mw/http:
  http-fetch: Tidy control flow in process_alternate_response
  http: Turn on verbose Curl messages if GIT_CURL_VERBOSE set in environment
  http-fetch: Fix message reporting rename of object file.
  http-fetch: Fix object list corruption in fill_active_slots().

18 years agogit-diff-tree --stdin: show all parents.
Junio C Hamano [Mon, 6 Feb 2006 07:00:41 +0000 (23:00 -0800)]
git-diff-tree --stdin: show all parents.

git-diff-tree --stdin ignored second and subsequent parents when
fed git-rev-list --parents output.  Update diff_tree_commit()
function to take a commit object, and pass a fabricated commit
object after grafting the fake parents from diff_tree_stdin().

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agocombine-diff: remove misguided --show-empty hack.
Junio C Hamano [Mon, 6 Feb 2006 06:25:00 +0000 (22:25 -0800)]
combine-diff: remove misguided --show-empty hack.

Now --always flag is available in diff-tree, there is no reason
to have that hack in the diffcore side.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agorev-parse lstat() workaround cleanup.
Linus Torvalds [Mon, 6 Feb 2006 05:41:47 +0000 (21:41 -0800)]
rev-parse lstat() workaround cleanup.

Earlier we had a workaround to avoid misspelled revision name to
be taken as a filename when "--no-revs --no-flags" are in
effect.  This cleans up the logic.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-send-email: Fully implement --quiet and document it.
Ryan Anderson [Mon, 6 Feb 2006 01:13:52 +0000 (20:13 -0500)]
git-send-email: Fully implement --quiet and document it.

Also reorganizes the man page to list options alphabetically.

Signed-off-by: Ryan Anderson <ryan@michonline.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodaemon: extend user-relative path notation.
Junio C Hamano [Sun, 5 Feb 2006 06:27:29 +0000 (22:27 -0800)]
daemon: extend user-relative path notation.

Earlier, we made --base-path to automatically forbid
user-relative paths, which was probably a mistake.  This
introduces --user-path (or --user-path=path) option to control
the use of user-relative paths independently.  The latter form
of the option can be used to restrict accesses to a part of each
user's home directory, similar to "public_html" some webservers
supports.

If we're invoked with --user-path=FOO option, then a URL of the
form git://~USER/PATH/... resolves to the path HOME/FOO/PATH/...,
where HOME is USER's home directory.

[jc: This is much reworked by me so bugs are mine, but the
 original patch was done by Mark Wooding.]

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodaemon: Set SO_REUSEADDR on listening sockets.
Mark Wooding [Fri, 3 Feb 2006 20:27:04 +0000 (20:27 +0000)]
daemon: Set SO_REUSEADDR on listening sockets.

Without this, you can silently lose the ability to receive IPv4
connections if you stop and restart the daemon.

[jc: tweaked code organization a bit and made this controllable
 from a command line option.]

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodaemon: do not forbid user relative paths unconditionally under --base-path
Junio C Hamano [Sat, 4 Feb 2006 07:50:55 +0000 (23:50 -0800)]
daemon: do not forbid user relative paths unconditionally under --base-path

Using base-path to relocate the server public space does not
have anything to do with allowing or forbidding user relative
paths.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agohttp-fetch: Tidy control flow in process_alternate_response
Mark Wooding [Wed, 1 Feb 2006 11:44:39 +0000 (11:44 +0000)]
http-fetch: Tidy control flow in process_alternate_response

It's a bit convoluted.  Tidy it up.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agohttp: Turn on verbose Curl messages if GIT_CURL_VERBOSE set in environment
Mark Wooding [Wed, 1 Feb 2006 11:44:37 +0000 (11:44 +0000)]
http: Turn on verbose Curl messages if GIT_CURL_VERBOSE set in environment

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agohttp-fetch: Fix message reporting rename of object file.
Mark Wooding [Wed, 1 Feb 2006 11:44:35 +0000 (11:44 +0000)]
http-fetch: Fix message reporting rename of object file.

move_temp_to_file returns 0 or -1.  This is not a good thing to pass to
strerror(3).  Fortunately, someone already reported the error, so don't
worry too much.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agohttp-fetch: Fix object list corruption in fill_active_slots().
Mark Wooding [Wed, 1 Feb 2006 11:44:28 +0000 (11:44 +0000)]
http-fetch: Fix object list corruption in fill_active_slots().

In fill_active_slots() -- if we find an object which has already arrived,
say as part of a pack, /don't/ remove it from the list.  It's already been
prefetched and someone will ask for it later.  Just label it as done and
carry blithely on.  (As it was, the code would dereference a freed object
to continue through the list anyway.)

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-show
Junio C Hamano [Mon, 6 Feb 2006 00:42:49 +0000 (16:42 -0800)]
git-show

This is essentially 'git whatchanged -n1 --always --cc "$@"'.
Just like whatchanged takes default flags from
whatchanged.difftree configuration, this uses show.difftree
configuration.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-diff: use --cc instead of -p.
Junio C Hamano [Mon, 6 Feb 2006 00:37:18 +0000 (16:37 -0800)]
git-diff: use --cc instead of -p.

The --cc output is much nicer when dealing with merges, so use
it by default.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-index: make --cc a synonym for -p for now.
Junio C Hamano [Mon, 6 Feb 2006 00:36:12 +0000 (16:36 -0800)]
diff-index: make --cc a synonym for -p for now.

It could be made later to show unmerged state nicer than the
default as we did for diff-files later, but this would suffice
for now.  We would like to make --cc the default for 'git diff'.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodiff-tree --always flag
Linus Torvalds [Sun, 5 Feb 2006 19:42:09 +0000 (11:42 -0800)]
diff-tree --always flag

It _might_ make sense for certain users like gitk and gitview if
we had a single tool that gives --pretty and its diff even if
the diff is empty.  Having said that, the flag --cc -m is too
specific.  If some uses want to see the commit log even for an
empty diff, that flag should not be something only --cc honors.

Here's an "--always" flag that does that.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoUse adler32() from zlib instead of defining our own.
Peter Eriksen [Sat, 4 Feb 2006 22:54:01 +0000 (23:54 +0100)]
Use adler32() from zlib instead of defining our own.

Since we already depend on zlib, we don't need to define our
own adler32().  Spotted by oprofile.

Signed-off-by: Peter Eriksen <s022018@student.dtu.dk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoFix git-rev-parse over-eager errors
Linus Torvalds [Sun, 5 Feb 2006 19:58:34 +0000 (11:58 -0800)]
Fix git-rev-parse over-eager errors

Using "--verify" together with "--no-flags" makes perfect sense, but
git-rev-parse would complain about it when it saw a flag, even though it
would never actually use/output that flag.

This fixes it.

Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoDo not fall back on vi on dumb terminals.
Junio C Hamano [Sun, 5 Feb 2006 06:10:32 +0000 (22:10 -0800)]
Do not fall back on vi on dumb terminals.

When TERM is set to 'dumb', do not start vi to edit the commit log
message.

Suggested by Amos Waterland.

Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agoMerge branches 'jc/sha1', 'jc/diff' and 'jc/ws'
Junio C Hamano [Sat, 4 Feb 2006 07:52:20 +0000 (23:52 -0800)]
Merge branches 'jc/sha1', 'jc/diff' and 'jc/ws'

* jc/sha1:
  get_sha1_1: allow octopus^12 to be properly parsed.

* jc/diff:
  combine-diff: finishing touches to git-diff-tree --cc

* jc/ws:
  whitespace cleanup.

18 years agoUse sha1_file.c's mkdir-like routine in apply.c.
Jason Riedy [Sat, 4 Feb 2006 06:50:57 +0000 (22:50 -0800)]
Use sha1_file.c's mkdir-like routine in apply.c.

As far as I can see, create_subdirectories() in apply.c just
duplicates the functionality of safe_create_leading_directories() from
sha1_file.c.  The former has a warm, fuzzy const parameter, but that's
not important.

The potential problem with EEXIST and creating directories should
never occur here, but will be removed by future
safe_create_leading_directories() changes.  Other uses of EEXIST in
apply.c should be fine barring intentionally malicious behavior.

Signed-off-by: Jason Riedy <ejr@cs.berkeley.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agodaemon: Provide missing argument for logerror() call.
Mark Wooding [Fri, 3 Feb 2006 20:27:02 +0000 (20:27 +0000)]
daemon: Provide missing argument for logerror() call.

Could cause a crash if --base-path set.  Unlikely to be a security the
concern: message doesn't go to the client, so we can't leak anything
(except by dumping core), and we've already forked, so it's not a denial
of service.

Signed-off-by: Mark Wooding <mdw@distorted.org.uk>
Signed-off-by: Junio C Hamano <junkio@cox.net>
18 years agogit-merge: Properly quote $merge_msg variable.
Junio C Hamano [Sat, 4 Feb 2006 03:41:33 +0000 (19:41 -0800)]
git-merge: Properly quote $merge_msg variable.

Otherwise it would go though shell expansion...

Signed-off-by: Junio C Hamano <junkio@cox.net>