threads / patch / 30364

patchTestsuite: get rid of "Broken pipe" errors from yes.

Subject: [GUILT] [PATCH] Testsuite: get rid of "Broken pipe" errors from yes.

## tl;dr

3 messages between Apr 30, 2012 and Apr 30, 2012. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Per Cederqvist· Apr 30, 2012, 10:27 UTC · lore

Signed-off-by: Per Cederqvist <cederp@opera.com>
---
 regression/t-029.sh |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Show changes to regression/t-029.sh +3 −3

diff --git a/regression/t-029.sh b/regression/t-029.sh
index 83e1d2b..09450c6 100755
--- a/regression/t-029.sh
+++ b/regression/t-029.sh
@@ -21,11 +21,11 @@ echo | shouldfail guilt repair --full
 
 cmd list_files
 
-yes n | shouldfail guilt repair --full
+yes n 2>/dev/null | shouldfail guilt repair --full
 
 cmd list_files
 
-yes y | cmd guilt repair --full
+yes y 2>/dev/null | cmd guilt repair --full
 
 cmd list_files
 
@@ -33,6 +33,6 @@ cmd guilt push -a
 
 cmd list_files
 
-yes Y | cmd guilt repair --full
+yes Y 2>/dev/null | cmd guilt repair --full
 
 cmd list_files

-- 
1.7.10

Jeff Sipek· Apr 30, 2012, 15:06 UTC · re: Per Cederqvist · lore

Re: [GUILT] [PATCH] Testsuite: get rid of "Broken pipe" errors from yes.

On Mon, Apr 30, 2012 at 12:27:21PM +0200, Per Cederqvist wrote:

Looks good. Interestingly, I haven't seen those errors before. What's your setup like?

Jeff.

Show 34 quoted lines

> Signed-off-by: Per Cederqvist <cederp@opera.com>
> ---
>  regression/t-029.sh |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/regression/t-029.sh b/regression/t-029.sh
> index 83e1d2b..09450c6 100755
> --- a/regression/t-029.sh
> +++ b/regression/t-029.sh
> @@ -21,11 +21,11 @@ echo | shouldfail guilt repair --full
>  
>  cmd list_files
>  
> -yes n | shouldfail guilt repair --full
> +yes n 2>/dev/null | shouldfail guilt repair --full
>  
>  cmd list_files
>  
> -yes y | cmd guilt repair --full
> +yes y 2>/dev/null | cmd guilt repair --full
>  
>  cmd list_files
>  
> @@ -33,6 +33,6 @@ cmd guilt push -a
>  
>  cmd list_files
>  
> -yes Y | cmd guilt repair --full
> +yes Y 2>/dev/null | cmd guilt repair --full
>  
>  cmd list_files
> -- 
> 1.7.10
>

-- 
You measure democracy by the freedom it gives its dissidents, not the
freedom it gives its assimilated conformists.
		- Abbie Hoffman

Jeff Sipek· Apr 30, 2012, 16:46 UTC · lore

Re: [GUILT] [PATCH] Testsuite: get rid of "Broken pipe" errors from yes.

On Mon, Apr 30, 2012 at 05:34:40PM +0200, Per Cederqvist wrote:

Show 14 quoted lines

> On Apr 30, 2012 5:06 PM, "Jeff Sipek" <jeffpc@josefsipek.net> wrote:
> >
> > On Mon, Apr 30, 2012 at 12:27:21PM +0200, Per Cederqvist wrote:
> > >
> >
> > Looks good.  Interestingly, I haven't seen those errors before.  What's
> your
> > setup like?
> >
> > Jeff.
> 
> I'm running the tests using M-x compile inside Emacs on Ubuntu 11.04. I
> cannot reproduce it in a plain terminal. yes might inherit some signal
> setup that Emacs makes.

Makes sense.

Thanks,

Jeff.

-- 
Don't drink and derive. Alcohol and algebra don't mix.