test: remove httpd tests that ask for user

Until we figure a way to pass the user correctly.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: remove cruft

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: fix ambiguous tracking test

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: fix aliases tests

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Merge tag 'v2.8.0'

Git 2.8

* tag 'v2.8.0': (532 commits)
  Git 2.8
  Documentation: fix git-p4 AsciiDoc formatting
  mingw: skip some tests in t9115 due to file name issues
  t1300: fix the new --show-origin tests on Windows
  t1300-repo-config: make it resilient to being run via 'sh -x'
  config --show-origin: report paths with forward slashes
  submodule: fix regression for deinit without submodules
  l10n: pt_PT: Update and add new translations
  l10n: ca.po: update translation
  Git 2.8-rc4
  Documentation: fix broken linkgit to git-config
  Documentation: use ASCII quotation marks in git-p4
  Revert "config.mak.uname: use clang for Mac OS X 10.6"
  git-compat-util: st_add4: work around gcc 4.2.x compiler crash
  l10n: vi.po (2530t): Update translation
  l10n: zh_CN: review for git v2.8.0 l10n round 2
  gitk: Follow themed bgcolor in help dialogs
  gitk: fr.po: Sync translations with git
  gitk: Update French translation (311t)
  gitk: Update German translation
  ...

travis: trivial improvements

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: fix

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: improve test selection

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: remote: add test shortcuts

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Merge branch 'integration'

* integration: (108 commits)
  cherry-pick: copy notes and run hooks
  cherry-pick: add --action-name option
  builtin: rewrite: add copy_rewrite_notes()
  builtin: move run_rewrite_hook() to rewrite.c
  cherry-pick: don't store skipped commit
  cherry-pick: store rewritten commits
  builtin: add rewrite helper
  cherry-pick: remember rerere-autoupdate
  remote: move tests
  travis: add more Mercurial versions
  travis: use old versions of hg and hg-git
  travis: add cache
  travis: test development version of hg
  travis: add specific test for hg v3.0
  Add Travis CI configuration
  revert/cherry-pick: cleanup rerere info on --abort
  revert/cherry-pick: add --skip option
  revert/cherry-pick: add --quiet option
  cherry-pick: add --skip-empty option
  cherry-pick: don't barf when there's nothing to do
  ...

Merge branch 'travis-ci' into integration

* travis-ci:
  travis: add more Mercurial versions
  travis: use old versions of hg and hg-git
  travis: add cache
  travis: test development version of hg
  travis: add specific test for hg v3.0
  Add Travis CI configuration

Merge branch 'fc/cherry-pick' into integration

* fc/cherry-pick:
  cherry-pick: copy notes and run hooks
  cherry-pick: add --action-name option
  builtin: rewrite: add copy_rewrite_notes()
  builtin: move run_rewrite_hook() to rewrite.c
  cherry-pick: don't store skipped commit
  cherry-pick: store rewritten commits
  builtin: add rewrite helper
  cherry-pick: remember rerere-autoupdate
  revert/cherry-pick: cleanup rerere info on --abort
  revert/cherry-pick: add --skip option
  revert/cherry-pick: add --quiet option
  cherry-pick: add --skip-empty option
  cherry-pick: don't barf when there's nothing to do

Merge branch 'fc/readme' into integration

* fc/readme:
  test: fix README location
  Add our own README

Merge branch 'fc/trivial' into integration

* fc/trivial:
  add: remove dead code
  contrib: svn-fe: fix python shebang
  add: avoid yoda condition
  config: avoid yoda conditions
  doc: trivial cleanup for `git pull`

Merge branch 'fc/fixes' into integration

* fc/fixes:
  credential: fix wrong double password prompt
  rebase: disable fork-point by default
  Silence gcc warnings with -O3
  Revert "silence some -Wuninitialized false positives"
  Revert "make error()'s constant return value more visible"
  version-gen: fix versions
  version-gen: cleanup
  build: fix installation of scripts

# Conflicts:
# Makefile

Merge branch 'fc/comp/graduate' into integration

* fc/comp/graduate:
  completion: graduate out of contrib

Merge branch 'fc/comp/fixes' into integration

* fc/comp/fixes:
  Revert "Update documentation occurrences of filename .sh"
  completion: prompt: fix for Zsh
  completion: zsh: fix for directories with spaces
  completion: zsh: improve main function selection
  completion: add missing fetch options
  completion: bash: cleanup cygwin check
  completion: zsh: trivial cleanups
  completion: remove zsh hack
  completion: zsh: don't hide ourselves
  completion: bash: remove zsh wrapper
  completion: bash: remove old wrappers

Merge branch 'fc/ruby' into integration

* fc/ruby:
  ruby: add simpler option parser
  ruby: add useful helpers
  ruby: bind log_tree_diff_flush()
  ruby: bind diff_tree_sha1()
  ruby: bind shortlog()
  ruby: add RevInfo and DiffOptions
  shortlog: split builtin from common code
  shortlog: add missing declaration
  ruby: bind read_sha1_file()
  ruby: bind find_unique_abbrev()
  ruby: bind remote and transport stuff
  ruby: bind get_merge_bases()
  ruby: add Commit class
  ruby: bind get_sha1()
  ruby: bind read_ref()/peel_ref()
  ruby: bind git_config()
  ruby: bind dwim_ref()
  ruby: bind setup_git_directory()
  ruby: add support for internal ruby programs
  Add support for ruby commands

Merge branch 'fc/publish' into integration

* fc/publish:
  remote: show published status and help
  branch: add quick marker to see published status
  sha1_name: add support for @{publish} marks
  sha1_name: simplify track finding
  branch: display publish branch
  push: add --set-publish option
  branch: add --set-publish-to option
  Add concept of 'publish' branch
  t5516 (fetch-push): fix test restoration

Merge branch 'fc/fetch/default' into integration

* fc/fetch/default:
  config: add fetch.default=simple to progress mode
  fetch: add new fetch.default configuration

Merge branch 'fc/stage' into integration

* fc/stage:
  completion: update 'git reset' new stage options
  reset: allow --keep with --stage
  reset: add --stage and --work options
  completion: update --staged options
  apply: add --work, --no-work options
  apply: add --stage option
  submodule: add --staged options
  stash: add --stage to pop and apply
  stash: add --stage option to save
  rm: add --staged option
  grep: add --staged option
  diff: document --staged
  stage: add edit command
  Add proper 'stage' command

Merge branch 'fc/update' into integration

* fc/update:
  Add new `git update` tool
  merge: add --reverse-parents option
  commit: support for multiple MERGE_MODE

Merge branch 'fc/config/mode' into integration

* fc/config/mode:
  config: add core.mode = progress pseudo-config

Merge branch 'fc/config/mode-next' into integration

* fc/config/mode-next:
  Add core.mode configuration

Merge branch 'fc/remote/mainline' into integration

* fc/remote/mainline:
  remote: move tests
  remote-hg: add missing && chain to tests
  remote-hg: avoid deprecated bookmarks.write()
  remote-hd: temporarily disable hg-git tests
  git-remote-bzr: update to v0.3
  git-remote-hg: update to v0.3
  Include git-remote-hg and git-remote-bzr

Merge branch 'fc/default-aliases' into integration

* fc/default-aliases:
  test: fix aliases
  Add more default aliases
  Add default aliases

Merge branch 'fc/branch/nice-verbose' into integration

* fc/branch/nice-verbose:
  branch: reorganize verbose options

Merge branch 'fc/backports' into integration

* fc/backports:
  tests: skip RFC1991 tests for gnupg 2.1

cherry-pick: copy notes and run hooks

If no action-name is specified, nothing is done.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: add --action-name option

So it can be used by other tools (e.g. git rebase), and the right action
is passed to the hooks and notes rewrite stuff.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

builtin: rewrite: add copy_rewrite_notes()

And use it on commit.c.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

builtin: move run_rewrite_hook() to rewrite.c

And use struct rewrite.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: don't store skipped commit

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: store rewritten commits

Will be useful for the next commits.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

builtin: add rewrite helper

So that we can load and store rewrites, as well as other operations on a
list of rewritten commits.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: remember rerere-autoupdate

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

remote: move tests

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: fix cherry pick rerere

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

remote: move tests

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Merge tag 'v2.7.0'

Git 2.7

* tag 'v2.7.0': (539 commits)
  Git 2.7
  Git 2.6.5
  l10n: de.po: translate 68 new messages
  l10n: de.po: improve some translations
  Git 2.7-rc3
  l10n: ru.po: update Russian translation
  l10n: Updated Bulgarian translation of git (2477t,0f,0u)
  l10n: ca.po: update translation
  l10n: zh_CN: for git v2.7.0 l10n round 2
  l10n: sv.po: Update Swedish translation (2477t0f0u)
  l10n: sv: Fix bad translation
  l10n: fr.po v2.7.0 round 2 (2477t)
  Git 2.7-rc2
  push: don't mark options of recurse-submodules for translation
  l10n: git.pot: v2.7.0 round 2 (2 new, 2 removed)
  t1401: test reflog creation for git-symbolic-ref
  symbolic-ref: propagate error code from create_symref()
  git-p4: reduce number of server queries for fetches
  git-p4: support multiple depot paths in p4 submit
  Update release notes to 2.7
  ...

travis: add more Mercurial versions

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: use old versions of hg and hg-git

Only for the moment.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: add cache

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: test development version of hg

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

travis: add specific test for hg v3.0

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Add Travis CI configuration

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

revert/cherry-pick: cleanup rerere info on --abort

As we should.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

revert/cherry-pick: add --skip option

Akin to 'am --skip' and 'rebase --skip'.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

revert/cherry-pick: add --quiet option

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: add --skip-empty option

Pretty much what it says on the tin.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

cherry-pick: don't barf when there's nothing to do

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

test: fix README location

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Add our own README

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

add: remove dead code

addremove is already 1 by default.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

contrib: svn-fe: fix python shebang

It's better if all our scripts use the same '/usr/bin/env python'.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

add: avoid yoda condition

18 is younger than person's age.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

config: avoid yoda conditions

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

doc: trivial cleanup for `git pull`

The user doesn't need to be burdened with irrelevant details.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

credential: fix wrong double password prompt

We shouldn't use ssh-askpass to ask for the user, that causes most tools
to ask for "password: username" followed by "password: password".
Totally confusing for the user.

Instead, use a normal prompt to ask for the user.

Reported-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

rebase: disable fork-point by default

It is possible that the upstream branch was at some point in the wrong
commit. The user might do 'git reset --hard' to restore that branch to
the right place, the wrong location is already in the reflog.

If the user tries to do a rebase without arguments the fork-point
feature will be used automatically and the wrong point in the reflog
used as one of the possible heads to check for bases.

This is especially bad when upstream was already in the same place as
the current branch (say upstream was fast-forwarded to topic by
mistake), in which case the rebase would silently ignore all the commits
and reset to upstream. The user would wonder what happened with his
commits, or worst; assume they were already in upstream.

This is a very bad situation we should try to avoid.

We cannot rely on the fact that the reflog will always contain only
good heads, therefore the fork-point should be disabled by default.

The fork-point feature is still useful, but it should be enabled
manually, only when the user knows what he is doing, and that all the
points in the reflog of the upstream branch are desirable.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Silence gcc warnings with -O3

Commit a469a10 (silence some -Wuninitialized false positives) tried to
silence some uninitialized false-positives with a hack. This hack
triggered many more warnings on clang, and eventually gcc 4.9.0 as well.

While more hacks can be used to silence the new warnings, it's much
better to decouple ourselves from such hacks and help the compilers
directly.

We can do that by either initializing the variables, or manually
returning the error value (-1) without depending on error() to do it.

This fixes warnings while compiling with gcc -O3.

  parse-options.c: In function ‘get_value’:
  parse-options.c:183:24: warning: ‘arg’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     *(int *)opt->value = strtol(arg, (char **)&s, 10);
  ^
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Revert "silence some -Wuninitialized false positives"

In recent versions of gcc (4.9.0), we get a few of these:

  notes.c: In function ‘notes_display_config’:
  notes.c:970:28: warning: right-hand operand of comma expression has no effect [-Wunused-value]
      config_error_nonbool(k);
                            ^
Previous commit explains the reason.

This reverts commit a469a1019352b8efc4bd7003b0bd59eb60fc428c.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Conflicts:
cache.h
parse-options.h

Revert "make error()'s constant return value more visible"

In recent versions of gcc (4.9.0), we get hundreds of these:

  advice.c: In function ‘error_resolve_conflict’:
  advice.c:79:69: warning: right-hand operand of comma expression has no effect [-Wunused-value]
    error("'%s' is not possible because you have unmerged files.", me);
                                                                     ^
The original patch intended to help in situations like this:

  if (error(...))
    /* do stuff */

However, when there's no conditional statement this gets translated to:

  (error(..), 1);

And the right hand of the expression has no effect.

So it looks like gcc is smarter now, and in trying to fix a few warnings
we generated hundreds more.

This reverts commit e208f9cc7574f5980faba498d0aa30b4defeb34f.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
Conflicts:
git-compat-util.h

version-gen: fix versions

Virtually all packaging guidelines would prefer 1.8.4~rc1, over
1.8.4.rc1 or 1.8.4-rc1, so it makes sense to use that instead.

In particular, the only packaging we provide, git.spec, generates a
wrong version, because git-1.8.4 < git-1.8.4.rc1, changing to ~rc1 fixes
the problem as it's considered newer.

The same happens in dpkg.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

version-gen: cleanup

No functional changes.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

build: fix installation of scripts

They need the gitexecdir.

Otherwise the child scripts fail when installing:

  % make -C contrib/remote-helpers install
  install: target ‘$d/libexec/git-core’ is not a directory
  Makefile:513: recipe for target 'install-python-script' failed

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: graduate out of contrib

These have been stable and widely used for quite a long time, they even
have tests outside of the contrib area, and most distributions ship
them, so they can be considered part of the core already.

We should be consistent and either we move the tests to contrib, or we
move the completions out of contrib.

Let's move them out of contrib and install them by default.

By default bash-completion installs the completions to
$(pkgdatadir)/completions, which is
$(prefix)/share/bash-completion/completions. And since most distributions do
not change this, it is obviously the right default. However,
bash-completion recommends to use pkg-config to find the right location,
so we could use that, but it's much easier to let distributions do it:

  bashcomdir=$pkg-config --variable=completionsdir bash-completion)

By default zsh looks for completions in
$(prefix)/share/zsh/site-functions.

The distributions use these locations to install Git completions:

  ArchLinux: /usr/share/bash-completion/completions/
  Debian: /usr/share/bash-completion/completions/
  Fedora: /etc/bash_completion.d/
  Ubuntu: /usr/share/bash-completion/completions/
  Mint: /usr/share/bash-completion/completions/
  OpenSuse: /etc/bash_completion.d/

However, in both Fedora and OpenSUSE their bash-completion intallations
use /usr/share/bash-completion/completions/ (they don't change the
default). So it  seems to a be a bug from the packager, since they
recommend the bash-completion location[1]

If some weird distribution has a problem with the location, they can
override 'bashcompdir' anyway.

[1] http://article.gmane.org/gmane.linux.redhat.fedora.devel/177405

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Revert "Update documentation occurrences of filename .sh"

The original code was correct: the example location ~/.git-completion.sh
is correct, because it's not only used by Bash. And zstyle command in
Zsh should use that same location; the Bash script.

This reverts commit 0e5ed7cca3c51c821c2bb0465617e75d994f432f.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: prompt: fix for Zsh

We can add colour in Zsh without the need of pcmode.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: zsh: fix for directories with spaces

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: zsh: improve main function selection

Sometimes we want to use the function directly (e.g. _git_checkout), for
example when zsh has the option 'complete_aliases', this way, we can do
something like:

  compdef _git gco=git_checkout

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: add missing fetch options

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: bash: cleanup cygwin check

Avoid Yoda conditions, use test, and cleaner statement.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: zsh: trivial cleanups

We don't need to override IFS, zsh has a native way of splitting by new
lines: the expansion flag (f).

Also, we don't need to split files by ':' or '='; that's only for words.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: remove zsh hack

We don't want to override the 'complete()' function in zsh, which can be
used by bashcomp.

Reported-by: Mark Lodato <lodato@google.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: zsh: don't hide ourselves

There's no need to hide the fact that we are on zsh any more.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: bash: remove zsh wrapper

It has been deprecated for one year and a half. It's time to move on.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

completion: bash: remove old wrappers

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: add simpler option parser

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: add useful helpers

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind log_tree_diff_flush()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind diff_tree_sha1()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind shortlog()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: add RevInfo and DiffOptions

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

shortlog: split builtin from common code

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

shortlog: add missing declaration

Otherwise we would have to include commit.h.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind read_sha1_file()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind find_unique_abbrev()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind remote and transport stuff

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind get_merge_bases()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: add Commit class

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind get_sha1()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind read_ref()/peel_ref()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind git_config()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind dwim_ref()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: bind setup_git_directory()

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

ruby: add support for internal ruby programs

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

Add support for ruby commands

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

remote: show published status and help

So when switching branches or doing `git status` we see relevant
information about the publish branch.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>

branch: add quick marker to see published status

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>