From: Peter Seebach <peter.seebach@windriver.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>, yocto@yoctoproject.org
Subject: Re: pseudo interaction issue
Date: Tue, 27 Mar 2012 09:26:43 -0500 [thread overview]
Message-ID: <20120327092643.78517bc5@wrlaptop> (raw)
In-Reply-To: <1332798316.28414.138.camel@ted>
On Mon, 26 Mar 2012 22:45:16 +0100
Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
> I'm still a little surprised we don't make any system() calls though.
> I just tried putting a os.system("true") call into the "breakit"
> class and it doesn't trigger the warnings. Could that be down to the
> lack of a popen wrapper?
Conclusion: Yes, it could. Once I understood why the popen wrapper
wasn't working, it got better; using the popen wrapper fixes things for
the breakit class, and probably for hob also.
However, the popen() wrapper is not really restoring the environment
after it runs. (In fact, I just looked at system(), and it does a
setupenv() but not a dropenv(), so it will, I think, be running
anything it runs with pseudo loaded but disabled, I think.)
I still think it might be wiser for us to move PSEUDO_PREFIX into the
list of things that are always set. I'm not sure; if we don't, we have
a good chance of finding weird problems faster, I guess!
-s
--
Listen, get this. Nobody with a good compiler needs to be justified.
next prev parent reply other threads:[~2012-03-27 14:26 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-17 17:35 pseudo interaction issue Paul Eggleton
2012-02-17 18:50 ` Mark Hatle
2012-03-14 9:02 ` Xu, Dongxiao
2012-03-22 1:49 ` Xu, Dongxiao
2012-03-22 16:18 ` Peter Seebach
2012-03-23 1:01 ` Xu, Dongxiao
2012-03-23 2:29 ` Peter Seebach
2012-03-23 3:21 ` Xu, Dongxiao
2012-03-23 7:16 ` Peter Seebach
2012-03-23 12:20 ` Paul Eggleton
2012-03-23 20:06 ` Peter Seebach
2012-03-23 22:45 ` Peter Seebach
2012-03-24 17:15 ` Richard Purdie
2012-03-24 17:41 ` Richard Purdie
2012-03-26 16:44 ` Peter Seebach
2012-03-26 16:47 ` Richard Purdie
2012-03-26 17:18 ` Peter Seebach
2012-03-26 21:45 ` Richard Purdie
2012-03-27 3:47 ` Peter Seebach
2012-03-27 14:26 ` Peter Seebach [this message]
2012-03-26 7:43 ` Peter Seebach
2012-03-26 9:23 ` Richard Purdie
2012-03-26 20:36 ` Peter Seebach
2012-03-26 20:41 ` Peter Seebach
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120327092643.78517bc5@wrlaptop \
--to=peter.seebach@windriver.com \
--cc=paul.eggleton@linux.intel.com \
--cc=richard.purdie@linuxfoundation.org \
--cc=yocto@yoctoproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.