git
17 years agoMerge branch 'maint'
Junio C Hamano [Tue, 3 Jul 2007 04:05:59 +0000 (21:05 -0700)] 
Merge branch 'maint'

* maint:
  Correctly document the name of the global excludes file configuration

17 years agoformat-patch: Add format.subjectprefix config option
Adam Roben [Mon, 2 Jul 2007 00:48:59 +0000 (17:48 -0700)] 
format-patch: Add format.subjectprefix config option

This change lets you use the format.subjectprefix config option to override the
default subject prefix.

Signed-off-by: Adam Roben <aroben@apple.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoTest 'git add' for unmerged entries when core.symlinks=false.
Johannes Sixt [Mon, 2 Jul 2007 11:28:42 +0000 (13:28 +0200)] 
Test 'git add' for unmerged entries when core.symlinks=false.

In 20314271679e169f324c118c69c8d9e0399feec9 git add was fixed if unmerged
entries are in the index and core.filemode=false. core.symlinks=false is
a similar case, which touches the same code path. Here is a test that
makes sure that the symlink property in the index is preserved, too.

Signed-off-by: Johannes Sixt <johannes.sixt@telecom.at>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years ago"git-push $URL" without refspecs pushes only matching branches
Junio C Hamano [Mon, 2 Jul 2007 02:00:08 +0000 (19:00 -0700)] 
"git-push $URL" without refspecs pushes only matching branches

When "git push" is run without any refspec (neither on the
command line nor in the config), we used to push "matching refs"
in the sense that anything under refs/ hierarchy that exist on
both ends were updated.  This used to be a sane default for
publishing your repository to another back when we did not have
refs/remotes/ hierarchy, but it does not make much sense these
days.

This changes the semantics to push only "matching branches".

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoCorrectly document the name of the global excludes file configuration
Michael Hendricks [Mon, 2 Jul 2007 16:48:34 +0000 (10:48 -0600)] 
Correctly document the name of the global excludes file configuration

Signed-off-by: Michael Hendricks <michael@ndrix.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'maint'
Junio C Hamano [Mon, 2 Jul 2007 23:41:44 +0000 (16:41 -0700)] 
Merge branch 'maint'

* maint:
  Make git-prune submodule aware (and fix a SEGFAULT in the process)

17 years agoMake git-prune submodule aware (and fix a SEGFAULT in the process)
Andy Parkins [Mon, 2 Jul 2007 12:56:58 +0000 (13:56 +0100)] 
Make git-prune submodule aware (and fix a SEGFAULT in the process)

I ran git-prune on a repository and got this:

 $ git-prune
 error: Object 228f8065b930120e35fc0c154c237487ab02d64a is a blob, not a commit
 Segmentation fault (core dumped)

This repository was a strange one in that it was being used to provide
its own submodule.  That is, the repository was cloned into a
subdirectory, an independent branch checked out in that subdirectory,
and then it was marked as a submodule.  git-prune then failed in the
above manner.

The problem was that git-prune was not submodule aware in two areas.

Linus said:

 > So what happens is that something traverses a tree object, looks at each
 > entry, sees that it's not a tree, and tries to look it up as a blob. But
 > subprojects are commits, not blobs, and then when you look at the object
 > more closely, you get the above kind of object type confusion.

and included a patch to add an S_ISGITLINK() test to reachable.c's
process_tree() function.  That fixed the first git-prune error, and
stopped it from trying to process the gitlink entries in trees as if
they were pointers to other trees (and of course failing, because
gitlinks _aren't_ trees).  That part of this patch is his.

The second area is add_cache_refs().  This is called before starting the
reachability analysis, and was calling lookup_blob() on every object
hash found in the index.  However, it is no longer true that every hash
in the index is a pointer to a blob, some of them are gitlinks, and are
not backed by any object at all, they are commits in another repository.
Normally this bug was not causing any problems, but in the case of the
self-referencing repository described above, it meant that the gitlink
hash was being marked as being of type OBJ_BLOB by add_cache_refs() call
to lookup_blob().  Then later, because that hash was also pointed to by
a ref, add_one_ref() would treat it as a commit; lookup_commit() would
return a NULL because that object was already noted as being an
OBJ_BLOB, not an OBJ_COMMIT; and parse_commit_buffer() would SEGFAULT on
that NULL pointer.

The fix made by this patch is to not blindly call lookup_blob() in
reachable.c's add_cache_refs(), and instead skip any index entries that
are S_ISGITLINK().

Signed-off-by: Andy Parkins <andyparkins@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'ew/svn'
Junio C Hamano [Mon, 2 Jul 2007 08:46:22 +0000 (01:46 -0700)] 
Merge branch 'ew/svn'

* ew/svn:
  git-svn: allow dcommit to retain local merge information

17 years agoMerge branch 'ns/stash'
Junio C Hamano [Mon, 2 Jul 2007 08:45:57 +0000 (01:45 -0700)] 
Merge branch 'ns/stash'

* ns/stash:
  Documentation: quote {non-attributes} for asciidoc
  git-stash: don't complain when listing in a repo with no stash
  git-stash: fix "can't shift that many" with no arguments
  git-stash: fix "no arguments" case in documentation
  git-stash: require "save" to be explicit and update documentation
  Document git-stash
  Add git-stash script

17 years agoMerge branch 'js/rebase'
Junio C Hamano [Mon, 2 Jul 2007 08:45:47 +0000 (01:45 -0700)] 
Merge branch 'js/rebase'

* js/rebase:
  Teach rebase -i about --preserve-merges
  rebase -i: provide reasonable reflog for the rebased branch
  rebase -i: several cleanups
  ignore git-rebase--interactive
  Teach rebase an interactive mode
  Move the pick_author code to git-sh-setup

17 years agoMerge branch 'jc/diffcore'
Junio C Hamano [Mon, 2 Jul 2007 08:45:12 +0000 (01:45 -0700)] 
Merge branch 'jc/diffcore'

* jc/diffcore:
  diffcore-delta.c: Ignore CR in CRLF for text files
  diffcore-delta.c: update the comment on the algorithm.
  diffcore_filespec: add is_binary
  diffcore_count_changes: pass diffcore_filespec

17 years agoDocumentation: minor cleanups to branch/checkout wording
Jeff King [Mon, 2 Jul 2007 05:28:20 +0000 (01:28 -0400)] 
Documentation: minor cleanups to branch/checkout wording

Change "to made" to "made to", which is a typo. Use "reflog"
instead of "ref log", which is used elsewhere throughout the
documentation.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoDocumentation: quote {non-attributes} for asciidoc
Jeff King [Mon, 2 Jul 2007 05:24:59 +0000 (01:24 -0400)] 
Documentation: quote {non-attributes} for asciidoc

Asciidoc treats {foo} as an attribute to be substituted; if
'foo' doesn't exist as an attribute, then the entire line
gets dropped. When the literal {foo} is desired, \{foo} is
required.

The exceptions to this rule are:
  - inside literal blocks
  - if the 'foo' contains non-alphanumeric characters (e.g.,
    {foo|bar} is assumed not to be an attribute)

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMake '!' aliases more useful
Johannes Schindelin [Sun, 1 Jul 2007 21:51:58 +0000 (22:51 +0100)] 
Make '!' aliases more useful

When an alias starts with an exclamation mark, the rest is interpreted
as a shell command. However, all arguments passed to git used to be
ignored.

Now you can have an alias like

$ git config alias.e '!echo'

and

$ git e Hello World

does what you expect it to do.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogetenv/setenv: use constants if available
Matthias Lederhofer [Thu, 28 Jun 2007 14:15:25 +0000 (16:15 +0200)] 
getenv/setenv: use constants if available

There were places using "GIT_DIR" instead of GIT_DIR_ENVIRONMENT and
"GIT_CONFIG" instead of CONFIG_ENVIRONMENT.  This makes it easier to
find all places touching an environment variable using git grep or
similar tools.

Signed-off-by: Matthias Lederhofer <matled@gmx.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-clone: fetch possibly detached HEAD over dumb http
Sven Verdoolaege [Fri, 29 Jun 2007 08:31:08 +0000 (10:31 +0200)] 
git-clone: fetch possibly detached HEAD over dumb http

git-clone supports cloning from a repo with detached HEAD,
but if this HEAD is not behind any branch tip then it
would not have been fetched over dumb http, resulting in a

fatal: Not a valid object name HEAD

Since 928c210a, this would also happen on a http repo
with a HEAD that is a symbolic link where someone has
forgotton to run update-server-info.

Signed-off-by: Sven Verdoolaege <skimo@liacs.nl>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-submodule: Instead of using only annotated tags, use any tags.
Emil Medve [Tue, 26 Jun 2007 23:40:58 +0000 (18:40 -0500)] 
git-submodule: Instead of using only annotated tags, use any tags.

Some repositories might not use/have annotated tags (for example the
ones created with git-cvsimport) and git-submodule status might fail
because git-describe might fail to find a tag.  This change allows the
status of a submodule to be described/displayed relative to lightweight
tags as well.

Signed-off-by: Emil Medve <Emilian.Medve@Freescale.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-submodule: provide easy way of adding new submodules
Sven Verdoolaege [Sun, 24 Jun 2007 21:06:07 +0000 (23:06 +0200)] 
git-submodule: provide easy way of adding new submodules

To make a submodule effectively usable, the path and
a URL where the submodule can be cloned need to be stored
in .gitmodules.  This subcommand takes care of setting
this information after cloning the new submodule.
Only the index is updated, so, if needed, the user may still
change the URL or switch to a different branch of the submodule
before committing.

Signed-off-by: Sven Verdoolaege <skimo@kotnet.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoUpdate public documentation links for 1.5.2.3
Junio C Hamano [Mon, 2 Jul 2007 08:14:00 +0000 (01:14 -0700)] 
Update public documentation links for 1.5.2.3

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge 1.5.2.3 in
Junio C Hamano [Mon, 2 Jul 2007 08:12:57 +0000 (01:12 -0700)] 
Merge 1.5.2.3 in

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoGIT 1.5.2.3 v1.5.2.3
Junio C Hamano [Mon, 2 Jul 2007 07:35:58 +0000 (00:35 -0700)] 
GIT 1.5.2.3

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoDocumentation: quote {non-attributes} for asciidoc
Jeff King [Mon, 2 Jul 2007 05:24:59 +0000 (01:24 -0400)] 
Documentation: quote {non-attributes} for asciidoc

Asciidoc treats {foo} as an attribute to be substituted; if
'foo' doesn't exist as an attribute, then the entire line
gets dropped. When the literal {foo} is desired, \{foo} is
required.

The exceptions to this rule are:
  - inside literal blocks
  - if the 'foo' contains non-alphanumeric characters (e.g.,
    {foo|bar} is assumed not to be an attribute)

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-stash: don't complain when listing in a repo with no stash
Jeff King [Mon, 2 Jul 2007 04:21:24 +0000 (00:21 -0400)] 
git-stash: don't complain when listing in a repo with no stash

Previously, the git-log invocation would complain if a repo
had not had any stashes created in it yet:

$ git-init
$ git-stash
fatal: ambiguous argument 'refs/stash': unknown revision or
  path not in the working tree.
Use '--' to separate paths from revisions

Instead, we only call git-log if we actually have a
refs/stash. We could alternatively create the ref when any
stash command is called, but it's better for the 'list'
command to not require write access to the repo.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-stash: fix "can't shift that many" with no arguments
Jeff King [Mon, 2 Jul 2007 04:20:34 +0000 (00:20 -0400)] 
git-stash: fix "can't shift that many" with no arguments

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-stash: fix "no arguments" case in documentation
Jeff King [Mon, 2 Jul 2007 04:20:06 +0000 (00:20 -0400)] 
git-stash: fix "no arguments" case in documentation

Commit 9488e875 changed this from 'save' to 'list', but
missed this spot in the documentation.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-stash: require "save" to be explicit and update documentation
Junio C Hamano [Sun, 1 Jul 2007 22:29:01 +0000 (15:29 -0700)] 
git-stash: require "save" to be explicit and update documentation

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'rs/diff'
Junio C Hamano [Sun, 1 Jul 2007 21:58:09 +0000 (14:58 -0700)] 
Merge branch 'rs/diff'

* rs/diff:
  diff: round down similarity index
  diffcore-rename: don't change similarity index based on basename equality

17 years agoMerge branch 'jc/quote'
Junio C Hamano [Sun, 1 Jul 2007 21:57:51 +0000 (14:57 -0700)] 
Merge branch 'jc/quote'

* jc/quote:
  Add core.quotepath configuration variable.

17 years agoFix t5516-fetch for systems where `wc -l` outputs whitespace.
Brian Gernhardt [Sun, 1 Jul 2007 15:48:54 +0000 (11:48 -0400)] 
Fix t5516-fetch for systems where `wc -l` outputs whitespace.

When wc outputs whitespace, the test "$(command | wc -l)" = 1 is
broken because "   1" != "1".  Let the shell eat the whitespace by
using test 1 = $(command | wc -l) instead.

Signed-off-by: Brian Gernhardt <benji@silverinsanity.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit add: respect core.filemode with unmerged entries
Johannes Schindelin [Fri, 29 Jun 2007 17:32:46 +0000 (18:32 +0100)] 
git add: respect core.filemode with unmerged entries

When a merge left unmerged entries, git add failed to pick up the
file mode from the index, when core.filemode == 0. If more than one
unmerged entry is there, the order of stage preference is 2, 1, 3.

Noticed by Johannes Sixt.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agot7004: ship trustdb to avoid gpg warnings
Johannes Schindelin [Sun, 1 Jul 2007 03:14:01 +0000 (20:14 -0700)] 
t7004: ship trustdb to avoid gpg warnings

This avoids warning messages from gpg while verifying the tags; without it,
the program complains that the key is not certified with a trusted signature.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'ml/worktree'
Junio C Hamano [Sun, 1 Jul 2007 20:11:01 +0000 (13:11 -0700)] 
Merge branch 'ml/worktree'

* ml/worktree:
  make git barf when an alias changes environment variables

17 years agoMerge branch 'ei/worktree+filter'
Junio C Hamano [Sun, 1 Jul 2007 20:10:42 +0000 (13:10 -0700)] 
Merge branch 'ei/worktree+filter'

* ei/worktree+filter:
  filter-branch: always export GIT_DIR if it is set
  setup_git_directory: fix segfault if repository is found in cwd
  test GIT_WORK_TREE
  extend rev-parse test for --is-inside-work-tree
  Use new semantics of is_bare/inside_git_dir/inside_work_tree
  introduce GIT_WORK_TREE to specify the work tree
  test git rev-parse
  rev-parse: introduce --is-bare-repository
  rev-parse: document --is-inside-git-dir

17 years agoMerge branch 'fl/config'
Junio C Hamano [Sun, 1 Jul 2007 06:49:01 +0000 (23:49 -0700)] 
Merge branch 'fl/config'

* fl/config:
  config: add support for --bool and --int while setting values

17 years agoMerge branch 'mk/svn'
Junio C Hamano [Sun, 1 Jul 2007 06:48:54 +0000 (23:48 -0700)] 
Merge branch 'mk/svn'

* mk/svn:
  git-svn: honor ~/.subversion/ client cert file settings.

17 years agoDocument git-stash
しらいしななこ [Sun, 1 Jul 2007 05:26:08 +0000 (14:26 +0900)] 
Document git-stash

This describes the git-stash command.

I borrowed a few paragraphs from Johannes's version, and added a few
examples.

Signed-off-by: Nanako Shiraishi <nanako3@bluebottle.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-svn: cache max revision in rev_db databases
Sam Vilain [Sat, 30 Jun 2007 08:56:14 +0000 (20:56 +1200)] 
git-svn: cache max revision in rev_db databases

Cache the maximum revision for each rev_db URL rather than looking it
up each time.  This saves a lot of time when rebuilding indexes on a
freshly cloned repository.

Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-svn: use git-log rather than rev-list | xargs cat-file
Sam Vilain [Sat, 30 Jun 2007 08:56:13 +0000 (20:56 +1200)] 
git-svn: use git-log rather than rev-list | xargs cat-file

This saves a bit of time when rebuilding the git-svn index.

Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
Acked-by: Eric Wong <normalperson@yhbt.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-svn: honor ~/.subversion/ client cert file settings.
Michael Krelin [Fri, 22 Jun 2007 09:15:03 +0000 (11:15 +0200)] 
git-svn: honor ~/.subversion/ client cert file settings.

Currently, whenever svn repository http server requests client
certificate, prompt provider is invoked, ignoring any
ssl-client-cert-file settings in ~/.subversion/servers.

Moreover, it happens more than once per session, which is quite
irritating.

Signed-off-by: Michael Krelin <hacker@klever.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agodiffcore-delta.c: Ignore CR in CRLF for text files
Junio C Hamano [Fri, 29 Jun 2007 06:14:13 +0000 (23:14 -0700)] 
diffcore-delta.c: Ignore CR in CRLF for text files

This ignores CR byte in CRLF sequence in text file when
computing similarity of two blobs.

Usually this should not matter as nobody sane would be checking
in a file with CRLF line endings to the repository (they would
use autocrlf so that the repository copy would have LF line
endings).

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agodiffcore-delta.c: update the comment on the algorithm.
Junio C Hamano [Fri, 29 Jun 2007 06:11:40 +0000 (23:11 -0700)] 
diffcore-delta.c: update the comment on the algorithm.

The comment at the top of the file described an old algorithm
that was neutral to text/binary differences (it hashed sliding
window of N-byte sequences and counted overlaps), but long time
ago we switched to a new heuristics that are more suitable for
line oriented (read: text) files that are much faster.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agodiffcore_filespec: add is_binary
Junio C Hamano [Fri, 29 Jun 2007 05:56:07 +0000 (22:56 -0700)] 
diffcore_filespec: add is_binary

diffcore-break and diffcore-rename would want to behave slightly
differently depending on the binary-ness of the data, so add one
bit to the filespec, as the structure is now passed down to
diffcore_count_changes() function.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agodiffcore_count_changes: pass diffcore_filespec
Junio C Hamano [Fri, 29 Jun 2007 05:54:37 +0000 (22:54 -0700)] 
diffcore_count_changes: pass diffcore_filespec

We may want to use richer information on the data we are dealing
with in this function, so instead of passing a buffer address
and length, just pass the diffcore_filespec structure.  Existing
callers always call this function with parameters taken from a
filespec anyway, so there is no functionality changes.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoAdd git-stash script
しらいしななこ [Sat, 30 Jun 2007 05:37:09 +0000 (14:37 +0900)] 
Add git-stash script

When my boss has something to show me and I have to update, for some
reason I am always in the middle of doing something else, and git pull
command refuses to work in such a case.

I wrote this little script to save the changes I made, perform the
update, and then come back to where I was, but on top of the updated
commit.

This is how you would use the script:

  $ git stash
  $ git pull
  $ git stash apply

[jc: with a few fixlets from the list]

Signed-off-by: Nanako Shiraishi <nanako3@bluebottle.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoDon't fflush(stdout) when it's not helpful
Theodore Ts'o [Fri, 29 Jun 2007 17:40:46 +0000 (13:40 -0400)] 
Don't fflush(stdout) when it's not helpful

This patch arose from a discussion started by Jim Meyering's patch
whose intention was to provide better diagnostics for failed writes.
Linus proposed a better way to do things, which also had the added
benefit that adding a fflush() to git-log-* operations and incremental
git-blame operations could improve interactive respose time feel, at
the cost of making things a bit slower when we aren't piping the
output to a downstream program.

This patch skips the fflush() calls when stdout is a regular file, or
if the environment variable GIT_FLUSH is set to "0".  This latter can
speed up a command such as:

GIT_FLUSH=0 strace -c -f -e write time git-rev-list HEAD | wc -l

a tiny amount.

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'cr/tag'
Junio C Hamano [Sat, 30 Jun 2007 19:06:44 +0000 (12:06 -0700)] 
Merge branch 'cr/tag'

* cr/tag:
  Add test-script for git-tag
  Add test script for git-stripspace.
  Fix git-stripspace to process correctly long lines and spaces.

17 years agogit: Try a bit harder not to lose errno in stdio
Linus Torvalds [Sat, 30 Jun 2007 18:44:20 +0000 (11:44 -0700)] 
git: Try a bit harder not to lose errno in stdio

This switches the checks around upon the exit codepath of the
git wrapper, so that we may recover at least non-transient errors.

It's still not perfect. As I've been harping on, stdio simply isn't very
good for error reporting. For example, if an IO error happened, you'd want
to see EIO, wouldn't you? And yes, that's what the kernel would return.
However, with buffered stdio (and flushing outside of our control), what
would likely happen is that some intermediate error return _does_ return
EIO, but then the kernel might decide to re-mount the filesystem read-only
due to the error, and the actual *report* for us might be

"write failure on standard output: read-only filesystem"

which lost the EIO.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'lt/run'
Junio C Hamano [Sat, 30 Jun 2007 18:22:08 +0000 (11:22 -0700)] 
Merge branch 'lt/run'

* lt/run:
  Check for IO errors after running a command
  Clean up internal command handling

17 years agoMerge branch 'maint'
Junio C Hamano [Sat, 30 Jun 2007 18:17:19 +0000 (11:17 -0700)] 
Merge branch 'maint'

* maint:
  Correct the name of NO_R_TO_GCC_LINKER in the comment describing it.
  git-remote: document -n
  repack: improve documentation on -a option

17 years agoCorrect the name of NO_R_TO_GCC_LINKER in the comment describing it.
Matt Kraai [Sat, 30 Jun 2007 17:05:03 +0000 (10:05 -0700)] 
Correct the name of NO_R_TO_GCC_LINKER in the comment describing it.

Signed-off-by: Matt Kraai <kraai@ftbfs.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-remote: document -n
Sam Vilain [Sat, 30 Jun 2007 08:56:16 +0000 (20:56 +1200)] 
git-remote: document -n

The 'show' and 'prune' commands accept an option '-n'; document what
it does.

Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agorepack: improve documentation on -a option
Sam Vilain [Sat, 30 Jun 2007 08:56:12 +0000 (20:56 +1200)] 
repack: improve documentation on -a option

Some minor enhancements to the git-repack manual page.

Signed-off-by: Sam Vilain <sam.vilain@catalyst.net.nz>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-tag: Fix "can't shift that many".
Alexandre Vassalotti [Thu, 28 Jun 2007 16:56:57 +0000 (12:56 -0400)] 
git-tag: Fix "can't shift that many".

This stop git-tag from emitting a "shift: can't shift that many"
error, when listing tags.

[jc: with further fixups from Sam Vilain merged in; it passes
 the tests under dash now]

Signed-off-by: Alexandre Vassalotti <alexandre@peadrop.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoCorrect usages of sed in git-tag for Mac OS X
Shawn O. Pearce [Sat, 30 Jun 2007 06:42:47 +0000 (02:42 -0400)] 
Correct usages of sed in git-tag for Mac OS X

Both `git-tag -l` and `git tag -v` fail on Mac OS X due to their
non-standard uses of sed.  Actually `git tag -v` fails because the
underlying git-tag-verify uses a non-standard sed command.

We now stick to only standard sed, which does make our sed scripts
slightly more complicated, but we can actually list tags with more
than 0 lines of additional context and we can verify signed tags
with gpg.  These major Git functions are much more important than
saving two or three lines of a simple sed script.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoUpdate draft Release Notes for 1.5.3
Junio C Hamano [Fri, 29 Jun 2007 16:20:06 +0000 (09:20 -0700)] 
Update draft Release Notes for 1.5.3

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'maint'
Junio C Hamano [Fri, 29 Jun 2007 05:05:30 +0000 (22:05 -0700)] 
Merge branch 'maint'

* maint:

17 years agoMerge branch 'master' of git://repo.or.cz/git-gui
Junio C Hamano [Fri, 29 Jun 2007 04:32:41 +0000 (21:32 -0700)] 
Merge branch 'master' of git://repo.or.cz/git-gui

* 'master' of git://repo.or.cz/git-gui:
  git-gui: Don't require a .pvcsrc to create Tools/Migrate menu hack
  git-gui: Don't nice git blame on MSYS as nice is not supported
  git-gui: Don't require $DISPLAY just to get --version
  git-gui: Quiet our installation process
  git-gui: Bind Tab/Shift-Tab to cycle between panes in blame
  git-gui: Correctly install to /usr/bin on Cygwin

17 years agoMerge branch 'maint' of git://repo.or.cz/git-gui into maint
Junio C Hamano [Fri, 29 Jun 2007 04:28:36 +0000 (21:28 -0700)] 
Merge branch 'maint' of git://repo.or.cz/git-gui into maint

* 'maint' of git://repo.or.cz/git-gui:
  git-gui: Don't require a .pvcsrc to create Tools/Migrate menu hack
  git-gui: Don't nice git blame on MSYS as nice is not supported
  git-gui: Don't require $DISPLAY just to get --version
  git-gui: Bind Tab/Shift-Tab to cycle between panes in blame
  git-gui: Correctly install to /usr/bin on Cygwin

17 years agoAvoid perl in t1300-repo-config
Alex Riesen [Wed, 27 Jun 2007 12:40:41 +0000 (14:40 +0200)] 
Avoid perl in t1300-repo-config

It fixes the test on system where ActiveState Perl is used.

Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-cvsimport: force checkout of working tree after initial import
Gerrit Pape [Thu, 28 Jun 2007 11:12:07 +0000 (11:12 +0000)] 
git-cvsimport: force checkout of working tree after initial import

When creating a brand new git repository through git-cvsimport (not
incremental import), force a checkout of HEAD of master as working tree
after successful import using the -f switch to git checkout.  Otherwise
the working tree is empty, and all files are reported as 'deleted' by
git status.

This was noticed and reported by Cameron Dale through
 http://bugs.debian.org/430903

Signed-off-by: Gerrit Pape <pape@smarden.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoAdd test-script for git-tag
Carlos Rica [Thu, 28 Jun 2007 20:09:12 +0000 (22:09 +0200)] 
Add test-script for git-tag

These tests are useful to develop the C version for git-tag.sh,
ensuring that the future builtin-tag.c will not break previous
behaviour.

The tests are focused on listing, verifying, deleting and creating
tags, checking always that the correct status value is returned
and everything remains as expected.

In order to verify and create signed tags, a PGP key was also
added, being created this way: gpg --homedir t/t7004 --gen-key
Type DSA and Elgamal, size 2048 bits, no expiration date.
Name and email: C O Mitter <committer@example.com>
No password given, to enable non-interactive operation.

Signed-off-by: Carlos Rica <jasampler@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-log: detect dup and fdopen failure
Jim Meyering [Wed, 27 Jun 2007 14:28:53 +0000 (16:28 +0200)] 
git-log: detect dup and fdopen failure

This defines xdup() and xfdopen() in git-compat-util.h to give
us error-catching variants of them without cluttering the code
too much.

Signed-off-by: Jim Meyering <jim@meyering.net>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agogit-send-email: make options easier to configure.
Adam Roben [Thu, 28 Jun 2007 03:59:37 +0000 (20:59 -0700)] 
git-send-email: make options easier to configure.

This change makes git-send-email's behavior easier to modify by adding config
equivalents for two more of git-send-email's flags.

The mapping of flag to config setting is:
  --[no-]supress-from => sendemail.suppressfrom
  --[no-]signed-off-cc => sendemail.signedoffcc

It renames the --threaded option to --thread/--no-thread; the
config variable is also called sendemail.thread.

Signed-off-by: Adam Roben <aroben@apple.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'master' of git://git.kernel.org/pub/scm/gitk/gitk
Junio C Hamano [Thu, 28 Jun 2007 03:48:17 +0000 (20:48 -0700)] 
Merge branch 'master' of git://git./gitk/gitk

* 'master' of git://git.kernel.org/pub/scm/gitk/gitk:
  gitk: Update selection background colorbar in prefs dialog
  gitk: Use a spinbox for setting tabstop settings

17 years agogitk: Update selection background colorbar in prefs dialog
Mark Levedahl [Wed, 27 Jun 2007 01:51:34 +0000 (21:51 -0400)] 
gitk: Update selection background colorbar in prefs dialog

The callback function was incorrectly set to update the background
colorbar when updated the selection background. This did not affect the
colors chosen or their use, just their presentation in the preferences
dialog box.

Signed-off-by: Mark Levedahl <mdl123@verizon.net>
17 years agogitk: Use a spinbox for setting tabstop settings
Mark Levedahl [Wed, 27 Jun 2007 01:51:35 +0000 (21:51 -0400)] 
gitk: Use a spinbox for setting tabstop settings

The tabstop must be a smallish positive integer, and a spinbox is the
accepted UI control to accomplish this limiting rather than the text
entry box previously used.

Signed-off-by: Mark Levedahl <mdl123@verizon.net>
17 years agoAdd test script for git-stripspace.
Carlos Rica [Mon, 25 Jun 2007 19:24:21 +0000 (21:24 +0200)] 
Add test script for git-stripspace.

These tests check some features that git-stripspace already has
and those that it should manage well: Removing trailing spaces
from lines, removing blank lines at the beginning and end,
unifying multiple lines between paragraphs, doing the correct
when there is no newline at the last line, etc.

It seems that the implementation needs to save the whole line
in memory to be able to manage correctly long lines with
text and spaces conveniently distribuited on them.

Signed-off-by: Carlos Rica <jasampler@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoconfig: add support for --bool and --int while setting values
Frank Lichtenheld [Mon, 25 Jun 2007 14:00:24 +0000 (16:00 +0200)] 
config: add support for --bool and --int while setting values

Signed-off-by: Frank Lichtenheld <frank@lichtenheld.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'master' of git://repo.or.cz/git/fastimport
Junio C Hamano [Wed, 27 Jun 2007 05:23:29 +0000 (22:23 -0700)] 
Merge branch 'master' of git://repo.or.cz/git/fastimport

* 'master' of git://repo.or.cz/git/fastimport:
  Teach bash how to complete +refspec on git-push

17 years agogit-send-email: Add --threaded option
Adam Roben [Tue, 26 Jun 2007 22:48:30 +0000 (15:48 -0700)] 
git-send-email: Add --threaded option

The --threaded option controls whether the In-Reply-To header will be set on
any emails sent. The current behavior is to always set this header, so this
option is most useful in its negated form, --no-threaded. This behavior can
also be controlled through the 'sendemail.threaded' config setting.

Signed-off-by: Adam Roben <aroben@apple.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoFix git-stripspace to process correctly long lines and spaces.
Carlos Rica [Mon, 25 Jun 2007 19:28:01 +0000 (21:28 +0200)] 
Fix git-stripspace to process correctly long lines and spaces.

Now the implementation gets more memory to store completely
each line before removing trailing spaces, and does it right
when the last line of the file ends with spaces and no newline
at the end.

Function stripspace needs again to be non-static in order to call
it from "builtin-tag.c" and the upcoming "builtin-commit.c".
A new parameter skip_comments was also added to the stripspace
function to optionally strips every shell #comment from the input,
needed for doing this task on those programs.

Signed-off-by: Carlos Rica <jasampler@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <jim@meyering.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <jim@meyering.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'maint'
Shawn O. Pearce [Wed, 27 Jun 2007 04:36:38 +0000 (00:36 -0400)] 
Merge branch 'maint'

* maint:
  git-gui: Don't require a .pvcsrc to create Tools/Migrate menu hack
  git-gui: Don't nice git blame on MSYS as nice is not supported
  git-gui: Don't require $DISPLAY just to get --version

17 years agogit-gui: Don't require a .pvcsrc to create Tools/Migrate menu hack gitgui-0.7.4
Shawn O. Pearce [Tue, 26 Jun 2007 19:27:35 +0000 (15:27 -0400)] 
git-gui: Don't require a .pvcsrc to create Tools/Migrate menu hack

The Tools/Migrate menu option is a hack just for me.  Yes, that's
right, git-gui has a hidden feature that really only works for me,
and the users that I support within my day-job's great firewall.
The menu option is not supported outside of that environment.

In the past we only enabled Tools/Migrate if our special local
script 'gui-miga' existed in the proper location, and if there
was a special '.pvcsrc' in the top level of the working directory.
This latter test for the '.pvcsrc' file is now failing, as the file
was removed from all Git repositories due to changes made to other
tooling within the great firewall's realm.

I have changed the test to only work on Cygwin, and only if the
special 'gui-miga' is present.  This works around the configuration
changes made recently within the great firewall's realm, but really
this entire Tools/Migrate thing should be abstracted out into some
sort of plugin system so other users can extend git-gui as they need.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agogit-gui: Don't nice git blame on MSYS as nice is not supported
Shawn O. Pearce [Wed, 27 Jun 2007 04:27:13 +0000 (00:27 -0400)] 
git-gui: Don't nice git blame on MSYS as nice is not supported

Johannes Sixt reported that MinGW/MSYS does not have a nice.exe to
drop the priority of a child process when it gets spawned.  So we
have to avoid trying to start `git blame` through nice when we are
on Windows and do not have Cygwin available to us.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 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 <frank@lichtenheld.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoTeach rebase -i about --preserve-merges
Johannes Schindelin [Mon, 25 Jun 2007 17:59:43 +0000 (18:59 +0100)] 
Teach rebase -i about --preserve-merges

The option "-p" (or long "--preserve-merges") makes it possible to
rebase side branches including merges, without straightening the
history.

Example:

           X
            \
         A---M---B
        /
---o---O---P---Q

When the current HEAD is "B", "git rebase -i -p --onto Q O" will yield

               X
                 \
---o---O---P---Q---A'---M'---B'

Note that this will

- _not_ touch X [*1*], it does

- _not_ work without the --interactive flag [*2*], it does

- _not_ guess the type of the merge, but blindly uses recursive or
  whatever strategy you provided with "-s <strategy>" for all merges it
  has to redo, and it does

- _not_ make use of the original merge commit via git-rerere.

*1*: only commits which reach a merge base between <upstream> and HEAD
     are reapplied. The others are kept as-are.

*2*: git-rebase without --interactive is inherently patch based (at
     least at the moment), and therefore merges cannot be preserved.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agorebase -i: provide reasonable reflog for the rebased branch
Johannes Schindelin [Mon, 25 Jun 2007 17:58:28 +0000 (18:58 +0100)] 
rebase -i: provide reasonable reflog for the rebased branch

If your rebase succeeded, the HEAD's reflog will still show the whole
mess, but "<branchname>@{1}" now shows the state _before_ the rebase,
so that you can reset (or compare) the original and the rebased
revisions more easily.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agorebase -i: several cleanups
Johannes Schindelin [Mon, 25 Jun 2007 17:56:55 +0000 (18:56 +0100)] 
rebase -i: several cleanups

Support "--verbose" in addition to "-v", show short names in the list
comment, clean up if there is nothing to do, and add several "test_ticks"
in the test script.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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

17 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 <frank@lichtenheld.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <frank@lichtenheld.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <simon@lst.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <sam.vilain@catalyst.net.nz>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <qtonthat@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <tjernlund@tjernlund.se>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <skimo@liacs.nl>
Acked-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoignore git-rebase--interactive
Matthias Lederhofer [Tue, 26 Jun 2007 13:38:42 +0000 (15:38 +0200)] 
ignore git-rebase--interactive

Signed-off-by: Matthias Lederhofer <matled@gmx.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMerge branch 'master' of git://git.kernel.org/pub/scm/gitk/gitk 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://git.kernel.org/pub/scm/gitk/gitk: (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
  ...

Conflicts:

gitk

17 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 <paulus@samba.org>
17 years agodiff: round down similarity index
René Scharfe [Sun, 24 Jun 2007 22:23:34 +0000 (00:23 +0200)] 
diff: round down similarity index

Rounding down the printed (dis)similarity index allows us to use
"100%" as a special value that indicates complete rewrites and
fully equal file contents, respectively.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agodiffcore-rename: don't change similarity index based on basename equality
René Scharfe [Sun, 24 Jun 2007 22:23:28 +0000 (00:23 +0200)] 
diffcore-rename: don't change similarity index based on basename equality

This implements a suggestion from Johannes.  It uses a separate field in
struct diff_score to keep the result of the file name comparison in the
rename detection logic.  This reverts the value of the similarity index
to be a function of file contents, only, and basename comparison is only
used to decide between files with equal amounts of content changes.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoCheck for IO errors after running a command
Linus Torvalds [Sun, 24 Jun 2007 17:29:33 +0000 (10:29 -0700)] 
Check for IO errors after running a command

This is trying to implement the strict IO error checks that Jim Meyering
suggested, but explicitly limits it to just regular files. If a pipe gets
closed on us, we shouldn't complain about it.

If the subcommand already returned an error, that takes precedence (and we
assume that the subcommand already printed out any relevant messages
relating to it)

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoClean up internal command handling
Linus Torvalds [Sun, 24 Jun 2007 17:10:40 +0000 (10:10 -0700)] 
Clean up internal command handling

This should change no code at all, it just moves the definition of "struct
cmd_struct" out, and then splits out the running of the right command into
the "run_command()" function.

It also removes the long-unused 'envp' pointer passing.

This is just preparation for adding some more error checking.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoTeach rebase an interactive mode
Johannes Schindelin [Mon, 25 Jun 2007 00:11:14 +0000 (01:11 +0100)] 
Teach rebase an interactive mode

Don't you just hate the fact sometimes, that git-rebase just applies
the patches, without any possibility to edit them, or rearrange them?
With "--interactive", git-rebase now lets you edit the list of patches,
so that you can reorder, edit and delete patches.

Such a list will typically look like this:

pick deadbee The oneline of this commit
pick fa1afe1 The oneline of the next commit
...

By replacing the command "pick" with the command "edit", you can amend
that patch and/or its commit message, and by replacing it with "squash"
you can tell rebase to fold that patch into the patch before that.

It is derived from the script sent to the list in
<Pine.LNX.4.63.0702252156190.22628@wbgn013.biozentrum.uni-wuerzburg.de>

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoMove the pick_author code to git-sh-setup
Johannes Schindelin [Mon, 25 Jun 2007 00:04:11 +0000 (01:04 +0100)] 
Move the pick_author code to git-sh-setup

At the moment, only git-commit uses that code, to pick the author name,
email and date from a given commit.

This code will be reused in git rebase --interactive.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 years agoTeach bash how to complete +refspec on git-push
Shawn O. Pearce [Sun, 24 Jun 2007 23:42:16 +0000 (19:42 -0400)] 
Teach bash how to complete +refspec on git-push

Using `git push origin +foo` to forcefully overwrite the remote
branch named foo is a common idiom, especially since + is shorter
than the long option --force and can be specified on a per-branch
basis.

We now complete `git push origin +foo` just like we do the standard
`git push origin foo`.  The leading + on a branch refspec does not
alter the completion.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
17 years agoAdd core.quotepath configuration variable.
Junio C Hamano [Sun, 24 Jun 2007 22:11:24 +0000 (15:11 -0700)] 
Add core.quotepath configuration variable.

We always quote "unusual" byte values in a pathname using
C-string style, to make it safer for parsing scripts that do not
handle NUL separated records well (or just too lazy to bother).
The absolute minimum bytes that need to be quoted for this
purpose are TAB, LF (and other control characters), double quote
and backslash.

However, we have also always quoted the bytes in high 8-bit
range; this was partly because we were lazy and partly because
we were being cautious.

This introduces an internal "quote_path_fully" variable, and
core.quotepath configuration variable to control it.  When set
to false, it does not quote bytes in high 8-bit range anymore
but passes them intact.

The variable defaults to "true" to retain the traditional
behaviour for now.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
17 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 <gitster@pobox.com>