Signed-off-by: Josef 'Jeff' Sipek <jeffpc@xxxxxxxxxxxxxx> On Fri, Mar 21, 2014 at 08:31:49AM +0100, Per Cederqvist wrote: > The shouldfail function already redirects stderr to stdout, so there > is no need to do the same in t-028.sh and t-021.sh. > > Signed-off-by: Per Cederqvist <cederp@xxxxxxxxx> > --- > regression/t-021.sh | 2 +- > regression/t-025.sh | 2 +- > regression/t-028.sh | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/regression/t-021.sh b/regression/t-021.sh > index 6337d7b..614e870 100755 > --- a/regression/t-021.sh > +++ b/regression/t-021.sh > @@ -61,7 +61,7 @@ for n in `_seq -2 $npatches`; do > if [ $n -gt 0 ]; then > cmd guilt pop -n $n > else > - shouldfail guilt pop -n $n 2>&1 > + shouldfail guilt pop -n $n > fi > > cmd list_files > diff --git a/regression/t-025.sh b/regression/t-025.sh > index 3824608..985fed4 100755 > --- a/regression/t-025.sh > +++ b/regression/t-025.sh > @@ -44,7 +44,7 @@ shouldfail guilt new "white space" > cmd list_files > > for pname in prepend mode /abc ./blah ../blah abc/./blah abc/../blah abc/. abc/.. abc/ ; do > - shouldfail guilt new "$pname" 2>&1 > + shouldfail guilt new "$pname" > > cmd list_files > done > diff --git a/regression/t-028.sh b/regression/t-028.sh > index 8480100..88e9adb 100755 > --- a/regression/t-028.sh > +++ b/regression/t-028.sh > @@ -29,6 +29,6 @@ guilt series | while read n; do > cmd guilt header $n > done > > -shouldfail guilt header non-existant 2>&1 > +shouldfail guilt header non-existant > > # FIXME: how do we check that -e works? > -- > 1.8.3.1 > -- Failure is not an option, It comes bundled with your Microsoft product. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html