Re: [PATCH 1/2] commands: send stderr to a new pipe

[Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>]

On Wed, 2017-06-21 at 13:56 +0300, Jussi Kukkonen wrote:
> On 19 June 2017 at 17:39, <leonardo.sandoval.gonzalez@linux.intel.com>
> wrote:
> >
> > From: Leonardo Sandoval <leonardo.sandoval.gonzalez@linux.intel.com>
> >
> > Do not mix the stderr into stdout, allowing test cases to query
> > the specific output.
> 
> I wonder if this patch could be related to the strange selftest
> failures where command output seems to be missing completely, e.g.
>     AssertionError: '' != 'ERROR: Unable to find any package producing
> path /not/exist'
> 
> https://autobuilder.yocto.io/builders/nightly-oe-selftest/builds/349/steps/Running%20oe-selftest/logs/stdio


I double check this series on local AB and I saw green in all tests, so
I am not sure what is going on.

This patch basically separates stdout and stderr, so now tests look into
stderr for errors instead of stdout.

Leo

> 
> Jussi
> 
> >
> > [YOCTO #9693]
> >
> > Signed-off-by: Leonardo Sandoval
> <leonardo.sandoval.gonzalez@linux.intel.com>
> > ---
> >  meta/lib/oeqa/utils/commands.py | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/lib/oeqa/utils/commands.py
> b/meta/lib/oeqa/utils/commands.py
> > index 57286fcb10..4ef0e874ff 100644
> > --- a/meta/lib/oeqa/utils/commands.py
> > +++ b/meta/lib/oeqa/utils/commands.py
> > @@ -29,7 +29,7 @@ class Command(object):
> >
> >          self.defaultopts = {
> >              "stdout": subprocess.PIPE,
> > -            "stderr": subprocess.STDOUT,
> > +            "stderr": subprocess.PIPE,
> >              "stdin": None,
> >              "shell": False,
> >              "bufsize": -1,
> > --
> > 2.12.0
> >
> > --
> > _______________________________________________
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
>