[PATCH] guilt: fix "from: xxx" filtering in the patch description

Commit 2cc8d35 ([PATCH] guilt: improve patch header handling) caused
a regression with patch header handling.  When using a hand written
patch description in the format:

> Frobnozzle: this is a patch subject
>
> From: Fred McNurk <fred@mcnurt.foo>

the old code filtered the From: line out of the git commit
description, while the new code leaves it in there. The result is
that when using guilt patchbomb, the From line is emitted twice in
the email that is sent - once from the author of the patch, and
again in the body of the commit message.

Fix this by unconditionally filtering the From: field out of the
constructed commit message that is used for git.

This patch was sent out via my local version of guilt that has this
patch applied, so the sending of this patch is demonstration that
this fix has been tested and works...

Signed-off-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[PATCH]  get rid of "cat: write error: Broken pipe" error message

In some circumstances (like running guilt in a M-x shell buffer in
Emacs) cat may give the above error message when the reader of the
output from cat exits without reading all input from cat.  (In other
circumstances cat is just silently terminated with SIGPIPE.)

Get rid of the error by removing the useless use of cat in
do_get_header and do_get_full_header.

Signed-off-by: Per Cederqvist <cederp@opera.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[PATCH] Documentation/Makefile: Run xmlto with --skip-validation

Signed-off-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT] handle branches with slashes in guilt-graph

Avoid sed errors when the branch name contains a slash.

Signed-off-by: Per Cederqvist <cederp@opera.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 1/6] Refuse to push corrupt patches

"guilt push" would treat corrupt patches as empty,
because "git apply --numstat" prints nothing on stdout.

(You do get an error message on stderr,
 but then guilt says "Patch applied" etc,
 and I didn't notice the earlier error message
 for quite some time.)

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 2/6] guilt-header: fix patch corruption

Hunks containing the string "END{}" were being corrupted.

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 3/6] Handle paths that contain spaces

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 4/6] Run regression tests in a directory which contains spaces

This helped me in finding some of the issues in the previous patch.

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 5/6] Allow guilt scripts to be run from a directory which contains spaces

`dirname $0` breaks if $0 includes spaces.

I don't need this; it was the result of a mis-understanding.
But here it is anyway; take it if you think it's useful.

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[GUILT 6/6] Allow the regression tests to be run from a directory with spaces in

Alan Jenkins <alan.christopher.jenkins@googlemail.com>
sed -i regression/*.sh 's|source $REG_DIR/scaffold|source "$REG_DIR/scaffold"|'

Same as the previous patch: it turned out I don't need this,
but you might think it's a good idea anyway.

Signed-off-by: Alan Jenkins <alan.christopher.jenkins@googlemail.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
guilt: remove a useless cat

Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[PATCH] Testsuite: get rid of "Broken pipe" errors from yes.

Signed-off-by: Per Cederqvist <cederp@opera.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
[PATCH] The tests should not fail if log.date or log.decorate are set.

Explicitly set log.date and log.decorate to their Git default values,
so that git produces the expected output even if log.date and
log.decorate are set by the user in his .gitconfig.

Signed-off-by: Per Cederqvist <cederp@opera.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
Merge branch 'mob' of git://repo.or.cz/guilt/mob

Conflicts:
	guilt
add simple bash-completion for commands.
add simple bash-completion for commands.

Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>
Merge branch 'master' into mob
fix "guilt --help"

Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>
add -h option to show usage for all guilt commands.

Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>
replace bash completion with one based on quilt's

Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com>