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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 23:07 +0200, Patrick Ohly wrote: > On Thu, 2017-06-22 at 15:47 -0500, Leonardo Sandoval wrote: > > On Thu, 2017-06-22 at 19:39 +0200, Patrick Ohly wrote: > > > On Thu, 2017-06-22 at 11:18 -0500, Leonardo Sandoval wrote: > > > > On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly w

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

2017-06-22 Thread Patrick Ohly
On Thu, 2017-06-22 at 15:47 -0500, Leonardo Sandoval wrote: > On Thu, 2017-06-22 at 19:39 +0200, Patrick Ohly wrote: > > On Thu, 2017-06-22 at 11:18 -0500, Leonardo Sandoval wrote: > > > On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly wrote: > > > > On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sando

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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 21:35 +0200, Patrick Ohly wrote: > > And please, whatever we do, let's not change oeqa in a way that breaks > existing tests, whether they are in OE-core or some other layer. > The AB issue that was reported recently may be related to this change but locally I have not see

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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 19:39 +0200, Patrick Ohly wrote: > On Thu, 2017-06-22 at 11:18 -0500, Leonardo Sandoval wrote: > > On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly wrote: > > > On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sandoval wrote: > > > > On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly w

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

2017-06-22 Thread Patrick Ohly
On Thu, 2017-06-22 at 17:59 +0100, Richard Purdie wrote: > The problem here is that stdout and stderr are being mixed and that > mixing is not accounting for newlines. bitbake itself does print > different messages to both. The mixing depends on kernel page size and > when the buffers are flushed.

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

2017-06-22 Thread Patrick Ohly
On Thu, 2017-06-22 at 11:18 -0500, Leonardo Sandoval wrote: > On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly wrote: > > On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sandoval wrote: > > > On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly wrote: > > > > On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sando

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

2017-06-22 Thread Richard Purdie
On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly wrote: > On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sandoval wrote: > > > > On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly wrote: > > > > > > On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sandoval wrote: > > > > > > > > On Thu, 2017-06-22 at 16:17

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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 17:59 +0200, Patrick Ohly wrote: > On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sandoval wrote: > > On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly wrote: > > > On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sandoval wrote: > > > > On Thu, 2017-06-22 at 16:17 +0200, Patrick Ohly w

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

2017-06-22 Thread Patrick Ohly
On Thu, 2017-06-22 at 10:37 -0500, Leonardo Sandoval wrote: > On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly wrote: > > On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sandoval wrote: > > > On Thu, 2017-06-22 at 16:17 +0200, Patrick Ohly wrote: > > > > On Mon, 2017-06-19 at 07:39 -0700, > > > > leonar

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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 17:14 +0200, Patrick Ohly wrote: > On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sandoval wrote: > > On Thu, 2017-06-22 at 16:17 +0200, Patrick Ohly wrote: > > > On Mon, 2017-06-19 at 07:39 -0700, > > > leonardo.sandoval.gonza...@linux.intel.com wrote: > > > > From: Leonardo San

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

2017-06-22 Thread Patrick Ohly
On Thu, 2017-06-22 at 09:58 -0500, Leonardo Sandoval wrote: > On Thu, 2017-06-22 at 16:17 +0200, Patrick Ohly wrote: > > On Mon, 2017-06-19 at 07:39 -0700, > > leonardo.sandoval.gonza...@linux.intel.com wrote: > > > From: Leonardo Sandoval > > > > > > Do not mix the stderr into stdout, allowing t

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

2017-06-22 Thread Leonardo Sandoval
On Thu, 2017-06-22 at 16:17 +0200, Patrick Ohly wrote: > On Mon, 2017-06-19 at 07:39 -0700, > leonardo.sandoval.gonza...@linux.intel.com wrote: > > From: Leonardo Sandoval > > > > Do not mix the stderr into stdout, allowing test cases to query > > the specific output. > > This changes the behavi

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

2017-06-22 Thread Patrick Ohly
On Mon, 2017-06-19 at 07:39 -0700, leonardo.sandoval.gonza...@linux.intel.com wrote: > From: Leonardo Sandoval > > Do not mix the stderr into stdout, allowing test cases to query > the specific output. This changes the behavior of functions that are also used outside of OE-core in a way that won

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

2017-06-21 Thread Leonardo Sandoval
On Wed, 2017-06-21 at 13:56 +0300, Jussi Kukkonen wrote: > On 19 June 2017 at 17:39, > wrote: > > > > From: Leonardo Sandoval > > > > 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 > fai

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

2017-06-21 Thread Burton, Ross
On 21 June 2017 at 11:56, Jussi Kukkonen wrote: > 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' > I'll admit that my

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

2017-06-21 Thread Jussi Kukkonen
On 19 June 2017 at 17:39, wrote: > > From: Leonardo Sandoval > > 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. Assertion

[OE-core] [PATCH 1/2] commands: send stderr to a new pipe

2017-06-19 Thread leonardo . sandoval . gonzalez
From: Leonardo Sandoval Do not mix the stderr into stdout, allowing test cases to query the specific output. [YOCTO #9693] Signed-off-by: Leonardo Sandoval --- meta/lib/oeqa/utils/commands.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/utils/commands.py b