Re: [GUILT v4 32/33] Improved doc and tests for guilt header.
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
On Mon, May 19, 2014 at 12:00:08AM +0200, Per Cederqvist wrote:
Show 54 quoted lines
> ---
> Documentation/guilt-header.txt | 5 ++++-
> regression/t-028.out | 9 +++++++++
> regression/t-028.sh | 3 +++
> 3 files changed, 16 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/guilt-header.txt b/Documentation/guilt-header.txt
> index 870bfaf..71b2e66 100644
> --- a/Documentation/guilt-header.txt
> +++ b/Documentation/guilt-header.txt
> @@ -18,7 +18,10 @@ Prints either the topmost patch's header or the header of a specified patch.
> -E::
> Open the raw patch in an editor, instead of printing it.
> <patchname>::
> - Name of the patch.
> + Name of the patch. If a patch with exactly this name exists,
> + use it. Otherwise, treat the name as a regexp; if the regexp
> + matches a single patch, use it. Otherwise, list all matching
> + patch names to stderr and fail.
>
> Author
> ------
> diff --git a/regression/t-028.out b/regression/t-028.out
> index ea72a3a..39ac900 100644
> --- a/regression/t-028.out
> +++ b/regression/t-028.out
> @@ -56,3 +56,12 @@ Patch non-existant is not in the series
> remove
> mode
> patch-with-some-desc
> +% guilt header de
> +de does not uniquely identify a patch. Did you mean any of these?
> + mode
> + patch-with-some-desc
> +% guilt header des
> +blah blah blah
> +
> +Signed-off-by: Commiter Name <commiter@email>
> +
> diff --git a/regression/t-028.sh b/regression/t-028.sh
> index 2ce0378..cd3088c 100755
> --- a/regression/t-028.sh
> +++ b/regression/t-028.sh
> @@ -35,4 +35,7 @@ shouldfail guilt header non-existant
> # patch name is a regexp that just happens to match an existing patch.
> shouldfail guilt header '.*'
>
> +shouldfail guilt header de
> +cmd guilt header des
> +
> # FIXME: how do we check that -e works?
> --
> 1.8.3.1
>
--
You measure democracy by the freedom it gives its dissidents, not the
freedom it gives its assimilated conformists.
- Abbie Hoffman