[OE-core] [PATCH 0/1] pseudo: add prefix for log and make log consistent
Seebs
seebs at seebs.net
Thu Sep 29 03:15:01 UTC 2016
On 28 Sep 2016, at 21:31, Robert Yang wrote:
> Thanks for the reply, I tested your patch from git repo, it doesn't
> work,
> the message goes to the terminal:
> Child process exit status 4: lock_held
This, it turns out, is a result of a much worse bug which somehow
previously evaded detection,
and just happened to work by blind luck or coincidence or something.
Which is that pseudo_logfile() was returning, not a file descriptor, but
"either 0 or -1", but I was then treating it as a file descriptor.
So actually the messages were going to fd 0, not to fd 2, in this case.
Fixed with a one-line patch:
- return 0;
+ return fd;
(And actually, I think it may make sense to go change those messages
from pseudo_diag to pseudo_debug, since they're basically useless unless
you're debugging the startup code.)
-s
More information about the Openembedded-core
mailing list