13 years agoDon't ignore a pack-refs write failure
Jim Meyering [Sun, 24 Jun 2007 19:13:11 +0000 (21:13 +0200)]
Don't ignore a pack-refs write failure

Without this, if the size of refs_file at that point is ever an exact
multiple of BUFSIZ, then an EIO or ENOSPC error on the final write would
not be diagnosed.

It's not worth worrying about EPIPE here.
Although theoretically possible that someone kill this process
with a manual SIGPIPE, it's not at all likely.

Signed-off-by: Jim Meyering <>
Signed-off-by: Junio C Hamano <>
13 years agodetect close failure on just-written file handles
Jim Meyering [Sun, 24 Jun 2007 19:20:41 +0000 (21:20 +0200)]
detect close failure on just-written file handles

I audited git for potential undetected write failures.
In the cases fixed below, the diagnostics I add mimic the diagnostics
used in surrounding code, even when that means not reporting
the precise strerror(errno) cause of the error.

Signed-off-by: Jim Meyering <>
Signed-off-by: Junio C Hamano <>
13 years agoconfig: Add --null/-z option for null-delimted output
Frank Lichtenheld [Mon, 25 Jun 2007 14:03:55 +0000 (16:03 +0200)]
config: Add --null/-z option for null-delimted output

Use \n as delimiter between key and value and \0 as
delimiter after each key/value pair. This should be
easily parsable output.

Signed-off-by: Frank Lichtenheld <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'maint'
Junio C Hamano [Wed, 27 Jun 2007 01:33:24 +0000 (18:33 -0700)]
Merge branch 'maint'

* maint:
  config: Change output of --get-regexp for valueless keys
  config: Complete documentation of --get-regexp
  cleanup merge-base test script
  Fix zero-object version-2 packs
  Ignore submodule commits when fetching over dumb protocols

13 years agoconfig: Change output of --get-regexp for valueless keys
Frank Lichtenheld [Mon, 25 Jun 2007 14:03:54 +0000 (16:03 +0200)]
config: Change output of --get-regexp for valueless keys

Print no space after the name of a key without value.
Otherwise keys without values are printed exactly the
same as keys with empty values.

Signed-off-by: Frank Lichtenheld <>
Signed-off-by: Junio C Hamano <>
13 years agoconfig: Complete documentation of --get-regexp
Frank Lichtenheld [Mon, 25 Jun 2007 14:03:53 +0000 (16:03 +0200)]
config: Complete documentation of --get-regexp

The asciidoc documentation of the --get-regexp option was
incomplete. Add some missing pieces:
 - List the option in SYNOPSIS
 - Mention that key names are printed

Signed-off-by: Frank Lichtenheld <>
Signed-off-by: Junio C Hamano <>
13 years agogit-new-workdir: Fix shell warning about operator == used with test.
Simon Hausmann [Mon, 25 Jun 2007 11:04:26 +0000 (13:04 +0200)]
git-new-workdir: Fix shell warning about operator == used with test.

Use = instead of == with test to test for equality.

Signed-off-by: Simon Hausmann <>
Signed-off-by: Junio C Hamano <>
13 years agocleanup merge-base test script
Sam Vilain [Tue, 26 Jun 2007 23:44:22 +0000 (11:44 +1200)]
cleanup merge-base test script

Add a picture, and keep the setup and the tests together.

Signed-off-by: Sam Vilain <>
Signed-off-by: Junio C Hamano <>
13 years agogit.spec: RPM failed, looking for wrong files.
Quy Tonthat [Tue, 26 Jun 2007 15:39:07 +0000 (01:39 +1000)]
git.spec: RPM failed, looking for wrong files.

RPM build broke with "File not found" error on git-gui.1 and git-citool.1
They actually are git-gui.1.gz and git-citool.1.gz

Signed-off-by: Quy Tonthat <>
Signed-off-by: Junio C Hamano <>
13 years agoFix zero-object version-2 packs
Linus Torvalds [Tue, 26 Jun 2007 21:34:02 +0000 (14:34 -0700)]
Fix zero-object version-2 packs

A pack-file can get created without any objects in it (to transfer "no
data" - which can happen if you use a reference git repo, for example,
or just otherwise just end up transferring only branch head information
and already have all the objects themselves).

And while we probably should never create an index for such a pack, if we
do (and we do), the index file size sanity checking was incorrect.

This fixes it.

Reported-and-tested-by: Jocke Tjernlund <>
Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoIgnore submodule commits when fetching over dumb protocols
Sven Verdoolaege [Tue, 26 Jun 2007 21:19:41 +0000 (23:19 +0200)]
Ignore submodule commits when fetching over dumb protocols

Without this patch, the code would look for the submodule
commits in the superproject and (needlessly) fail when it
couldn't find them.

Signed-off-by: Sven Verdoolaege <>
Acked-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'master' of git:// into pm/gitk
Junio C Hamano [Tue, 26 Jun 2007 22:41:06 +0000 (15:41 -0700)]
Merge branch 'master' of git://git./gitk/gitk into pm/gitk

* 'master' of git:// (21 commits)
  gitk: Add a progress bar to show progress while resetting
  gitk: Improve handling of whitespace and special chars in filenames
  gitk: Fix bug causing nearby tags/heads to sometimes not be displayed
  gitk: Limit how often we change the canvas scrolling region
  gitk: Add a "reset branch to here" row context-menu operation
  gitk: Get rid of the childlist variable
  gitk: Speed up the reading of references
  gitk: Show local uncommitted changes as a fake commit
  gitk: New algorithm for drawing the graph lines
  gitk: Store ids in rowrangelist and idrowranges rather than row numbers
  gitk: Disable the head context menu entries for the checked-out branch
  gitk: Cope with commit messages with carriage-returns and initial blank lines
  gitk: Implement a simple scheduler for the compute-intensive stuff
  gitk: Improve the behaviour of the initial selection
  gitk: Add some more comments to the optimize_rows procedure
  gitk: Don't try to list large numbers of tags or heads in the details pane
  gitk: New infrastructure for working out branches & previous/next tags
  [PATCH] gitk: Allow specifying tabstop as other than default 8 characters.
  [PATCH] gitk: Update fontsize in patch / tree list
  [PATCH] gitk: Make selection highlight color configurable



13 years agogitk: Add a progress bar to show progress while resetting
Paul Mackerras [Tue, 26 Jun 2007 01:09:49 +0000 (11:09 +1000)]
gitk: Add a progress bar to show progress while resetting

Since git reset now gets chatty while resetting, we were getting errors
reported when a reset was done using the "reset branch to here" menu
item.  With this we now read the progress messages from git reset and
update a progress bar.  Because git reset outputs the progress messages
to standard error, and Tcl treats messages to standard error as error
messages, we have to invoke git reset via a shell and redirect standard
error into standard output.

This also fixes a bug in computing descendent heads when head ids
are changed via a reset.

Signed-off-by: Paul Mackerras <>
13 years agogit-send-email: Do not make @-less message ID
Junio C Hamano [Wed, 20 Jun 2007 20:47:34 +0000 (13:47 -0700)]
git-send-email: Do not make @-less message ID

When the original $from address fails to yield a valid-looking
e-mail address, we created a bogus looking message ID, formatted
like this:

Message-Id: <11823357623688-git-send-email->

This commit fixes it by moving call to make_message_id() to
where it matters, namely, before the $message_id is needed to be
placed in the generated e-mail header; this has an important
side effect of making it clear that $from is already available.

Also throw in Sys::Hostname::hostname() just for fun, although I
suspect that the code would never trigger due to the modified
call sequence that makes sure $from is always available.  This
is based on a suggestion by Michael Hendricks.

Signed-off-by: Junio C Hamano <>
13 years agogit-svn: trailing slash in prefix is mandatory with --branches/-b
Gerrit Pape [Tue, 19 Jun 2007 16:47:41 +0000 (16:47 +0000)]
git-svn: trailing slash in prefix is mandatory with --branches/-b

Make clear in the documentation that when using --branches/-b and
--prefix with 'init', the prefix must include a trailing slash.
This matches the actual behavior of git-svn, e.g.:

 $ git svn init -Ttrunk -treleases -bbranches --prefix xxx \
 --prefix='xxx' must have a trailing slash '/'

This was noticed by R. Vanicat and reported through

Signed-off-by: Gerrit Pape <>
Signed-off-by: Junio C Hamano <>
13 years agonew-workdir: handle rev-parse --git-dir not always giving full path
Julian Phillips [Tue, 19 Jun 2007 11:44:43 +0000 (12:44 +0100)]
new-workdir: handle rev-parse --git-dir not always giving full path

rev-parse --git-dir outputs a full path - except for the single case
of when the path would be $(pwd)/.git, in which case it outputs simply
.git.  Check for this special case and handle it.

Signed-off-by: Julian Phillips <>
Signed-off-by: Junio C Hamano <>
13 years agomake dist: include configure script in tarball
Matthias Lederhofer [Mon, 18 Jun 2007 21:30:36 +0000 (23:30 +0200)]
make dist: include configure script in tarball

Signed-off-by: Matthias Lederhofer <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'lt/follow'
Junio C Hamano [Sun, 24 Jun 2007 09:08:31 +0000 (02:08 -0700)]
Merge branch 'lt/follow'

* lt/follow:
  Fix up "git log --follow" a bit..
  Finally implement "git log --follow"

13 years agot9500: skip gitweb tests if perl version is too old
Sven Verdoolaege [Fri, 22 Jun 2007 15:49:08 +0000 (17:49 +0200)]
t9500: skip gitweb tests if perl version is too old

gitweb calls Encode::decode_utf8 with two arguments,
but old versions of perl only allow this function to be called
with one argument.  Even older versions of perl do not even
have an Encode module.

Signed-off-by: Sven Verdoolaege <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'master' of git://
Junio C Hamano [Sun, 24 Jun 2007 06:54:41 +0000 (23:54 -0700)]
Merge branch 'master' of git://

* 'master' of git:// (260 commits)
  Avoid src:dst syntax as default bash completion for git push
  Make it possible to specify the HEAD for the internal findUpstreamBranchPoint function.
  Added git-p4 branches command that shows the mapping of perforce depot paths to imported git branches.
  Warn about conflicting p4 branch mappings and use the first one found.
  Fix the branch mapping detection to be independent from the order of the "p4 branches" output.
  git-p4 fails when cloning a p4 depo.
  Fix initial multi-branch import.
  Only use double quotes on Windows
  Fix git-p4 rebase to detect the correct upstream branch instead of unconditionally
  Moved the code from git-p4 submit to figure out the upstream branch point
  git-p4 submit: Fix missing quotes around p4 commands to make them work with spaces in filenames
  Mention remotes/p4/master also in the documentation.
  Provide some information for single branch imports where the commits go
  git-p4: check for existence of repo dir before trying to create
  Write out the options tag in the log message of imports only if we actually have
  Fix support for explicit disabling of syncing with the origin
  Fix depot-paths encoding for multi-path imports (don't split up //depot/path/foo)
  Fix project name guessing
  Fix updating/creating remotes/p4/* heads from origin/p4/*
  Fixed the check to make sure to exclude the HEAD symbolic refs when updating

13 years agogitk: Improve handling of whitespace and special chars in filenames
Paul Mackerras [Sat, 23 Jun 2007 10:28:15 +0000 (20:28 +1000)]
gitk: Improve handling of whitespace and special chars in filenames

The main thing here is better parsing of the diff --git lines in the
output of git diff-tree -p.  We now cope with filenames in quotes with
special chars escaped.  If the filenames contain spaces they aren't
quoted, however, which can create difficulties in parsing.  We get
around the difficulties by detecting the case when the filename hasn't
changed (chop the part after "diff --git " in two and see if the halves
match apart from a/ in one and b/ in the other), and if it hasn't
changed, we just use one half.  If the filename has changed we wait
for the "rename from" and "rename to" lines, which give the old and
new filenames unambiguously.

This also improves the parsing of the output of git diff-tree.
Instead of using lindex to extract the filename, we take the part from
the first tab on, and if it starts with a quote, we use [lindex $str 0]
to remove the quotes and convert the escapes.

This also gets rid of some unused tagging of the diff text, uses
[string compare] instead of [regexp] in some places, and fixes the
regexp for detecting the @@ hunk-separator lines (the regexp wasn't
accepting a single number, as in "-0,0 +1" for example).

Signed-off-by: Paul Mackerras <>
13 years agogitk: Fix bug causing nearby tags/heads to sometimes not be displayed
Paul Mackerras [Mon, 18 Jun 2007 12:39:21 +0000 (22:39 +1000)]
gitk: Fix bug causing nearby tags/heads to sometimes not be displayed

When we compute descendent heads and descendent/ancestor tags, we
cache the results.  We need to be careful to invalidate the cache
when we add stuff to the graph.  Also make sure that when we cache
descendent heads for a node we only cache the heads that are actually
descendents of that node.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Limit how often we change the canvas scrolling region
Paul Mackerras [Tue, 31 Oct 2006 04:00:53 +0000 (15:00 +1100)]
gitk: Limit how often we change the canvas scrolling region

For some unknown reason, changing the scrolling region on the canvases
provokes multiple milliseconds worth of computation in the X server,
and this can end up slowing gitk down significantly.  This works around
the problem by limiting the rate at which we update the scrolling region
after the first 100 rows to at most 2 per second.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Add a "reset branch to here" row context-menu operation
Paul Mackerras [Thu, 19 Oct 2006 00:09:06 +0000 (10:09 +1000)]
gitk: Add a "reset branch to here" row context-menu operation

This adds an entry to the menu that comes up when the user does a
right-click on a row.  The new entry allows the user to reset the
currently checked-out head to the commit for the row that they did
the right-click on.  The user has to select what type of reset to
do, and confirm the reset, via a dialog box that pops up.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Get rid of the childlist variable
Paul Mackerras [Sun, 17 Jun 2007 23:48:23 +0000 (09:48 +1000)]
gitk: Get rid of the childlist variable

The information in childlist is a duplicate of what's in the children
array, and it wasn't being accessed often enough to be really worth
keeping the list around as well.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Speed up the reading of references
Paul Mackerras [Mon, 11 Sep 2006 00:36:53 +0000 (10:36 +1000)]
gitk: Speed up the reading of references

We were doing two execs for each tag - one to map the tag ID to a
commit ID and one to read the contents of the tag for later display.
This speeds up the process by not reading the contents of the tag
(instead it is read later if needed), and by using the -d flag to
git show-ref, which gives us refs/tags/foo^{} lines which give us
the commit ID.  Also this uses string operations instead of regexps.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Show local uncommitted changes as a fake commit
Paul Mackerras [Thu, 7 Sep 2006 00:21:39 +0000 (10:21 +1000)]
gitk: Show local uncommitted changes as a fake commit

If there are local changes in the repository, i.e., git-diff-index HEAD
produces some output, then this optionally displays an extra row in
the graph as a child of the HEAD commit (but with a red circle to
indicate that it's not a real commit).  There is a checkbox in the
preferences window to control whether gitk does this or not.

Clicking on the extra row shows the diffs between the working directory
and the HEAD (using git diff-index -p).  The right-click menu on the
extra row allows the user to generate a patch containing the local diffs,
or to display the diffs between the working directory and any commit.

Signed-off-by: Paul Mackerras <>
13 years agogitk: New algorithm for drawing the graph lines
Paul Mackerras [Sun, 15 Oct 2006 08:03:46 +0000 (18:03 +1000)]
gitk: New algorithm for drawing the graph lines

This only draws as much of the graph lines as is visible.  This can
happen by adding coordinates on to an existing graph line or by
creating a new line.  This means that we only need to have laid out
and optimized as much of the graph as is actually visible in order to
draw it, including the lines (previously we didn't draw a graph
line until we had laid out and optimized to the end of a segment of
the line, i.e. down to a down-arrow or to the row where the line's
commit is displayed).  This also lets us get rid of the linesegends
list, and gives us an easy workaround for the X server bug that
causes long lines to be misdrawn.  This also gets rid of the use
of rowoffsets in drawlineseg et al.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Store ids in rowrangelist and idrowranges rather than row numbers
Paul Mackerras [Sat, 14 Oct 2006 09:21:02 +0000 (19:21 +1000)]
gitk: Store ids in rowrangelist and idrowranges rather than row numbers

This removes the need for insertrow to go through rowrangelist and
idrowranges and adjust a lot of entries.  The first entry for a given
id is now the row number of the first child, not that row number + 1,
and rowranges compensates for that so its callers didn't have to
change.  This adds a ranges argument to drawlineseg so that we can
avoid calling rowranges a second time inside drawlineseg (all its
callers already called rowranges).

Signed-off-by: Paul Mackerras <>
13 years agogitk: Disable the head context menu entries for the checked-out branch
Paul Mackerras [Sun, 17 Jun 2007 07:08:35 +0000 (17:08 +1000)]
gitk: Disable the head context menu entries for the checked-out branch

Neither the "check out this branch" nor the "remove this branch"
menu item can be used on the currently-checked out branch, so disable

Signed-off-by: Paul Mackerras <>
13 years agogitk: Cope with commit messages with carriage-returns and initial blank lines
Paul Mackerras [Wed, 27 Sep 2006 00:56:02 +0000 (10:56 +1000)]
gitk: Cope with commit messages with carriage-returns and initial blank lines

In some repositories imported from other systems we can get carriage
return characters in the commit message, which leads to a multi-line
headline being displayed in the summary window, which looks bad.
Also some commit messages start with one or more blank lines, which
leads to an empty headline.  This fixes these problems.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Implement a simple scheduler for the compute-intensive stuff
Paul Mackerras [Sun, 17 Jun 2007 04:45:00 +0000 (14:45 +1000)]
gitk: Implement a simple scheduler for the compute-intensive stuff

This allows us to do compute-intensive processing, such as laying out
the graph, relatively efficiently while also having the GUI be
reasonably responsive.  The problem previously was that file events
were serviced before X events, so reading from another process which
supplies data quickly (hi git rev-list :) could mean that X events
didn't get processed for a long time.

With this, gitk finishes laying out the graph slightly sooner and
still responds to the GUI while doing so.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Improve the behaviour of the initial selection
Paul Mackerras [Sat, 16 Jun 2007 11:51:08 +0000 (21:51 +1000)]
gitk: Improve the behaviour of the initial selection

It used to be that if you clicked on a line while gitk was still drawing
stuff, it would immediately re-select the first line of the display.
This fixes that.

Signed-off-by: Paul Mackerras <>
13 years agogitk: Add some more comments to the optimize_rows procedure
Paul Mackerras [Thu, 14 Sep 2006 23:45:23 +0000 (09:45 +1000)]
gitk: Add some more comments to the optimize_rows procedure

Signed-off-by: Paul Mackerras <>
13 years agogitk: Don't try to list large numbers of tags or heads in the details pane
Paul Mackerras [Sat, 16 Jun 2007 11:21:57 +0000 (21:21 +1000)]
gitk: Don't try to list large numbers of tags or heads in the details pane

With some large repositories, a commit can end up on thousands of
branches, which results in an extremely long "Branches:" line in the
details window, and that results in the window being extremely slow
to scroll.

This fixes it by just showing "many (N)" after "Branches:", "Follows:"
or "Precedes:", where N is the number of heads or tags.  The limit
is currently set at 20 but could be made configurable (and the "many"
could be a link to pop up a window listing them all in case anyone
really wants to know).

Signed-off-by: Paul Mackerras <>
13 years agogitk: New infrastructure for working out branches & previous/next tags
Paul Mackerras [Sat, 16 Jun 2007 10:29:25 +0000 (20:29 +1000)]
gitk: New infrastructure for working out branches & previous/next tags

Instead of working out descendent heads and descendent & ancestor
branches in a two-pass algorithm, this reads and stores a simplified
version of the graph topology, and works out descendent/ancestor
tags and descendent heads on demand (with a bit of caching).

The advantages of this are, first, that we now don't have to use
--topo-order on the git rev-list process.  Secondly, we don't have
to re-read the whole graph when tags or heads change or even when
the graph changes.  Since we can cope with parents coming before
children, we can update the graph by running a git rev-list with
arguments that just give us the new commits, and merge the new
commits into the simplified graph.

The graph is simplified in the sense that commits with exactly one
parent and one child (which is >90% of them in most cases) are grouped
together into arcs joining nodes or 'branch/merge points', which are
the commits that don't have exactly 1 parent and 1 child.  This reduces
the size of the graph substantially and decreases the time to traverse
it correspondingly.

Signed-off-by: Paul Mackerras <>
13 years agoFix up "git log --follow" a bit..
Linus Torvalds [Thu, 21 Jun 2007 17:22:59 +0000 (10:22 -0700)]
Fix up "git log --follow" a bit..

This fixes "git log --follow" to hopefully not leak memory any more, and
also cleans it up a bit to look more like some of the other functions that
use "diff_queued_diff" (by *not* using it directly as a global in the
code, but by instead just taking a pointer to the diff queue and using

As to "diff_queued_diff", I think it would be better off not as a global
at all, but as being just an entry in the "struct diff_options" structure,
but that's a separate issue, and there may be some subtle reason for why
it's currently a global.

Anyway, no real changes. Instead of having a magical first entry in the
diff-queue, we now end up just keeping the diff-queue clean, and keeping
our "preferred" file pairing in an internal "choice" variable. That makes
it easy to switch the choice around when we find a better one.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoFinally implement "git log --follow"
Linus Torvalds [Tue, 19 Jun 2007 21:22:46 +0000 (14:22 -0700)]
Finally implement "git log --follow"

Ok, I've really held off doing this too damn long, because I'm lazy, and I
was always hoping that somebody else would do it.

But no, people keep asking for it, but nobody actually did anything, so I
decided I might as well bite the bullet, and instead of telling people
they could add a "--follow" flag to "git log" to do what they want to do,
I decided that it looks like I just have to do it for them..

The code wasn't actually that complicated, in that the diffstat for this
patch literally says "70 insertions(+), 1 deletions(-)", but I will have
to admit that in order to get to this fairly simple patch, you did have to
know and understand the internal git diff generation machinery pretty
well, and had to really be able to follow how commit generation interacts
with generating patches and generating the log.

So I suspect that while I was right that it wasn't that hard, I might have
been expecting too much of random people - this patch does seem to be
firmly in the core "Linus or Junio" territory.

To make a long story short: I'm sorry for it taking so long until I just
did it.

I'm not going to guarantee that this works for everybody, but you really
can just look at the patch, and after the appropriate appreciative noises
("Ooh, aah") over how clever I am, you can then just notice that the code
itself isn't really that complicated.

All the real new code is in the new "try_to_follow_renames()" function. It
really isn't rocket science: we notice that the pathname we were looking
at went away, so we start a full tree diff and try to see if we can
instead make that pathname be a rename or a copy from some other previous
pathname. And if we can, we just continue, except we show *that*
particular diff, and ever after we use the _previous_ pathname.

One thing to look out for: the "rename detection" is considered to be a
singular event in the _linear_ "git log" output! That's what people want
to do, but I just wanted to point out that this patch is *not* carrying
around a "commit,pathname" kind of pair and it's *not* going to be able to
notice the file coming from multiple *different* files in earlier history.

IOW, if you use "git log --follow", then you get the stupid CVS/SVN kind
of "files have single identities" kind of semantics, and git log will just
pick the identity based on the normal move/copy heuristics _as_if_ the
history could be linearized.

Put another way: I think the model is broken, but given the broken model,
I think this patch does just about as well as you can do. If you have
merges with the same "file" having different filenames over the two
branches, git will just end up picking _one_ of the pathnames at the point
where the newer one goes away. It never looks at multiple pathnames in

And if you understood all that, you probably didn't need it explained, and
if you didn't understand the above blathering, it doesn't really mtter to
you. What matters to you is that you can now do

git log -p --follow builtin-rev-list.c

and it will find the point where the old "rev-list.c" got renamed to
"builtin-rev-list.c" and show it as such.

Signed-off-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'jc/oneline'
Junio C Hamano [Sat, 23 Jun 2007 06:33:08 +0000 (23:33 -0700)]
Merge branch 'jc/oneline'

* jc/oneline:
  pp_header(): work around possible memory corruption

13 years agoMerge branch 'ei/oneline+add-empty'
Junio C Hamano [Sat, 23 Jun 2007 06:32:19 +0000 (23:32 -0700)]
Merge branch 'ei/oneline+add-empty'

* ei/oneline+add-empty:
  Fix ALLOC_GROW calls with obsolete semantics
  Fix ALLOC_GROW off-by-one
  builtin-add: simplify (and increase accuracy of) exclude handling
  dir_struct: add collect_ignored option
  Extend --pretty=oneline to cover the first paragraph,
  Lift 16kB limit of log message output

13 years agofilter-branch: add example to move everything into a subdirectory
Johannes Schindelin [Thu, 21 Jun 2007 17:05:30 +0000 (18:05 +0100)]
filter-branch: add example to move everything into a subdirectory

This is based on Jeff King's example in

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoMerge branch 'js/filter'
Junio C Hamano [Sat, 23 Jun 2007 06:20:40 +0000 (23:20 -0700)]
Merge branch 'js/filter'

* js/filter:
  filter-branch: subdirectory filter needs --full-history
  filter-branch: Simplify parent computation.
  Teach filter-branch about subdirectory filtering
  filter-branch: also don't fail in map() if a commit cannot be mapped
  filter-branch: Use rev-list arguments to specify revision ranges.
  filter-branch: fix behaviour of '-k'
  filter-branch: use $(($i+1)) instead of $((i+1))
  chmod +x
  filter-branch: prevent filters from reading from stdin
  t7003: make test repeatable
  Add git-filter-branch

13 years agoTwo trivial -Wcast-qual fixes
Junio C Hamano [Sat, 23 Jun 2007 06:19:43 +0000 (23:19 -0700)]
Two trivial -Wcast-qual fixes

Luiz Fernando N. Capitulino noticed the one in tree-walk.h where
we cast away constness while computing the legnth of a tree

Signed-off-by: Junio C Hamano <>
13 years agodiffcore-rename: favour identical basenames
Johannes Schindelin [Thu, 21 Jun 2007 11:52:11 +0000 (12:52 +0100)]
diffcore-rename: favour identical basenames

When there are several candidates for a rename source, and one of them
has an identical basename to the rename target, take that one.

Noticed by Govind Salinas, posted by Shawn O. Pearce, partial patch
by Linus Torvalds.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
13 years agoDocument git-gui, git-citool as mainporcelain manual pages
Shawn O. Pearce [Thu, 21 Jun 2007 04:51:00 +0000 (00:51 -0400)]
Document git-gui, git-citool as mainporcelain manual pages

Jakub Narebski pointed out that the git-gui blame viewer is not a
widely known feature, but is incredibly useful.  Part of the issue
is advertising.  Up until now we haven't even referenced git-gui from
within the core Git manual pages, mostly because I just wasn't sure
how I wanted to supply git-gui documentation to end-users, or how
that documentation should integrate with the core Git documentation.

Based upon Jakub's comment that many users may not even know that
the gui is available in a stock Git distribution I'm offering up
two basic manual pages: git-citool and git-gui.  These should offer
enough of a starting point for users to identify that the gui exists,
and how to start it.  Future releases of git-gui may contain their
own documentation system available from within a running git-gui.
But not today.

Signed-off-by: Shawn O. Pearce <>
Signed-off-by: Junio C Hamano <>
13 years agoGenerate tags with correct timestamp (git-svnimport)
Dave O'Neill [Thu, 21 Jun 2007 21:48:56 +0000 (17:48 -0400)]
Generate tags with correct timestamp (git-svnimport)

Now uses git-tag instead of manually constructing the tag.  This gives us a
correct timestamp, removes some crufty code, and makes it work the same as

The generated tags are now lightweight tags instead of tag objects, which may
or may not be the behaviour we want.

Also, remove two unused variables from git-cvsimport.

Signed-off-by: Junio C Hamano <>
13 years agoImport branch 'git-p4' of git://
Shawn O. Pearce [Fri, 22 Jun 2007 23:08:36 +0000 (19:08 -0400)]
Import branch 'git-p4' of git://

Simon has asked that the git.git project include the git-p4 project
as at least a contrib/fast-import within git.git.  I think it makes
a lot of sense, as git-p4 nicely complements the only other in-tree
fast-import user: import-tars.perl.

git-p4 is offered under the MIT license by its authors.

13 years agoAvoid src:dst syntax as default bash completion for git push
Shawn O. Pearce [Fri, 22 Jun 2007 22:44:04 +0000 (18:44 -0400)]
Avoid src:dst syntax as default bash completion for git push

Raimund Bauer just discovered that the default bash completion for
a local branch name in a git-push line is not the best choice when
the branch does not exist on the remote system.

In the past we have always completed the local name 'test' as
"test:test", indicating that the destination name is the same as
the local name.  But this fails when "test" does not yet exist on
the remote system, as there is no "test" branch for it to match
the name against.

Fortunately git-push does the right thing when given just the
local branch, as it assumes you want to use the same name in the
destination repository.  So we now offer "test" as the completion
in a git-push line, and let git-push assume that is also the remote
branch name.

We also still support the remote branch completion after the :,
but only if the user manually adds the colon before trying to get
a completion.

Signed-off-by: Shawn O. Pearce <>
13 years agoMake it possible to specify the HEAD for the internal findUpstreamBranchPoint function.
Simon Hausmann [Thu, 21 Jun 2007 22:01:57 +0000 (00:01 +0200)]
Make it possible to specify the HEAD for the internal findUpstreamBranchPoint function.

This isn't used right now in git-p4 but I use it in an external script that loads git-p4 as module.

Signed-off-by: Simon Hausmann <>
13 years agoCloning from a repo without "current branch"
Nanako Shiraishi [Sat, 16 Jun 2007 22:26:08 +0000 (15:26 -0700)]
Cloning from a repo without "current branch"

If the remote repository does not have a "current branch", git-clone
was confused and did not set up the resulting new repository
correctly.  It did not reset HEAD from the default 'master', and did
not write the SHA1 to the master branch.

Signed-off-by: Nanako Shiraishi <>
Signed-off-by: Junio C Hamano <>
13 years agoChange default man page path to /usr/share/man
Ismail Dönmez [Tue, 8 May 2007 10:49:31 +0000 (13:49 +0300)]
Change default man page path to /usr/share/man

According to FHS,

default man page path is $prefix/share/man.

Signed-off-by: Ismail Donmez <>
Signed-off-by: Junio C Hamano <>
13 years agoINSTALL: explain how to build documentation
Junio C Hamano [Mon, 18 Jun 2007 08:43:34 +0000 (01:43 -0700)]
INSTALL: explain how to build documentation

Signed-off-by: Junio C Hamano <>
13 years agocvsserver: Actually implement --export-all
Alex Riesen [Sat, 16 Jun 2007 23:52:50 +0000 (01:52 +0200)]
cvsserver: Actually implement --export-all

Frank Lichtenheld, Fri, Jun 15, 2007 03:01:53 +0200:
> +test_expect_failure 'req_Root failure (export-all w/o whitelist)' \
> +  'cat request-anonymous | git-cvsserver --export-all pserver >log 2>&1
> +   || false'

This does not work, at least for bash in current Ubuntu:

    GNU bash, version 3.2.13(1)-release

You have to put "||" on the previous line:

Signed-off-by: Junio C Hamano <>
13 years agoAdded git-p4 branches command that shows the mapping of perforce depot paths to impor...
Simon Hausmann [Wed, 20 Jun 2007 21:10:28 +0000 (23:10 +0200)]
Added git-p4 branches command that shows the mapping of perforce depot paths to imported git branches.

Signed-off-by: Simon Hausmann <>
14 years agoWarn about conflicting p4 branch mappings and use the first one found.
Simon Hausmann [Sun, 17 Jun 2007 13:10:24 +0000 (15:10 +0200)]
Warn about conflicting p4 branch mappings and use the first one found.

Signed-off-by: Simon Hausmann <>
14 years agoFix the branch mapping detection to be independent from the order of the "p4 branches...
Simon Hausmann [Sun, 17 Jun 2007 09:25:34 +0000 (11:25 +0200)]
Fix the branch mapping detection to be independent from the order of the "p4 branches" output.

Collect "unknown" source branches separately and register them at the end.

Also added a minor speed up to splitFilesIntoBranches by breaking out of the loop through all branches when it's safe.

Signed-off-by: Simon Hausmann <>
14 years agoFix ALLOC_GROW calls with obsolete semantics
Jeff King [Sat, 16 Jun 2007 22:43:40 +0000 (18:43 -0400)]
Fix ALLOC_GROW calls with obsolete semantics

ALLOC_GROW now expects the 'nr' argument to be "how much you
want" and not "how much you have". This fixes all cases
where we weren't previously adding anything to the 'nr'.

Signed-off-by: Jeff King <>
Signed-off-by: Junio C Hamano <>
14 years agoMerge branch 'jk/add-empty' into ei/oneline+add-empty
Junio C Hamano [Sun, 17 Jun 2007 00:58:28 +0000 (17:58 -0700)]
Merge branch 'jk/add-empty' into ei/oneline+add-empty

* jk/add-empty:
  builtin-add: simplify (and increase accuracy of) exclude handling
  dir_struct: add collect_ignored option

14 years agopp_header(): work around possible memory corruption
Johannes Schindelin [Fri, 15 Jun 2007 12:19:07 +0000 (13:19 +0100)]
pp_header(): work around possible memory corruption

add_user_info() possibly adds way more than just the commit header line.
In fact, it sometimes needs so much more space that there is a buffer
overrun, leading to an ugly crash. For example, the date is printed in its
own line, and usually takes up more space than the equivalent Unix epoch.

So, for good measure, add 80 characters (a full line) to the allocated
space, in addition to the header line length.

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
14 years agoFix ALLOC_GROW off-by-one
Jeff King [Sat, 16 Jun 2007 22:37:39 +0000 (18:37 -0400)]
Fix ALLOC_GROW off-by-one

The ALLOC_GROW macro will never let us fill the array completely,
instead allocating an extra chunk if that would be the case. This is
because the 'nr' argument was originally treated as "how much we do have
now" instead of "how much do we want". The latter makes much more
sense because you can grow by more than one item.

This off-by-one never resulted in an error because it meant we were
overly conservative about when to allocate. Any callers which passed
"how much we have now" need to be updated, or they will fail to allocate

Signed-off-by: Jeff King <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git log --full-diff
Jakub Narebski [Sat, 16 Jun 2007 19:03:38 +0000 (21:03 +0200)]
Document git log --full-diff

Based on description of commit 477f2b41310c4b1040a9e7f72720b9c39d82caf9
"git log --full-diff" adding this option.

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git log --abbrev-commit, as a kind of pretty option
Jakub Narebski [Sat, 16 Jun 2007 19:03:39 +0000 (21:03 +0200)]
Document git log --abbrev-commit, as a kind of pretty option

Documentation taken from paraphrased description of "--abbrev[=<n>]"
diff option, and from description of commit 5c51c985 introducing
this option.

Note that to change number of digits one must use "--abbrev=<n>",
which affects [also] diff output.

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoUse tabs for indenting definition list for options in git-log.txt
Jakub Narebski [Sat, 16 Jun 2007 19:03:37 +0000 (21:03 +0200)]
Use tabs for indenting definition list for options in git-log.txt

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git rev-list --timestamp
Jakub Narebski [Sat, 16 Jun 2007 19:03:42 +0000 (21:03 +0200)]
Document git rev-list --timestamp

Note that git log does not understand this option yet:

  $ git log --timestamp
  fatal: unrecognized argument: --timestamp

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git reflog --stale-fix
Jakub Narebski [Sat, 16 Jun 2007 19:03:46 +0000 (21:03 +0200)]
Document git reflog --stale-fix

Document --stale-fix, used in "git reflog expire --stale-fix --all"
to remove invalid reflog entries, to fix situation after running
non reflog-aware git-prune from an older git in the presence of
reflogs (see RelNotes-1.5.0.txt).

Based on description of commit 1389d9ddaa68a4cbf5018d88f971b9bbb7aaa3c9
  "reflog expire --fix-stale"
which introduced this option.

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git rev-parse --is-inside-git-dir
Jakub Narebski [Sat, 16 Jun 2007 19:03:44 +0000 (21:03 +0200)]
Document git rev-parse --is-inside-git-dir

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git read-tree --trivial
Jakub Narebski [Sat, 16 Jun 2007 19:03:45 +0000 (21:03 +0200)]
Document git read-tree --trivial

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agoDocument git rev-list --full-history
Jakub Narebski [Sat, 16 Jun 2007 19:03:43 +0000 (21:03 +0200)]
Document git rev-list --full-history

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agogit-p4 fails when cloning a p4 depo.
Benjamin Sergeant [Fri, 8 Jun 2007 18:13:55 +0000 (11:13 -0700)]
git-p4 fails when cloning a p4 depo.

A perforce command with all the files in the repo is generated to get
all the file content.
Here is a patch to break it into multiple successive perforce command
who uses 4K of parameter max, and collect the output for later.

It works, but not for big depos, because the whole perforce depo
content is stored in memory in, and it looks like mine is
bigger than 2 Gigs, so I had to kill the process.

[Simon: I added the bit about using SC_ARG_MAX, as suggested by Han-Wen]

Signed-off-by: Benjamin Sergeant <>
Signed-off-by: Simon Hausmann <>
14 years agoDo not use h_errno after connect(2): the function does not set it
Alex Riesen [Tue, 12 Jun 2007 20:52:10 +0000 (22:52 +0200)]
Do not use h_errno after connect(2): the function does not set it

Randal L. Schwartz noticed compilation problems on SunOS, which made
me look at the code again. The thing is, h_errno is not used by
connect(2), it is only for functions from netdb.h, like gethostbyname.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
14 years agoDocumentation: update "stale" links for
Junio C Hamano [Sat, 16 Jun 2007 16:10:18 +0000 (09:10 -0700)]
Documentation: update "stale" links for

Signed-off-by: Junio C Hamano <>
14 years agoFix initial multi-branch import.
Simon Hausmann [Sat, 16 Jun 2007 11:09:21 +0000 (13:09 +0200)]
Fix initial multi-branch import.

The list of existing p4 branches in git wasn't initialized.

Signed-off-by: Simon Hausmann <>
14 years agoMerge branch 'jc/remote'
Junio C Hamano [Sat, 16 Jun 2007 08:22:45 +0000 (01:22 -0700)]
Merge branch 'jc/remote'

* jc/remote:
  git-push: Update description of refspecs and add examples
  remote.c: "git-push frotz" should update what matches at the source.
  remote.c: fix "git push" weak match disambiguation
  remote.c: minor clean-up of match_explicit()
  remote.c: refactor creation of new dst ref
  remote.c: refactor match_explicit_refs()

14 years agoMerge branch 'gp/branch'
Junio C Hamano [Sat, 16 Jun 2007 08:22:43 +0000 (01:22 -0700)]
Merge branch 'gp/branch'

* gp/branch:
  git-branch: cleanup config file when deleting branches

14 years agoMerge branch 'fl/cvsserver'
Junio C Hamano [Sat, 16 Jun 2007 08:22:38 +0000 (01:22 -0700)]
Merge branch 'fl/cvsserver'

* fl/cvsserver:
  cvsserver: Actually implement --export-all
  cvsserver: Let --base-path and pserver get along just fine
  cvsserver: Add some useful commandline options

14 years agoMerge branch 'lh/submodule'
Junio C Hamano [Sat, 16 Jun 2007 08:22:35 +0000 (01:22 -0700)]
Merge branch 'lh/submodule'

* lh/submodule:
  gitmodules(5): remove leading period from synopsis
  Add gitmodules(5)
  git-submodule: give submodules proper names
  Rename sections from "module" to "submodule" in .gitmodules
  git-submodule: remember to checkout after clone
  t7400: barf if git-submodule removes or replaces a file

14 years agoMerge branch 'maint' to sync with GIT
Junio C Hamano [Sat, 16 Jun 2007 08:22:10 +0000 (01:22 -0700)]
Merge branch 'maint' to sync with GIT

14 years agogit-svn: avoid string eval for defining functions
Sam Vilain [Fri, 15 Jun 2007 03:43:59 +0000 (15:43 +1200)]
git-svn: avoid string eval for defining functions

You don't need to use string eval to define new functions; assigning a
code reference to the target symbol table is enough.

Acked-by: Eric Wong <>
Signed-off-by: Junio C Hamano <>
14 years agoFix pushing to a pattern with no dst
Daniel Barkalow [Fri, 15 Jun 2007 14:22:37 +0000 (10:22 -0400)]
Fix pushing to a pattern with no dst

Refspecs with no colons are left with no dst value, because they are
interepreted differently for fetch and push. For push, they mean to
reuse the src side. Fix this for patterns.

Signed-off-by: Daniel Barkalow <>
Acked-by: Linus Torvalds <>
Signed-off-by: Junio C Hamano <>
14 years agoGIT v1.5.2.2
Junio C Hamano [Sat, 16 Jun 2007 06:58:18 +0000 (23:58 -0700)]

Signed-off-by: Junio C Hamano <>
14 years agoDocumentation: adjust to AsciiDoc 8
Junio C Hamano [Fri, 15 Jun 2007 05:20:16 +0000 (22:20 -0700)]
Documentation: adjust to AsciiDoc 8

It turns out that the attribute definition we have had for a
long time to hide "^" character from AsciiDoc 7 was not honored
by AsciiDoc 8 even under "-a asciidoc7compatible" mode.  There is
a similar breakage with the "compatible" mode with + characters.

The double colon at the end of definition list term needs
to be attached to the term, without a whitespace.  After this
minimum fixups, AsciiDoc 8 (I used 8.2.1 on Debian) with
compatibility mode seems to produce reasonably good results.

Signed-off-by: Junio C Hamano <>
14 years agoAvoid diff cost on "git log -z"
Junio C Hamano [Sat, 16 Jun 2007 06:48:35 +0000 (23:48 -0700)]
Avoid diff cost on "git log -z"

Johannes and Marco discovered that "git log -z" spent cycles in diff even
though there is no need to actually compute diffs.

Signed-off-by: Junio C Hamano <>
14 years agogit-branch --track: fix tracking branch computation.
Junio C Hamano [Sat, 16 Jun 2007 06:35:32 +0000 (23:35 -0700)]
git-branch --track: fix tracking branch computation.

The original code did not take hierarchical branch names into account at all.

[jc: cherry-picked 11f68d9 from 'master']

Tested-by: Gerrit Pape <>
Signed-off-by: Junio C Hamano <>
14 years ago$EMAIL is a last resort fallback, as it's system-wide.
Junio C Hamano [Sat, 16 Jun 2007 06:33:06 +0000 (23:33 -0700)]
$EMAIL is a last resort fallback, as it's system-wide.

$EMAIL is a system-wide setup that is used for many many many
applications. If the git user chose a specific setup,
then _this_ should be honoured rather than $EMAIL.

[jc: cherry-picked ec563e8 from 'master']

Signed-off-by: Pierre Habouzit <>
Signed-off-by: Junio C Hamano <>
14 years agomerge-recursive: refuse to merge binary files
Johannes Schindelin [Tue, 5 Jun 2007 02:36:49 +0000 (03:36 +0100)]
merge-recursive: refuse to merge binary files

[jc: cherry-picked 9f30855 from 'master']

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
14 years agoMove buffer_is_binary() to xdiff-interface.h
Johannes Schindelin [Tue, 5 Jun 2007 02:36:11 +0000 (03:36 +0100)]
Move buffer_is_binary() to xdiff-interface.h

We already have two instances where we want to determine if a buffer
contains binary data as opposed to text.

[jc: cherry-picked 6bfce93e from 'master']

Signed-off-by: Johannes Schindelin <>
Signed-off-by: Junio C Hamano <>
14 years agoAdd a local implementation of hstrerror for the system which do not have it
Alex Riesen [Wed, 13 Jun 2007 18:54:32 +0000 (20:54 +0200)]
Add a local implementation of hstrerror for the system which do not have it

The function converts the value of h_errno (last error of name
resolver library, see netdb.h).
One of systems which supposedly do not have the function is SunOS.
POSIX does not mandate its presence.

Signed-off-by: Alex Riesen <>
Signed-off-by: Junio C Hamano <>
14 years agogitmodules(5): remove leading period from synopsis
Lars Hjemli [Wed, 13 Jun 2007 22:01:10 +0000 (00:01 +0200)]
gitmodules(5): remove leading period from synopsis

Asciidoc treats a line starting with a period followed by a title as a
blocktitle element. My introduction of gitmodules(5) unfortunatly broke
the documentation build process due to this processing, since it made
asciidoc generate an illegal (empty) synopsis element. Removing the leading
period fixes the problem and also makes gitmodules(5) use the same synopsis
notation as gitattributes(5).

Noticed-by: Matthias Lederhofer <>
Signed-off-by: Lars Hjemli <>
Signed-off-by: Junio C Hamano <>
14 years agoGenerated spec file to be ignored is named git.spec and not git-core.spec
Jakub Narebski [Wed, 13 Jun 2007 23:12:20 +0000 (01:12 +0200)]
Generated spec file to be ignored is named git.spec and not git-core.spec

Signed-off-by: Jakub Narebski <>
Signed-off-by: Junio C Hamano <>
14 years agocvsserver: Actually implement --export-all
Frank Lichtenheld [Fri, 15 Jun 2007 01:01:53 +0000 (03:01 +0200)]
cvsserver: Actually implement --export-all

Embarrassing bug number two in my options patch.

Also enforce that --export-all is only ever used together with an
explicit whitelist. Otherwise people might export every git repository
on the whole system without realising.

Signed-off-by: Frank Lichtenheld <>
Signed-off-by: Junio C Hamano <>
14 years agocvsserver: Let --base-path and pserver get along just fine
Frank Lichtenheld [Fri, 15 Jun 2007 01:01:52 +0000 (03:01 +0200)]
cvsserver: Let --base-path and pserver get along just fine

Embarassing bug number one in my options patch.

Since the code for --base-path support rewrote
the cvsroot value after comparing it with a possible
existing value (i.e. from pserver authentication)
the check always failed.

Signed-off-by: Frank Lichtenheld <>
Signed-off-by: Junio C Hamano <>
14 years agogit-svn: reduce stat() calls for a backwards compatibility check
Eric Wong [Wed, 13 Jun 2007 09:37:05 +0000 (02:37 -0700)]
git-svn: reduce stat() calls for a backwards compatibility check

Also, this fixes a bug where in an odd case a remote named
"config" could get renamed to ".metadata".

Signed-off-by: Eric Wong <>
Signed-off-by: Junio C Hamano <>
14 years agogit-svn: test for creating new directories over svn://
Eric Wong [Wed, 13 Jun 2007 09:37:04 +0000 (02:37 -0700)]
git-svn: test for creating new directories over svn://

As reported by Matthieu Moy, this is causing svnserve to
terminate connections, because it segfaults.

This test is disabled by default and can be enabled by setting
SVNSERVE_PORT to an unbound (for TCP port in the
environment (in addition to SVN_TESTS=1).  I'm not comfortable
with having a test start a daemon by default and take up a port
that could potentially stay running if the test failed.

Signed-off-by: Eric Wong <>
Signed-off-by: Junio C Hamano <>
14 years agogit-svn: cleanup: factor out longest_common_path() function
Eric Wong [Wed, 13 Jun 2007 09:37:03 +0000 (02:37 -0700)]
git-svn: cleanup: factor out longest_common_path() function

I hadn't looked at this code in a while and had to read this
again to figure out what it did.  To avoid having to do this
again in the future, I just gave gave the hunk a descriptive

Signed-off-by: Eric Wong <>
Signed-off-by: Junio C Hamano <>
14 years agogitview: run blame with -C -C
Aneesh Kumar K.V [Wed, 13 Jun 2007 08:46:16 +0000 (14:16 +0530)]
gitview: run blame with -C -C

pass -C -C option to git-blame so that blame browsing
works when the data is copied over from other files.

Signed-off-by: Aneesh Kumar K.V <>
Signed-off-by: Junio C Hamano <>
14 years agogitview: Fix the blame interface.
Aneesh Kumar K.V [Wed, 13 Jun 2007 08:46:15 +0000 (14:16 +0530)]
gitview: Fix the blame interface.

The async reading from the pipe was skipping some of the
input lines. Fix the same by making sure that we add the
partial content of the previous read to the newly read

Signed-off-by: Aneesh Kumar K.V <>
Signed-off-by: Junio C Hamano <>
14 years agoMore static
Junio C Hamano [Wed, 13 Jun 2007 08:42:05 +0000 (01:42 -0700)]
More static

There still are quite a few symbols that ought to be static.

Signed-off-by: Junio C Hamano <>
14 years ago-Wold-style-definition fix
Junio C Hamano [Wed, 13 Jun 2007 08:22:51 +0000 (01:22 -0700)]
-Wold-style-definition fix

Signed-off-by: Junio C Hamano <>
14 years agoMakefile: allow generating git.o for debugging purposes
Junio C Hamano [Wed, 13 Jun 2007 08:28:21 +0000 (01:28 -0700)]
Makefile: allow generating git.o for debugging purposes

Signed-off-by: Junio C Hamano <>
14 years agoMakefile: common-cmds.h depends on script
Junio C Hamano [Wed, 13 Jun 2007 09:00:01 +0000 (02:00 -0700)]
Makefile: common-cmds.h depends on script

Signed-off-by: Junio C Hamano <>