Re: [GUILT v4 32/33] Improved doc and tests for guilt header.

2015-01-22 Thread Jeff Sipek
Signed-off-by: Josef 'Jeff' Sipek jef...@josefsipek.net


On Mon, May 19, 2014 at 12:00:08AM +0200, Per Cederqvist wrote:
 ---
  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
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[GUILT v4 32/33] Improved doc and tests for guilt header.

2014-05-18 Thread Per Cederqvist
---
 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

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html