Code

git.git
16 years agoautoconf: Add checking for unsetenv function
Jakub Narebski [Fri, 18 Jan 2008 01:03:51 +0000 (02:03 +0100)]
autoconf: Add checking for unsetenv function

Update configure.ac (and config.mak.in) by adding test for unsetenv
(NO_UNSETENV).  Add comment about NO_UNSETENV to Makefile header, as
original commit 731043fd adding compat/unsetenv.c didn't do that.

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agocore-tutorial typofix
Thomas Zander [Wed, 16 Jan 2008 22:48:21 +0000 (23:48 +0100)]
core-tutorial typofix

Signed-off-by: Thomas Zander <zander@kde.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoOfficially deprecate repo-config.
Junio C Hamano [Fri, 18 Jan 2008 06:52:40 +0000 (22:52 -0800)]
Officially deprecate repo-config.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoFix random fast-import errors when compiled with NO_MMAP
Shawn O. Pearce [Fri, 18 Jan 2008 03:57:00 +0000 (22:57 -0500)]
Fix random fast-import errors when compiled with NO_MMAP

fast-import was relying on the fact that on most systems mmap() and
write() are synchronized by the filesystem's buffer cache.  We were
relying on the ability to mmap() 20 bytes beyond the current end
of the file, then later fill in those bytes with a future write()
call, then read them through the previously obtained mmap() address.

This isn't always true with some implementations of NFS, but it is
especially not true with our NO_MMAP=YesPlease build time option used
on some platforms.  If fast-import was built with NO_MMAP=YesPlease
we used the malloc()+pread() emulation and the subsequent write()
call does not update the trailing 20 bytes of a previously obtained
"mmap()" (aka malloc'd) address.

Under NO_MMAP that behavior causes unpack_entry() in sha1_file.c to
be unable to read an object header (or data) that has been unlucky
enough to be written to the packfile at a location such that it
is in the trailing 20 bytes of a window previously opened on that
same packfile.

This bug has gone unnoticed for a very long time as it is highly data
dependent.  Not only does the object have to be placed at the right
position, but it also needs to be positioned behind some other object
that has been accessed due to a branch cache invalidation.  In other
words the stars had to align just right, and if you did run into
this bug you probably should also have purchased a lottery ticket.

Fortunately the workaround is a lot easier than the bug explanation.

Before we allow unpack_entry() to read data from a pack window
that has also (possibly) been modified through write() we force
all existing windows on that packfile to be closed.  By closing
the windows we ensure that any new access via the emulated mmap()
will reread the packfile, updating to the current file content.

This comes at a slight performance degredation as we cannot reuse
previously cached windows when we update the packfile.  But it
is a fairly minor difference as the window closes happen at only
two points:

 - When the packfile is finalized and its .idx is generated:

   At this stage we are getting ready to update the refs and any
   data access into the packfile is going to be random, and is
   going after only the branch tips (to ensure they are valid).
   Our existing windows (if any) are not likely to be positioned
   at useful locations to access those final tip commits so we
   probably were closing them before anyway.

 - When the branch cache missed and we need to reload:

   At this point fast-import is getting change commands for the next
   commit and it needs to go re-read a tree object it previously
   had written out to the packfile.  What windows we had (if any)
   are not likely to cover the tree in question so we probably were
   closing them before anyway.

We do try to avoid unnecessarily closing windows in the second case
by checking to see if the packfile size has increased since the
last time we called unpack_entry() on that packfile.  If the size
has not changed then we have not written additional data, and any
existing window is still vaild.  This nicely handles the cases where
fast-import is going through a branch cache reload and needs to read
many trees at once.  During such an event we are not likely to be
updating the packfile so we do not cycle the windows between reads.

With this change in place t9301-fast-export.sh (which was broken
by c3b0dec509fe136c5417422f31898b5a4e2d5e02) finally works again.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agofast-import.c: don't try to commit marks file if write failed
Brandon Casey [Thu, 17 Jan 2008 16:58:34 +0000 (10:58 -0600)]
fast-import.c: don't try to commit marks file if write failed

We also move the assignment of -1 to the lock file descriptor
up, so that rollback_lock_file() can be called safely after a
possible attempt to fclose(). This matches the contents of
the 'if' statement just above testing success of fdopen().

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Acked-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agorefs.c: rework ref_locks by abstracting from underlying struct lock_file
Brandon Casey [Wed, 16 Jan 2008 19:14:30 +0000 (13:14 -0600)]
refs.c: rework ref_locks by abstracting from underlying struct lock_file

Instead of calling close_lock_file() and commit_lock_file() directly,
which take a struct lock_file argument, add two new functions:
close_ref() and commit_ref(), which handle calling the previous
lock_file functions and modifying the ref_lock structure.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoImprove use of lockfile API
Brandon Casey [Wed, 16 Jan 2008 19:12:46 +0000 (13:12 -0600)]
Improve use of lockfile API

Remove remaining double close(2)'s.  i.e. close() before
commit_locked_index() or commit_lock_file().

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoclose_lock_file(): new function in the lockfile API
Brandon Casey [Wed, 16 Jan 2008 19:05:32 +0000 (11:05 -0800)]
close_lock_file(): new function in the lockfile API

The lockfile API is a handy way to obtain a file that is cleaned
up if you die().  But sometimes you would need this sequence to
work:

 1. hold_lock_file_for_update() to get a file descriptor for
    writing;

 2. write the contents out, without being able to decide if the
    results should be committed or rolled back;

 3. do something else that makes the decision --- and this
    "something else" needs the lockfile not to have an open file
    descriptor for writing (e.g. Windows do not want a open file
    to be renamed);

 4. call commit_lock_file() or rollback_lock_file() as
    appropriately.

This adds close_lock_file() you can call between step 2 and 3 in
the above sequence.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocument lockfile API
Junio C Hamano [Wed, 16 Jan 2008 19:00:13 +0000 (11:00 -0800)]
Document lockfile API

We have nice set of placeholders, but nobody stepped in to fill
the gap in the API documentation, so I am doing it myself.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoBe more careful about updating refs
Linus Torvalds [Tue, 15 Jan 2008 23:50:17 +0000 (15:50 -0800)]
Be more careful about updating refs

This makes write_ref_sha1() more careful: it actually checks the SHA1 of
the ref it is updating, and refuses to update a ref with an object that it
cannot find.

Perhaps more importantly, it also refuses to update a branch head with a
non-commit object. I don't quite know *how* the stable series maintainers
were able to corrupt their repository to have a HEAD that pointed to a tag
rather than a commit object, but they did. Which results in a totally
broken repository that cannot be cloned or committed on.

So make it harder for people to shoot themselves in the foot like that.

The test t1400-update-ref.sh is fixed at the same time, as it
assumed that the commands involved in the particular test would
not care about corrupted repositories whose refs point at
nonexistant bogus objects.  That assumption does not hold true
anymore.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoCorrect spelling in diff.c comment
Bill Lear [Wed, 16 Jan 2008 12:46:53 +0000 (06:46 -0600)]
Correct spelling in diff.c comment

Correct a spelling mistake in a comment.

Signed-off-by: Bill Lear <rael@zopyra.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: fix and clarify grammar in git-merge docs.
Dave Peticolas [Wed, 16 Jan 2008 05:16:05 +0000 (21:16 -0800)]
Documentation: fix and clarify grammar in git-merge docs.

Signed-off-by: Dave Peticolas <dave@krondo.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMake 'git fsck' complain about non-commit branches
Linus Torvalds [Wed, 16 Jan 2008 00:34:17 +0000 (16:34 -0800)]
Make 'git fsck' complain about non-commit branches

Since having non-commits in branches is a no-no, and just means you cannot
commit on them, let's make fsck tell you when a branch is bad.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMake builtin-commit.c more careful about parenthood
Linus Torvalds [Wed, 16 Jan 2008 00:12:33 +0000 (16:12 -0800)]
Make builtin-commit.c more careful about parenthood

When creating the commit object, be a whole lot more careful about making
sure that the parent lines really are valid parent lines. Check things
like MERGE_HEAD having proper SHA1 lines in it, and double-check that all
the parents exist and are actually commits.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agotreat any file with NUL as binary
Dmitry Potapov [Wed, 16 Jan 2008 01:59:12 +0000 (04:59 +0300)]
treat any file with NUL as binary

There are two heuristics in Git to detect whether a file is binary
or text. One in xdiff-interface.c (which is taken from GNU diff)
relies on existence of the NUL byte at the beginning. However,
convert.c used a different heuristic, which relied on the percent
of non-printable symbols (less than 1% for text files).

Due to differences in detection whether a file is binary or not,
it was possible that a file that diff treats as binary could be
treated as text by CRLF conversion. This is very confusing for a
user who sees that 'git diff' shows the file as binary expects it
to be added as binary.

This patch makes is_binary to consider any file that contains at
least one NUL character as binary, to ensure that the heuristics
used for CRLF conversion is tighter than what is used by diff.

Signed-off-by: Dmitry Potapov <dpotapov@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-commit: fix double close(2) that can close a wrong file descriptor
Kristian Høgsberg [Tue, 15 Jan 2008 20:00:02 +0000 (15:00 -0500)]
git-commit: fix double close(2) that can close a wrong file descriptor

The codepath to prepare index files for the temporary and next
index file was closing file descriptor it obtained from the
lockfile API by hand, without letting the API know that the fd
should not be doubly closed.

This is not usually a problem (except it may get EBADFD) but if
we opened another fd for an entirely unrelated purpose (say, an
fd used to mmap a packfile) between the time we close the fd to
the index file and the time we commit or rollback the lockfile
(causing it to also try closing the recorded fd), the lockfile
API will close an incorrect file descriptor that is still used
for an entirely unrelated purpose.

There's four close(fd) calls in prepare_index() and they're all
incorrect.  The open fd's are cleaned up in rollback_index_files() and
shouldn't be closed manually.  The patch below gets rid of the extra
close() calls and should fix the problem.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agohg-to-git: improve popen calls
Mark Drago [Tue, 15 Jan 2008 01:11:19 +0000 (20:11 -0500)]
hg-to-git: improve popen calls

This patch improves all of the popen calls in hg-to-git.py by specifying the
template 'hg log' should use instead of calling 'hg log' and grepping for the
desired data.

Signed-off-by: Mark Drago <markdrago@gmail.com>
Acked-by: Stelian Pop <stelian@popies.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agols-remote: add -t and -h options.
Miklos Vajna [Tue, 15 Jan 2008 20:34:54 +0000 (21:34 +0100)]
ls-remote: add -t and -h options.

These options are listed in the manpage (aliases for --tags/--heads) but they
were not handled.

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoSquelch bogus progress output from git-rebase--interactive
Junio C Hamano [Tue, 15 Jan 2008 04:01:21 +0000 (20:01 -0800)]
Squelch bogus progress output from git-rebase--interactive

The command repeats "Rebasing (1/1)" many times even when
there is only one task remaining, because mark_action_done() is
called to skip comment and empty lines in the TODO file.

This should fix it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoAdd using merge subtree How-To
Miklos Vajna [Tue, 15 Jan 2008 00:35:02 +0000 (01:35 +0100)]
Add using merge subtree How-To

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoFix git-rerere documentation
Junio C Hamano [Mon, 14 Jan 2008 23:10:38 +0000 (15:10 -0800)]
Fix git-rerere documentation

rerere.enabled is _not_ on by default.  The command is enabled if rr-cache
exists even when rerere.enabled is missing, and enabled or disabled by
explicitly setting the rerere.enabled variable.

16 years agoRevert "builtin-commit.c: remove useless check added by faulty cut and paste"
Junio C Hamano [Mon, 14 Jan 2008 21:54:24 +0000 (13:54 -0800)]
Revert "builtin-commit.c: remove useless check added by faulty cut and paste"

This reverts commit 16335fdd7ace78a8285ba25fd7a81177a48e7c9b.

We are calling overlay_tree_on_cache() which does use CE_UPDATE
flag to mark duplicated entries, which is the same as the
codepath in git-ls-files with its --with-tree option.

Because the pathname ce->name is given to path_list_insert()
which does not allow duplicates, there is no breakage either way
from the correctness point of view in this codepath, unlike the
one in ls-files.  But avoiding unnecessary processing with a
single bit check is certainly better.

16 years agoMake default pre-commit hook less noisy
Jean-Luc Herren [Mon, 14 Jan 2008 21:47:42 +0000 (22:47 +0100)]
Make default pre-commit hook less noisy

This hook thought to have found a conflict marker any time it saw
a 7-character combination of any of the characters '<>=' at the
beginning of a line, whereas it should only look for the *same*
character to appear repeatedly.

Also, restrict it to match exactly 7 times, to avoid matching the
underlining with '='-characters often used in documentation.

Signed-off-by: Jean-Luc Herren <jlh@gmx.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agocvsimport: remove last use of repo-config from git standard tools
Dan McGee [Mon, 14 Jan 2008 04:51:10 +0000 (22:51 -0600)]
cvsimport: remove last use of repo-config from git standard tools

git cvsimport was the last tool to use repo-config instead of config. Update
it to use plain git config.

Signed-off-by: Dan McGee <dpmcgee@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoRemove usage of git- (dash) commands from email hook
Dan McGee [Mon, 14 Jan 2008 04:51:01 +0000 (22:51 -0600)]
Remove usage of git- (dash) commands from email hook

Switch all git command calls to use the git (space) command format, and
remove the use of git-repo-config in place of git config.

Signed-off-by: Dan McGee <dpmcgee@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agobuiltin-commit.c: remove useless check added by faulty cut and paste
Junio C Hamano [Sun, 13 Jan 2008 10:38:46 +0000 (02:38 -0800)]
builtin-commit.c: remove useless check added by faulty cut and paste

2888605c649ccd423232161186d72c0e6c458a48 (builtin-commit: fix
partial-commit support) mindlessly cut and pasted from
builtin-ls-files.c, and included a part that was meant to
exclude redundant path after "ls-files --with-tree" overlayed
the HEAD commit on top of the index.  This logic does not apply
to what git-commit does and should not have been copied, even
though it would not hurt.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoFix performance regression for partial commits
Linus Torvalds [Sun, 13 Jan 2008 08:30:56 +0000 (00:30 -0800)]
Fix performance regression for partial commits

When running "git commit paths" to create a partial commit, we
used to carefully build the temporary index so that we do not
lose the cached stat information.  The rewrite of the command in
C lost it by carelessly using read_tree().

This resurrects the earlier behaviour to keep the cached stat
information as much as possible by using one-tree merge logic.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-clean: fix off-by-one memory access when given no arguments
Jeff King [Sat, 12 Jan 2008 09:04:32 +0000 (04:04 -0500)]
git-clean: fix off-by-one memory access when given no arguments

The "seen" variable is used by match_pathspec, and must have
as many elements as there are in the given pathspec. We
create the pathspec either from the command line arguments
_or_ from just the current prefix.

Thus allocating "seen" based upon just argc is wrong, since
if argc == 0, then we still have one pathspec, the prefix,
but we don't allocate any space in "seen".

Signed-off-by: Jeff King <peff@peff.net>
Tested-by: İsmail Dönmez <ismail@pardus.org.tr>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-svn: handle leading/trailing whitespace from svnsync revprops
Eric Wong [Sat, 12 Jan 2008 07:13:55 +0000 (23:13 -0800)]
git-svn: handle leading/trailing whitespace from svnsync revprops

Repositories generated by svnsync cannot be relied on to have
properly set revprops without newlines in UUIDs and URLs.  There
may be broken versions of svnsync out there that append extra
newlines to UUIDs, or the revprops could've been changed by
repository administrators at any time, too.

At least one repository we've come across has an embedded
newline erroneously set in the svnsync-uuid prop.  This is bad
because the trailing newline is taken as another record by the
Git.pm library, and the wantarray detection causes tmp_config()
to return an array with an empty-but-existing second element.

We will now strip leading and trailing whitespace both before
setting and after reading the uuid and url for svnsync values.
We will also force tmp_config to return a single scalar when
reading existing values.

SVN UUIDs should never have whitespace in them, and SVN
repository URLs should be URI-escaped, so neither of those
values we ever see in git-svn should actually have whitespace
in them.

Thanks to Dennis Schridde for the bug report and Junio for
helping diagnose this.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoGIT 1.5.4-rc3 v1.5.4-rc3
Junio C Hamano [Sat, 12 Jan 2008 05:14:39 +0000 (21:14 -0800)]
GIT 1.5.4-rc3

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoRPM spec: include gitk message files.
Junio C Hamano [Sat, 12 Jan 2008 05:40:15 +0000 (21:40 -0800)]
RPM spec: include gitk message files.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-relink.txt: describe more clearly how hard linking occurs
Brandon Casey [Fri, 11 Jan 2008 18:14:19 +0000 (12:14 -0600)]
git-relink.txt: describe more clearly how hard linking occurs

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocument some default values in config.txt
Michele Ballabio [Fri, 11 Jan 2008 21:11:13 +0000 (22:11 +0100)]
Document some default values in config.txt

This documents the default values of gc.auto, gc.autopacklimit
fetch.unpacklimit, receive.unpacklimit and transfer.unpacklimit.

Signed-off-by: Michele Ballabio <barra_cuda@katamail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMerge ../gitk
Junio C Hamano [Fri, 11 Jan 2008 08:51:30 +0000 (00:51 -0800)]
Merge ../gitk

* ../gitk:
  gitk: Update German translation.
  gitk: Fix typo in user message.
  gitk: Fix the Makefile to cope with systems lacking msgfmt

16 years agogitk: Update German translation.
Christian Stimming [Wed, 9 Jan 2008 21:21:58 +0000 (22:21 +0100)]
gitk: Update German translation.

Now 100% complete (163 strings).

Signed-off-by: Christian Stimming <stimming@tuhh.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogitk: Fix typo in user message.
Christian Stimming [Wed, 9 Jan 2008 21:23:18 +0000 (22:23 +0100)]
gitk: Fix typo in user message.

Signed-off-by: Christian Stimming <stimming@tuhh.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogitk: Fix the Makefile to cope with systems lacking msgfmt
Charles Bailey [Wed, 9 Jan 2008 14:31:09 +0000 (14:31 +0000)]
gitk: Fix the Makefile to cope with systems lacking msgfmt

The po2msg.sh script and the .gitignore in the po directory have been
shamelessly copied from the current git-gui.  This enables the top
level "make NO_MSGFMT" to work consistently for git across the git-gui
and gitk sub-projects.

This is the same effective patch that has previously been posted as a
git.git patch which more succinctly described the copying of
po/.gitignore and po/po2msg.sh from git-gui.

Signed-off-by: Charles Bailey <charles@hashpling.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agopack-objects: remove redundant and wrong call to deflateEnd()
Junio C Hamano [Fri, 11 Jan 2008 07:26:09 +0000 (23:26 -0800)]
pack-objects: remove redundant and wrong call to deflateEnd()

We somehow called deflateEnd() on a stream that we have called
deflateEnd() on already.

In fact, the second deflateEnd() has always been returning
Z_STREAM_ERROR.  We just never checked the error return from
that particular deflateEnd().

The first one returns 0 for success.  We might want to tighten
the check even more to check that.

Noticed by Marco.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agobundle, fast-import: detect write failure
Jim Meyering [Thu, 10 Jan 2008 08:54:25 +0000 (09:54 +0100)]
bundle, fast-import: detect write failure

I noticed some unchecked writes.  This fixes them.

* bundle.c (create_bundle): Die upon write failure.
* fast-import.c (keep_pack): Die upon write or close failure.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoTrim leading / off of paths in git-svn prop_walk
Kevin Ballard [Wed, 9 Jan 2008 06:37:20 +0000 (01:37 -0500)]
Trim leading / off of paths in git-svn prop_walk

prop_walk adds a leading / to all subdirectory paths. Unfortunately
this causes a problem when the remote repo lives in a subdirectory itself,
as the leading / causes subsequent PROPFIND calls to be executed on
the wrong path. Trimming the / before calling the PROPFIND fixes this problem.

Signed-off-by: Kevin Ballard <kevin@sb.org>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoshortlog: mention the "-e" option in the usage
Johannes Schindelin [Wed, 9 Jan 2008 11:39:54 +0000 (11:39 +0000)]
shortlog: mention the "-e" option in the usage

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoChange git-gc documentation to reflect gc.packrefs implementation.
Florian La Roche [Wed, 9 Jan 2008 16:05:16 +0000 (17:05 +0100)]
Change git-gc documentation to reflect gc.packrefs implementation.

56752391a8c0c591853b276e4fa0b45c34ced181 (Make "git gc" pack all
refs by default) changed the default of gc.packrefs to true, to
pack all refs by default in any repository.  IOW, the users need
to disable it explicitly if they want to by setting the config
variable, since 1.5.3.

However, we forgot to update the documentation.  This fixes it.

Signed-off-by: Florian La Roche <laroche@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agorecv_sideband: Do not use ANSI escape sequence on dumb terminals.
Johannes Sixt [Tue, 8 Jan 2008 16:24:53 +0000 (17:24 +0100)]
recv_sideband: Do not use ANSI escape sequence on dumb terminals.

The "clear to end of line" sequence is used to nicely output the progress
indicator without leaving garbage on the terminal. However, this works
only on ANSI capable terminals. We use the same check as in color.c to
find out whether the terminal supports this feature and use a workaround
(a few spaces in a row) if it does not.

[jc: as an old fashoned git myself, and given the fact that the
possible prefix and suffix are small number of short constant strings,
I actually prefer a simpler-and-more-stupid approach.  This is with
Nico's clean-up.]

Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMerge branch 'master' of git://git.kernel.org/pub/scm/gitk/gitk
Junio C Hamano [Wed, 9 Jan 2008 06:41:15 +0000 (22:41 -0800)]
Merge branch 'master' of git://git./gitk/gitk

* 'master' of git://git.kernel.org/pub/scm/gitk/gitk:
  [PATCH] gitk: use user-configured background in view definition dialog
  [PATCH] gitk: Update German translation
  [PATCH] gitk: Update and fix Makefile
  gitk: Restore some widget options whose defaults changed in Tk 8.5
  gitk: Recode de.po to UTF-8
  [PATCH] gitk i18n: Recode gitk from latin1 to utf8 so that the (c) copyright character is valid utf8.
  [PATCH] gitk i18n: More markup -- various options menus
  [PATCH] gitk i18n: Initial German translation
  [PATCH] gitk i18n: Markup several strings for translation
  [PATCH] gitk i18n: Import msgcat for message string translation; load translation catalogs
  [PATCH] gitk i18n: Add Makefile with rules for po file creation and installation

16 years ago[PATCH] gitk: use user-configured background in view definition dialog
Gerrit Pape [Wed, 9 Jan 2008 03:19:47 +0000 (14:19 +1100)]
[PATCH] gitk: use user-configured background in view definition dialog

Have the text fields in the view definition dialog (View->New view...)
use the background color as configured through the preferences, instead
of hard-coded 'white'.

This was suggested by Paul Wise through http://bugs.debian.org/457124

Signed-off-by: Gerrit Pape <pape@smarden.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
16 years ago[PATCH] gitk: Update German translation
Christian Stimming [Wed, 9 Jan 2008 03:13:22 +0000 (14:13 +1100)]
[PATCH] gitk: Update German translation

Signed-off-by: Christian Stimming <stimming@tuhh.de>
Signed-off-by: Paul Mackerras <paulus@samba.org>
16 years ago[PATCH] gitk: Update and fix Makefile
Christian Stimming [Wed, 9 Jan 2008 03:12:18 +0000 (14:12 +1100)]
[PATCH] gitk: Update and fix Makefile

This Makefile uses the template provided at git.git/gitk-git/Makefile
by Junio and adds the rules for the i18n files.

Signed-off-by: Christian Stimming <stimming@tuhh.de>
Signed-off-by: Paul Mackerras <paulus@samba.org>
16 years agogit.el: Make status refresh faster.
Alexandre Julliard [Tue, 8 Jan 2008 13:49:09 +0000 (14:49 +0100)]
git.el: Make status refresh faster.

Don't set the needs-refresh flag when inserting a new file info, since
ewoc refreshes it upon insert already; this makes a full refresh twice
as fast.

Also make git-fileinfo-prettyprint a little faster by not retrieving
permission values twice.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit.el: Refresh files from their real state upon commit.
Alexandre Julliard [Tue, 8 Jan 2008 13:46:22 +0000 (14:46 +0100)]
git.el: Refresh files from their real state upon commit.

Instead of just setting the state to up-to-date, retrieve the full
state again, so that the file type can be displayed properly.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit.el: Make sure we never insert the same file twice.
Alexandre Julliard [Tue, 8 Jan 2008 13:45:46 +0000 (14:45 +0100)]
git.el: Make sure we never insert the same file twice.

Skip non-zero stage files during git-ls-files -c, they are handled
later. Also fix git-insert-info-list to merge duplicate file names.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years ago"git-apply --check" should not report "fixed"
Junio C Hamano [Tue, 8 Jan 2008 23:46:18 +0000 (15:46 -0800)]
"git-apply --check" should not report "fixed"

When running "git apply --check" while --whitespace=fix is
enabled (either from the command line or via the configuration),
we reported that "N line(s) applied after _fixing_", but --check
by itself does not apply and this message was alarming.

We could even reword the message to say "N line(s) would have
been applied after fixing...", but this patch does not go that
far.  Instead, we just make it use the "N lines add whitespace
errors" warning, which happens to be a good diagnostic message a
user would expect from the --check option.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMerge in GIT 1.5.3.8
Junio C Hamano [Tue, 8 Jan 2008 07:39:02 +0000 (23:39 -0800)]
Merge in GIT 1.5.3.8

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoGIT 1.5.3.8 v1.5.3.8
Junio C Hamano [Tue, 8 Jan 2008 06:41:43 +0000 (22:41 -0800)]
GIT 1.5.3.8

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: config: add 'help.*' and 'instaweb.*' variables.
Christian Couder [Tue, 8 Jan 2008 03:55:14 +0000 (04:55 +0100)]
Documentation: config: add 'help.*' and 'instaweb.*' variables.

Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoAllow git-mergetool to handle paths with a leading space
Rogan Dawes [Mon, 7 Jan 2008 07:37:38 +0000 (09:37 +0200)]
Allow git-mergetool to handle paths with a leading space

Signed-off-by: Rogan Dawes <rogan@dawes.za.net>
16 years agogit-svn: clarify the "Ignoring error from SVN" piece
Eric Wong [Mon, 7 Jan 2008 10:40:40 +0000 (02:40 -0800)]
git-svn: clarify the "Ignoring error from SVN" piece

I've heard of several users puzzled by this, and it sometimes it
appears as if git-svn is doing nothing on slower connections and
larger repositories.

Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-svn: support for funky branch and project names over HTTP(S)
Eric Wong [Mon, 12 Nov 2007 07:37:42 +0000 (23:37 -0800)]
git-svn: support for funky branch and project names over HTTP(S)

SVN requires that paths be URI-escaped for HTTP(S) repositories.
file:// and svn:// repositories do not need these rules.

Additionally, accessing individual paths inside repositories
(check_path() and get_log() do NOT require escapes to function
and in fact it breaks things).

Noticed-by: Michael J. Cohen <mjc@cruiseplanners.com>
Signed-off-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoslightly better auto gc message
Nicolas Pitre [Mon, 7 Jan 2008 21:05:22 +0000 (16:05 -0500)]
slightly better auto gc message

Signed-off-by: Nicolas Pitre <nico@cam.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: typofix
Ralf Wildenhues [Mon, 7 Jan 2008 21:43:27 +0000 (22:43 +0100)]
Documentation: typofix

Signed-off-by: Ralf Wildenhues <Ralf.Wildenhues@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoUpdate draft release notes for 1.5.4
Junio C Hamano [Mon, 7 Jan 2008 06:20:40 +0000 (22:20 -0800)]
Update draft release notes for 1.5.4

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agodiff: do not chomp hunk-header in the middle of a character
Junio C Hamano [Wed, 2 Jan 2008 09:50:11 +0000 (01:50 -0800)]
diff: do not chomp hunk-header in the middle of a character

We truncate hunk-header line at 80 bytes, but that 80th byte
could be in the middle of a character, which is bad.  This uses
pick_one_utf8_char() function to make sure we do not cut a character
in the middle.

This assumes that the internal representation of the text is
UTF-8.  This needs to be extended in the future but the optimal
direction has not been decided yet.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoutf8_width(): allow non NUL-terminated input
Junio C Hamano [Wed, 2 Jan 2008 09:49:58 +0000 (01:49 -0800)]
utf8_width(): allow non NUL-terminated input

The original interface assumed that the input string is
always terminated with a NUL, but that wasn't too useful.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoutf8: pick_one_utf8_char()
Junio C Hamano [Mon, 7 Jan 2008 03:02:22 +0000 (19:02 -0800)]
utf8: pick_one_utf8_char()

utf8_width() function was doing two different things.  To pick a
valid character from UTF-8 stream, and compute the display width of
that character.  This splits the former to a separate function
pick_one_utf8_char().

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit.el: Display file types and type changes.
Alexandre Julliard [Sun, 6 Jan 2008 11:13:36 +0000 (12:13 +0100)]
git.el: Display file types and type changes.

Handle the T status from git-diff-index to display type changes
between file/symlink/subproject. Also always show the file type for
symlink and subprojects to indicate that they are not normal files.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit.el: Retrieve the permissions for up-to-date files.
Alexandre Julliard [Sun, 6 Jan 2008 11:13:01 +0000 (12:13 +0100)]
git.el: Retrieve the permissions for up-to-date files.

This allows displaying correctly the executable flag for the initial
commit, and will make it possible to show the file type for up-to-date
symlinks and subprojects.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit.el: Support for getting diffs from inside the log-edit buffer.
Alexandre Julliard [Sun, 6 Jan 2008 11:12:24 +0000 (12:12 +0100)]
git.el: Support for getting diffs from inside the log-edit buffer.

Take advantage of the new log-edit feature that allows to show a diff
with C-c C-d while editing the log message.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMake the git metapackage require the same version of the subpackages.
James Bowes [Sun, 6 Jan 2008 17:35:02 +0000 (12:35 -0500)]
Make the git metapackage require the same version of the subpackages.

Without explicit version deps in the rpm spec file, 'yum update git'
effectively does nothing. Require explicit versions of the subpackages,
so that they get pulled in on an update.

Signed-off-by: James Bowes <jbowes@dangerouslyinc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agotree-walk: don't parse incorrect entries
Martin Koegler [Sun, 6 Jan 2008 17:21:10 +0000 (18:21 +0100)]
tree-walk: don't parse incorrect entries

The current code can access memory outside of the tree buffer in the
case of malformed tree entries.

This patch prevents this by:

 * The rest of the buffer must be at least 24 bytes (at least 1 byte
   mode, 1 blank, at least one byte path name, 1 NUL, 20 bytes sha1).

 * Check that the last NUL (21 bytes before the end) is present.
   This ensures that strlen() and get_mode() calls stay within the
   buffer.

 * The mode may not be empty. We have only to reject a blank at the
   begin, as the rest is handled by if (c < '0' || c > '7').

 * The blank is ensured by get_mode().

 * The path must contain at least one character.

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocument the color.interactive semantics
Jeff King [Sat, 5 Jan 2008 09:57:44 +0000 (04:57 -0500)]
Document the color.interactive semantics

There are two possible confusions with the color.interactive
description:

  1. the short name "interactive" implies that it covers all
     interactive commands; let's explicitly make it so, even
     though there are no other interactive commands which
     currently use it

  2. Not all parts of "git add --interactive" are controlled
     by color.interactive (specifically, the diffs require
     tweaking color.diff). So let's clarify that it applies
     only to displays and prompts.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoadd--interactive: allow diff colors without interactive colors
Jeff King [Fri, 4 Jan 2008 08:35:21 +0000 (03:35 -0500)]
add--interactive: allow diff colors without interactive colors

Users with color.diff set to true/auto will not see color in
"git add -i" unless they also set color.interactive.

This changes the semantics of color.interactive to control only the
coloring of the interaction aspect of the command and let color.diff
to control the color of hunk picker, which would arguably be more
convenient.

Old $use_color variable is now renamed to $menu_use_color to make it
clear that it is about coloring the interaction.

The "colored" subroutine now checks if the passed color is defined,
instead of checking $use_color variable, to decide if the lines should
be colored.  The various variables that define colors for different
parts of the output are set or unset depending on the setting of
color.interactive and color.diff configuration variables.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoadd--interactive: remove unused diff colors
Jeff King [Sun, 6 Jan 2008 04:24:09 +0000 (23:24 -0500)]
add--interactive: remove unused diff colors

When color support was added, we colored the diffs ourselves.
However, 4af756f3 changed this to simply run "git diff-files"
twice, keeping the colored output separately.

This makes the internal diff color variables obsolete with
one exception: when splitting hunks, we have to manually
recreate the fragment for each part of the split. Thus we
keep $fraginfo_color around to do that correctly.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoparse_tag_buffer: don't parse invalid tags
Martin Koegler [Sun, 6 Jan 2008 19:03:10 +0000 (20:03 +0100)]
parse_tag_buffer: don't parse invalid tags

The current tag parsing code can access memory outside the tag buffer,
if \n are missing. This patch prevent this behaviour.

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: rename gitlink macro to linkgit
Dan McGee [Sat, 29 Dec 2007 06:20:38 +0000 (00:20 -0600)]
Documentation: rename gitlink macro to linkgit

Between AsciiDoc 8.2.2 and 8.2.3, the following change was made to the stock
Asciidoc configuration:

@@ -149,7 +153,10 @@
 # Inline macros.
 # Backslash prefix required for escape processing.
 # (?s) re flag for line spanning.
-(?su)[\\]?(?P<name>\w(\w|-)*?):(?P<target>\S*?)(\[(?P<attrlist>.*?)\])=
+
+# Explicit so they can be nested.
+(?su)[\\]?(?P<name>(http|https|ftp|file|mailto|callto|image|link)):(?P<target>\S*?)(\[(?P<attrlist>.*?)\])=
+
 # Anchor: [[[id]]]. Bibliographic anchor.
 (?su)[\\]?\[\[\[(?P<attrlist>[\w][\w-]*?)\]\]\]=anchor3
 # Anchor: [[id,xreflabel]]

This default regex now matches explicit values, and unfortunately in this
case gitlink was being matched by just 'link', causing the wrong inline
macro template to be applied. By renaming the macro, we can avoid being
matched by the wrong regex.

Signed-off-by: Dan McGee <dpmcgee@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: fix "gitlink::foobar[s]"
Junio C Hamano [Sun, 6 Jan 2008 03:10:35 +0000 (19:10 -0800)]
Documentation: fix "gitlink::foobar[s]"

They should be spelled with a single colon.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation: remove gitman.info with "make clean"
Junio C Hamano [Sun, 6 Jan 2008 03:09:09 +0000 (19:09 -0800)]
Documentation: remove gitman.info with "make clean"

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agocustom pretty format: tolerate empty e-mail address
Junio C Hamano [Sun, 6 Jan 2008 12:21:07 +0000 (04:21 -0800)]
custom pretty format: tolerate empty e-mail address

When e-mail address is empty (e.g. "A U Thor <>"), --pretty=format
misparsed the commit header and did not pick up the date field correctly.

Noticed by Marco, fixed slightly differently with additional sanity
check and with a test.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDocumentation/Makefile - honor $DESTDIR for quick-install target
Mark Levedahl [Sat, 5 Jan 2008 22:32:21 +0000 (17:32 -0500)]
Documentation/Makefile - honor $DESTDIR for quick-install target

Signed-off-by: Mark Levedahl <mdl123@verizon.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agofilter-branch: work correctly with ambiguous refnames
Junio C Hamano [Sat, 5 Jan 2008 20:18:43 +0000 (12:18 -0800)]
filter-branch: work correctly with ambiguous refnames

'git-filter-branch branch' could fail producing the error:
"Which ref do you want to rewrite?" if existed another branch
or tag, which name was 'branch-something' or 'something/branch'.

[jc: original report and fix were done between Dmitry Potapov
and Dscho; I rewrote it using "rev-parse --symbolic-full-name"]

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-rev-parse --symbolic-full-name
Junio C Hamano [Sat, 5 Jan 2008 20:09:55 +0000 (12:09 -0800)]
git-rev-parse --symbolic-full-name

The plumbing level can understand that the user meant
"refs/heads/master" when the user says "master" or
"heads/master", but there is no easy way for the scripts to
figure it out without duplicating the dwim_ref() logic.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-stash clear: refuse to work with extra parameter for now
Junio C Hamano [Sat, 5 Jan 2008 09:35:54 +0000 (01:35 -0800)]
git-stash clear: refuse to work with extra parameter for now

Because it is so tempting to expect "git stash clear stash@{4}"
to remove the fourth element in the stash while leaving other
elements intact, we should not blindly throw away everything
upon seeing such a command.

This may change when we start using "git reflog delete" to
selectively nuke a single (or multiple, for that matter) stash
entries with such a command line.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogitk: Restore some widget options whose defaults changed in Tk 8.5
Paul Mackerras [Sun, 6 Jan 2008 04:54:46 +0000 (15:54 +1100)]
gitk: Restore some widget options whose defaults changed in Tk 8.5

The default options for panedwindows in Tk 8.5 make the sash
virtually invisible -- the handle is not shown and the relief is
flat.  This puts the defaults back to showing the handle and a
raised relief on the sash, as in Tk 8.4.

This uses the option command to do this, and also uses the option
command to set the default font for various UI elements to the
UI font ("uifont").

Signed-off-by: Paul Mackerras <paulus@samba.org>
16 years agogit-stash: use stdout instead of stderr for non error messages
Marco Costalba [Sat, 5 Jan 2008 08:57:24 +0000 (09:57 +0100)]
git-stash: use stdout instead of stderr for non error messages

Some scripts and libraries check stderr to detect a failing command,
instead of checking the exit code.  Because the output from git-status
is not primarily for machine consumption, it would not hurt to send
these messages to stdout instead and it will make it easier to drive
the command for such callers.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoUpdate draft release notes for 1.5.4
Junio C Hamano [Sat, 5 Jan 2008 07:13:47 +0000 (23:13 -0800)]
Update draft release notes for 1.5.4

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-am: Run git gc only once and not for every patch.
Michael Stefaniuc [Fri, 4 Jan 2008 18:59:26 +0000 (19:59 +0100)]
git-am: Run git gc only once and not for every patch.

With "too many unreachable loose objects" git gc --auto will always
trigger. This clutters the output of git am and thus git rebase.

Signed-off-by: Michael Stefaniuc <mstefani@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agot/t3800: do not use a temporary file to hold expected result.
Junio C Hamano [Sat, 5 Jan 2008 06:45:08 +0000 (22:45 -0800)]
t/t3800: do not use a temporary file to hold expected result.

It is a good practice to write program output to a temporary file
during the test, as it would allow easier postmortem when the tested
program does break.  But there is no benefit in writing the expected
output out to the temporary.

This actually fixes a bug in check_verify_failure() routine.
The intention of the test seems to make sure the "git mktag" command
fails, and it spits out the expected error message.  But if the
command did not fail as expected, the shell function as originally
written would not have detected the failure.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agot/t{3600,3800,5401}: do not use egrep when grep would do
Junio C Hamano [Sat, 5 Jan 2008 06:28:13 +0000 (22:28 -0800)]
t/t{3600,3800,5401}: do not use egrep when grep would do

There is nothing _wrong_ with egrep per se, but this way we
would have less dependency on external tools.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agot/t7001: avoid unnecessary ERE when using grep
Miklos Vajna [Sat, 5 Jan 2008 04:06:48 +0000 (05:06 +0100)]
t/t7001: avoid unnecessary ERE when using grep

As pointed out by Junio, it's unnecessary to use "grep -E" and ".+" when we can
just use "grep" and "..*".

Signed-off-by: Miklos Vajna <vmiklos@frugalware.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agot/t7600: avoid GNUism in grep
Junio C Hamano [Sat, 5 Jan 2008 06:22:55 +0000 (22:22 -0800)]
t/t7600: avoid GNUism in grep

Using \+ to mean "one or more" in grep without -E is a GNU
extension outside POSIX.  Avoid it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoMerge git://repo.or.cz/git-gui
Junio C Hamano [Sat, 5 Jan 2008 06:03:42 +0000 (22:03 -0800)]
Merge git://repo.or.cz/git-gui

* git://repo.or.cz/git-gui:
  git-gui: Make commit log messages end with a newline
  Added Swedish translation.
  git-gui: Unconditionally use absolute paths with Cygwin
  git-gui: Handle file mode changes (644->755) in diff viewer
  git-gui: Move frequently used commands to the top of the context menu.

16 years agoDocument git-reset defaults to HEAD if no commit is given
Marco Costalba [Fri, 4 Jan 2008 09:53:32 +0000 (10:53 +0100)]
Document git-reset defaults to HEAD if no commit is given

Signed-off by: Marco Costalba <mcostalba@gmail.com>

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agodiff: remove lazy config loading
Jeff King [Fri, 4 Jan 2008 09:45:31 +0000 (04:45 -0500)]
diff: remove lazy config loading

There is no point to this. Either:

  1. The program has already loaded git_diff_ui_config, in
     which case this is a noop.
  2. The program didn't, which means it is plumbing that
     does not _want_ git_diff_ui_config to be loaded.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agodiff: load funcname patterns in "basic" config
Jeff King [Fri, 4 Jan 2008 09:16:14 +0000 (04:16 -0500)]
diff: load funcname patterns in "basic" config

The funcname patterns influence the "comment" on @@ lines of
the diff. They are safe to use with plumbing since they
don't fundamentally change the meaning of the diff in any
way.

Since all diff users call either diff_ui_config or
diff_basic_config, we can get rid of the lazy reading of the
config.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoadd a "basic" diff config callback
Jeff King [Fri, 4 Jan 2008 08:59:34 +0000 (03:59 -0500)]
add a "basic" diff config callback

The diff porcelain uses git_diff_ui_config to set
porcelain-ish config options, like automatically turning on
color. The plumbing specifically avoids calling this
function, since it doesn't want things like automatic color
or rename detection.

However, some diff options should be set for both plumbing
and porcelain. For example, one can still turn on color in
git-diff-files using the --color command line option. This
means we want the color config from color.diff.* (so that
once color is on, we use the user's preferred scheme), but
_not_ the color.diff variable.

We split the diff config into "ui" and "basic", where
"basic" is suitable for use by plumbing (so _most_ things
affecting the output should still go into the "ui" part).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoreceive-pack: reject invalid refnames
Martin Koegler [Fri, 4 Jan 2008 19:37:17 +0000 (20:37 +0100)]
receive-pack: reject invalid refnames

Signed-off-by: Martin Koegler <mkoegler@auto.tuwien.ac.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoDon't access line[-1] for a zero-length "line" from fgets.
Jim Meyering [Fri, 4 Jan 2008 17:37:41 +0000 (18:37 +0100)]
Don't access line[-1] for a zero-length "line" from fgets.

A NUL byte at beginning of file, or just after a newline
would provoke an invalid buf[-1] access in a few places.

* builtin-grep.c (cmd_grep): Don't access buf[-1].
* builtin-pack-objects.c (get_object_list): Likewise.
* builtin-rev-list.c (read_revisions_from_stdin): Likewise.
* bundle.c (read_bundle_header): Likewise.
* server-info.c (read_pack_info_file): Likewise.
* transport.c (insert_packed_refs): Likewise.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-clean: make "Would remove ..." path relative to cwd again
Junio C Hamano [Fri, 4 Jan 2008 03:04:37 +0000 (19:04 -0800)]
git-clean: make "Would remove ..." path relative to cwd again

The rewrite changed the output to use the path relative to the
top of the work tree without a good reason.  This fixes it.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoFix grammar nits in documentation and in code comments.
Jim Meyering [Thu, 3 Jan 2008 14:18:07 +0000 (15:18 +0100)]
Fix grammar nits in documentation and in code comments.

Signed-off-by: Jim Meyering <meyering@redhat.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoUninline prefixcmp()
Junio C Hamano [Thu, 3 Jan 2008 09:23:12 +0000 (01:23 -0800)]
Uninline prefixcmp()

Now the routine is an open-coded loop that avoids an extra
strlen() in the previous implementation, it got a bit too big to
be inlined.  Uninlining it makes code footprint smaller but the
result still retains the avoidance of strlen() cost.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agoUpdate callers of check_ref_format()
Junio C Hamano [Wed, 2 Jan 2008 19:14:40 +0000 (11:14 -0800)]
Update callers of check_ref_format()

This updates send-pack and fast-import to use symbolic constants
for checking the return values from check_ref_format(), and also
futureproof the logic in lock_any_ref_for_update() to explicitly
name the case that is usually considered an error but is Ok for
this particular use.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
16 years agogit-svn: unlink index files that were globbed, too
Eric Wong [Wed, 2 Jan 2008 18:10:03 +0000 (10:10 -0800)]
git-svn: unlink index files that were globbed, too

commit 3157dd9e89a71e80673d0bc21b5c0630f3b1fe68 (git-svn: unlink
internal index files after operations) introduced unlinking
index files after fetching.  However, this missed indices for
refs that were created by globbing branches and tags.  This will
track all refs we ever touch during a fetch and unlink them at
exit time.

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